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

[Jason Cooper]

On Sat, Mar 30, 2013 at 07:51:53AM +0100, Chris Moore wrote:
> 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

In this case, we usually look for a model number on the back and append
that to the compatible string.  There are three model numbers listed
here [1] under specifications, based on capacity.

thx,

Jason.

[1] http://www.lacie.com/products/product.htm?id=10597