Subject: [PATCH] ARM: kirkwood: DT board setup for CloudBox

From: [email-masked] (Chris Moore)
Date: Sat, 30 Mar 2013 07:51:53 +0100

Hi Simon et al.

Le 29/03/2013 19:41, Simon Guinot a ?crit :
> On Fri, Mar 29, 2013 at 11:31:32AM -0400, Jason Cooper wrote:
>> Neat! another new board!
> Others are coming :)

That's great, thank you.

>> On Fri, Mar 29, 2013 at 12:06:46PM +0100, Simon Guinot wrote:
>>> This patch adds DT board setup for the LaCie NAS CloudBox. The CloudBox
>>> is a low cost NAS based on the Network Space v2.
>>>
>>> Chipset list:
>>> - CPU MARVELL 88F6702 1Ghz
>>> - SDRAM memory: 256MB DDR2-800 (2x128MB x8) 400Mhz
>>> - 1 Ethernet Gigabit port (PHY MARVELL 88E1318)
>>> - SPI flash, NOR 512KB
>>> - 1 push button
>>> - 2 LEDs (red and blue)
>>>
>>> +
>>> +/ {
>>> +	model = "LaCie CloudBox";
>>> +	compatible = "lacie,cloudbox", "marvell,kirkwood-88f6702", "marvell,kirkwood";
>> Is there a more specific string we can add here?  eg
>> "lacie,cloudbox-MODEL", "lacie,cloudbox", ...
> Unfortunately no. CloudBox is a product name which is also used by LaCie
> to name the board and the project.
>

I think it really is necessary to have a more specific name.
I have two, very different, products corresponding to LaCie CloudBox:
- The first one was a small black one with a 2.5" disk reviewed here: 
http://www.smallnetbuilder.com/cloud/cloud-storage/462-lacie-cloudbox-reviewed
- The later one is a larger white one with a 3.5" disk described here: 
http://www.smallnetbuilder.com/nas/nas-news/31927-lacie-relaunches-its-cloud-connected-nas

IIRC the later one was originally called "CloudBox Family" although 
maybe Family was dropped when it was officially released.

Cheers,
Chris