[PATCH] arm: dtsi: sama5d2: Add uart4 definition to sama5d2.dtsi

[PATCH] arm: dtsi: sama5d2: Add uart4 definition to sama5d2.dtsi

[Tiaki Rice]

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>
---
 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 at fc00c000 {
+				compatible = "atmel,at91sam9260-usart";
+				reg = <0xfc00c000 0x100>;
+				clocks = <&uart4_clk>;
+				clock-names = "usart";
+				status = "disabled";
+			};
+
 			i2c1: i2c at fc028000 {
 				compatible = "atmel,sama5d2-i2c";
 				reg = <0xfc028000 0x100>;
-- 
2.22.0.windows.1

-------------- next part --------------
A non-text attachment was scrubbed...
Name: winmail.dat
Type: application/ms-tnef
Size: 41077 bytes
Desc: not available
URL: <https://lists.denx.de/pipermail/u-boot/attachments/20200507/66968c62/attachment.bin>

[PATCH] arm: dtsi: sama5d2: Add uart4 definition to sama5d2.dtsi

[Eugen.Hristev at microchip.com]

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 at fc00c000 {
> +compatible = "atmel,at91sam9260-usart";
> +reg = <0xfc00c000 0x100>;
> +clocks = <&uart4_clk>;
> +clock-names = "usart";
> +status = "disabled";
> +};
> +
>   i2c1: i2c at fc028000 {
>   compatible = "atmel,sama5d2-i2c";
>   reg = <0xfc028000 0x100>;
> --
> 2.22.0.windows.1
> 

[PATCH] arm: dtsi: sama5d2: Add uart4 definition to sama5d2.dtsi

[Eugen.Hristev at microchip.com]

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 at 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 at microchip.com <Eugen.Hristev@microchip.com>
> Sent: Thursday, 7 May 2020 7:12 PM
> To: tiakirice at hotmail.com; u-boot at 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 at fc00c000 {
>> +compatible = "atmel,at91sam9260-usart"; reg = <0xfc00c000 0x100>;
>> +clocks = <&uart4_clk>; clock-names = "usart"; status = "disabled"; };
>> +
>>    i2c1: i2c at fc028000 {
>>    compatible = "atmel,sama5d2-i2c";
>>    reg = <0xfc028000 0x100>;
>> --
>> 2.22.0.windows.1
>>
> 

[PATCH] arm: dtsi: sama5d2: Add uart4 definition to sama5d2.dtsi

[Tiaki Rice]

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...


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 at hotmail.com
Cc: u-boot at 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 at 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 at microchip.com <Eugen.Hristev@microchip.com>
> Sent: Thursday, 7 May 2020 7:12 PM
> To: tiakirice at hotmail.com; u-boot at 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 at fc00c000 {
>> +compatible = "atmel,at91sam9260-usart"; reg = <0xfc00c000 0x100>; 
>> +clocks = <&uart4_clk>; clock-names = "usart"; status = "disabled"; 
>> +};
>> +
>>    i2c1: i2c at fc028000 {
>>    compatible = "atmel,sama5d2-i2c";
>>    reg = <0xfc028000 0x100>;
>> --
>> 2.22.0.windows.1
>>
> 

[PATCH] arm: dtsi: sama5d2: Add uart4 definition to sama5d2.dtsi

[Eugen.Hristev at microchip.com]

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 at microchip.com <Eugen.Hristev@microchip.com>
> Sent: Thursday, 7 May 2020 8:19 PM
> To: tiakirice at hotmail.com
> Cc: u-boot at 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 at 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 at microchip.com <Eugen.Hristev@microchip.com>
>> Sent: Thursday, 7 May 2020 7:12 PM
>> To: tiakirice at hotmail.com; u-boot at 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 at fc00c000 {
>>> +compatible = "atmel,at91sam9260-usart"; reg = <0xfc00c000 0x100>;
>>> +clocks = <&uart4_clk>; clock-names = "usart"; status = "disabled";
>>> +};
>>> +
>>>     i2c1: i2c at fc028000 {
>>>     compatible = "atmel,sama5d2-i2c";
>>>     reg = <0xfc028000 0x100>;
>>> --
>>> 2.22.0.windows.1
>>>
>>
> 

[PATCH] arm: dtsi: sama5d2: Add uart4 definition to sama5d2.dtsi

[Tiaki Rice]

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?


Regards, Tiaki

-----Original Message-----
From: Eugen.Hristev@microchip.com <Eugen.Hristev@microchip.com> 
Sent: Thursday, 7 May 2020 9:55 PM
To: tiakirice at hotmail.com
Cc: u-boot at 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 at microchip.com <Eugen.Hristev@microchip.com>
> Sent: Thursday, 7 May 2020 8:19 PM
> To: tiakirice at hotmail.com
> Cc: u-boot at 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 at 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 at microchip.com <Eugen.Hristev@microchip.com>
>> Sent: Thursday, 7 May 2020 7:12 PM
>> To: tiakirice at hotmail.com; u-boot at 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 at fc00c000 {
>>> +compatible = "atmel,at91sam9260-usart"; reg = <0xfc00c000 0x100>; 
>>> +clocks = <&uart4_clk>; clock-names = "usart"; status = "disabled"; 
>>> +};
>>> +
>>>     i2c1: i2c at fc028000 {
>>>     compatible = "atmel,sama5d2-i2c";
>>>     reg = <0xfc028000 0x100>;
>>> --
>>> 2.22.0.windows.1
>>>
>>
> 

-------------- next part --------------
A non-text attachment was scrubbed...
Name: 0001-dtsi-sama5d2-Add-uart4-definition-to-sama5d2.dtsi.patch
Type: application/octet-stream
Size: 1023 bytes
Desc: 0001-dtsi-sama5d2-Add-uart4-definition-to-sama5d2.dtsi.patch
URL: <https://lists.denx.de/pipermail/u-boot/attachments/20200507/771b1f15/attachment.obj>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: 0001-arm-dtsi-sama5d2-Add-uart4-definition-to-sama5d2.dts.patch
Type: application/octet-stream
Size: 1013 bytes
Desc: 0001-arm-dtsi-sama5d2-Add-uart4-definition-to-sama5d2.dts.patch
URL: <https://lists.denx.de/pipermail/u-boot/attachments/20200507/771b1f15/attachment-0001.obj>

[PATCH] arm: dtsi: sama5d2: Add uart4 definition to sama5d2.dtsi

[Eugen.Hristev at microchip.com]

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 at microchip.com <Eugen.Hristev@microchip.com>
> Sent: Thursday, 7 May 2020 9:55 PM
> To: tiakirice at hotmail.com
> Cc: u-boot at 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 at microchip.com <Eugen.Hristev@microchip.com>
>> Sent: Thursday, 7 May 2020 8:19 PM
>> To: tiakirice at hotmail.com
>> Cc: u-boot at 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 at 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 at microchip.com <Eugen.Hristev@microchip.com>
>>> Sent: Thursday, 7 May 2020 7:12 PM
>>> To: tiakirice at hotmail.com; u-boot at 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 at fc00c000 {
>>>> +compatible = "atmel,at91sam9260-usart"; reg = <0xfc00c000 0x100>;
>>>> +clocks = <&uart4_clk>; clock-names = "usart"; status = "disabled";
>>>> +};
>>>> +
>>>>      i2c1: i2c at fc028000 {
>>>>      compatible = "atmel,sama5d2-i2c";
>>>>      reg = <0xfc028000 0x100>;
>>>> --
>>>> 2.22.0.windows.1
>>>>
>>>
>>
>