Skip to contentSkip to conversation

MAINTAINERS: add me as a maintainer of MTD

PATCH New Heiko Schocher 6 participants 11 messages Oct 31, 2014 – Nov 6, 2014

lore
Share & tools
lore

Heiko Schocher

author
#0 lore raw

PATCH 1/1 MAINTAINERS: add me as a maintainer of MTD

Add MAINTAINERS and doc/git-mailrc entry.

Signed-off-by Heiko Schocher <[email protected]>

---
as discussed this step here and there first of all with Scott Wood for example here:
http://lists.denx.de/pipermail/u-boot/2014-August/186005.html

I am now ready for taking over the ownership for MTD ...

@Scott: I would be happy, if you can still have a look in nand patches ;-)
please inform me, if you have no longer time for it, so we can set u-boot-nand into not maintained state

@all: Please send me a list of pending patches, so I can go through
them, thanks!

 MAINTAINERS    | 8 ++++++++
 doc/git-mailrc | 1 +
 2 files changed, 9 insertions(+)
MAINTAINERS +8 -0
index fd346c9..e81a08a 100644
@@ -258,6 +258,14 @@ S:	Maintained
 T:	git git://git.denx.de/u-boot-mmc.git
 F:	drivers/mmc/
 
+MTD
+M:	Heiko Schocher <[email protected]>
+S:	Maintained
+T:	git git://git.denx.de/u-boot-mtd.git
+F:	drivers/mtd*
+F:	include/linux/mtd*
+F:	include/mtd*
+
 OPENRISC
 M:	Stefan Kristiansson <[email protected]>
 S:	Maintained
doc/git-mailrc +1 -0
index ad22763..166fe37 100644
@@ -116,6 +116,7 @@ alias fdt            uboot, sjg
 alias i2c            uboot, hs
 alias kconfig        uboot, masahiro
 alias mmc            uboot, panto
+alias mtd            uboot, hs
 alias nand           uboot, scottwood
 alias net            uboot, jhersh
 alias spi	     uboot, jagan
Signature · 1 line
1.8.3.1
#1 lore raw

PATCH 1/1 MAINTAINERS: add me as a maintainer of MTD

Hi Heiko,

Quoted 10 lines · Heiko Schocher

Add MAINTAINERS and doc/git-mailrc entry.

Signed-off-by Heiko Schocher <[email protected]>

---
as discussed this step here and there first of all with Scott Wood for example here:
http://lists.denx.de/pipermail/u-boot/2014-August/186005.html

I am now ready for taking over the ownership for MTD ...

Is it for entire drivers/mtd ? or any specific parts?

Quoted 17 lines

@Scott: I would be happy, if you can still have a look in nand patches ;-)
please inform me, if you have no longer time for it, so we can set u-boot-nand into not maintained state

@all: Please send me a list of pending patches, so I can go through
them, thanks!

 MAINTAINERS    | 8 ++++++++
 doc/git-mailrc | 1 +
 2 files changed, 9 insertions(+)
MAINTAINERS +0 -0
index fd346c9..e81a08a 100644

MAINTAINERS S: Maintained · line 265

Quoted 8 lines
MAINTAINERS +5 -0
@@ -258,6 +258,14 @@ S: Maintained
 T:     git git://git.denx.de/u-boot-mmc.git
 F:     drivers/mmc/
 
+MTD
+M:     Heiko Schocher <[email protected]>
+S:     Maintained
+T:     git git://git.denx.de/u-boot-mtd.git
+F:     drivers/mtd*

Don't add entire mtd - drivers/mtd/spi is owned by me. and few spi-flash mtd stuff is being under review.

Quoted 19 lines
MAINTAINERS +3 -0
+F:     include/linux/mtd*
+F:     include/mtd*
+
 OPENRISC
 M:     Stefan Kristiansson <[email protected]>
 S:     Maintained
doc/git-mailrc +1 -1
index ad22763..166fe37 100644
@@ -116,6 +116,7 @@ alias fdt            uboot, sjg
 alias i2c            uboot, hs
 alias kconfig        uboot, masahiro
 alias mmc            uboot, panto
+alias mtd            uboot, hs
 alias nand           uboot, scottwood
 alias net            uboot, jhersh
 alias spi           uboot, jagan
--
Quoted 2 lines

1.8.3.1

thanks!

Signature · 1 line
Jagan.
#2 lore raw

PATCH 1/1 MAINTAINERS: add me as a maintainer of MTD

Dear Jagan,

Quoted 4 lines · In message <CAD6G_RRkDhNHC=+OtgtXPd7WdfJzq2bokbMKhKnL=BLSx-AdMw@mail.gmail.com>…
Quoted 2 lines

I am now ready for taking over the ownership for MTD ...

Is it for entire drivers/mtd ? or any specific parts?

This is meant for all parts of drivers/mtd that are not already covered by other custodians.

Quoted 6 lines

MAINTAINERS S: Maintained · line 265

Quoted 4 lines
MAINTAINERS +4 -0
+M:     Heiko Schocher <[email protected]>
+S:     Maintained
+T:     git git://git.denx.de/u-boot-mtd.git
+F:     drivers/mtd*

Don't add entire mtd - drivers/mtd/spi is owned by me.

I don't know if we have a better notation for "all the rest that is not already covered by somebody else"?

Best regards,

Wolfgang Denk

Signature · 6 lines
DENX Software Engineering GmbH,     MD: Wolfgang Denk & Detlev Zundel
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: (+49)-8142-66989-10 Fax: (+49)-8142-66989-80 Email: wd at denx.de
Humanity has the  stars  in  its  future,  and  that  future  is  too
important  to be lost under the burden of juvenile folly and ignorant
superstition.                                          - Isaac Asimov
#3 lore raw

PATCH 1/1 MAINTAINERS: add me as a maintainer of MTD

Hi Heiko, Wolfgang,

Quoted 8 lines · On Fri, 31 Oct 2014 12:44:07 +0100 Wolfgang Denk <[email protected]> wrote:

Dear Jagan,

Quoted 4 lines · In message <CAD6G_RRkDhNHC=+OtgtXPd7WdfJzq2bokbMKhKnL=BLSx-AdMw@mail.gmail.com>…
Quoted 2 lines

I am now ready for taking over the ownership for MTD ...

Is it for entire drivers/mtd ? or any specific parts?

Quoted 7 lines

This is meant for all parts of drivers/mtd that are not already covered by other custodians.

Quoted 4 lines

MAINTAINERS S: Maintained · line 265

Quoted 4 lines
MAINTAINERS +4 -0
+M:     Heiko Schocher <[email protected]>
+S:     Maintained
+T:     git git://git.denx.de/u-boot-mtd.git
+F:     drivers/mtd*

I am afraid this does not work at all.

Do you mean this?

F: drivers/mtd/

Quoted 4 lines
Quoted 1 line

Don't add entire mtd - drivers/mtd/spi is owned by me.

I don't know if we have a better notation for "all the rest that is not already covered by somebody else"?

Let's see the last part of the MAINTAINERS:

THE REST

M:      Tom Rini <[email protected]>
L:      u-boot at lists.denx.de
Q:      http://patchwork.ozlabs.org/project/uboot/list/
S:      Maintained
T:      git git://git.denx.de/u-boot.git
F:      *
F:      */

This entry does __not__ mean everything is maintained by Tom, but Tom owns all the rest that is not covered by somebody else.

MTD
M: Heiko Schocher <[email protected]>
S: Maintained
T: git git://git.denx.de/u-boot-mtd.git
F: drivers/mtd/

I think this makes sense.

Let's study from linux/MAINTAINERS!

MEMORY TECHNOLOGY DEVICES (MTD)

M:      David Woodhouse <[email protected]>
M:      Brian Norris <[email protected]>
L:      linux-mtd at lists.infradead.org
W:      http://www.linux-mtd.infradead.org/
Q:      http://patchwork.ozlabs.org/project/linux-mtd/list/
T:      git git://git.infradead.org/linux-mtd.git
T:      git git://git.infradead.org/l2-mtd.git
S:      Maintained
F:      drivers/mtd/
F:      include/linux/mtd/
F:      include/uapi/mtd/

[snip]

ONENAND FLASH DRIVER

M:      Kyungmin Park <[email protected]>
L:      linux-mtd at lists.infradead.org
S:      Maintained
F:      drivers/mtd/onenand/
F:      include/linux/mtd/onenand*.h

BTW, "CFI FLASH" entry is questionable.

CFI FLASH
M: Stefan Roese <[email protected]>
S: Maintained
T: git git://git.denx.de/u-boot-cfi-flash.git
F: drivers/mtd/*

Do all the files in drivers/mtd
belong to CFI FLASH subsystem?

Best Regards
Masahiro Yamada

author
#4 lore raw

PATCH 1/1 MAINTAINERS: add me as a maintainer of MTD

Hello Masahiro Yamada,

Am 31.10.2014 13:29, schrieb Masahiro Yamada:

Quoted 27 lines

Hi Heiko, Wolfgang,

Quoted 8 lines · On Fri, 31 Oct 2014 12:44:07 +0100 Wolfgang Denk<[email protected]> wrote:

Dear Jagan,

Quoted 4 lines · In message<CAD6G_RRkDhNHC=+OtgtXPd7WdfJzq2bokbMKhKnL=BLSx-AdMw@mail.gmail.com>…
Quoted 2 lines

I am now ready for taking over the ownership for MTD ...

Is it for entire drivers/mtd ? or any specific parts?

Quoted 7 lines

This is meant for all parts of drivers/mtd that are not already covered by other custodians.

Quoted 4 lines

MAINTAINERS S: Maintained · line 265

Quoted 4 lines
MAINTAINERS +4 -0
+M:     Heiko Schocher<[email protected]>
+S:     Maintained
+T:     git git://git.denx.de/u-boot-mtd.git
+F:     drivers/mtd*

I am afraid this does not work at all.

Quoted 3 lines

Do you mean this?

F: drivers/mtd/

Yes, I think this is correct. I will change it to:

F:     drivers/mtd/
F:     include/linux/mtd/
F:     include/mtd/
Quoted 25 lines
Quoted 4 lines
Quoted 1 line

Don't add entire mtd - drivers/mtd/spi is owned by me.

I don't know if we have a better notation for "all the rest that is not already covered by somebody else"?

Let's see the last part of the MAINTAINERS:

THE REST

M:      Tom Rini<[email protected]>
L:      u-boot at lists.denx.de
Q:      http://patchwork.ozlabs.org/project/uboot/list/
S:      Maintained
T:      git git://git.denx.de/u-boot.git
F:      *
F:      */

This entry does __not__ mean everything is maintained by Tom, but Tom owns all the rest that is not covered by somebody else.

Quoted 8 lines

MTD
M: Heiko Schocher<[email protected]>
S: Maintained
T: git git://git.denx.de/u-boot-mtd.git
F: drivers/mtd/

I think this makes sense.

Yep.

Quoted 5 lines

Let's study from linux/MAINTAINERS!

MEMORY TECHNOLOGY DEVICES (MTD)

Quoted 11 lines
M:      David Woodhouse<[email protected]>
M:      Brian Norris<[email protected]>
L:      linux-mtd at lists.infradead.org
W:      http://www.linux-mtd.infradead.org/
Q:      http://patchwork.ozlabs.org/project/linux-mtd/list/
T:      git git://git.infradead.org/linux-mtd.git
T:      git git://git.infradead.org/l2-mtd.git
S:      Maintained
F:      drivers/mtd/
F:      include/linux/mtd/
F:      include/uapi/mtd/

Ok, adapted.

Quoted 26 lines

[snip]

ONENAND FLASH DRIVER

M:      Kyungmin Park<[email protected]>
L:      linux-mtd at lists.infradead.org
S:      Maintained
F:      drivers/mtd/onenand/
F:      include/linux/mtd/onenand*.h

BTW, "CFI FLASH" entry is questionable.

CFI FLASH
M: Stefan Roese<[email protected]>
S: Maintained
T: git git://git.denx.de/u-boot-cfi-flash.git
F: drivers/mtd/*

Quoted 2 lines

Do all the files in drivers/mtd
belong to CFI FLASH subsystem?

I think no ... Stefan, can you send an update for this?

Thanks!

bye,
Heiko

Signature · 2 lines
DENX Software Engineering GmbH,     MD: Wolfgang Denk & Detlev Zundel
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
#5 lore raw

PATCH 1/1 MAINTAINERS: add me as a maintainer of MTD

Quoted 9 lines · Heiko Schocher
Quoted 9 lines

BTW, "CFI FLASH" entry is questionable.

CFI FLASH

M:    Stefan Roese<[email protected]>
S:    Maintained
T:    git git://git.denx.de/u-boot-cfi-flash.git
F:    drivers/mtd/*
Quoted 4 lines
Quoted 2 lines

Do all the files in drivers/mtd
belong to CFI FLASH subsystem?

I think no ... Stefan, can you send an update for this?

Yes, will do.

Thanks,
Stefan

#6 lore raw

PATCH 1/1 MAINTAINERS: add me as a maintainer of MTD

In reply to Masahiro Yamada

Hi Masahiro Yamada,

Quoted 18 lines · Masahiro Yamada

Hi Heiko, Wolfgang,

Quoted 11 lines · On Fri, 31 Oct 2014 12:44:07 +0100 Wolfgang Denk <[email protected]> wrote:

Dear Jagan,

Quoted 4 lines · In message <CAD6G_RRkDhNHC=+OtgtXPd7WdfJzq2bokbMKhKnL=BLSx-AdMw@mail.gmail.com>…
Quoted 2 lines

I am now ready for taking over the ownership for MTD ...

Is it for entire drivers/mtd ? or any specific parts?

This is meant for all parts of drivers/mtd that are not already covered by other custodians.

Quoted 7 lines
Quoted 4 lines
Quoted 4 lines

MAINTAINERS S: Maintained · line 265

Quoted 4 lines
MAINTAINERS +4 -0
+M:     Heiko Schocher <[email protected]>
+S:     Maintained
+T:     git git://git.denx.de/u-boot-mtd.git
+F:     drivers/mtd*

I am afraid this does not work at all.

I too feel that this mayn't work - for everything but uncovered part.

Quoted 53 lines

Do you mean this?

F: drivers/mtd/

Quoted 4 lines
Quoted 1 line

Don't add entire mtd - drivers/mtd/spi is owned by me.

I don't know if we have a better notation for "all the rest that is not already covered by somebody else"?

Let's see the last part of the MAINTAINERS:

THE REST

M:      Tom Rini <[email protected]>
L:      u-boot at lists.denx.de
Q:      http://patchwork.ozlabs.org/project/uboot/list/
S:      Maintained
T:      git git://git.denx.de/u-boot.git
F:      *
F:      */

This entry does __not__ mean everything is maintained by Tom, but Tom owns all the rest that is not covered by somebody else.

MTD

M:      Heiko Schocher <[email protected]>
S:      Maintained
T:      git git://git.denx.de/u-boot-mtd.git
F:      drivers/mtd/

I think this makes sense.

Let's study from linux/MAINTAINERS!

MEMORY TECHNOLOGY DEVICES (MTD)

Quoted 11 lines
M:      David Woodhouse <[email protected]>
M:      Brian Norris <[email protected]>
L:      linux-mtd at lists.infradead.org
W:      http://www.linux-mtd.infradead.org/
Q:      http://patchwork.ozlabs.org/project/linux-mtd/list/
T:      git git://git.infradead.org/linux-mtd.git
T:      git git://git.infradead.org/l2-mtd.git
S:      Maintained
F:      drivers/mtd/
F:      include/linux/mtd/
F:      include/uapi/mtd/

Linux in MTD is shared by two maintainer where whole drivers/mtd

Quoted 22 lines

[snip]

ONENAND FLASH DRIVER

M:      Kyungmin Park <[email protected]>
L:      linux-mtd at lists.infradead.org
S:      Maintained
F:      drivers/mtd/onenand/
F:      include/linux/mtd/onenand*.h

BTW, "CFI FLASH" entry is questionable.

CFI FLASH

Quoted 9 lines
M:      Stefan Roese <[email protected]>
S:      Maintained
T:      git git://git.denx.de/u-boot-cfi-flash.git
F:      drivers/mtd/*

Do all the files in drivers/mtd
belong to CFI FLASH subsystem?

No - it has cfi, ubifs, nand, sf

thanks!

Signature · 1 line
Jagan.
#7 lore raw

PATCH 1/1 MAINTAINERS: add me as a maintainer of MTD

In reply to Heiko Schocher
Quoted 9 lines · Heiko Schocher

Add MAINTAINERS and doc/git-mailrc entry.

Signed-off-by Heiko Schocher <[email protected]>

---
as discussed this step here and there first of all with Scott Wood for example here:
http://lists.denx.de/pipermail/u-boot/2014-August/186005.html

Quoted 5 lines

I am now ready for taking over the ownership for MTD ...

@Scott: I would be happy, if you can still have a look in nand patches ;-)
please inform me, if you have no longer time for it, so we can set u-boot-nand into not maintained state

If you're only looking to be custodian for mtd infrastructure and not NAND, I guess I'll continue to do it... I hope there won't be any more from-scratch updates of the NAND code, though. :-)

-Scott

#8 lore raw

PATCH 1/1 MAINTAINERS: add me as a maintainer of MTD

Quoted 12 lines · Scott Wood
Quoted 11 lines · Heiko Schocher

Add MAINTAINERS and doc/git-mailrc entry.

Signed-off-by Heiko Schocher <[email protected]>

---
as discussed this step here and there first of all with Scott Wood for example here:
http://lists.denx.de/pipermail/u-boot/2014-August/186005.html

I am now ready for taking over the ownership for MTD ...

Quoted 7 lines
Quoted 3 lines

@Scott: I would be happy, if you can still have a look in nand patches ;-)
please inform me, if you have no longer time for it, so we can set u-boot-nand into not maintained state

If you're only looking to be custodian for mtd infrastructure and not NAND, I guess I'll continue to do it... I hope there won't be any more from-scratch updates of the NAND code, though. :-)

I thought separation of mtd is not quite good as it related to all flash stuff, it may
find difficulty to reviewer's which in-sync with the mtd maintainer (if separated)

And the drivers/mtd has nor, nand, and sf are the key and all together as MTD subsystem
atleast from code point of.

IMHO, why can't we follow a maintainer list as "MEMORY TECHNOLOGY DEVICES (MTD)" which can list a maintainer's who owned the drivers/mtd - I guess this could be a defined way.

thanks!

Signature · 1 line
Jagan.
author
#9 lore raw

PATCH 1/1 MAINTAINERS: add me as a maintainer of MTD

In reply to Scott Wood

Hello Scott,

Am 03.11.2014 22:34, schrieb Scott Wood:

Quoted 10 lines · Heiko Schocher
Quoted 9 lines

Add MAINTAINERS and doc/git-mailrc entry.

Signed-off-by Heiko Schocher<[email protected]>

---
as discussed this step here and there first of all with Scott Wood for example here:
http://lists.denx.de/pipermail/u-boot/2014-August/186005.html

Quoted 8 lines
Quoted 5 lines

I am now ready for taking over the ownership for MTD ...

@Scott: I would be happy, if you can still have a look in nand patches ;-)
please inform me, if you have no longer time for it, so we can set u-boot-nand into not maintained state

If you're only looking to be custodian for mtd infrastructure and not NAND, I guess I'll continue to do it... I hope there won't be any more

Puuh .. this are good news ;-)

I thought you mentioned you did not have enough time for this job ... Great to hear, that you will continue! Thanks for your work!

Quoted 1 line

from-scratch updates of the NAND code, though. :-)

Hopefully not.

bye,
Heiko

Signature · 2 lines
DENX Software Engineering GmbH,     MD: Wolfgang Denk & Detlev Zundel
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
#10 lore raw

PATCH 1/1 MAINTAINERS: add me as a maintainer of MTD

Quoted 19 lines · Heiko Schocher

Hello Scott,

Am 03.11.2014 22:34, schrieb Scott Wood:

Quoted 16 lines · Heiko Schocher
Quoted 14 lines

Add MAINTAINERS and doc/git-mailrc entry.

Signed-off-by Heiko Schocher<[email protected]>

---
as discussed this step here and there first of all with Scott Wood for example here:
http://lists.denx.de/pipermail/u-boot/2014-August/186005.html

I am now ready for taking over the ownership for MTD ...

@Scott: I would be happy, if you can still have a look in nand patches ;-)
please inform me, if you have no longer time for it, so we can set u-boot-nand into not maintained state

Quoted 7 lines
Quoted 2 lines

If you're only looking to be custodian for mtd infrastructure and not NAND, I guess I'll continue to do it... I hope there won't be any more

Puuh .. this are good news ;-)

I thought you mentioned you did not have enough time for this job ... Great to hear, that you will continue! Thanks for your work!

Well, I was particularly busy when I said that. :-)

Plus it seemed like there were several others more active in the NAND code than I was.

-Scott