lkml.org 
[lkml]   [2022]   [May]   [31]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
Date
SubjectRe: [PATCH 1/3] ARM: dts: aspeed: Adding Jabil Rubypass BMC
From
On 31/05/2022 03:10, David Wang wrote:
> The initial introduction of the jabil server with AST2600 BMC SoC.
>
> ---
>
> v5
> - Compatible value chamged to "jabil,rbp-bmc".
>
> v4
> - Add document the new compatible.
> - Delete redundant blank lines.
> - Add a new line at end of file.
>
> v3
> - Adjustment entries are ordered by name.
> - Remove the bootargs property and modify the stdout-path property.
> - Group multiple LED devices into a group.
> - Modify the "gpio3_71" node name format.
> - Delete unneeded blank line at the end.
>
> v2
> - Disable empty i2c bus.
> - Remove gfx node because aspeed-g6.dtsi isn't supported.
> - Modify the led format and add some properties.
> - Fix wrong indentation.
> - Add stdout property in chosen node.
> - Rename temp to temperature-sensor in i2c bus 0.

Thanks for the changes.

> +&lpc_snoop {
> + snoop-ports = <0x80>;
> + status = "okay";
> +};
> +

You still have here additional blank line about which I already
commented. Then you removed both blank lines - this one and the usual
trailing one.

I mean... really... Just remove the unneeded one, leaving the proper
patch like every other file in Linux kernel.

Best regards,
Krzysztof

\
 
 \ /
  Last update: 2022-05-31 12:24    [W:0.870 / U:0.456 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site