Re: [PATCH 1/2] MAINTAINERS: Add entry for SPI NAND framework and drivers

From: Tom Rini <trini@konsulko.com>
To: Frieder Schrempf <frieder.schrempf@kontron.de>
Cc: "Dario Binacchi" <dario.binacchi@amarulasolutions.com>,
	"Frieder Schrempf" <frieder@fris.de>,
	u-boot@lists.denx.de, "Jagan Teki" <jagan@amarulasolutions.com>,
	"Michael Nazzareno Trimarchi" <michael@amarulasolutions.com>,
	"Damien Le Moal" <damien.lemoal@opensource.wdc.com>,
	"Ilias Apalodimas" <ilias.apalodimas@linaro.org>,
	"Marek Behún" <kabel@kernel.org>,
	"Michal Simek" <michal.simek@amd.com>,
	"Philippe Reynes" <philippe.reynes@softathome.com>,
	"Sean Anderson" <sean.anderson@seco.com>,
	"Simon Glass" <sjg@chromium.org>, "Stefan Roese" <sr@denx.de>,
	"William Zhang" <william.zhang@broadcom.com>
Subject: Re: [PATCH 1/2] MAINTAINERS: Add entry for SPI NAND framework and drivers
Date: Tue, 28 Feb 2023 09:48:55 -0500	[thread overview]
Message-ID: <Y/4UV9YJcyFoxzZU@bill-the-cat> (raw)
In-Reply-To: <6a871b33-d8ff-2cf7-0415-b4fd172ad1d8@kontron.de>

[-- Attachment #1: Type: text/plain, Size: 3557 bytes --]

On Tue, Feb 28, 2023 at 09:52:45AM +0100, Frieder Schrempf wrote:
> On 27.02.23 16:24, Dario Binacchi wrote:
> > Hi Frieder,
> > 
> > On Mon, Feb 13, 2023 at 10:47 AM Frieder Schrempf <frieder@fris.de> wrote:
> >>
> >> From: Frieder Schrempf <frieder.schrempf@kontron.de>
> >>
> >> In [1] Michael agreed on taking patches for SPI NAND through the RAW
> >> NAND tree. Add a dedicated entry to the MAINTAINERS file which adds
> >> Michael and Dario as maintainers and myself as reviewer.
> >>
> >> [1] https://eur04.safelinks.protection.outlook.com/?url=https%3A%2F%2Flists.denx.de%2Fpipermail%2Fu-boot%2F2023-February%2F508571.html&data=05%7C01%7Cfrieder.schrempf%40kontron.de%7C0ec7e2ff745f4303d31208db18d6c423%7C8c9d3c973fd941c8a2b1646f3942daf1%7C0%7C0%7C638131082918732044%7CUnknown%7CTWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D%7C3000%7C%7C%7C&sdata=4HDR%2BwouzdpqPFWPdvnZWRXSyjc94T%2F0pPHaXTYNrsM%3D&reserved=0
> >>
> >> Signed-off-by: Frieder Schrempf <frieder.schrempf@kontron.de>
> >> Cc: Jagan Teki <jagan@amarulasolutions.com>
> >> Cc: Dario Binacchi <dario.binacchi@amarulasolutions.com>
> >> Cc: Michael Nazzareno Trimarchi <michael@amarulasolutions.com>
> >> Cc: Tom Rini <trini@konsulko.com>
> >> ---
> >>  MAINTAINERS | 8 ++++++++
> >>  1 file changed, 8 insertions(+)
> >>
> >> diff --git a/MAINTAINERS b/MAINTAINERS
> >> index b9c505d5fa..e40c401337 100644
> >> --- a/MAINTAINERS
> >> +++ b/MAINTAINERS
> >> @@ -1375,6 +1375,14 @@ T:       git https://eur04.safelinks.protection.outlook.com/?url=https%3A%2F%2Fsource.denx.de%2Fu-boot%2Fcustodians%2Fu-boot-spi.git&data=05%7C01%7Cfrieder.schrempf%40kontron.de%7C0ec7e2ff745f4303d31208db18d6c423%7C8c9d3c973fd941c8a2b1646f3942daf1%7C0%7C0%7C638131082918732044%7CUnknown%7CTWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D%7C3000%7C%7C%7C&sdata=RPFcfbNZYy7sKTu8PruDhcIDts7ufkzrUtPr1cR8yhw%3D&reserved=0
> >>  F:     drivers/spi/
> >>  F:     include/spi*
> >>
> >> +SPI NAND
> >> +M:     Dario Binacchi <dario.binacchi@amarulasolutions.com>
> >> +M:     Michael Trimarchi <michael@amarulasolutions.com>
> >> +R:     Frieder Schrempf <frieder.schrempf@kontron.de>
> >> +S:     Maintained
> >> +T:     git https://eur04.safelinks.protection.outlook.com/?url=https%3A%2F%2Fsource.denx.de%2Fu-boot%2Fcustodians%2Fu-boot-nand-flash.git&data=05%7C01%7Cfrieder.schrempf%40kontron.de%7C0ec7e2ff745f4303d31208db18d6c423%7C8c9d3c973fd941c8a2b1646f3942daf1%7C0%7C0%7C638131082918732044%7CUnknown%7CTWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D%7C3000%7C%7C%7C&sdata=pGjKSC7chQZ4%2BvT7QVNDdselY2P4llM5AeEMLTjzYUI%3D&reserved=0
> >> +F:     drivers/mtd/nand/spi/
> >> +
> > 
> > Please re-send a single patch where you only consider drivers/mtd/nand
> > for NAND FLASH
> > in order to be as similar as possible to the Linux kernel MAINTAINERS file.
> > Add to it "+R:     Frieder Schrempf <frieder.schrempf@kontron.de>"
> 
> Actually, I don't really like to be listed as reviewer for the whole
> NAND subsystem. I only have limited time and knowledge and SPI NAND is
> an area I feel I can provide help.
> 
> > Neither SPI NAND nor  RAW NAND exists in the Linux kernel MAINTAINERS file.
> 
> Is there a requirement to have MAINTAINERS in sync with Linux?

What matters most is that get_maintainers.pl returns something useful
and correct. So if you're only comfortable with SPI NAND, the above is
fine, we have similar for SPI-NOR already.

-- 
Tom

[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 659 bytes --]