platform-driver-x86.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
* [PATCH] MAINTAINERS: repair file entry in MICROSOFT SURFACE AGGREGATOR TABLET-MODE SWITCH
@ 2022-07-13  4:09 Lukas Bulwahn
  2022-07-13  9:35 ` Andy Shevchenko
                   ` (2 more replies)
  0 siblings, 3 replies; 4+ messages in thread
From: Lukas Bulwahn @ 2022-07-13  4:09 UTC (permalink / raw)
  To: Maximilian Luz, Hans de Goede, Mark Gross, platform-driver-x86
  Cc: kernel-janitors, linux-kernel, Lukas Bulwahn

Commit 9f794056db5b ("platform/surface: Add KIP/POS tablet-mode switch
driver") adds the section MICROSOFT SURFACE AGGREGATOR TABLET-MODE SWITCH
with a file entry, but the file that is added with this commit is actually
named slightly differently.

  file entry name: drivers/platform/surface/surface_aggregator_tablet_switch.c
  added file name: drivers/platform/surface/surface_aggregator_tabletsw.c

Hence, ./scripts/get_maintainer.pl --self-test=patterns complains about a
broken reference.

Repair this file entry to the actual file name added with the commit above.

Signed-off-by: Lukas Bulwahn <lukas.bulwahn@gmail.com>
---
 MAINTAINERS | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/MAINTAINERS b/MAINTAINERS
index 9bfd69e2b83d..9ec738419526 100644
--- a/MAINTAINERS
+++ b/MAINTAINERS
@@ -13440,7 +13440,7 @@ MICROSOFT SURFACE AGGREGATOR TABLET-MODE SWITCH
 M:	Maximilian Luz <luzmaximilian@gmail.com>
 L:	platform-driver-x86@vger.kernel.org
 S:	Maintained
-F:	drivers/platform/surface/surface_aggregator_tablet_switch.c
+F:	drivers/platform/surface/surface_aggregator_tabletsw.c
 
 MICROSOFT SURFACE BATTERY AND AC DRIVERS
 M:	Maximilian Luz <luzmaximilian@gmail.com>
-- 
2.17.1


^ permalink raw reply related	[flat|nested] 4+ messages in thread

* Re: [PATCH] MAINTAINERS: repair file entry in MICROSOFT SURFACE AGGREGATOR TABLET-MODE SWITCH
  2022-07-13  4:09 [PATCH] MAINTAINERS: repair file entry in MICROSOFT SURFACE AGGREGATOR TABLET-MODE SWITCH Lukas Bulwahn
@ 2022-07-13  9:35 ` Andy Shevchenko
  2022-07-13  9:48 ` Maximilian Luz
  2022-07-14 19:49 ` Hans de Goede
  2 siblings, 0 replies; 4+ messages in thread
From: Andy Shevchenko @ 2022-07-13  9:35 UTC (permalink / raw)
  To: Lukas Bulwahn
  Cc: Maximilian Luz, Hans de Goede, Mark Gross, Platform Driver,
	kernel-janitors, Linux Kernel Mailing List

On Wed, Jul 13, 2022 at 6:19 AM Lukas Bulwahn <lukas.bulwahn@gmail.com> wrote:
>
> Commit 9f794056db5b ("platform/surface: Add KIP/POS tablet-mode switch
> driver") adds the section MICROSOFT SURFACE AGGREGATOR TABLET-MODE SWITCH
> with a file entry, but the file that is added with this commit is actually
> named slightly differently.
>
>   file entry name: drivers/platform/surface/surface_aggregator_tablet_switch.c
>   added file name: drivers/platform/surface/surface_aggregator_tabletsw.c
>
> Hence, ./scripts/get_maintainer.pl --self-test=patterns complains about a
> broken reference.
>
> Repair this file entry to the actual file name added with the commit above.

Fixes tag?

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


With Best Regards,
Andy Shevchenko

^ permalink raw reply	[flat|nested] 4+ messages in thread

* Re: [PATCH] MAINTAINERS: repair file entry in MICROSOFT SURFACE AGGREGATOR TABLET-MODE SWITCH
  2022-07-13  4:09 [PATCH] MAINTAINERS: repair file entry in MICROSOFT SURFACE AGGREGATOR TABLET-MODE SWITCH Lukas Bulwahn
  2022-07-13  9:35 ` Andy Shevchenko
@ 2022-07-13  9:48 ` Maximilian Luz
  2022-07-14 19:49 ` Hans de Goede
  2 siblings, 0 replies; 4+ messages in thread
From: Maximilian Luz @ 2022-07-13  9:48 UTC (permalink / raw)
  To: Lukas Bulwahn, Hans de Goede, Mark Gross, platform-driver-x86
  Cc: kernel-janitors, linux-kernel

On 7/13/22 06:09, Lukas Bulwahn wrote:
> Commit 9f794056db5b ("platform/surface: Add KIP/POS tablet-mode switch
> driver") adds the section MICROSOFT SURFACE AGGREGATOR TABLET-MODE SWITCH
> with a file entry, but the file that is added with this commit is actually
> named slightly differently.
> 
>    file entry name: drivers/platform/surface/surface_aggregator_tablet_switch.c
>    added file name: drivers/platform/surface/surface_aggregator_tabletsw.c
> 
> Hence, ./scripts/get_maintainer.pl --self-test=patterns complains about a
> broken reference.
> 
> Repair this file entry to the actual file name added with the commit above.
> 
> Signed-off-by: Lukas Bulwahn <lukas.bulwahn@gmail.com>

Thanks for fixing this and sorry for messing this up, I should have noticed.

With the appropriate Fixes tag, as mentioned by Andy:

Reviewed-by: Maximilian Luz <luzmaximilian@gmail.com>

> ---
>   MAINTAINERS | 2 +-
>   1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/MAINTAINERS b/MAINTAINERS
> index 9bfd69e2b83d..9ec738419526 100644
> --- a/MAINTAINERS
> +++ b/MAINTAINERS
> @@ -13440,7 +13440,7 @@ MICROSOFT SURFACE AGGREGATOR TABLET-MODE SWITCH
>   M:	Maximilian Luz <luzmaximilian@gmail.com>
>   L:	platform-driver-x86@vger.kernel.org
>   S:	Maintained
> -F:	drivers/platform/surface/surface_aggregator_tablet_switch.c
> +F:	drivers/platform/surface/surface_aggregator_tabletsw.c
>   
>   MICROSOFT SURFACE BATTERY AND AC DRIVERS
>   M:	Maximilian Luz <luzmaximilian@gmail.com>

^ permalink raw reply	[flat|nested] 4+ messages in thread

* Re: [PATCH] MAINTAINERS: repair file entry in MICROSOFT SURFACE AGGREGATOR TABLET-MODE SWITCH
  2022-07-13  4:09 [PATCH] MAINTAINERS: repair file entry in MICROSOFT SURFACE AGGREGATOR TABLET-MODE SWITCH Lukas Bulwahn
  2022-07-13  9:35 ` Andy Shevchenko
  2022-07-13  9:48 ` Maximilian Luz
@ 2022-07-14 19:49 ` Hans de Goede
  2 siblings, 0 replies; 4+ messages in thread
From: Hans de Goede @ 2022-07-14 19:49 UTC (permalink / raw)
  To: Lukas Bulwahn, Maximilian Luz, Mark Gross, platform-driver-x86
  Cc: kernel-janitors, linux-kernel

Hi,

On 7/13/22 06:09, Lukas Bulwahn wrote:
> Commit 9f794056db5b ("platform/surface: Add KIP/POS tablet-mode switch
> driver") adds the section MICROSOFT SURFACE AGGREGATOR TABLET-MODE SWITCH
> with a file entry, but the file that is added with this commit is actually
> named slightly differently.
> 
>   file entry name: drivers/platform/surface/surface_aggregator_tablet_switch.c
>   added file name: drivers/platform/surface/surface_aggregator_tabletsw.c
> 
> Hence, ./scripts/get_maintainer.pl --self-test=patterns complains about a
> broken reference.
> 
> Repair this file entry to the actual file name added with the commit above.
> 
> Signed-off-by: Lukas Bulwahn <lukas.bulwahn@gmail.com>

Thank you for your patch, I've applied this patch to my review-hans 
branch:
https://git.kernel.org/pub/scm/linux/kernel/git/pdx86/platform-drivers-x86.git/log/?h=review-hans

With the requested Fixes tag added.

Once I've run some tests on this branch the patches there will be
added to the platform-drivers-x86/for-next branch and eventually
will be included in the pdx86 pull-request to Linus for the next
merge-window.

Regards,

Hans

> ---
>  MAINTAINERS | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/MAINTAINERS b/MAINTAINERS
> index 9bfd69e2b83d..9ec738419526 100644
> --- a/MAINTAINERS
> +++ b/MAINTAINERS
> @@ -13440,7 +13440,7 @@ MICROSOFT SURFACE AGGREGATOR TABLET-MODE SWITCH
>  M:	Maximilian Luz <luzmaximilian@gmail.com>
>  L:	platform-driver-x86@vger.kernel.org
>  S:	Maintained
> -F:	drivers/platform/surface/surface_aggregator_tablet_switch.c
> +F:	drivers/platform/surface/surface_aggregator_tabletsw.c
>  
>  MICROSOFT SURFACE BATTERY AND AC DRIVERS
>  M:	Maximilian Luz <luzmaximilian@gmail.com>


^ permalink raw reply	[flat|nested] 4+ messages in thread

end of thread, other threads:[~2022-07-14 19:49 UTC | newest]

Thread overview: 4+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2022-07-13  4:09 [PATCH] MAINTAINERS: repair file entry in MICROSOFT SURFACE AGGREGATOR TABLET-MODE SWITCH Lukas Bulwahn
2022-07-13  9:35 ` Andy Shevchenko
2022-07-13  9:48 ` Maximilian Luz
2022-07-14 19:49 ` Hans de Goede

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).