Re: [PATCH 0/5] m25p80,spi-nor: Fix module aliases for m25p80; clean up chip identification

2014-10-09 Thread Brian Norris 
On Tue, Sep 30, 2014 at 02:47:53AM +0100, Ben Hutchings wrote:
 On Sun, 2014-09-28 at 15:03 -0700, Brian Norris wrote:
  On Sun, Sep 14, 2014 at 06:13:15PM +0100, Ben Hutchings wrote:
   Note that these patch are:
   - Based on your 'testing' branch
  
  Which testing branch? These two are the only official
  repos for MTD:
  
  git git://git.infradead.org/linux-mtd.git
  git git://git.infradead.org/l2-mtd.git
 
 You had a testing branch in
 git://git.infradead.org/users/norris/linux-mtd.git, and that included
 commit af249c3a0ca8 ('mtd: spi-nor: remove duplicated w25q128 entry')
 which this patch series depended on.

That repository is very much unofficial. I have a MAINTAINERS patch
queued up to add git://git.infradead.org/l2-mtd.git (our -next repo) to
reduce future confusion.

Regards,
Brian


-- 
To UNSUBSCRIBE, email to debian-kernel-requ...@lists.debian.org
with a subject of unsubscribe. Trouble? Contact listmas...@lists.debian.org
Archive: https://lists.debian.org/20141010045200.GC22169@norris-Latitude-E6410

Re: [PATCH 0/5] m25p80,spi-nor: Fix module aliases for m25p80; clean up chip identification

2014-09-29 Thread Ben Hutchings 
On Sun, 2014-09-28 at 15:03 -0700, Brian Norris wrote:
 On Sun, Sep 14, 2014 at 06:13:15PM +0100, Ben Hutchings wrote:
  On Sun, 2014-09-14 at 18:10 +0100, Ben Hutchings wrote:
   The first patch in the series restores the module aliases to m25p80, but
   it does so by duplicating the list of names.  This should be suitable
   for stable, but it isn't viable in the longer term.
   
   The following patches change the spi-nor interface so that this
   duplication is no longer necessary.  This includes removing
   spi_nor::read_id, but it could be re-added after this with a different
   interface, e.g. returning a flash_info structure (which would need to be
   defined in spi_nor.h).
  
  Note that these patch are:
  - Based on your 'testing' branch
 
 Which testing branch? These two are the only official
 repos for MTD:
 
 git git://git.infradead.org/linux-mtd.git
 git git://git.infradead.org/l2-mtd.git

You had a testing branch in
git://git.infradead.org/users/norris/linux-mtd.git, and that included
commit af249c3a0ca8 ('mtd: spi-nor: remove duplicated w25q128 entry')
which this patch series depended on.

Ben.

  - Untested by me, aside from compiling and checking that m25p80 has the
expected module aliases
 
 Brian

-- 
Ben Hutchings
The two most common things in the universe are hydrogen and stupidity.


signature.asc
Description: This is a digitally signed message part

Re: [PATCH 0/5] m25p80,spi-nor: Fix module aliases for m25p80; clean up chip identification

2014-09-28 Thread Mark Brown 
On Sun, Sep 14, 2014 at 06:10:24PM +0100, Ben Hutchings wrote:
 The first patch in the series restores the module aliases to m25p80, but
 it does so by duplicating the list of names.  This should be suitable
 for stable, but it isn't viable in the longer term.

Please don't CC patches not for the SPI subsystem to linux-spi, they
just clog up patchwork for no benefit.


signature.asc
Description: Digital signature

Re: [PATCH 0/5] m25p80,spi-nor: Fix module aliases for m25p80; clean up chip identification

2014-09-28 Thread Brian Norris 
On Sun, Sep 14, 2014 at 06:13:15PM +0100, Ben Hutchings wrote:
 On Sun, 2014-09-14 at 18:10 +0100, Ben Hutchings wrote:
  The first patch in the series restores the module aliases to m25p80, but
  it does so by duplicating the list of names.  This should be suitable
  for stable, but it isn't viable in the longer term.
  
  The following patches change the spi-nor interface so that this
  duplication is no longer necessary.  This includes removing
  spi_nor::read_id, but it could be re-added after this with a different
  interface, e.g. returning a flash_info structure (which would need to be
  defined in spi_nor.h).
 
 Note that these patch are:
 - Based on your 'testing' branch

Which testing branch? These two are the only official
repos for MTD:

git git://git.infradead.org/linux-mtd.git
git git://git.infradead.org/l2-mtd.git

 - Untested by me, aside from compiling and checking that m25p80 has the
   expected module aliases

Brian


-- 
To UNSUBSCRIBE, email to debian-kernel-requ...@lists.debian.org
with a subject of unsubscribe. Trouble? Contact listmas...@lists.debian.org
Archive: https://lists.debian.org/20140928220300.GB3248@norris-Latitude-E6410

Re: [PATCH 0/5] m25p80,spi-nor: Fix module aliases for m25p80; clean up chip identification

2014-09-14 Thread Ben Hutchings 
On Sun, 2014-09-14 at 18:10 +0100, Ben Hutchings wrote:
 The first patch in the series restores the module aliases to m25p80, but
 it does so by duplicating the list of names.  This should be suitable
 for stable, but it isn't viable in the longer term.
 
 The following patches change the spi-nor interface so that this
 duplication is no longer necessary.  This includes removing
 spi_nor::read_id, but it could be re-added after this with a different
 interface, e.g. returning a flash_info structure (which would need to be
 defined in spi_nor.h).

Note that these patch are:
- Based on your 'testing' branch
- Untested by me, aside from compiling and checking that m25p80 has the
  expected module aliases

Ben.

-- 
Ben Hutchings
The world is coming to an end.  Please log off.


signature.asc
Description: This is a digitally signed message part