diff mbox

[U-Boot] MAINTAINERS: add me as a maintainer of MTD

Message ID 1415600124-25916-1-git-send-email-hs@denx.de
State Deferred
Delegated to: Tom Rini
Headers show

Commit Message

Heiko Schocher Nov. 10, 2014, 6:15 a.m. UTC
Add MAINTAINERS and doc/git-mailrc entry.

Signed-off-by: Heiko Schocher <hs@denx.de>

---
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 guesses he could continue with NAND custodianship, thanks!
So no need for taking this into the MTD custodianship ... puuh ;-)

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

Changes in v2:
add comment from Jagan Teki and Masahiro Yamada:
correct files entry in MAINTAINERS

 MAINTAINERS    | 8 ++++++++
 doc/git-mailrc | 1 +
 2 files changed, 9 insertions(+)

Comments

Jagan Teki Nov. 10, 2014, 6:38 a.m. UTC | #1
On 10 November 2014 11:45, Heiko Schocher <hs@denx.de> wrote:
> Add MAINTAINERS and doc/git-mailrc entry.
>
> Signed-off-by: Heiko Schocher <hs@denx.de>
>
> ---
> 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 guesses he could continue with NAND custodianship, thanks!
> So no need for taking this into the MTD custodianship ... puuh ;-)
>
> @all: Please send me a list of pending patches, so I can go through
>       them, thanks!
>
> Changes in v2:
> add comment from Jagan Teki and Masahiro Yamada:
> correct files entry in MAINTAINERS
>
>  MAINTAINERS    | 8 ++++++++
>  doc/git-mailrc | 1 +
>  2 files changed, 9 insertions(+)
>
> diff --git a/MAINTAINERS b/MAINTAINERS
> index 4e2f8a9..2ac55f8 100644
> --- a/MAINTAINERS
> +++ b/MAINTAINERS
> @@ -258,6 +258,14 @@ S: Maintained
>  T:     git git://git.denx.de/u-boot-mmc.git
>  F:     drivers/mmc/
>
> +MTD
> +M:     Heiko Schocher <hs@denx.de>
> +S:     Maintained
> +T:     git git://git.denx.de/u-boot-mtd.git
> +F:     drivers/mtd/
> +F:     include/linux/mtd/
> +F:     include/mtd/
> +

This approach, I feel it's sounds not good to me.
[1] as per my comments, it's good to add the maintainers who  belongs to
drivers/mtd as "MTD" maintainer so-that it is more defined way for any one
in opensource to understand it - Linux is operating same i guess.

Comments?

[1] http://u-boot.10912.n7.nabble.com/PATCH-MAINTAINERS-add-me-as-a-maintainer-of-MTD-td194237.html#a194619

>  OPENRISC
>  M:     Stefan Kristiansson <stefan.kristiansson@saunalahti.fi>
>  S:     Maintained
> diff --git a/doc/git-mailrc b/doc/git-mailrc
> index ad22763..166fe37 100644
> --- a/doc/git-mailrc
> +++ b/doc/git-mailrc
> @@ -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
> --
> 1.8.3.1
>

thanks!
Heiko Schocher Nov. 10, 2014, 7:32 a.m. UTC | #2
Hello Jagan,

Am 10.11.2014 07:38, schrieb Jagan Teki:
> On 10 November 2014 11:45, Heiko Schocher <hs@denx.de> wrote:
>> Add MAINTAINERS and doc/git-mailrc entry.
>>
>> Signed-off-by: Heiko Schocher <hs@denx.de>
>>
>> ---
>> 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 guesses he could continue with NAND custodianship, thanks!
>> So no need for taking this into the MTD custodianship ... puuh ;-)
>>
>> @all: Please send me a list of pending patches, so I can go through
>>        them, thanks!
>>
>> Changes in v2:
>> add comment from Jagan Teki and Masahiro Yamada:
>> correct files entry in MAINTAINERS
>>
>>   MAINTAINERS    | 8 ++++++++
>>   doc/git-mailrc | 1 +
>>   2 files changed, 9 insertions(+)
>>
>> diff --git a/MAINTAINERS b/MAINTAINERS
>> index 4e2f8a9..2ac55f8 100644
>> --- a/MAINTAINERS
>> +++ b/MAINTAINERS
>> @@ -258,6 +258,14 @@ S: Maintained
>>   T:     git git://git.denx.de/u-boot-mmc.git
>>   F:     drivers/mmc/
>>
>> +MTD
>> +M:     Heiko Schocher <hs@denx.de>
>> +S:     Maintained
>> +T:     git git://git.denx.de/u-boot-mtd.git
>> +F:     drivers/mtd/
>> +F:     include/linux/mtd/
>> +F:     include/mtd/
>> +
>
> This approach, I feel it's sounds not good to me.
> [1] as per my comments, it's good to add the maintainers who  belongs to

Hups... sorry. Seems I missed this EMail ...

> drivers/mtd as "MTD" maintainer so-that it is more defined way for any one
> in opensource to understand it - Linux is operating same i guess.

I could not parse you here ...

Looking in linux MAINTAINERS:

MEMORY TECHNOLOGY DEVICES (MTD)
M:      David Woodhouse <dwmw2@infradead.org>
M:      Brian Norris <computersforpeace@gmail.com>
L:      linux-mtd@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
S:      Maintained
F:      drivers/mtd/
F:      include/linux/mtd/
F:      include/uapi/mtd/
[...]
ONENAND FLASH DRIVER
M:      Kyungmin Park <kyungmin.park@samsung.com>
L:      linux-mtd@lists.infradead.org
S:      Maintained
F:      drivers/mtd/onenand/
F:      include/linux/mtd/onenand*.h
[...]
PHRAM MTD DRIVER
M:      Joern Engel <joern@lazybastard.org>
L:      linux-mtd@lists.infradead.org
S:      Maintained
F:      drivers/mtd/devices/phram.c
...
and a lot more ...

So where is the difference? The mtd mailinglist? I am sure
we do not want to introduce a seperate mailinglist...

I do not want to maintain your parts!

> Comments?

@Kyungmin Park: Could you comment here too, espacially for ubi/ubifs
                 IIRC I see no comments for the "MTD/UBI/UBIFS sync with
                 Linux 3.14" patches from me ... do you have longer time
                 for maintaining this?

                 There are also patches (some here for example which are
                 pending, and seeing no comments from you ...)

                 for example 2 examples from me:
                 [U-Boot] mtd, ubi: errormessage handling
                 http://patchwork.ozlabs.org/patch/405259/

                 [U-Boot,v2] ubi: enable error reporting in initialization
                 http://patchwork.ozlabs.org/patch/407136/
bye,
Heiko

> [1] http://u-boot.10912.n7.nabble.com/PATCH-MAINTAINERS-add-me-as-a-maintainer-of-MTD-td194237.html#a194619
>
>>   OPENRISC
>>   M:     Stefan Kristiansson <stefan.kristiansson@saunalahti.fi>
>>   S:     Maintained
>> diff --git a/doc/git-mailrc b/doc/git-mailrc
>> index ad22763..166fe37 100644
>> --- a/doc/git-mailrc
>> +++ b/doc/git-mailrc
>> @@ -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
>> --
>> 1.8.3.1
>>
>
> thanks!
>
diff mbox

Patch

diff --git a/MAINTAINERS b/MAINTAINERS
index 4e2f8a9..2ac55f8 100644
--- a/MAINTAINERS
+++ b/MAINTAINERS
@@ -258,6 +258,14 @@  S:	Maintained
 T:	git git://git.denx.de/u-boot-mmc.git
 F:	drivers/mmc/
 
+MTD
+M:	Heiko Schocher <hs@denx.de>
+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 <stefan.kristiansson@saunalahti.fi>
 S:	Maintained
diff --git a/doc/git-mailrc b/doc/git-mailrc
index ad22763..166fe37 100644
--- a/doc/git-mailrc
+++ b/doc/git-mailrc
@@ -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