On 16/11/2022 15:56, Luca Weiss wrote:
> From: Julian Braha <julianbraha@xxxxxxxxx>
>
> Add device tree for the Motorola Moto G6 (ali) smartphone. This device
> is based on Snapdragon 450 (sdm450) SoC which is a variant of MSM8953.
>
> Signed-off-by: Julian Braha <julianbraha@xxxxxxxxx>
Missing SoB.
Please run scripts/checkpatch.pl and fix reported warnings.
> ---
> arch/arm64/boot/dts/qcom/Makefile | 1 +
> .../boot/dts/qcom/sdm450-motorola-ali.dts | 256 ++++++++++++++++++
> 2 files changed, 257 insertions(+)
> create mode 100644 arch/arm64/boot/dts/qcom/sdm450-motorola-ali.dts
>
> diff --git a/arch/arm64/boot/dts/qcom/Makefile b/arch/arm64/boot/dts/qcom/Makefile
> index d7669a7cee9f..a0b537414593 100644
> --- a/arch/arm64/boot/dts/qcom/Makefile
> +++ b/arch/arm64/boot/dts/qcom/Makefile
> @@ -115,6 +115,7 @@ dtb-$(CONFIG_ARCH_QCOM) += sc7280-crd-r3.dtb
> dtb-$(CONFIG_ARCH_QCOM) += sc8280xp-crd.dtb
> dtb-$(CONFIG_ARCH_QCOM) += sc8280xp-lenovo-thinkpad-x13s.dtb
> dtb-$(CONFIG_ARCH_QCOM) += sda660-inforce-ifc6560.dtb
> +dtb-$(CONFIG_ARCH_QCOM) += sdm450-motorola-ali.dtb
> dtb-$(CONFIG_ARCH_QCOM) += sdm630-sony-xperia-ganges-kirin.dtb
> dtb-$(CONFIG_ARCH_QCOM) += sdm630-sony-xperia-nile-discovery.dtb
> dtb-$(CONFIG_ARCH_QCOM) += sdm630-sony-xperia-nile-pioneer.dtb
> diff --git a/arch/arm64/boot/dts/qcom/sdm450-motorola-ali.dts b/arch/arm64/boot/dts/qcom/sdm450-motorola-ali.dts
> new file mode 100644
> index 000000000000..62874306fc24
> --- /dev/null
> +++ b/arch/arm64/boot/dts/qcom/sdm450-motorola-ali.dts
> @@ -0,0 +1,256 @@
> +// SPDX-License-Identifier: GPL-2.0-only
> +/*
> + * Copyright (c) 2020, The Linux Foundation. All rights reserved.
> + */
> +/dts-v1/;
> +
> +#include "msm8953.dtsi"
> +#include "pm8953.dtsi"
> +#include "pmi8950.dtsi"
> +
> +/ {
> + model = "Motorola Moto G6";
> + compatible = "motorola,ali", "qcom,sdm450";
> + chassis-type = "handset";
> + qcom,msm-id = <338 0>;
> + qcom,board-id = <0x41 0xb1a0>,
> + <0x42 0xb1a0>,
> + <0x43 0xc100>,
> + <0x42 0xb1b0>,
> + <0x43 0xc200>,
> + <0x44 0xc200>,
> + <0x42 0xb200>,
> + <0x43 0xb200>;
Both properties are not allowed, unless exception list is extended.
Does not look like you tested the DTS against bindings. Please run `make
dtbs_check` (see Documentation/devicetree/bindings/writing-schema.rst
for instructions).
Best regards,
Krzysztof
