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

[Simon Guinot]

On Tue, Apr 02, 2013 at 06:17:10AM +0200, Chris Moore wrote:
> Hi Simon,
> 
> Le 01/04/2013 18:33, Simon Guinot a écrit :
> >On Sat, Mar 30, 2013 at 07:51:53AM +0100, Chris Moore wrote:
> >
> >Le 29/03/2013 19:41, Simon Guinot a écrit :
> >>>On Fri, Mar 29, 2013 at 11:31:32AM -0400, Jason Cooper wrote
> >>>>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.
> >Indeed, there is two (or rather one and half) released products called
> >CloudBox.
> >
> >The old one (the first smallnetbuilder link) has been first called
> >Network Space v2 mini, then SafeBox and finally released under the name
> >Cloudbox. Very very few products have been sold. I don't know any owner
> >and I suspect you got yours from the LaCie beta test program :)
> >
> 
> Yes, in fact I was given both by LaCie for beta testing :)
> Thank you, LaCie ;-)
> 
> >The new CloudBox is a very different product which has been first named
> >FamilyBox and then released under the CloudBox name. Because the previous
> >CloudBox was a really confidential product, the marketing guys have
> >decided to reuse the name. This was a bad idea and despite the warnings
> >they did that even so.
> >
> >Now, the fact is that the new CloudBox is a huge-selling LaCie product.
> >That's why I propose to keep the names just as they are.
> >
> 
> That is surprising: I greatly prefer the old black one, mainly
> because it uploads autonomously with the PC powered off.

I suspect the problem was the disk capacity, 200GB only. The product
price was the same as 1|2TB NASes. Despite the fashioned cloud effect,
maybe this was a little bit over expensive...

> 
> Are all the new ones white and all the old ones black?

Arf. I don't think we can trust the color case. Indeed, the old ones
are black but there is still no guarantee that the new ones will stay
white.

> If so, can't we add white to the name or at least specify in Kconfig
> that it is the white version with a 3.5" disk?

Maybe we could append some suffixes with version:
netspace_mini_v2 -> cloudbox_v1
cloudbox         -> cloudbox_v2

But again, I am quite reluctant to do this change. I think this would
be confusing for the users.

Simon
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 198 bytes
Desc: Digital signature
URL: <http://lists.infradead.org/pipermail/linux-arm-kernel/attachments/20130402/1dd2fcb2/attachment.sig>