Re: [PATCH for-4.4 1/2] mtd: spi-nor: fix Spansion regressions (aliased with Winbond)

2016-04-25 Thread Brian Norris
Hi Cyrille, On Mon, Apr 04, 2016 at 05:33:30PM +0200, Cyrille Pitchen wrote: > Le 01/04/2016 22:27, Brian Norris a écrit : > > On Wed, Mar 30, 2016 at 02:47:48PM +0200, Cyrille Pitchen wrote: > >> Just a general remark: maybe reading the JEDEC ID is not a so reliable > >> mean to > >> discover

Re: [PATCH for-4.4 1/2] mtd: spi-nor: fix Spansion regressions (aliased with Winbond)

2016-04-25 Thread Brian Norris
Hi Cyrille, On Mon, Apr 04, 2016 at 05:33:30PM +0200, Cyrille Pitchen wrote: > Le 01/04/2016 22:27, Brian Norris a écrit : > > On Wed, Mar 30, 2016 at 02:47:48PM +0200, Cyrille Pitchen wrote: > >> Just a general remark: maybe reading the JEDEC ID is not a so reliable > >> mean to > >> discover

Re: [PATCH for-4.4 1/2] mtd: spi-nor: fix Spansion regressions (aliased with Winbond)

2016-04-04 Thread Cyrille Pitchen
Hi Brian, Le 01/04/2016 22:27, Brian Norris a écrit : > On Wed, Mar 30, 2016 at 02:47:48PM +0200, Cyrille Pitchen wrote: >> Hi all, >> >> [...] >>> But this is interesting: I see the latest datasheet for Spansion >>> s25fl064k says it supports the Block Protect bits in the Status >>> Register,

Re: [PATCH for-4.4 1/2] mtd: spi-nor: fix Spansion regressions (aliased with Winbond)

2016-04-04 Thread Cyrille Pitchen
Hi Brian, Le 01/04/2016 22:27, Brian Norris a écrit : > On Wed, Mar 30, 2016 at 02:47:48PM +0200, Cyrille Pitchen wrote: >> Hi all, >> >> [...] >>> But this is interesting: I see the latest datasheet for Spansion >>> s25fl064k says it supports the Block Protect bits in the Status >>> Register,

Re: [PATCH for-4.4 1/2] mtd: spi-nor: fix Spansion regressions (aliased with Winbond)

2016-04-01 Thread Brian Norris
On Wed, Mar 30, 2016 at 02:47:48PM +0200, Cyrille Pitchen wrote: > Hi all, > > [...] > > But this is interesting: I see the latest datasheet for Spansion > > s25fl064k says it supports the Block Protect bits in the Status > > Register, so presumably *some* version of s25fl064k should support > >

Re: [PATCH for-4.4 1/2] mtd: spi-nor: fix Spansion regressions (aliased with Winbond)

2016-04-01 Thread Brian Norris
On Wed, Mar 30, 2016 at 02:47:48PM +0200, Cyrille Pitchen wrote: > Hi all, > > [...] > > But this is interesting: I see the latest datasheet for Spansion > > s25fl064k says it supports the Block Protect bits in the Status > > Register, so presumably *some* version of s25fl064k should support > >

Re: [PATCH for-4.4 1/2] mtd: spi-nor: fix Spansion regressions (aliased with Winbond)

2016-03-31 Thread James Cameron
On Wed, Mar 30, 2016 at 02:47:48PM +0200, Cyrille Pitchen wrote: > Hi all, > > [...] > > But this is interesting: I see the latest datasheet for Spansion > > s25fl064k says it supports the Block Protect bits in the Status > > Register, so presumably *some* version of s25fl064k should support > >

Re: [PATCH for-4.4 1/2] mtd: spi-nor: fix Spansion regressions (aliased with Winbond)

2016-03-31 Thread James Cameron
On Wed, Mar 30, 2016 at 02:47:48PM +0200, Cyrille Pitchen wrote: > Hi all, > > [...] > > But this is interesting: I see the latest datasheet for Spansion > > s25fl064k says it supports the Block Protect bits in the Status > > Register, so presumably *some* version of s25fl064k should support > >

Re: [PATCH for-4.4 1/2] mtd: spi-nor: fix Spansion regressions (aliased with Winbond)

2016-03-30 Thread Cyrille Pitchen
Hi all, [...] > But this is interesting: I see the latest datasheet for Spansion > s25fl064k says it supports the Block Protect bits in the Status > Register, so presumably *some* version of s25fl064k should support > write_sr(nor, 0) to unlock it at boot... > > If Felix's initial report is

Re: [PATCH for-4.4 1/2] mtd: spi-nor: fix Spansion regressions (aliased with Winbond)

2016-03-30 Thread Cyrille Pitchen
Hi all, [...] > But this is interesting: I see the latest datasheet for Spansion > s25fl064k says it supports the Block Protect bits in the Status > Register, so presumably *some* version of s25fl064k should support > write_sr(nor, 0) to unlock it at boot... > > If Felix's initial report is

Re: [PATCH for-4.4 1/2] mtd: spi-nor: fix Spansion regressions (aliased with Winbond)

2016-03-29 Thread Matthias Schiffer
On 03/28/2016 10:56 PM, Brian Norris wrote: > On Mon, Mar 28, 2016 at 12:52:51AM +0200, Matthias Schiffer wrote: >> On 03/26/2016 07:57 PM, Matthias Schiffer wrote: >>> On 12/15/2015 07:48 PM, Brian Norris wrote: Spansion and Winbond have occasionally used the same manufacturer ID, and

Re: [PATCH for-4.4 1/2] mtd: spi-nor: fix Spansion regressions (aliased with Winbond)

2016-03-29 Thread Matthias Schiffer
On 03/28/2016 10:56 PM, Brian Norris wrote: > On Mon, Mar 28, 2016 at 12:52:51AM +0200, Matthias Schiffer wrote: >> On 03/26/2016 07:57 PM, Matthias Schiffer wrote: >>> On 12/15/2015 07:48 PM, Brian Norris wrote: Spansion and Winbond have occasionally used the same manufacturer ID, and

Re: [PATCH for-4.4 1/2] mtd: spi-nor: fix Spansion regressions (aliased with Winbond)

2016-03-28 Thread Brian Norris
On Mon, Mar 28, 2016 at 12:52:51AM +0200, Matthias Schiffer wrote: > On 03/26/2016 07:57 PM, Matthias Schiffer wrote: > > On 12/15/2015 07:48 PM, Brian Norris wrote: > >> Spansion and Winbond have occasionally used the same manufacturer ID, > >> and they don't support the same features.

Re: [PATCH for-4.4 1/2] mtd: spi-nor: fix Spansion regressions (aliased with Winbond)

2016-03-28 Thread Brian Norris
On Mon, Mar 28, 2016 at 12:52:51AM +0200, Matthias Schiffer wrote: > On 03/26/2016 07:57 PM, Matthias Schiffer wrote: > > On 12/15/2015 07:48 PM, Brian Norris wrote: > >> Spansion and Winbond have occasionally used the same manufacturer ID, > >> and they don't support the same features.

Re: [PATCH for-4.4 1/2] mtd: spi-nor: fix Spansion regressions (aliased with Winbond)

2016-03-27 Thread Matthias Schiffer
On 03/26/2016 07:57 PM, Matthias Schiffer wrote: > On 12/15/2015 07:48 PM, Brian Norris wrote: >> Spansion and Winbond have occasionally used the same manufacturer ID, >> and they don't support the same features. Particularly, writing SR=0 >> seems to break read access for Spansion's s25fl064k.

Re: [PATCH for-4.4 1/2] mtd: spi-nor: fix Spansion regressions (aliased with Winbond)

2016-03-27 Thread Matthias Schiffer
On 03/26/2016 07:57 PM, Matthias Schiffer wrote: > On 12/15/2015 07:48 PM, Brian Norris wrote: >> Spansion and Winbond have occasionally used the same manufacturer ID, >> and they don't support the same features. Particularly, writing SR=0 >> seems to break read access for Spansion's s25fl064k.

Re: [PATCH for-4.4 1/2] mtd: spi-nor: fix Spansion regressions (aliased with Winbond)

2016-03-26 Thread Matthias Schiffer
On 12/15/2015 07:48 PM, Brian Norris wrote: > Spansion and Winbond have occasionally used the same manufacturer ID, > and they don't support the same features. Particularly, writing SR=0 > seems to break read access for Spansion's s25fl064k. Unfortunately, we > don't currently have a way to

Re: [PATCH for-4.4 1/2] mtd: spi-nor: fix Spansion regressions (aliased with Winbond)

2016-03-26 Thread Matthias Schiffer
On 12/15/2015 07:48 PM, Brian Norris wrote: > Spansion and Winbond have occasionally used the same manufacturer ID, > and they don't support the same features. Particularly, writing SR=0 > seems to break read access for Spansion's s25fl064k. Unfortunately, we > don't currently have a way to

Re: [PATCH for-4.4 1/2] mtd: spi-nor: fix Spansion regressions (aliased with Winbond)

2016-01-05 Thread bayi cheng
On Wed, 2016-01-06 at 01:03 +0100, Felix Fietkau wrote: > On 2016-01-06 01:02, Brian Norris wrote: > > On Mon, Jan 04, 2016 at 06:29:28PM -0800, Brian Norris wrote: > >> On Tue, Dec 15, 2015 at 10:48:20AM -0800, Brian Norris wrote: > >> > Spansion and Winbond have occasionally used the same

Re: [PATCH for-4.4 1/2] mtd: spi-nor: fix Spansion regressions (aliased with Winbond)

2016-01-05 Thread Felix Fietkau
On 2016-01-06 01:02, Brian Norris wrote: > On Mon, Jan 04, 2016 at 06:29:28PM -0800, Brian Norris wrote: >> On Tue, Dec 15, 2015 at 10:48:20AM -0800, Brian Norris wrote: >> > Spansion and Winbond have occasionally used the same manufacturer ID, >> > and they don't support the same features.

Re: [PATCH for-4.4 1/2] mtd: spi-nor: fix Spansion regressions (aliased with Winbond)

2016-01-05 Thread Brian Norris
On Mon, Jan 04, 2016 at 06:29:28PM -0800, Brian Norris wrote: > On Tue, Dec 15, 2015 at 10:48:20AM -0800, Brian Norris wrote: > > Spansion and Winbond have occasionally used the same manufacturer ID, > > and they don't support the same features. Particularly, writing SR=0 > > seems to break read

Re: [PATCH for-4.4 1/2] mtd: spi-nor: fix Spansion regressions (aliased with Winbond)

2016-01-05 Thread Brian Norris
On Mon, Jan 04, 2016 at 06:29:28PM -0800, Brian Norris wrote: > On Tue, Dec 15, 2015 at 10:48:20AM -0800, Brian Norris wrote: > > Spansion and Winbond have occasionally used the same manufacturer ID, > > and they don't support the same features. Particularly, writing SR=0 > > seems to break read

Re: [PATCH for-4.4 1/2] mtd: spi-nor: fix Spansion regressions (aliased with Winbond)

2016-01-05 Thread Felix Fietkau
On 2016-01-06 01:02, Brian Norris wrote: > On Mon, Jan 04, 2016 at 06:29:28PM -0800, Brian Norris wrote: >> On Tue, Dec 15, 2015 at 10:48:20AM -0800, Brian Norris wrote: >> > Spansion and Winbond have occasionally used the same manufacturer ID, >> > and they don't support the same features.

Re: [PATCH for-4.4 1/2] mtd: spi-nor: fix Spansion regressions (aliased with Winbond)

2016-01-05 Thread bayi cheng
On Wed, 2016-01-06 at 01:03 +0100, Felix Fietkau wrote: > On 2016-01-06 01:02, Brian Norris wrote: > > On Mon, Jan 04, 2016 at 06:29:28PM -0800, Brian Norris wrote: > >> On Tue, Dec 15, 2015 at 10:48:20AM -0800, Brian Norris wrote: > >> > Spansion and Winbond have occasionally used the same

Re: [PATCH for-4.4 1/2] mtd: spi-nor: fix Spansion regressions (aliased with Winbond)

2016-01-04 Thread Brian Norris
On Tue, Dec 15, 2015 at 10:48:20AM -0800, Brian Norris wrote: > Spansion and Winbond have occasionally used the same manufacturer ID, > and they don't support the same features. Particularly, writing SR=0 > seems to break read access for Spansion's s25fl064k. Unfortunately, we > don't currently

Re: [PATCH for-4.4 1/2] mtd: spi-nor: fix Spansion regressions (aliased with Winbond)

2016-01-04 Thread Brian Norris
On Tue, Dec 15, 2015 at 10:48:20AM -0800, Brian Norris wrote: > Spansion and Winbond have occasionally used the same manufacturer ID, > and they don't support the same features. Particularly, writing SR=0 > seems to break read access for Spansion's s25fl064k. Unfortunately, we > don't currently

[PATCH for-4.4 1/2] mtd: spi-nor: fix Spansion regressions (aliased with Winbond)

2015-12-15 Thread Brian Norris
Spansion and Winbond have occasionally used the same manufacturer ID, and they don't support the same features. Particularly, writing SR=0 seems to break read access for Spansion's s25fl064k. Unfortunately, we don't currently have a way to differentiate these Spansion and Winbond parts, so rather

[PATCH for-4.4 1/2] mtd: spi-nor: fix Spansion regressions (aliased with Winbond)

2015-12-15 Thread Brian Norris
Spansion and Winbond have occasionally used the same manufacturer ID, and they don't support the same features. Particularly, writing SR=0 seems to break read access for Spansion's s25fl064k. Unfortunately, we don't currently have a way to differentiate these Spansion and Winbond parts, so rather