On 20/07/2022 10:52, Quan Nguyen wrote:
> The Mt. Mitchell BMC is an ASPEED AST2600-based BMC for the Mt. Mitchell
> hardware reference platform with AmpereOne(TM) processor.
>
> Signed-off-by: Quan Nguyen <quan@xxxxxxxxxxxxxxxxxxxxxx>
> Signed-off-by: Phong Vo <phong@xxxxxxxxxxxxxxxxxxxxxx>
> Signed-off-by: Thang Q. Nguyen <thang@xxxxxxxxxxxxxxxxxxxxxx>
> ---
> v2 :
> + Remove bootargs [Krzysztof]
> + Fix gpio-keys nodes name to conform with device tree binding
> documents [Krzysztof]
> + Fix some nodes to use generic name [Krzysztof]
> + Remove unnecessary blank line [Krzysztof]
> + Fix typo "LTC" to "LLC" in license info and corrected license
> info to GPL-2.0-only
>
> arch/arm/boot/dts/Makefile | 1 +
> .../boot/dts/aspeed-bmc-ampere-mtmitchell.dts | 577 ++++++++++++++++++
> 2 files changed, 578 insertions(+)
> create mode 100644 arch/arm/boot/dts/aspeed-bmc-ampere-mtmitchell.dts
>
(...)
> +
> +&i2c3 {
> + status = "okay";
> +};
> +
> +&i2c4 {
> + status = "okay";
> +
> + adc_i2c: adc-i2c@16 {
Node name: just "adc"
> + compatible = "lltc,ltc2497";
> + reg = <0x16>;
> + vref-supply = <&voltage_mon_reg>;
> + #io-channel-cells = <1>;
> + status = "okay";
Isn't this new node? Why do you need status here?
> + };
> +
> + eeprom@50 {
> + compatible = "atmel,24c64";
> + reg = <0x50>;
> + pagesize = <32>;
> + };
> +
(...)
> +
> +&gpio1 {
> + gpio-line-names =
> + /*18A0-18A7*/ "","","","","","","","",
> + /*18B0-18B7*/ "","","","","emmc-rst-n","","s0-soc-pgood","",
> + /*18C0-18C7*/ "uart1-mode0","uart1-mode1","uart2-mode0","uart2-mode1",
> + "uart3-mode0","uart3-mode1","uart4-mode0","uart4-mode1",
> + /*18D0-18D7*/ "","","","","","","","",
> + /*18E0-18E3*/ "","","","";
> +};
> +
You have a trailing line error.
Best regards,
Krzysztof
