mbox series

[0/3] AFS move and OF support

Message ID 20190123122910.19024-1-linus.walleij@linaro.org
Headers show
Series AFS move and OF support | expand

Message

Linus Walleij Jan. 23, 2019, 12:29 p.m. UTC
As a starting point to fixing up the AFS support on ARM
reference designs, let's move the implementation down to
the "parsers" subdirectory and add OF probing support.

More refactorings will be done on top of this change.

Linus Walleij (3):
  mtd: afs: Move AFS partition parser to parsers subdir
  mtd: partitions: Add AFS partitions DT bindings
  mtd: partitions: Add OF support to AFS partitions

 .../mtd/partitions/arm,arm-firmware-suite.txt   | 17 +++++++++++++++++
 drivers/mtd/Kconfig                             | 16 ----------------
 drivers/mtd/Makefile                            |  1 -
 drivers/mtd/parsers/Kconfig                     | 16 ++++++++++++++++
 drivers/mtd/parsers/Makefile                    |  1 +
 drivers/mtd/{ => parsers}/afs.c                 |  7 +++++++
 6 files changed, 41 insertions(+), 17 deletions(-)
 create mode 100644 Documentation/devicetree/bindings/mtd/partitions/arm,arm-firmware-suite.txt
 rename drivers/mtd/{ => parsers}/afs.c (96%)

-- 
2.20.1


______________________________________________________
Linux MTD discussion mailing list
http://lists.infradead.org/mailman/listinfo/linux-mtd/

Comments

Linus Walleij Feb. 13, 2019, 7:59 a.m. UTC | #1
On Wed, Jan 23, 2019 at 1:29 PM Linus Walleij <linus.walleij@linaro.org> wrote:

> As a starting point to fixing up the AFS support on ARM

> reference designs, let's move the implementation down to

> the "parsers" subdirectory and add OF probing support.

>

> More refactorings will be done on top of this change.

>

> Linus Walleij (3):

>   mtd: afs: Move AFS partition parser to parsers subdir

>   mtd: partitions: Add AFS partitions DT bindings

>   mtd: partitions: Add OF support to AFS partitions


Can we merge this for v5.1?

If what you're waiting for is DT binding review I am pretty
sure they are uncontroversial and will not happen at all since
noone responded for three weeks on that list.

The follow-up AFSv2 series would be nice to get in as well
so I can patch up the device trees to finally have all
DT platforms detect the software binaries on the flash.

Yours,
Linus Walleij

______________________________________________________
Linux MTD discussion mailing list
http://lists.infradead.org/mailman/listinfo/linux-mtd/
Linus Walleij Feb. 13, 2019, 8:40 a.m. UTC | #2
Top posting to Boris' kernelorg address, sorry for using the wrong
address, maybe this is why the series was missed in the first
place :/

Yours,
Linus Walleij


On Wed, Feb 13, 2019 at 8:59 AM Linus Walleij <linus.walleij@linaro.org> wrote:
>

> On Wed, Jan 23, 2019 at 1:29 PM Linus Walleij <linus.walleij@linaro.org> wrote:

>

> > As a starting point to fixing up the AFS support on ARM

> > reference designs, let's move the implementation down to

> > the "parsers" subdirectory and add OF probing support.

> >

> > More refactorings will be done on top of this change.

> >

> > Linus Walleij (3):

> >   mtd: afs: Move AFS partition parser to parsers subdir

> >   mtd: partitions: Add AFS partitions DT bindings

> >   mtd: partitions: Add OF support to AFS partitions

>

> Can we merge this for v5.1?

>

> If what you're waiting for is DT binding review I am pretty

> sure they are uncontroversial and will not happen at all since

> noone responded for three weeks on that list.

>

> The follow-up AFSv2 series would be nice to get in as well

> so I can patch up the device trees to finally have all

> DT platforms detect the software binaries on the flash.

>

> Yours,

> Linus Walleij


______________________________________________________
Linux MTD discussion mailing list
http://lists.infradead.org/mailman/listinfo/linux-mtd/
Boris Brezillon Feb. 13, 2019, 9:15 a.m. UTC | #3
Hi Linus,

On Wed, 13 Feb 2019 09:40:11 +0100
Linus Walleij <linus.walleij@linaro.org> wrote:

> Top posting to Boris' kernelorg address, sorry for using the wrong

> address, maybe this is why the series was missed in the first

> place :/


Nope, it's on my todo list. Just didn't have much time to dedicate to
MTD lately. Sorry for the delay, I'll see if I can review/merge it this
week.

Boris

______________________________________________________
Linux MTD discussion mailing list
http://lists.infradead.org/mailman/listinfo/linux-mtd/
Boris Brezillon Feb. 13, 2019, 1:36 p.m. UTC | #4
On Wed, 23 Jan 2019 13:29:07 +0100
Linus Walleij <linus.walleij@linaro.org> wrote:

> As a starting point to fixing up the AFS support on ARM

> reference designs, let's move the implementation down to

> the "parsers" subdirectory and add OF probing support.

> 

> More refactorings will be done on top of this change.

> 

> Linus Walleij (3):

>   mtd: afs: Move AFS partition parser to parsers subdir

>   mtd: partitions: Add AFS partitions DT bindings

>   mtd: partitions: Add OF support to AFS partitions


Let's try to use a consistent prefix for patches touching partition
parsers code, like "mtd: partitions: <format>: ". Same applies to the
other patch series. Other than that, patches look good to me.

> 

>  .../mtd/partitions/arm,arm-firmware-suite.txt   | 17 +++++++++++++++++

>  drivers/mtd/Kconfig                             | 16 ----------------

>  drivers/mtd/Makefile                            |  1 -

>  drivers/mtd/parsers/Kconfig                     | 16 ++++++++++++++++

>  drivers/mtd/parsers/Makefile                    |  1 +

>  drivers/mtd/{ => parsers}/afs.c                 |  7 +++++++

>  6 files changed, 41 insertions(+), 17 deletions(-)

>  create mode 100644 Documentation/devicetree/bindings/mtd/partitions/arm,arm-firmware-suite.txt

>  rename drivers/mtd/{ => parsers}/afs.c (96%)

> 



______________________________________________________
Linux MTD discussion mailing list
http://lists.infradead.org/mailman/listinfo/linux-mtd/