mbox series

[GIT,PULL] SoCFPGA DTS Agilex fix for v5.6

Message ID 20200302054701.21658-1-dinguyen@kernel.org (mailing list archive)
State Superseded
Headers show
Series [GIT,PULL] SoCFPGA DTS Agilex fix for v5.6 | expand

Pull-request

git://git.kernel.org/pub/scm/linux/kernel/git/dinguyen/linux.git socfpga_fixes_for_v5.6

Message

Dinh Nguyen March 2, 2020, 5:47 a.m. UTC
Hi Arnd, Kevin, and Olof:

Please pull in this SoCFPGA Agilex fix for v5.6.

Thanks,
Dinh

The following changes since commit 98d54f81e36ba3bf92172791eba5ca5bd813989b:

  Linux 5.6-rc4 (2020-03-01 16:38:46 -0600)

are available in the Git repository at:

  git://git.kernel.org/pub/scm/linux/kernel/git/dinguyen/linux.git socfpga_fixes_for_v5.6

for you to fetch changes up to ac3e92744b3c8c45fab44baa9307cf7b3ad9c078:

  arm64: dts: socfpga: agilex: Fix gmac compatible (2020-03-01 23:19:58 -0600)

----------------------------------------------------------------
Ley Foon Tan (1):
      arm64: dts: socfpga: agilex: Fix gmac compatible

 arch/arm64/boot/dts/intel/socfpga_agilex.dtsi | 6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)

Comments

Olof Johansson March 2, 2020, 6:18 p.m. UTC | #1
Hi Dinh,

On Sun, Mar 01, 2020 at 11:47:01PM -0600, Dinh Nguyen wrote:
> Hi Arnd, Kevin, and Olof:
> 
> Please pull in this SoCFPGA Agilex fix for v5.6.
> 
> Thanks,
> Dinh
> 
> The following changes since commit 98d54f81e36ba3bf92172791eba5ca5bd813989b:
> 
>   Linux 5.6-rc4 (2020-03-01 16:38:46 -0600)
> 
> are available in the Git repository at:
> 
>   git://git.kernel.org/pub/scm/linux/kernel/git/dinguyen/linux.git socfpga_fixes_for_v5.6
> 
> for you to fetch changes up to ac3e92744b3c8c45fab44baa9307cf7b3ad9c078:
> 
>   arm64: dts: socfpga: agilex: Fix gmac compatible (2020-03-01 23:19:58 -0600)
> 
> ----------------------------------------------------------------
> Ley Foon Tan (1):
>       arm64: dts: socfpga: agilex: Fix gmac compatible

This patch has two minor formatting errors:

1) The "Fixes:" tag should go in the same block as Signed-off-by/Cc at the end,
   no need to do an empty line between.

2) The Fixes line should be in one line, even if it's longer than 80 characters.

I'll hold off sending a pull request out if you do a quick respin to fix that.
If I merge it as-is we'll get a nag note from Stephen Rothwell. :)


-Olof
Dinh Nguyen March 3, 2020, 3:36 p.m. UTC | #2
Hi Olof,

On 3/2/20 12:18 PM, Olof Johansson wrote:
> Hi Dinh,
> 
> On Sun, Mar 01, 2020 at 11:47:01PM -0600, Dinh Nguyen wrote:
>> Hi Arnd, Kevin, and Olof:
>>
>> Please pull in this SoCFPGA Agilex fix for v5.6.
>>
>> Thanks,
>> Dinh
>>
>> The following changes since commit 98d54f81e36ba3bf92172791eba5ca5bd813989b:
>>
>>   Linux 5.6-rc4 (2020-03-01 16:38:46 -0600)
>>
>> are available in the Git repository at:
>>
>>   git://git.kernel.org/pub/scm/linux/kernel/git/dinguyen/linux.git socfpga_fixes_for_v5.6
>>
>> for you to fetch changes up to ac3e92744b3c8c45fab44baa9307cf7b3ad9c078:
>>
>>   arm64: dts: socfpga: agilex: Fix gmac compatible (2020-03-01 23:19:58 -0600)
>>
>> ----------------------------------------------------------------
>> Ley Foon Tan (1):
>>       arm64: dts: socfpga: agilex: Fix gmac compatible
> 
> This patch has two minor formatting errors:
> 
> 1) The "Fixes:" tag should go in the same block as Signed-off-by/Cc at the end,
>    no need to do an empty line between.
> 
> 2) The Fixes line should be in one line, even if it's longer than 80 characters.
> 
> I'll hold off sending a pull request out if you do a quick respin to fix that.
> If I merge it as-is we'll get a nag note from Stephen Rothwell. :)
> 

Respun and resent. Sorry for that error.

thanks,

Dinh