On 07.05.2020 14:32, Tiaki Rice wrote:

Eugen,

Thanks for the speedy reply, I did not know about this script! I must have glossed over it in the uboot patch documentation page.

The commit line is 82 characters which is longer than the preferred max of 75. Oops. Don't think this warning would be stopping the patch going through though...

Hi,

For sure that won't be a problem with patchwork. however , it looks like your patch is windows-generated ? That might be an issue, I do not know. Can you send me your patch as attachment file ?

Thanks

Here is the output for the patch:

 $ ./scripts/checkpatch.pl 0001-dtsi-sama5d2-Add-uart4-definition-to-sama5d2.dtsi.patch
 WARNING: Possible unwrapped commit description (prefer a maximum 75 chars per line)
 #7:
 This patch adds support for uart4 to the processor level device tree include file.

 total: 0 errors, 1 warnings, 0 checks, 14 lines checked

 NOTE: For some of the reported defects, checkpatch may be able to
       mechanically convert to the typical style using --fix or --fix-inplace.

 0001-dtsi-sama5d2-Add-uart4-definition-to-sama5d2.dtsi.patch has style problems, please review.

 NOTE: Ignored message types: COMPLEX_MACRO CONSIDER_KSTRTO MINMAX MULTISTATEMENT_MACRO_USE_DO_WHILE NETWORKING_BLOCK_COMMENT_STYLE PREFER_ETHER_ADDR_COPY USLEEP_RANGE

 NOTE: If any of the errors are false positives, please report
       them to the maintainer, see CHECKPATCH in MAINTAINERS.

---

Regards,

Tiaki Rice

-----Original Message----- From: Eugen.Hristev@microchip.com Eugen.Hristev@microchip.com Sent: Thursday, 7 May 2020 8:19 PM To: tiakirice@hotmail.com Cc: u-boot@lists.denx.de Subject: Re: [PATCH] arm: dtsi: sama5d2: Add uart4 definition to sama5d2.dtsi

On 07.05.2020 12:21, Tiaki Rice wrote:

...

Hello Eugen,

I'm new to this whole mailing list patchwork thingy so I suspect I didn't quite submit the patch correctly, maybe you can point me in the right direction.

What I did:

• Made the changes on master and locally commited them
• Created a patch using 'git format-patch' and modified the patch file email body.
• Used 'git send-email' to send the patch to u-boot@lists.denx.de and

Cc'd you

This opened my Outlook of which I checked to make sure the email was plain text and then sent it. Any ideas?

Hi Tiaki,

From my perspective you did everything very well. The patch looks sent fine. I would only have one question, did you check your patch with the built-in perl script that automatically checks your patch for formatting ? ./scripts/checkpatch.pl --strict <your patch file>

About patchwork, I think it may be an issue with patchwork.

Eugen

...

Thanks in advance,

Tiaki Rice

-----Original Message----- From: Eugen.Hristev@microchip.com Eugen.Hristev@microchip.com Sent: Thursday, 7 May 2020 7:12 PM To: tiakirice@hotmail.com; u-boot@lists.denx.de Subject: Re: [PATCH] arm: dtsi: sama5d2: Add uart4 definition to sama5d2.dtsi

On 07.05.2020 07:43, Tiaki Rice wrote:

...

This patch adds support for uart4 to the processor level device tree include file.

Signed-off-by: Tiaki Rice tiakirice@hotmail.com Cc: Eugen Hristev Eugen.Hristev@microchip.com

---

Hi,

Thanks for contributing !

I have an issue with this patch missing from patchwork. Anyone has any idea why that happened ?

Eugen

...

arch/arm/dts/sama5d2.dtsi | 8 ++++++++
1 file changed, 8 insertions(+)

diff --git a/arch/arm/dts/sama5d2.dtsi b/arch/arm/dts/sama5d2.dtsi index 5adc47b906..6fb2cb25f9 100644 --- a/arch/arm/dts/sama5d2.dtsi +++ b/arch/arm/dts/sama5d2.dtsi @@ -746,6 +746,14 @@ status = "disabled"; };

+uart4: serial@fc00c000 { +compatible = "atmel,at91sam9260-usart"; reg = <0xfc00c000 0x100>; +clocks = <&uart4_clk>; clock-names = "usart"; status = "disabled"; +};

• i2c1: i2c@fc028000 { compatible = "atmel,sama5d2-i2c"; reg = <0xfc028000 0x100>;

-- 2.22.0.windows.1

On 07.05.2020 17:23, Tiaki Rice wrote:

Eugen,

Ah, I suspect the issue might be might be because I changed the subject inside the file but the did not change the filename. I shouldn't have done it that way anyway. Regarding windows vs linux, doing a diff between them showed no differences aside from the tag at the end describing the git version.

I have amended the commit and attached both the windows git patch I originally submitted for reference (0001-dtsi...) and the new patch created under CentOS8 (0001-arm-dtsi...). If you can't see any problems, should I try emailing the patch to the list again?

Ok, your patch looks good. I can see it now in patchwork. Maybe it was a patchwork glitch.

I will modify the patch a little bit to comply with the shortlog rules: ARM must be in caps then dts, no need to put dtsi, and no need to mention sama5d2.dtsi again in shortlog (ARM: dts: sama5d2 means it's sama5d2.dtsi )

like:

ARM: dts: sama5d2: add uart4 definition

if you feel like doing that yourself, modify and send a v2 patch (git format-patch -v 2) and send it again with git send-email.

Thanks again ! Eugen

Regards, Tiaki

-----Original Message----- From: Eugen.Hristev@microchip.com Eugen.Hristev@microchip.com Sent: Thursday, 7 May 2020 9:55 PM To: tiakirice@hotmail.com Cc: u-boot@lists.denx.de Subject: Re: [PATCH] arm: dtsi: sama5d2: Add uart4 definition to sama5d2.dtsi

On 07.05.2020 14:32, Tiaki Rice wrote:

...

Eugen,

Thanks for the speedy reply, I did not know about this script! I must have glossed over it in the uboot patch documentation page.

The commit line is 82 characters which is longer than the preferred max of 75. Oops. Don't think this warning would be stopping the patch going through though...

Hi,

For sure that won't be a problem with patchwork. however , it looks like your patch is windows-generated ? That might be an issue, I do not know. Can you send me your patch as attachment file ?

Thanks

...

Here is the output for the patch:

  $ ./scripts/checkpatch.pl 0001-dtsi-sama5d2-Add-uart4-definition-to-sama5d2.dtsi.patch
  WARNING: Possible unwrapped commit description (prefer a maximum 75 chars per line)
  #7:
  This patch adds support for uart4 to the processor level device tree include file.

  total: 0 errors, 1 warnings, 0 checks, 14 lines checked

  NOTE: For some of the reported defects, checkpatch may be able to
        mechanically convert to the typical style using --fix or --fix-inplace.

  0001-dtsi-sama5d2-Add-uart4-definition-to-sama5d2.dtsi.patch has style problems, please review.

  NOTE: Ignored message types: COMPLEX_MACRO CONSIDER_KSTRTO MINMAX

MULTISTATEMENT_MACRO_USE_DO_WHILE NETWORKING_BLOCK_COMMENT_STYLE PREFER_ETHER_ADDR_COPY USLEEP_RANGE

  NOTE: If any of the errors are false positives, please report
        them to the maintainer, see CHECKPATCH in MAINTAINERS.

---

Regards,

Tiaki Rice

-----Original Message----- From: Eugen.Hristev@microchip.com Eugen.Hristev@microchip.com Sent: Thursday, 7 May 2020 8:19 PM To: tiakirice@hotmail.com Cc: u-boot@lists.denx.de Subject: Re: [PATCH] arm: dtsi: sama5d2: Add uart4 definition to sama5d2.dtsi

On 07.05.2020 12:21, Tiaki Rice wrote:

...

Hello Eugen,

I'm new to this whole mailing list patchwork thingy so I suspect I didn't quite submit the patch correctly, maybe you can point me in the right direction.

What I did:

• Made the changes on master and locally commited them
• Created a patch using 'git format-patch' and modified the patch file email body.
• Used 'git send-email' to send the patch to u-boot@lists.denx.de and

Cc'd you

This opened my Outlook of which I checked to make sure the email was plain text and then sent it. Any ideas?

Hi Tiaki,

From my perspective you did everything very well. The patch looks sent fine. I would only have one question, did you check your patch with the built-in perl script that automatically checks your patch for formatting ? ./scripts/checkpatch.pl --strict <your patch file>

About patchwork, I think it may be an issue with patchwork.

Eugen

...

Thanks in advance,

Tiaki Rice

-----Original Message----- From: Eugen.Hristev@microchip.com Eugen.Hristev@microchip.com Sent: Thursday, 7 May 2020 7:12 PM To: tiakirice@hotmail.com; u-boot@lists.denx.de Subject: Re: [PATCH] arm: dtsi: sama5d2: Add uart4 definition to sama5d2.dtsi

On 07.05.2020 07:43, Tiaki Rice wrote:

...

This patch adds support for uart4 to the processor level device tree include file.

Signed-off-by: Tiaki Rice tiakirice@hotmail.com Cc: Eugen Hristev Eugen.Hristev@microchip.com

---

Hi,

Thanks for contributing !

I have an issue with this patch missing from patchwork. Anyone has any idea why that happened ?

Eugen

...

 arch/arm/dts/sama5d2.dtsi | 8 ++++++++
 1 file changed, 8 insertions(+)

diff --git a/arch/arm/dts/sama5d2.dtsi b/arch/arm/dts/sama5d2.dtsi index 5adc47b906..6fb2cb25f9 100644 --- a/arch/arm/dts/sama5d2.dtsi +++ b/arch/arm/dts/sama5d2.dtsi @@ -746,6 +746,14 @@ status = "disabled"; };

+uart4: serial@fc00c000 { +compatible = "atmel,at91sam9260-usart"; reg = <0xfc00c000 0x100>; +clocks = <&uart4_clk>; clock-names = "usart"; status = "disabled"; +};

• i2c1: i2c@fc028000 { compatible = "atmel,sama5d2-i2c"; reg = <0xfc028000 0x100>;

-- 2.22.0.windows.1