From mboxrd@z Thu Jan 1 00:00:00 1970 From: Jason Cooper Subject: Re: [PATCH reword] ARM: MVEBU: Netgear RN102: Use Hardware BCH ECC Date: Sun, 14 Sep 2014 00:18:47 -0400 Message-ID: <20140914041847.GB21986@titan.lakedaemon.net> References: <1410339341-3372-1-git-send-email-klightspeed@killerwolves.net> <20140913210829.GI30828@titan.lakedaemon.net> <54150349.7050503@killerwolves.net> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Return-path: Content-Disposition: inline In-Reply-To: <54150349.7050503-aslSrjg9ejhWX4hkXwHRhw@public.gmane.org> Sender: devicetree-owner-u79uwXL29TY76Z2rM5mHXA@public.gmane.org To: Ben Peddell Cc: Ezequiel Garcia , Brian Norris , Russell King , linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r@public.gmane.org, Arnaud Ebalard , linux-mtd-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r@public.gmane.org, Thomas Petazzoni , Gregory Clement , devicetree-u79uwXL29TY76Z2rM5mHXA@public.gmane.org List-Id: devicetree@vger.kernel.org On Sun, Sep 14, 2014 at 12:54:01PM +1000, Ben Peddell wrote: > On 14/09/14 07:08, Jason Cooper wrote: > > On Wed, Sep 10, 2014 at 06:55:41PM +1000, klightspeed-aslSrjg9ejhWX4hkXwHRhw@public.gmane.org wrote: > >> The bootloader on the Netgear ReadyNAS RN102 uses Hardware BCH ECC > >> (strength = 4), while the pxa3xx NAND driver by default uses > >> Hamming ECC (strength = 1). > >> > >> This patch changes the ECC mode on these machines to match that > >> of the bootloader and of the stock firmware. That way, it is > >> now possible to update the kernel from userland (e.g. using > >> standard tools from mtd-utils package); u-boot will happily > >> load and boot it. > >> > >> Fixes: 92beaccd8b49 ("ARM: mvebu: Enable NAND controller in ReadyNAS 102 .dts file") > >> Signed-off-by: Ben Peddell > >> Acked-by: Ezequiel Garcia > >> Tested-by: Arnaud Ebalard > >> --- > >> arch/arm/boot/dts/armada-370-netgear-rn102.dts | 4 ++++ > >> 1 file changed, 4 insertions(+) > > > > I've already applied this the mvebu/fixes: > > > > 500abb6ccb9e ARM: mvebu: Netgear RN2120: Use Hardware BCH ECC > > That patch only affects arch/arm/boot/dts/armada-xp-netgear-rn2120.dts > > The patch I posted affects arch/arm/boot/dts/armada-370-netgear-rn102.dts Gah! You're right. Applied to mvebu/fixes and Cc'd the stable v3.14+ thx, Jason. -- To unsubscribe from this list: send the line "unsubscribe devicetree" in the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org More majordomo info at http://vger.kernel.org/majordomo-info.html From mboxrd@z Thu Jan 1 00:00:00 1970 Date: Sun, 14 Sep 2014 00:18:47 -0400 From: Jason Cooper To: Ben Peddell Subject: Re: [PATCH reword] ARM: MVEBU: Netgear RN102: Use Hardware BCH ECC Message-ID: <20140914041847.GB21986@titan.lakedaemon.net> References: <1410339341-3372-1-git-send-email-klightspeed@killerwolves.net> <20140913210829.GI30828@titan.lakedaemon.net> <54150349.7050503@killerwolves.net> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <54150349.7050503@killerwolves.net> Cc: Thomas Petazzoni , devicetree@vger.kernel.org, Russell King , Arnaud Ebalard , linux-mtd@lists.infradead.org, Ezequiel Garcia , Gregory Clement , Brian Norris , linux-arm-kernel@lists.infradead.org List-Id: Linux MTD discussion mailing list List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , On Sun, Sep 14, 2014 at 12:54:01PM +1000, Ben Peddell wrote: > On 14/09/14 07:08, Jason Cooper wrote: > > On Wed, Sep 10, 2014 at 06:55:41PM +1000, klightspeed@killerwolves.net wrote: > >> The bootloader on the Netgear ReadyNAS RN102 uses Hardware BCH ECC > >> (strength = 4), while the pxa3xx NAND driver by default uses > >> Hamming ECC (strength = 1). > >> > >> This patch changes the ECC mode on these machines to match that > >> of the bootloader and of the stock firmware. That way, it is > >> now possible to update the kernel from userland (e.g. using > >> standard tools from mtd-utils package); u-boot will happily > >> load and boot it. > >> > >> Fixes: 92beaccd8b49 ("ARM: mvebu: Enable NAND controller in ReadyNAS 102 .dts file") > >> Signed-off-by: Ben Peddell > >> Acked-by: Ezequiel Garcia > >> Tested-by: Arnaud Ebalard > >> --- > >> arch/arm/boot/dts/armada-370-netgear-rn102.dts | 4 ++++ > >> 1 file changed, 4 insertions(+) > > > > I've already applied this the mvebu/fixes: > > > > 500abb6ccb9e ARM: mvebu: Netgear RN2120: Use Hardware BCH ECC > > That patch only affects arch/arm/boot/dts/armada-xp-netgear-rn2120.dts > > The patch I posted affects arch/arm/boot/dts/armada-370-netgear-rn102.dts Gah! You're right. Applied to mvebu/fixes and Cc'd the stable v3.14+ thx, Jason. From mboxrd@z Thu Jan 1 00:00:00 1970 From: jason@lakedaemon.net (Jason Cooper) Date: Sun, 14 Sep 2014 00:18:47 -0400 Subject: [PATCH reword] ARM: MVEBU: Netgear RN102: Use Hardware BCH ECC In-Reply-To: <54150349.7050503@killerwolves.net> References: <1410339341-3372-1-git-send-email-klightspeed@killerwolves.net> <20140913210829.GI30828@titan.lakedaemon.net> <54150349.7050503@killerwolves.net> Message-ID: <20140914041847.GB21986@titan.lakedaemon.net> To: linux-arm-kernel@lists.infradead.org List-Id: linux-arm-kernel.lists.infradead.org On Sun, Sep 14, 2014 at 12:54:01PM +1000, Ben Peddell wrote: > On 14/09/14 07:08, Jason Cooper wrote: > > On Wed, Sep 10, 2014 at 06:55:41PM +1000, klightspeed at killerwolves.net wrote: > >> The bootloader on the Netgear ReadyNAS RN102 uses Hardware BCH ECC > >> (strength = 4), while the pxa3xx NAND driver by default uses > >> Hamming ECC (strength = 1). > >> > >> This patch changes the ECC mode on these machines to match that > >> of the bootloader and of the stock firmware. That way, it is > >> now possible to update the kernel from userland (e.g. using > >> standard tools from mtd-utils package); u-boot will happily > >> load and boot it. > >> > >> Fixes: 92beaccd8b49 ("ARM: mvebu: Enable NAND controller in ReadyNAS 102 .dts file") > >> Signed-off-by: Ben Peddell > >> Acked-by: Ezequiel Garcia > >> Tested-by: Arnaud Ebalard > >> --- > >> arch/arm/boot/dts/armada-370-netgear-rn102.dts | 4 ++++ > >> 1 file changed, 4 insertions(+) > > > > I've already applied this the mvebu/fixes: > > > > 500abb6ccb9e ARM: mvebu: Netgear RN2120: Use Hardware BCH ECC > > That patch only affects arch/arm/boot/dts/armada-xp-netgear-rn2120.dts > > The patch I posted affects arch/arm/boot/dts/armada-370-netgear-rn102.dts Gah! You're right. Applied to mvebu/fixes and Cc'd the stable v3.14+ thx, Jason.