Message ID | 20220519074112.25600-3-ae40515@yahoo.com.tw (mailing list archive) |
---|---|
State | Superseded |
Headers | show |
Series | Add support for Xiaomi Mi 5s Plus | expand |
On 19/05/2022 09:41, Alec Su wrote: > Document Xiaomi Mi 5s Plus (xiaomi-natrium) smartphone which is based on > Snapdragon 821 SoC. > > Signed-off-by: Alec Su <ae40515@yahoo.com.tw> Acked-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org> Best regards, Krzysztof
On Thu, 19 May 2022 07:41:12 +0000, Alec Su wrote: > Document Xiaomi Mi 5s Plus (xiaomi-natrium) smartphone which is based on > Snapdragon 821 SoC. > > Signed-off-by: Alec Su <ae40515@yahoo.com.tw> > --- > Documentation/devicetree/bindings/arm/qcom.yaml | 1 + > 1 file changed, 1 insertion(+) > Running 'make dtbs_check' with the schema in this patch gives the following warnings. Consider if they are expected or the schema is incorrect. These may not be new warnings. Note that it is not yet a requirement to have 0 warnings for dtbs_check. This will change in the future. Full log is available here: https://patchwork.ozlabs.org/patch/ /: compatible: 'oneOf' conditional failed, one must be fixed: arch/arm64/boot/dts/qcom/msm8916-alcatel-idol347.dtb arch/arm64/boot/dts/qcom/msm8916-asus-z00l.dtb arch/arm64/boot/dts/qcom/msm8916-huawei-g7.dtb arch/arm64/boot/dts/qcom/msm8916-longcheer-l8150.dtb arch/arm64/boot/dts/qcom/msm8916-longcheer-l8910.dtb arch/arm64/boot/dts/qcom/msm8916-mtp.dtb arch/arm64/boot/dts/qcom/msm8916-samsung-j5.dtb arch/arm64/boot/dts/qcom/msm8916-samsung-serranove.dtb arch/arm64/boot/dts/qcom/msm8916-wingtech-wt88047.dtb arch/arm64/boot/dts/qcom/msm8994-huawei-angler-rev-101.dtb arch/arm64/boot/dts/qcom/msm8994-msft-lumia-octagon-cityman.dtb arch/arm64/boot/dts/qcom/msm8994-sony-xperia-kitakami-ivy.dtb arch/arm64/boot/dts/qcom/msm8994-sony-xperia-kitakami-karin.dtb arch/arm64/boot/dts/qcom/msm8994-sony-xperia-kitakami-satsuki.dtb arch/arm64/boot/dts/qcom/msm8994-sony-xperia-kitakami-sumire.dtb arch/arm64/boot/dts/qcom/msm8994-sony-xperia-kitakami-suzuran.dtb arch/arm64/boot/dts/qcom/sc7180-trogdor-coachz-r1.dtb arch/arm64/boot/dts/qcom/sc7180-trogdor-coachz-r1-lte.dtb arch/arm64/boot/dts/qcom/sc7180-trogdor-coachz-r3.dtb arch/arm64/boot/dts/qcom/sc7180-trogdor-coachz-r3-lte.dtb arch/arm64/boot/dts/qcom/sc7180-trogdor-homestar-r2.dtb arch/arm64/boot/dts/qcom/sc7180-trogdor-homestar-r3.dtb arch/arm64/boot/dts/qcom/sc7180-trogdor-homestar-r4.dtb arch/arm64/boot/dts/qcom/sc7180-trogdor-lazor-limozeen-nots-r4.dtb arch/arm64/boot/dts/qcom/sc7180-trogdor-lazor-limozeen-nots-r5.dtb arch/arm64/boot/dts/qcom/sc7180-trogdor-lazor-limozeen-nots-r9.dtb arch/arm64/boot/dts/qcom/sc7180-trogdor-lazor-limozeen-r4.dtb arch/arm64/boot/dts/qcom/sc7180-trogdor-lazor-limozeen-r9.dtb arch/arm64/boot/dts/qcom/sc7180-trogdor-lazor-r0.dtb arch/arm64/boot/dts/qcom/sc7180-trogdor-lazor-r1.dtb arch/arm64/boot/dts/qcom/sc7180-trogdor-lazor-r1-kb.dtb arch/arm64/boot/dts/qcom/sc7180-trogdor-lazor-r1-lte.dtb arch/arm64/boot/dts/qcom/sc7180-trogdor-lazor-r3.dtb arch/arm64/boot/dts/qcom/sc7180-trogdor-lazor-r3-kb.dtb arch/arm64/boot/dts/qcom/sc7180-trogdor-lazor-r3-lte.dtb arch/arm64/boot/dts/qcom/sc7180-trogdor-lazor-r9.dtb arch/arm64/boot/dts/qcom/sc7180-trogdor-lazor-r9-kb.dtb arch/arm64/boot/dts/qcom/sc7180-trogdor-lazor-r9-lte.dtb arch/arm64/boot/dts/qcom/sc7180-trogdor-pompom-r1.dtb arch/arm64/boot/dts/qcom/sc7180-trogdor-pompom-r1-lte.dtb arch/arm64/boot/dts/qcom/sc7180-trogdor-pompom-r2.dtb arch/arm64/boot/dts/qcom/sc7180-trogdor-pompom-r2-lte.dtb arch/arm64/boot/dts/qcom/sc7180-trogdor-pompom-r3.dtb arch/arm64/boot/dts/qcom/sc7180-trogdor-pompom-r3-lte.dtb arch/arm64/boot/dts/qcom/sc7180-trogdor-r1.dtb arch/arm64/boot/dts/qcom/sc7180-trogdor-r1-lte.dtb arch/arm64/boot/dts/qcom/sc7280-crd.dtb arch/arm64/boot/dts/qcom/sc7280-herobrine-herobrine-r0.dtb arch/arm64/boot/dts/qcom/sc7280-herobrine-herobrine-r1.dtb arch/arm64/boot/dts/qcom/sc7280-idp2.dtb arch/arm64/boot/dts/qcom/sc7280-idp.dtb arch/arm64/boot/dts/qcom/sm8150-hdk.dtb arch/arm64/boot/dts/qcom/sm8150-microsoft-surface-duo.dtb arch/arm64/boot/dts/qcom/sm8150-sony-xperia-kumano-bahamut.dtb arch/arm64/boot/dts/qcom/sm8150-sony-xperia-kumano-griffin.dtb arch/arm64/boot/dts/qcom/sm8250-hdk.dtb arch/arm64/boot/dts/qcom/sm8250-sony-xperia-edo-pdx203.dtb arch/arm64/boot/dts/qcom/sm8250-sony-xperia-edo-pdx206.dtb arch/arm64/boot/dts/qcom/sm8350-microsoft-surface-duo2.dtb arch/arm64/boot/dts/qcom/sm8350-sony-xperia-sagami-pdx214.dtb arch/arm64/boot/dts/qcom/sm8350-sony-xperia-sagami-pdx215.dtb arch/arm/boot/dts/qcom-msm8916-samsung-serranove.dtb
On 2022/5/19 21:05, Rob Herring wrote: > On Thu, 19 May 2022 07:41:12 +0000, Alec Su wrote: >> Document Xiaomi Mi 5s Plus (xiaomi-natrium) smartphone which is based on >> Snapdragon 821 SoC. >> >> Signed-off-by: Alec Su <ae40515@yahoo.com.tw> >> --- >> Documentation/devicetree/bindings/arm/qcom.yaml | 1 + >> 1 file changed, 1 insertion(+) >> > > Running 'make dtbs_check' with the schema in this patch gives the > following warnings. Consider if they are expected or the schema is > incorrect. These may not be new warnings. > > Note that it is not yet a requirement to have 0 warnings for dtbs_check. > This will change in the future. > > Full log is available here: https://patchwork.ozlabs.org/patch/ > > > /: compatible: 'oneOf' conditional failed, one must be fixed: > arch/arm64/boot/dts/qcom/msm8916-alcatel-idol347.dtb > arch/arm64/boot/dts/qcom/msm8916-asus-z00l.dtb > arch/arm64/boot/dts/qcom/msm8916-huawei-g7.dtb > arch/arm64/boot/dts/qcom/msm8916-longcheer-l8150.dtb > arch/arm64/boot/dts/qcom/msm8916-longcheer-l8910.dtb > arch/arm64/boot/dts/qcom/msm8916-mtp.dtb > arch/arm64/boot/dts/qcom/msm8916-samsung-j5.dtb > arch/arm64/boot/dts/qcom/msm8916-samsung-serranove.dtb > arch/arm64/boot/dts/qcom/msm8916-wingtech-wt88047.dtb > arch/arm64/boot/dts/qcom/msm8994-huawei-angler-rev-101.dtb > arch/arm64/boot/dts/qcom/msm8994-msft-lumia-octagon-cityman.dtb > arch/arm64/boot/dts/qcom/msm8994-sony-xperia-kitakami-ivy.dtb > arch/arm64/boot/dts/qcom/msm8994-sony-xperia-kitakami-karin.dtb > arch/arm64/boot/dts/qcom/msm8994-sony-xperia-kitakami-satsuki.dtb > arch/arm64/boot/dts/qcom/msm8994-sony-xperia-kitakami-sumire.dtb > arch/arm64/boot/dts/qcom/msm8994-sony-xperia-kitakami-suzuran.dtb > arch/arm64/boot/dts/qcom/sc7180-trogdor-coachz-r1.dtb > arch/arm64/boot/dts/qcom/sc7180-trogdor-coachz-r1-lte.dtb > arch/arm64/boot/dts/qcom/sc7180-trogdor-coachz-r3.dtb > arch/arm64/boot/dts/qcom/sc7180-trogdor-coachz-r3-lte.dtb > arch/arm64/boot/dts/qcom/sc7180-trogdor-homestar-r2.dtb > arch/arm64/boot/dts/qcom/sc7180-trogdor-homestar-r3.dtb > arch/arm64/boot/dts/qcom/sc7180-trogdor-homestar-r4.dtb > arch/arm64/boot/dts/qcom/sc7180-trogdor-lazor-limozeen-nots-r4.dtb > arch/arm64/boot/dts/qcom/sc7180-trogdor-lazor-limozeen-nots-r5.dtb > arch/arm64/boot/dts/qcom/sc7180-trogdor-lazor-limozeen-nots-r9.dtb > arch/arm64/boot/dts/qcom/sc7180-trogdor-lazor-limozeen-r4.dtb > arch/arm64/boot/dts/qcom/sc7180-trogdor-lazor-limozeen-r9.dtb > arch/arm64/boot/dts/qcom/sc7180-trogdor-lazor-r0.dtb > arch/arm64/boot/dts/qcom/sc7180-trogdor-lazor-r1.dtb > arch/arm64/boot/dts/qcom/sc7180-trogdor-lazor-r1-kb.dtb > arch/arm64/boot/dts/qcom/sc7180-trogdor-lazor-r1-lte.dtb > arch/arm64/boot/dts/qcom/sc7180-trogdor-lazor-r3.dtb > arch/arm64/boot/dts/qcom/sc7180-trogdor-lazor-r3-kb.dtb > arch/arm64/boot/dts/qcom/sc7180-trogdor-lazor-r3-lte.dtb > arch/arm64/boot/dts/qcom/sc7180-trogdor-lazor-r9.dtb > arch/arm64/boot/dts/qcom/sc7180-trogdor-lazor-r9-kb.dtb > arch/arm64/boot/dts/qcom/sc7180-trogdor-lazor-r9-lte.dtb > arch/arm64/boot/dts/qcom/sc7180-trogdor-pompom-r1.dtb > arch/arm64/boot/dts/qcom/sc7180-trogdor-pompom-r1-lte.dtb > arch/arm64/boot/dts/qcom/sc7180-trogdor-pompom-r2.dtb > arch/arm64/boot/dts/qcom/sc7180-trogdor-pompom-r2-lte.dtb > arch/arm64/boot/dts/qcom/sc7180-trogdor-pompom-r3.dtb > arch/arm64/boot/dts/qcom/sc7180-trogdor-pompom-r3-lte.dtb > arch/arm64/boot/dts/qcom/sc7180-trogdor-r1.dtb > arch/arm64/boot/dts/qcom/sc7180-trogdor-r1-lte.dtb > arch/arm64/boot/dts/qcom/sc7280-crd.dtb > arch/arm64/boot/dts/qcom/sc7280-herobrine-herobrine-r0.dtb > arch/arm64/boot/dts/qcom/sc7280-herobrine-herobrine-r1.dtb > arch/arm64/boot/dts/qcom/sc7280-idp2.dtb > arch/arm64/boot/dts/qcom/sc7280-idp.dtb > arch/arm64/boot/dts/qcom/sm8150-hdk.dtb > arch/arm64/boot/dts/qcom/sm8150-microsoft-surface-duo.dtb > arch/arm64/boot/dts/qcom/sm8150-sony-xperia-kumano-bahamut.dtb > arch/arm64/boot/dts/qcom/sm8150-sony-xperia-kumano-griffin.dtb > arch/arm64/boot/dts/qcom/sm8250-hdk.dtb > arch/arm64/boot/dts/qcom/sm8250-sony-xperia-edo-pdx203.dtb > arch/arm64/boot/dts/qcom/sm8250-sony-xperia-edo-pdx206.dtb > arch/arm64/boot/dts/qcom/sm8350-microsoft-surface-duo2.dtb > arch/arm64/boot/dts/qcom/sm8350-sony-xperia-sagami-pdx214.dtb > arch/arm64/boot/dts/qcom/sm8350-sony-xperia-sagami-pdx215.dtb > arch/arm/boot/dts/qcom-msm8916-samsung-serranove.dtb > I checked the full log. These warnings seem not the new warning caused by the patch series. Look like these device trees are trying to match with each of the compatible strings in qcom.yaml.
On 20/05/2022 03:23, Alec Su wrote: > On 2022/5/19 21:05, Rob Herring wrote: >> On Thu, 19 May 2022 07:41:12 +0000, Alec Su wrote: >>> Document Xiaomi Mi 5s Plus (xiaomi-natrium) smartphone which is based on >>> Snapdragon 821 SoC. > > I checked the full log. These warnings seem not the new warning caused > by the patch series. Look like these device trees are trying to match > with each of the compatible strings in qcom.yaml. Yes, I'll fix them. Best regards, Krzysztof
diff --git a/Documentation/devicetree/bindings/arm/qcom.yaml b/Documentation/devicetree/bindings/arm/qcom.yaml index 5c06d1bfc046..7ee852079d13 100644 --- a/Documentation/devicetree/bindings/arm/qcom.yaml +++ b/Documentation/devicetree/bindings/arm/qcom.yaml @@ -190,6 +190,7 @@ properties: - sony,kagura-row - sony,keyaki-row - xiaomi,gemini + - xiaomi,natrium - xiaomi,scorpio - const: qcom,msm8996
Document Xiaomi Mi 5s Plus (xiaomi-natrium) smartphone which is based on Snapdragon 821 SoC. Signed-off-by: Alec Su <ae40515@yahoo.com.tw> --- Documentation/devicetree/bindings/arm/qcom.yaml | 1 + 1 file changed, 1 insertion(+)