* [PATCH] MAINTAINERS: Update DW 8250 UART maintainership
@ 2022-06-17 9:39 Ilpo Järvinen
2022-06-17 9:45 ` Greg Kroah-Hartman
` (2 more replies)
0 siblings, 3 replies; 9+ messages in thread
From: Ilpo Järvinen @ 2022-06-17 9:39 UTC (permalink / raw)
To: Greg Kroah-Hartman, linux-serial, LKML, Andy Shevchenko; +Cc: Jiri Slaby
[-- Attachment #1: Type: text/plain, Size: 793 bytes --]
Add myself as maintainer for DW 8250 UART and up it to Supported.
Signed-off-by: Ilpo Järvinen <ilpo.jarvinen@linux.intel.com>
---
MAINTAINERS | 3 ++-
1 file changed, 2 insertions(+), 1 deletion(-)
diff --git a/MAINTAINERS b/MAINTAINERS
index a6d3bd9d2a8d..6c90484f395b 100644
--- a/MAINTAINERS
+++ b/MAINTAINERS
@@ -19230,8 +19230,9 @@ F: Documentation/devicetree/bindings/gpio/snps,creg-gpio.txt
F: drivers/gpio/gpio-creg-snps.c
SYNOPSYS DESIGNWARE 8250 UART DRIVER
+M: Ilpo Järvinen <ilpo.jarvinen@linux.intel.com>
R: Andy Shevchenko <andriy.shevchenko@linux.intel.com>
-S: Maintained
+S: Supported
F: drivers/tty/serial/8250/8250_dw.c
F: drivers/tty/serial/8250/8250_dwlib.*
F: drivers/tty/serial/8250/8250_lpss.c
--
tg: (f2906aa86338..) dw/maintainer (depends on: tty-next)
^ permalink raw reply related [flat|nested] 9+ messages in thread
* Re: [PATCH] MAINTAINERS: Update DW 8250 UART maintainership
2022-06-17 9:39 [PATCH] MAINTAINERS: Update DW 8250 UART maintainership Ilpo Järvinen
@ 2022-06-17 9:45 ` Greg Kroah-Hartman
2022-06-17 10:21 ` Andy Shevchenko
2022-06-17 9:47 ` Andy Shevchenko
2022-06-17 20:09 ` kernel test robot
2 siblings, 1 reply; 9+ messages in thread
From: Greg Kroah-Hartman @ 2022-06-17 9:45 UTC (permalink / raw)
To: Ilpo Järvinen; +Cc: linux-serial, LKML, Andy Shevchenko, Jiri Slaby
On Fri, Jun 17, 2022 at 12:39:29PM +0300, Ilpo Järvinen wrote:
> Add myself as maintainer for DW 8250 UART and up it to Supported.
>
> Signed-off-by: Ilpo Järvinen <ilpo.jarvinen@linux.intel.com>
Obviously I need an ack from Andy too. And thanks for doing this and
all the work you have already done so far, it's appreciated.
greg k-h
^ permalink raw reply [flat|nested] 9+ messages in thread
* Re: [PATCH] MAINTAINERS: Update DW 8250 UART maintainership
2022-06-17 9:39 [PATCH] MAINTAINERS: Update DW 8250 UART maintainership Ilpo Järvinen
2022-06-17 9:45 ` Greg Kroah-Hartman
@ 2022-06-17 9:47 ` Andy Shevchenko
2022-06-17 20:09 ` kernel test robot
2 siblings, 0 replies; 9+ messages in thread
From: Andy Shevchenko @ 2022-06-17 9:47 UTC (permalink / raw)
To: Ilpo Järvinen
Cc: Greg Kroah-Hartman, linux-serial, LKML, Andy Shevchenko, Jiri Slaby
On Fri, Jun 17, 2022 at 11:41 AM Ilpo Järvinen
<ilpo.jarvinen@linux.intel.com> wrote:
>
> Add myself as maintainer for DW 8250 UART and up it to Supported.
Thanks!
Reviewed-by: Andy Shevchenko <andy.shevchenko@gmail.com>
> Signed-off-by: Ilpo Järvinen <ilpo.jarvinen@linux.intel.com>
>
> ---
> MAINTAINERS | 3 ++-
> 1 file changed, 2 insertions(+), 1 deletion(-)
>
> diff --git a/MAINTAINERS b/MAINTAINERS
> index a6d3bd9d2a8d..6c90484f395b 100644
> --- a/MAINTAINERS
> +++ b/MAINTAINERS
> @@ -19230,8 +19230,9 @@ F: Documentation/devicetree/bindings/gpio/snps,creg-gpio.txt
> F: drivers/gpio/gpio-creg-snps.c
>
> SYNOPSYS DESIGNWARE 8250 UART DRIVER
> +M: Ilpo Järvinen <ilpo.jarvinen@linux.intel.com>
> R: Andy Shevchenko <andriy.shevchenko@linux.intel.com>
> -S: Maintained
> +S: Supported
> F: drivers/tty/serial/8250/8250_dw.c
> F: drivers/tty/serial/8250/8250_dwlib.*
> F: drivers/tty/serial/8250/8250_lpss.c
>
> --
> tg: (f2906aa86338..) dw/maintainer (depends on: tty-next)
--
With Best Regards,
Andy Shevchenko
^ permalink raw reply [flat|nested] 9+ messages in thread
* Re: [PATCH] MAINTAINERS: Update DW 8250 UART maintainership
2022-06-17 9:45 ` Greg Kroah-Hartman
@ 2022-06-17 10:21 ` Andy Shevchenko
0 siblings, 0 replies; 9+ messages in thread
From: Andy Shevchenko @ 2022-06-17 10:21 UTC (permalink / raw)
To: Greg Kroah-Hartman; +Cc: Ilpo Järvinen, linux-serial, LKML, Jiri Slaby
On Fri, Jun 17, 2022 at 11:45:00AM +0200, Greg Kroah-Hartman wrote:
> On Fri, Jun 17, 2022 at 12:39:29PM +0300, Ilpo Järvinen wrote:
> > Add myself as maintainer for DW 8250 UART and up it to Supported.
> >
> > Signed-off-by: Ilpo Järvinen <ilpo.jarvinen@linux.intel.com>
>
> Obviously I need an ack from Andy too.
I just gave my Rb, but just in case
Acked-by: From: Andy Shevchenko <andriy.shevchenko@linux.intel.com>
> And thanks for doing this and
> all the work you have already done so far, it's appreciated.
+1 and many, thanks, Ilpo!
--
With Best Regards,
Andy Shevchenko
^ permalink raw reply [flat|nested] 9+ messages in thread
* Re: [PATCH] MAINTAINERS: Update DW 8250 UART maintainership
2022-06-17 9:39 [PATCH] MAINTAINERS: Update DW 8250 UART maintainership Ilpo Järvinen
2022-06-17 9:45 ` Greg Kroah-Hartman
2022-06-17 9:47 ` Andy Shevchenko
@ 2022-06-17 20:09 ` kernel test robot
2022-06-17 20:17 ` Joe Perches
2 siblings, 1 reply; 9+ messages in thread
From: kernel test robot @ 2022-06-17 20:09 UTC (permalink / raw)
To: Ilpo Järvinen, Greg Kroah-Hartman, linux-serial, LKML,
Andy Shevchenko
Cc: kbuild-all, Jiri Slaby
Hi "Ilpo,
Thank you for the patch! Yet something to improve:
[auto build test ERROR on linus/master]
[also build test ERROR on v5.19-rc2 next-20220617]
[If your patch is applied to the wrong git tree, kindly drop us a note.
And when submitting patch, we suggest to use '--base' as documented in
https://git-scm.com/docs/git-format-patch]
url: https://github.com/intel-lab-lkp/linux/commits/Ilpo-J-rvinen/MAINTAINERS-Update-DW-8250-UART-maintainership/20220617-174145
base: https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git 47700948a4abb4a5ae13ef943ff682a7f327547a
reproduce: make htmldocs
If you fix the issue, kindly add following tag where applicable
Reported-by: kernel test robot <lkp@intel.com>
All errors (new ones prefixed by >>):
>> 'utf-8' codec can't decode byte 0xe4 in position 1396: invalid continuation byte
--
0-DAY CI Kernel Test Service
https://01.org/lkp
^ permalink raw reply [flat|nested] 9+ messages in thread
* Re: [PATCH] MAINTAINERS: Update DW 8250 UART maintainership
2022-06-17 20:09 ` kernel test robot
@ 2022-06-17 20:17 ` Joe Perches
0 siblings, 0 replies; 9+ messages in thread
From: Joe Perches @ 2022-06-17 20:17 UTC (permalink / raw)
To: kernel test robot, Ilpo Järvinen, Greg Kroah-Hartman,
linux-serial, LKML, Andy Shevchenko
Cc: kbuild-all, Jiri Slaby
On Sat, 2022-06-18 at 04:09 +0800, kernel test robot wrote:
> Hi "Ilpo,
>
> Thank you for the patch! Yet something to improve:
>
> [auto build test ERROR on linus/master]
> [also build test ERROR on v5.19-rc2 next-20220617]
> [If your patch is applied to the wrong git tree, kindly drop us a note.
> And when submitting patch, we suggest to use '--base' as documented in
> https://git-scm.com/docs/git-format-patch]
>
> url: https://github.com/intel-lab-lkp/linux/commits/Ilpo-J-rvinen/MAINTAINERS-Update-DW-8250-UART-maintainership/20220617-174145
> base: https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git 47700948a4abb4a5ae13ef943ff682a7f327547a
> reproduce: make htmldocs
>
> If you fix the issue, kindly add following tag where applicable
> Reported-by: kernel test robot <lkp@intel.com>
>
> All errors (new ones prefixed by >>):
>
> > > 'utf-8' codec can't decode byte 0xe4 in position 1396: invalid continuation byte
?
false positive?
^ permalink raw reply [flat|nested] 9+ messages in thread
* Re: [PATCH] MAINTAINERS: Update DW 8250 UART maintainership
@ 2022-06-17 20:17 ` Joe Perches
0 siblings, 0 replies; 9+ messages in thread
From: Joe Perches @ 2022-06-17 20:17 UTC (permalink / raw)
To: kbuild-all
[-- Attachment #1: Type: text/plain, Size: 996 bytes --]
On Sat, 2022-06-18 at 04:09 +0800, kernel test robot wrote:
> Hi "Ilpo,
>
> Thank you for the patch! Yet something to improve:
>
> [auto build test ERROR on linus/master]
> [also build test ERROR on v5.19-rc2 next-20220617]
> [If your patch is applied to the wrong git tree, kindly drop us a note.
> And when submitting patch, we suggest to use '--base' as documented in
> https://git-scm.com/docs/git-format-patch]
>
> url: https://github.com/intel-lab-lkp/linux/commits/Ilpo-J-rvinen/MAINTAINERS-Update-DW-8250-UART-maintainership/20220617-174145
> base: https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git 47700948a4abb4a5ae13ef943ff682a7f327547a
> reproduce: make htmldocs
>
> If you fix the issue, kindly add following tag where applicable
> Reported-by: kernel test robot <lkp@intel.com>
>
> All errors (new ones prefixed by >>):
>
> > > 'utf-8' codec can't decode byte 0xe4 in position 1396: invalid continuation byte
?
false positive?
^ permalink raw reply [flat|nested] 9+ messages in thread
* Re: [kbuild-all] Re: [PATCH] MAINTAINERS: Update DW 8250 UART maintainership
2022-06-17 20:17 ` Joe Perches
(?)
@ 2022-06-20 3:17 ` Chen, Rong A
2022-06-20 6:12 ` Ilpo Järvinen
-1 siblings, 1 reply; 9+ messages in thread
From: Chen, Rong A @ 2022-06-20 3:17 UTC (permalink / raw)
To: Joe Perches, kernel test robot, Ilpo Järvinen,
Greg Kroah-Hartman, linux-serial, LKML, Andy Shevchenko
Cc: kbuild-all, Jiri Slaby
On 6/18/2022 4:17 AM, Joe Perches wrote:
> On Sat, 2022-06-18 at 04:09 +0800, kernel test robot wrote:
>> Hi "Ilpo,
>>
>> Thank you for the patch! Yet something to improve:
>>
>> [auto build test ERROR on linus/master]
>> [also build test ERROR on v5.19-rc2 next-20220617]
>> [If your patch is applied to the wrong git tree, kindly drop us a note.
>> And when submitting patch, we suggest to use '--base' as documented in
>> https://git-scm.com/docs/git-format-patch]
>>
>> url: https://github.com/intel-lab-lkp/linux/commits/Ilpo-J-rvinen/MAINTAINERS-Update-DW-8250-UART-maintainership/20220617-174145
>> base: https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git 47700948a4abb4a5ae13ef943ff682a7f327547a
>> reproduce: make htmldocs
>>
>> If you fix the issue, kindly add following tag where applicable
>> Reported-by: kernel test robot <lkp@intel.com>
>>
>> All errors (new ones prefixed by >>):
>>
>>>> 'utf-8' codec can't decode byte 0xe4 in position 1396: invalid continuation byte
>
> ?
>
> false positive?
Hi,
I tried to apply the patch, and I can reproduce the error by 'make
htmldocs':
$ wget -q -O -
https://lore.kernel.org/lkml/be58b398-71ff-7c12-1bf1-a09181d9c80@linux.intel.com/raw
| git apply -v
$ git --no-pager diff
diff --git a/MAINTAINERS b/MAINTAINERS
index f52543aedd61e..3d64756814f09 100644
--- a/MAINTAINERS
+++ b/MAINTAINERS
@@ -19247,8 +19247,9 @@ F:
Documentation/devicetree/bindings/gpio/snps,creg-gpio.txt
F: drivers/gpio/gpio-creg-snps.c
SYNOPSYS DESIGNWARE 8250 UART DRIVER
+M: Ilpo Jrvinen <ilpo.jarvinen@linux.intel.com>
R: Andy Shevchenko <andriy.shevchenko@linux.intel.com>
-S: Maintained
+S: Supported
F: drivers/tty/serial/8250/8250_dw.c
F: drivers/tty/serial/8250/8250_dwlib.*
F: drivers/tty/serial/8250/8250_lpss.c
It seems the special character 'ä' in name 'Ilpo Järvinen' was parsed
wrongly.
Best Regards,
Rong Chen
^ permalink raw reply related [flat|nested] 9+ messages in thread
* Re: [kbuild-all] Re: [PATCH] MAINTAINERS: Update DW 8250 UART maintainership
2022-06-20 3:17 ` [kbuild-all] " Chen, Rong A
@ 2022-06-20 6:12 ` Ilpo Järvinen
0 siblings, 0 replies; 9+ messages in thread
From: Ilpo Järvinen @ 2022-06-20 6:12 UTC (permalink / raw)
To: Chen, Rong A
Cc: Joe Perches, kernel test robot, Greg Kroah-Hartman, linux-serial,
LKML, Andy Shevchenko, kbuild-all, Jiri Slaby
[-- Attachment #1: Type: text/plain, Size: 2853 bytes --]
On Mon, 20 Jun 2022, Chen, Rong A wrote:
> On 6/18/2022 4:17 AM, Joe Perches wrote:
> > On Sat, 2022-06-18 at 04:09 +0800, kernel test robot wrote:
> > > Hi "Ilpo,
> > >
> > > Thank you for the patch! Yet something to improve:
> > >
> > > [auto build test ERROR on linus/master]
> > > [also build test ERROR on v5.19-rc2 next-20220617]
> > > [If your patch is applied to the wrong git tree, kindly drop us a note.
> > > And when submitting patch, we suggest to use '--base' as documented in
> > > https://git-scm.com/docs/git-format-patch]
> > >
> > > url:
> > > https://github.com/intel-lab-lkp/linux/commits/Ilpo-J-rvinen/MAINTAINERS-Update-DW-8250-UART-maintainership/20220617-174145
> > > base: https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git
> > > 47700948a4abb4a5ae13ef943ff682a7f327547a
> > > reproduce: make htmldocs
> > >
> > > If you fix the issue, kindly add following tag where applicable
> > > Reported-by: kernel test robot <lkp@intel.com>
> > >
> > > All errors (new ones prefixed by >>):
> > >
> > > > > 'utf-8' codec can't decode byte 0xe4 in position 1396: invalid
> > > > > continuation byte
> >
> > ?
> >
> > false positive?
This is correctly identified problem, however, the problem is lkp's end.
It seemingly fails to handle non-UTF8 encoded emails correctly and just
goes to assume the input is UTF8 encoded (w/o checking the Content-Type
header).
> I tried to apply the patch, and I can reproduce the error by 'make htmldocs':
>
> $ wget -q -O -
> https://lore.kernel.org/lkml/be58b398-71ff-7c12-1bf1-a09181d9c80@linux.intel.com/raw
> | git apply -v
> $ git --no-pager diff
> diff --git a/MAINTAINERS b/MAINTAINERS
> index f52543aedd61e..3d64756814f09 100644
> --- a/MAINTAINERS
> +++ b/MAINTAINERS
> @@ -19247,8 +19247,9 @@ F:
> Documentation/devicetree/bindings/gpio/snps,creg-gpio.txt
> F: drivers/gpio/gpio-creg-snps.c
>
> SYNOPSYS DESIGNWARE 8250 UART DRIVER
> +M: Ilpo Jrvinen <ilpo.jarvinen@linux.intel.com>
> R: Andy Shevchenko <andriy.shevchenko@linux.intel.com>
> -S: Maintained
> +S: Supported
> F: drivers/tty/serial/8250/8250_dw.c
> F: drivers/tty/serial/8250/8250_dwlib.*
> F: drivers/tty/serial/8250/8250_lpss.c
>
> It seems the special character 'ä' in name 'Ilpo Järvinen' was parsed wrongly.
The original file was UTF-8 (0xc3 0xa4). The email is correctly encoded
ISO 8859-15 as can be see e.g. by downloading this (with a single 0xe4
char):
https://marc.info/?l=linux-kernel&m=165545870203513&q=raw
When saving patch/exporting email, MUA/exporter should do ISO 8859-15 ->
UTF-8 conversion. If I export email I sent, I get it back to UTF-8 just
fine (as per the locale I've in use).
Just in case, I've attached the original patch file.
I thought git's toolchain these days handles these things fine but
it seems not.
--
i.
[-- Warning: decoded text below may be mangled, UTF-8 assumed --]
[-- Attachment #2: Type: text/x-diff; NAME=maint.patch, Size: 944 bytes --]
From: Ilpo Järvinen <ilpo.jarvinen@linux.intel.com>
Subject: [PATCH] MAINTAINERS: Update DW 8250 UART maintainership
Add myself as maintainer for DW 8250 UART and up it to Supported.
Signed-off-by: Ilpo Järvinen <ilpo.jarvinen@linux.intel.com>
---
MAINTAINERS | 3 ++-
1 file changed, 2 insertions(+), 1 deletion(-)
diff --git a/MAINTAINERS b/MAINTAINERS
index a6d3bd9d2a8d..6c90484f395b 100644
--- a/MAINTAINERS
+++ b/MAINTAINERS
@@ -19230,8 +19230,9 @@ F: Documentation/devicetree/bindings/gpio/snps,creg-gpio.txt
F: drivers/gpio/gpio-creg-snps.c
SYNOPSYS DESIGNWARE 8250 UART DRIVER
+M: Ilpo Järvinen <ilpo.jarvinen@linux.intel.com>
R: Andy Shevchenko <andriy.shevchenko@linux.intel.com>
-S: Maintained
+S: Supported
F: drivers/tty/serial/8250/8250_dw.c
F: drivers/tty/serial/8250/8250_dwlib.*
F: drivers/tty/serial/8250/8250_lpss.c
--
tg: (65534736d9a5..) dw/maintainer (depends on: tty-next)
^ permalink raw reply related [flat|nested] 9+ messages in thread
end of thread, other threads:[~2022-06-20 6:12 UTC | newest]
Thread overview: 9+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2022-06-17 9:39 [PATCH] MAINTAINERS: Update DW 8250 UART maintainership Ilpo Järvinen
2022-06-17 9:45 ` Greg Kroah-Hartman
2022-06-17 10:21 ` Andy Shevchenko
2022-06-17 9:47 ` Andy Shevchenko
2022-06-17 20:09 ` kernel test robot
2022-06-17 20:17 ` Joe Perches
2022-06-17 20:17 ` Joe Perches
2022-06-20 3:17 ` [kbuild-all] " Chen, Rong A
2022-06-20 6:12 ` Ilpo Järvinen
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.