Re: [PATCH 0/8] Armada XP pinctrl consolidation and ix4-300d fixes

2014-09-21 Thread Benoit Masson
ada-xp.dtsi | 37 + > 8 files changed, 202 insertions(+), 189 deletions(-) > > --- > Cc: Jason Cooper > Cc: Andrew Lunn > Cc: Gregory Clement > Cc: Ezequiel Garcia > Cc: Thomas Petazzoni > Cc: Benoit Masson > Cc: devicet...@vger.kernel.org &g

[PATCH 2/2] ARM: mvebu: Added dts defintion for Lenovo Iomega ix4-300d NAS

2014-07-26 Thread Benoit Masson
The Lenovo Iomega ix4-300d is a 4-Bay sata NAS with dual Gb, USB2.0 & 3.0, powered by a Marvell Armada XP MV78230 dual core CPU. http://shop.lenovo.com/us/en/servers/network-storage/lenovoemc/ix4-300d/ Signed-off-by: Benoit Masson --- arch/arm/boot/dts/Makefile |

[PATCH 2/2] V3 ARM: mvebu: Added dts defintion for Lenovo Iomega ix4-300d NAS

2014-07-26 Thread Benoit Masson
The Lenovo Iomega ix4-300d is a 4-Bay sata NAS with dual Gb, USB2.0 & 3.0, powered by a Marvell Armada XP MV78230 dual core CPU. http://shop.lenovo.com/us/en/servers/network-storage/lenovoemc/ix4-300d/ Signed-off-by: Benoit Masson --- arch/arm/boot/dts/Makefile |

[PATCH 2/2] V4 ARM: mvebu: Added dts defintion for Lenovo Iomega ix4-300d NAS

2014-07-27 Thread Benoit Masson
The Lenovo Iomega ix4-300d is a 4-Bay sata NAS with dual Gb, USB2.0 & 3.0, powered by a Marvell Armada XP MV78230 dual core CPU. http://shop.lenovo.com/us/en/servers/network-storage/lenovoemc/ix4-300d/ Signed-off-by: Benoit Masson --- arch/arm/boot/dts/Makefile |

Re: [PATCH 2/2] V3 ARM: mvebu: Added dts defintion for Lenovo Iomega ix4-300d NAS

2014-07-27 Thread Benoit Masson
Le 27 juil. 2014 à 22:32, Thomas Petazzoni a écrit : > Dear Benoit Masson, > Dear Thomas, > On Sat, 26 Jul 2014 16:48:10 -0700, Benoit Masson wrote: > >> +gpio-keys { >> +compatible = "gpio-keys"; >> +pinctrl-

Re: [PATCH 2/2] ARM: mvebu: Added dts defintion for Lenovo Iomega ix4-300d NAS

2014-07-28 Thread Benoit Masson
Le 28 juil. 2014 à 10:49, Arnd Bergmann a écrit : > On Sunday 27 July 2014 03:52:44 Andrew Lunn wrote: >> Hi Benoit Hi Arnd, >> >>> + i2c@11000 { >>> + compatible = "marvell,mv78230-a0-i2c", >>> + "marvell,mv6

Re: [PATCH 2/2] ARM: mvebu: Added dts defintion for Lenovo Iomega ix4-300d NAS

2014-07-28 Thread Benoit Masson
Le 28 juil. 2014 à 13:02, Arnd Bergmann a écrit : > On Monday 28 July 2014 12:50:47 Benoit Masson wrote: >>> >>> It should still list both "marvell,mv78230-i2c" and "marvell,mv64xxx-i2c", >>> as the generic name may not be enough to describe t

Re: [PATCH 2/2] V4 ARM: mvebu: Added dts defintion for Lenovo Iomega ix4-300d NAS

2014-07-28 Thread Benoit Masson
Le 28 juil. 2014 à 16:08, Jason Cooper a écrit : > Benoit, > > On Sun, Jul 27, 2014 at 04:30:39PM -0700, Benoit Masson wrote: >> The Lenovo Iomega ix4-300d is a 4-Bay sata NAS with dual Gb, >> USB2.0 & 3.0, powered by a Marvell Armada XP MV78230 dual core CPU. >>

Re: [PATCH 1/2] Added dts defintion for Lenovo ix4-300d nas

2014-07-23 Thread Benoit Masson
Hi Andrew I've taken your comment into account, yet for the PHY init comment, see my answer below. Le 23 juil. 2014 à 16:14, Andrew Lunn a écrit : > >>> + /* warning: you need both eth1 & 0 to be initialize for poweroff to >>> shutdown otherwise it reboots */ > > What do you mean by initial

Re: [PATCH 2/2] Adding lenovo in vendor

2014-07-23 Thread Benoit Masson
Looking at the vendor list it should then be "Lenovo Group Ltd." since no other vendor except 1 use Limited all other use Ltd. Fine with this ? Le 23 juil. 2014 à 19:26, Jason Cooper a écrit : > On Wed, Jul 23, 2014 at 05:10:33PM +0200, Andrew Lunn wrote: +lenovoLENOVO >>> >>> is this

Re: [PATCH 1/2] Added dts defintion for Lenovo ix4-300d nas

2014-07-23 Thread Benoit Masson
Well after several test with the original BSP driver I found that this is not fully WOL related. It could be that the LED pin is linked to poweron for WOL, yet here the issue is on shutdown pulling up MPP24 should poweroff the deice but only reboots it except is both PHY have some reg written to

[PATCH 2/2] ARM: mvebu: Added dts defintion for Lenovo Iomega ix4-300d NAS

2014-07-23 Thread Benoit Masson
The Lenovo Iomega ix4-300d is a 4-Bay sata NAS with dual Gb, USB2.0 & 3.0, powered by a Marvell Armada XP MV78230 dual core CPU. http://shop.lenovo.com/fr/fr/servers/network-storage/lenovoemc/ix4-300d/ Signed-off-by: Benoit Masson --- arch/arm/boot/dts/Makefile |

[PATCH 1/2] Adding Lenovo - Lenovo Group Ltd. to the vendor-prefixes list.

2014-07-23 Thread Benoit Masson
Lenovo Group Ltd. (stylized as lenovo) is a Chinese multinational computer technology company with headquarters in Beijing, China, and Morrisville, North Carolina, United States. http://www.lenovo.com/ Signed-off-by: Benoit Masson --- Documentation/devicetree/bindings/vendor-prefixes.txt | 1

[PATCH 2/2] ARM: mvebu: Added dts defintion for Lenovo Iomega ix4-300d NAS

2014-07-23 Thread Benoit Masson
The Lenovo Iomega ix4-300d is a 4-Bay sata NAS with dual Gb, USB2.0 & 3.0, powered by a Marvell Armada XP MV78230 dual core CPU. http://shop.lenovo.com/fr/fr/servers/network-storage/lenovoemc/ix4-300d/ Signed-off-by: Benoit Masson --- arch/arm/boot/dts/Makefile |

Re: [PATCH 2/2] ARM: mvebu: Added dts defintion for Lenovo Iomega ix4-300d NAS

2014-07-23 Thread Benoit Masson
No worries, sorry it makes more email for you guys, I've added the colors for power and revert order hdd:blue instead of blue:hdd for the hdd led. For the marvell,mv78230-a0-i2c unfortunately i've spend 3 hours trying to understand this but it only works with this on the ix4-300d :(. There was

Re: [PATCH 1/2] Added dts defintion for Lenovo ix4-300d nas

2014-07-23 Thread Benoit Masson
Well I'm glad you spend time on this. So I'll explain more in detail where I am on this one. I have the vendor 3.0.29 kernel patched with LSP drivers from Marvell (which includes the PHY & Net drivers). With this version (called LSPkernel now on) pulling MPP24 GPIO output up shutdown the power

Re: [PATCH 0/8] Armada XP pinctrl consolidation and ix4-300d fixes

2014-10-14 Thread Benoit Masson
Le 6 oct. 2014 à 18:13, Sebastian Hesselbarth a écrit : > On 10/06/2014 01:11 AM, Benoit Masson wrote: >> Le 3 oct. 2014 à 17:41, Sebastian Hesselbarth >> a écrit : >>> On 10/03/2014 05:29 PM, Benoit Masson wrote: >>>> Le 3 oct. 2014 à 17:06, Sebastian H

Re: [PATCH 0/8] Armada XP pinctrl consolidation and ix4-300d fixes

2014-10-15 Thread Benoit Masson
Le 15 oct. 2014 à 16:41, Andrew Lunn a écrit : >>> All the 3 branch works WHEN APPLYING A0 patch (below), with both my custom >>> kernel config and the arch/arm/configs/mvebu_v7_defconfig >> Just one thing though, one of my cron behavior change with 3.17, when I use >> the cmd "/sbin/hwclock

Re: [PATCH 0/8] Armada XP pinctrl consolidation and ix4-300d fixes

2014-10-05 Thread Benoit Masson
Le 3 oct. 2014 à 17:41, Sebastian Hesselbarth a écrit : > On 10/03/2014 05:29 PM, Benoit Masson wrote: >> Le 3 oct. 2014 à 17:06, Sebastian Hesselbarth >> a écrit : >>> On 10/03/2014 04:11 PM, Jason Cooper wrote: >>>> On Sun, Sep 21, 2014 at 04:11:23PM +

Re: [PATCH 0/8] Armada XP pinctrl consolidation and ix4-300d fixes

2014-10-03 Thread Benoit Masson
Le 3 oct. 2014 à 17:06, Sebastian Hesselbarth a écrit : > On 10/03/2014 04:11 PM, Jason Cooper wrote: >> On Sun, Sep 21, 2014 at 04:11:23PM +0200, Benoit Masson wrote: >>>> Le 19 sept. 2014 à 22:14, Sebastian Hesselbarth >>>> a écrit : >>>> this

Re: [PATCH 1/4] ARM: dt: mvebu: ix4-300d: remove whole flash partition

2015-11-28 Thread Benoit Masson
>From mobile > Le 28 nov. 2015 à 18:38, Sebastian Hesselbarth > a écrit : > >> On 28.11.2015 17:52, Andrew Lunn wrote: >>> On Sat, Nov 28, 2015 at 12:14:05PM +0100, Sebastian Hesselbarth wrote: >>> Current NAND node has an additional flash partition for the whole >>> flash overlapping with real