RE: [PATCH v1 0/2] serial_mxc: fixing reception
Hello Fabio, > -Original Message- > From: U-Boot On Behalf Of Fabio Estevam > Sent: Monday, September 5, 2022 1:20 PM > To: ZHIZHIKIN Andrey > Cc: Pali Rohár ; u-boot@lists.denx.de; trini > ; Peng Fan (OSS) ; Fabio Estevam > ; Stefano Babic > Subject: Re: [PATCH v1 0/2] serial_mxc: fixing reception > > Hi Andrey, > > On Mon, Sep 5, 2022 at 6:21 AM ZHIZHIKIN Andrey > wrote: > > > I cannot modify this part since removing you from maintainer list would > > require a transfer of maintainership of this component to another name, > > which I do not know. > > > > I've Cc:'d Tom and people from imx world here so they can make a decision > > on who shall continue the maintenance of this file when your entry is > > removed. > > I have added the imx serial driver to the "ARM FREESCALE IMX" list: > https://lore.kernel.org/u-boot/20220905105700.3209658-1-feste...@gmail.com/T/#u I've seen your patch on the ML already, thanks a lot for taking care of this! Regards, Andrey
Re: [PATCH v1 0/2] serial_mxc: fixing reception
Hi Andrey, On Mon, Sep 5, 2022 at 6:21 AM ZHIZHIKIN Andrey wrote: > I cannot modify this part since removing you from maintainer list would > require a transfer of maintainership of this component to another name, > which I do not know. > > I've Cc:'d Tom and people from imx world here so they can make a decision > on who shall continue the maintenance of this file when your entry is removed. I have added the imx serial driver to the "ARM FREESCALE IMX" list: https://lore.kernel.org/u-boot/20220905105700.3209658-1-feste...@gmail.com/T/#u
RE: [PATCH v1 0/2] serial_mxc: fixing reception
Hello Pali, > -Original Message- > From: U-Boot On Behalf Of Pali Rohar > Sent: Monday, September 5, 2022 11:14 AM > To: ZHIZHIKIN Andrey > Cc: u-boot@lists.denx.de > Subject: Re: [PATCH v1 0/2] serial_mxc: fixing reception > > On Monday 05 September 2022 09:12:49 ZHIZHIKIN Andrey wrote: > > Hello Pali, > > > > > -Original Message- > > > From: U-Boot On Behalf Of Pali Rohar > > > Sent: Monday, September 5, 2022 10:59 AM > > > To: u-boot@lists.denx.de > > > Subject: Re: [PATCH v1 0/2] serial_mxc: fixing reception > > > > > > On Monday 05 September 2022 10:53:58 Johannes Schneider wrote: > > > > while writing to a imx-serial device is probably thoroughly tested - and > > > obviosly works for the debug-serial - using a serial driver to read data > received > > > over the serial interface does not work reliably. > > > > > > > > the patches in this series address issues found during the > > > > implementation > of a > > > custom uboot-command to query a coprocessor, connected to an imx8mm over > uart4, > > > for mainboard-identification strings > > > > > > > > > > > > Johannes Schneider (2): > > > > serial: mxc: enable the RX pipeline > > > > serial: mxc: have putc use the TXFIFO > > > > > > > > drivers/serial/serial_mxc.c | 13 - > > > > 1 file changed, 12 insertions(+), 1 deletion(-) > > > > > > Hello! I'm really not maintainer of mxc and I do not have time to review > > > those patches. So please do not send me them as I'm spammed with tons of > > > unrelated emails/patches. > > > > Then there is a problem with having your name as a maintainer for this file. > > > > If I execute: > > $ ./scripts/get_maintainer.pl drivers/serial/serial_mxc.c > > "Pali Rohár" > (commit_signer:1/1=100%,authored:1/1=100%,added_lines:2/2=100%,removed_lines:2/2= > 100%) > > u-boot@lists.denx.de (open list) > > > > I guess this has to be corrected then, right? > > > > Regards, > > Andrey > > Yes, please correct it. I have not added myself to the list of mxc > maintainers and I a really do not have time for reviewing mxc patches. I cannot modify this part since removing you from maintainer list would require a transfer of maintainership of this component to another name, which I do not know. I've Cc:'d Tom and people from imx world here so they can make a decision on who shall continue the maintenance of this file when your entry is removed. -- andrey
Re: [PATCH v1 0/2] serial_mxc: fixing reception
On Monday 05 September 2022 09:12:49 ZHIZHIKIN Andrey wrote: > Hello Pali, > > > -Original Message- > > From: U-Boot On Behalf Of Pali Rohar > > Sent: Monday, September 5, 2022 10:59 AM > > To: u-boot@lists.denx.de > > Subject: Re: [PATCH v1 0/2] serial_mxc: fixing reception > > > > On Monday 05 September 2022 10:53:58 Johannes Schneider wrote: > > > while writing to a imx-serial device is probably thoroughly tested - and > > obviosly works for the debug-serial - using a serial driver to read data > > received > > over the serial interface does not work reliably. > > > > > > the patches in this series address issues found during the implementation > > > of a > > custom uboot-command to query a coprocessor, connected to an imx8mm over > > uart4, > > for mainboard-identification strings > > > > > > > > > Johannes Schneider (2): > > > serial: mxc: enable the RX pipeline > > > serial: mxc: have putc use the TXFIFO > > > > > > drivers/serial/serial_mxc.c | 13 - > > > 1 file changed, 12 insertions(+), 1 deletion(-) > > > > Hello! I'm really not maintainer of mxc and I do not have time to review > > those patches. So please do not send me them as I'm spammed with tons of > > unrelated emails/patches. > > Then there is a problem with having your name as a maintainer for this file. > > If I execute: > $ ./scripts/get_maintainer.pl drivers/serial/serial_mxc.c > "Pali Rohár" > (commit_signer:1/1=100%,authored:1/1=100%,added_lines:2/2=100%,removed_lines:2/2=100%) > u-boot@lists.denx.de (open list) > > I guess this has to be corrected then, right? > > Regards, > Andrey Yes, please correct it. I have not added myself to the list of mxc maintainers and I a really do not have time for reviewing mxc patches.
RE: [PATCH v1 0/2] serial_mxc: fixing reception
Hello Pali, > -Original Message- > From: U-Boot On Behalf Of Pali Rohar > Sent: Monday, September 5, 2022 10:59 AM > To: u-boot@lists.denx.de > Subject: Re: [PATCH v1 0/2] serial_mxc: fixing reception > > On Monday 05 September 2022 10:53:58 Johannes Schneider wrote: > > while writing to a imx-serial device is probably thoroughly tested - and > obviosly works for the debug-serial - using a serial driver to read data > received > over the serial interface does not work reliably. > > > > the patches in this series address issues found during the implementation > > of a > custom uboot-command to query a coprocessor, connected to an imx8mm over > uart4, > for mainboard-identification strings > > > > > > Johannes Schneider (2): > > serial: mxc: enable the RX pipeline > > serial: mxc: have putc use the TXFIFO > > > > drivers/serial/serial_mxc.c | 13 - > > 1 file changed, 12 insertions(+), 1 deletion(-) > > Hello! I'm really not maintainer of mxc and I do not have time to review > those patches. So please do not send me them as I'm spammed with tons of > unrelated emails/patches. Then there is a problem with having your name as a maintainer for this file. If I execute: $ ./scripts/get_maintainer.pl drivers/serial/serial_mxc.c "Pali Rohár" (commit_signer:1/1=100%,authored:1/1=100%,added_lines:2/2=100%,removed_lines:2/2=100%) u-boot@lists.denx.de (open list) I guess this has to be corrected then, right? Regards, Andrey
Re: [PATCH v1 0/2] serial_mxc: fixing reception
On Monday 05 September 2022 10:53:58 Johannes Schneider wrote: > while writing to a imx-serial device is probably thoroughly tested - and > obviosly works for the debug-serial - using a serial driver to read data > received over the serial interface does not work reliably. > > the patches in this series address issues found during the implementation of > a custom uboot-command to query a coprocessor, connected to an imx8mm over > uart4, for mainboard-identification strings > > > Johannes Schneider (2): > serial: mxc: enable the RX pipeline > serial: mxc: have putc use the TXFIFO > > drivers/serial/serial_mxc.c | 13 - > 1 file changed, 12 insertions(+), 1 deletion(-) Hello! I'm really not maintainer of mxc and I do not have time to review those patches. So please do not send me them as I'm spammed with tons of unrelated emails/patches.
