Re: [PATCH v3 1/1] add support for Seagate BlackArmor NAS220

2014-12-26 Thread Evgeni Dobrev
Hello Sebastian, On Thu, Dec 25, 2014 at 02:08:12PM +0100, Sebastian Hesselbarth wrote: > On 22.12.2014 13:57, Evgeni Dobrev wrote: > >This patch adds support for Seagate BlackArmor NAS220. > > > >The Seagate BlackArmor NAS 220 is a NAS system based on Marvell 88f6192. It > >has > >32MB NAND and

Re: [PATCH v3 1/1] add support for Seagate BlackArmor NAS220

2014-12-25 Thread Sebastian Hesselbarth
On 25.12.2014 15:12, Andrew Lunn wrote: Hi Sebastian I'm not sure what you mean here. The binding Documentation says: I was hoping that using phys/phy-names would allow us to get rid of nr-ports property. I haven't checked the corresponding code and likely will not before next year, but we sho

Re: [PATCH v3 1/1] add support for Seagate BlackArmor NAS220

2014-12-25 Thread Andrew Lunn
> >Hi Sebastian > > > >I'm not sure what you mean here. The binding Documentation says: > > I was hoping that using phys/phy-names would allow us to get rid of > nr-ports property. I haven't checked the corresponding code and likely > will not before next year, but we should try to get rid of the

Re: [PATCH v3 1/1] add support for Seagate BlackArmor NAS220

2014-12-25 Thread Sebastian Hesselbarth
On 25.12.2014 14:31, Andrew Lunn wrote: On Thu, Dec 25, 2014 at 02:08:12PM +0100, Sebastian Hesselbarth wrote: + status = "okay"; + }; + + sata@8 { + status = "okay"; + nr-ports = <2>; I need some

Re: [PATCH v3 1/1] add support for Seagate BlackArmor NAS220

2014-12-25 Thread Andrew Lunn
On Thu, Dec 25, 2014 at 02:08:12PM +0100, Sebastian Hesselbarth wrote: > >+status = "okay"; > >+}; > >+ > >+sata@8 { > >+status = "okay"; > >+nr-ports = <2>; > > I need some update from the other mvebu guys her

Re: [PATCH v3 1/1] add support for Seagate BlackArmor NAS220

2014-12-25 Thread Sebastian Hesselbarth
On 22.12.2014 13:57, Evgeni Dobrev wrote: This patch adds support for Seagate BlackArmor NAS220. The Seagate BlackArmor NAS 220 is a NAS system based on Marvell 88f6192. It has 32MB NAND and 128MB DRAM. It has two SATA slots, one Gigabit Ethernet port, two USB 2.0 ports, two buttons and three LE

Re: [PATCH v3 1/1] add support for Seagate BlackArmor NAS220

2014-12-23 Thread Andrew Lunn
On Mon, Dec 22, 2014 at 01:57:50PM +0100, Evgeni Dobrev wrote: > This patch adds support for Seagate BlackArmor NAS220. > > The Seagate BlackArmor NAS 220 is a NAS system based on Marvell 88f6192. It > has > 32MB NAND and 128MB DRAM. It has two SATA slots, one Gigabit Ethernet port, > two > USB