MAINTAINERS: adjust to removing i2c designware platform data

Message ID 20210419061809.15045-1-lukas.bulwahn@gmail.com
State New
Headers show
Series
  • MAINTAINERS: adjust to removing i2c designware platform data
Related show

Commit Message

Lukas Bulwahn April 19, 2021, 6:18 a.m.
Commit 5a517b5bf687 ("i2c: designware: Get rid of legacy platform data")
removes ./include/linux/platform_data/i2c-designware.h, but misses to
adjust the SYNOPSYS DESIGNWARE I2C DRIVER section in MAINTAINERS.

Hence, ./scripts/get_maintainer.pl --self-test=patterns complains:

  warning: no file matches F: include/linux/platform_data/i2c-designware.h

Remove the file entry to this removed file as well.

Signed-off-by: Lukas Bulwahn <lukas.bulwahn@gmail.com>
---
applies cleanly on next-20210419

Andy, please ack.
Lee, please pick this minor patch on your -next tree.

 MAINTAINERS | 1 -
 1 file changed, 1 deletion(-)

Comments

Andy Shevchenko April 19, 2021, 8:33 a.m. | #1
On Mon, Apr 19, 2021 at 9:38 AM Lukas Bulwahn <lukas.bulwahn@gmail.com> wrote:
>
> Commit 5a517b5bf687 ("i2c: designware: Get rid of legacy platform data")
> removes ./include/linux/platform_data/i2c-designware.h, but misses to
> adjust the SYNOPSYS DESIGNWARE I2C DRIVER section in MAINTAINERS.
>
> Hence, ./scripts/get_maintainer.pl --self-test=patterns complains:
>
>   warning: no file matches F: include/linux/platform_data/i2c-designware.h
>
> Remove the file entry to this removed file as well.

Oops, I was under the impression I grepped all occurrences, but I have not.

Reviewed-by: Andy Shevchenko <andy.shevchenko@gmail.com>

Thanks for the catch!

> Signed-off-by: Lukas Bulwahn <lukas.bulwahn@gmail.com>
> ---
> applies cleanly on next-20210419
>
> Andy, please ack.
> Lee, please pick this minor patch on your -next tree.
>
>  MAINTAINERS | 1 -
>  1 file changed, 1 deletion(-)
>
> diff --git a/MAINTAINERS b/MAINTAINERS
> index bbe356508f29..6b903aad27f4 100644
> --- a/MAINTAINERS
> +++ b/MAINTAINERS
> @@ -17667,7 +17667,6 @@ R:      Mika Westerberg <mika.westerberg@linux.intel.com>
>  L:     linux-i2c@vger.kernel.org
>  S:     Maintained
>  F:     drivers/i2c/busses/i2c-designware-*
> -F:     include/linux/platform_data/i2c-designware.h
>
>  SYNOPSYS DESIGNWARE MMC/SD/SDIO DRIVER
>  M:     Jaehoon Chung <jh80.chung@samsung.com>
> --
> 2.17.1
>
Andy Shevchenko May 28, 2021, 2:34 p.m. | #2
On Mon, Apr 19, 2021 at 11:33:16AM +0300, Andy Shevchenko wrote:
> On Mon, Apr 19, 2021 at 9:38 AM Lukas Bulwahn <lukas.bulwahn@gmail.com> wrote:

> >

> > Commit 5a517b5bf687 ("i2c: designware: Get rid of legacy platform data")

> > removes ./include/linux/platform_data/i2c-designware.h, but misses to

> > adjust the SYNOPSYS DESIGNWARE I2C DRIVER section in MAINTAINERS.

> >

> > Hence, ./scripts/get_maintainer.pl --self-test=patterns complains:

> >

> >   warning: no file matches F: include/linux/platform_data/i2c-designware.h

> >

> > Remove the file entry to this removed file as well.

> 

> Oops, I was under the impression I grepped all occurrences, but I have not.

> 

> Reviewed-by: Andy Shevchenko <andy.shevchenko@gmail.com>

> 

> Thanks for the catch!


Wolfram, isn't it forgotten somehow?

-- 
With Best Regards,
Andy Shevchenko
Wolfram Sang May 28, 2021, 2:48 p.m. | #3
> > > Remove the file entry to this removed file as well.

> > 

> > Oops, I was under the impression I grepped all occurrences, but I have not.

> > 

> > Reviewed-by: Andy Shevchenko <andy.shevchenko@gmail.com>

> > 

> > Thanks for the catch!

> 

> Wolfram, isn't it forgotten somehow?


I can pick it. I refrained from doing so because Lukas explicitly asked
Lee to pick it.
Wolfram Sang May 28, 2021, 2:49 p.m. | #4
On Mon, Apr 19, 2021 at 08:18:09AM +0200, Lukas Bulwahn wrote:
> Commit 5a517b5bf687 ("i2c: designware: Get rid of legacy platform data")

> removes ./include/linux/platform_data/i2c-designware.h, but misses to

> adjust the SYNOPSYS DESIGNWARE I2C DRIVER section in MAINTAINERS.

> 

> Hence, ./scripts/get_maintainer.pl --self-test=patterns complains:

> 

>   warning: no file matches F: include/linux/platform_data/i2c-designware.h

> 

> Remove the file entry to this removed file as well.

> 

> Signed-off-by: Lukas Bulwahn <lukas.bulwahn@gmail.com>


Applied to for-current, thanks!
Lukas Bulwahn May 28, 2021, 2:54 p.m. | #5
On Fri, May 28, 2021 at 4:48 PM Wolfram Sang <wsa@kernel.org> wrote:
>

>

> > > > Remove the file entry to this removed file as well.

> > >

> > > Oops, I was under the impression I grepped all occurrences, but I have not.

> > >

> > > Reviewed-by: Andy Shevchenko <andy.shevchenko@gmail.com>

> > >

> > > Thanks for the catch!

> >

> > Wolfram, isn't it forgotten somehow?

>

> I can pick it. I refrained from doing so because Lukas explicitly asked

> Lee to pick it.

>


Wolfram, take it. Nobody compiles MAINTAINERS, so it cannot break any build :)

Thanks, Lukas
Andy Shevchenko May 28, 2021, 5:19 p.m. | #6
On Fri, May 28, 2021 at 04:48:16PM +0200, Wolfram Sang wrote:
> 

> > > > Remove the file entry to this removed file as well.

> > > 

> > > Oops, I was under the impression I grepped all occurrences, but I have not.

> > > 

> > > Reviewed-by: Andy Shevchenko <andy.shevchenko@gmail.com>

> > > 

> > > Thanks for the catch!

> > 

> > Wolfram, isn't it forgotten somehow?

> 

> I can pick it. I refrained from doing so because Lukas explicitly asked

> Lee to pick it.


Ah, I see. AFAIR Lee doesn't update his branch frequently, so it might be that
he already applied that but it's not visible to Linux Next.


-- 
With Best Regards,
Andy Shevchenko
Lee Jones May 28, 2021, 6:19 p.m. | #7
On Fri, 28 May 2021, Andy Shevchenko wrote:

> On Fri, May 28, 2021 at 04:48:16PM +0200, Wolfram Sang wrote:

> > 

> > > > > Remove the file entry to this removed file as well.

> > > > 

> > > > Oops, I was under the impression I grepped all occurrences, but I have not.

> > > > 

> > > > Reviewed-by: Andy Shevchenko <andy.shevchenko@gmail.com>

> > > > 

> > > > Thanks for the catch!

> > > 

> > > Wolfram, isn't it forgotten somehow?

> > 

> > I can pick it. I refrained from doing so because Lukas explicitly asked

> > Lee to pick it.

> 

> Ah, I see. AFAIR Lee doesn't update his branch frequently, so it might be that

> he already applied that but it's not visible to Linux Next.


I'm not sure whether to find this funny or be offended.

Let's go with funny!

AFAICT, this patch does not enter my area of responsibility.

Please take it via I2C.

-- 
Lee Jones [李琼斯]
Senior Technical Lead - Developer Services
Linaro.org │ Open source software for Arm SoCs
Follow Linaro: Facebook | Twitter | Blog
Andy Shevchenko May 29, 2021, 7:49 a.m. | #8
On Fri, May 28, 2021 at 9:20 PM Lee Jones <lee.jones@linaro.org> wrote:
> On Fri, 28 May 2021, Andy Shevchenko wrote:

> > On Fri, May 28, 2021 at 04:48:16PM +0200, Wolfram Sang wrote:


...

> > Ah, I see. AFAIR Lee doesn't update his branch frequently, so it might be that

> > he already applied that but it's not visible to Linux Next.

>

> I'm not sure whether to find this funny or be offended.


Definitely not about offending anyone. Let's go with funny!

-- 
With Best Regards,
Andy Shevchenko

Patch

diff --git a/MAINTAINERS b/MAINTAINERS
index bbe356508f29..6b903aad27f4 100644
--- a/MAINTAINERS
+++ b/MAINTAINERS
@@ -17667,7 +17667,6 @@  R:	Mika Westerberg <mika.westerberg@linux.intel.com>
 L:	linux-i2c@vger.kernel.org
 S:	Maintained
 F:	drivers/i2c/busses/i2c-designware-*
-F:	include/linux/platform_data/i2c-designware.h
 
 SYNOPSYS DESIGNWARE MMC/SD/SDIO DRIVER
 M:	Jaehoon Chung <jh80.chung@samsung.com>