* Re: [PATCH] dt-bindings: cpufreq: cpufreq-qcom-hw: Add SM8550 compatible
2022-11-16 11:50 [PATCH] dt-bindings: cpufreq: cpufreq-qcom-hw: Add SM8550 compatible Abel Vesa
@ 2022-11-16 13:04 ` Krzysztof Kozlowski
2022-11-16 15:29 ` Rob Herring
1 sibling, 0 replies; 3+ messages in thread
From: Krzysztof Kozlowski @ 2022-11-16 13:04 UTC (permalink / raw)
To: Abel Vesa, Rafael J. Wysocki, Viresh Kumar, Rob Herring
Cc: Linux Kernel Mailing List, devicetree, linux-arm-msm, linux-pm
On 16/11/2022 12:50, Abel Vesa wrote:
> Add compatible for EPSS CPUFREQ-HW on SM8550.
> Also document the interrupts.
None of other variants had interrupts so far, so you are adding it for
new SoC, right? Or documenting existing usage? If the first, you need
allOf:if:then.
>
> Signed-off-by: Abel Vesa <abel.vesa@linaro.org>
> ---
> .../devicetree/bindings/cpufreq/cpufreq-qcom-hw.yaml | 7 +++++++
> 1 file changed, 7 insertions(+)
>
Best regards,
Krzysztof
^ permalink raw reply [flat|nested] 3+ messages in thread
* Re: [PATCH] dt-bindings: cpufreq: cpufreq-qcom-hw: Add SM8550 compatible
2022-11-16 11:50 [PATCH] dt-bindings: cpufreq: cpufreq-qcom-hw: Add SM8550 compatible Abel Vesa
2022-11-16 13:04 ` Krzysztof Kozlowski
@ 2022-11-16 15:29 ` Rob Herring
1 sibling, 0 replies; 3+ messages in thread
From: Rob Herring @ 2022-11-16 15:29 UTC (permalink / raw)
To: Abel Vesa
Cc: Rafael J. Wysocki, devicetree, Linux Kernel Mailing List,
linux-arm-msm, linux-pm, Krzysztof Kozlowski, Viresh Kumar
On Wed, 16 Nov 2022 13:50:46 +0200, Abel Vesa wrote:
> Add compatible for EPSS CPUFREQ-HW on SM8550.
> Also document the interrupts.
>
> Signed-off-by: Abel Vesa <abel.vesa@linaro.org>
> ---
> .../devicetree/bindings/cpufreq/cpufreq-qcom-hw.yaml | 7 +++++++
> 1 file changed, 7 insertions(+)
>
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/
cpufreq@17d43000: interrupts-extended: [[215, 0], [216, 0]] is too short
arch/arm64/boot/dts/qcom/sdm845-sony-xperia-tama-akari.dtb
arch/arm64/boot/dts/qcom/sdm845-sony-xperia-tama-akatsuki.dtb
arch/arm64/boot/dts/qcom/sdm845-sony-xperia-tama-apollo.dtb
cpufreq@17d43000: interrupts-extended: [[221, 0], [222, 0]] is too short
arch/arm64/boot/dts/qcom/sdm845-lg-judyp.dtb
cpufreq@17d43000: interrupts-extended: [[223, 0], [224, 0]] is too short
arch/arm64/boot/dts/qcom/sdm845-lg-judyln.dtb
cpufreq@17d43000: interrupts-extended: [[226, 0], [227, 0]] is too short
arch/arm64/boot/dts/qcom/sdm850-samsung-w737.dtb
cpufreq@17d43000: interrupts-extended: [[227, 0], [228, 0]] is too short
arch/arm64/boot/dts/qcom/sdm845-mtp.dtb
cpufreq@17d43000: interrupts-extended: [[228, 0], [229, 0]] is too short
arch/arm64/boot/dts/qcom/sdm845-oneplus-enchilada.dtb
arch/arm64/boot/dts/qcom/sdm845-oneplus-fajita.dtb
cpufreq@17d43000: interrupts-extended: [[229, 0], [230, 0]] is too short
arch/arm64/boot/dts/qcom/sdm845-xiaomi-polaris.dtb
cpufreq@17d43000: interrupts-extended: [[230, 0], [231, 0]] is too short
arch/arm64/boot/dts/qcom/sdm845-xiaomi-beryllium.dtb
cpufreq@17d43000: interrupts-extended: [[231, 0], [232, 0]] is too short
arch/arm64/boot/dts/qcom/sdm845-shift-axolotl.dtb
cpufreq@17d43000: interrupts-extended: [[234, 0], [235, 0]] is too short
arch/arm64/boot/dts/qcom/sdm845-cheza-r1.dtb
arch/arm64/boot/dts/qcom/sdm845-cheza-r2.dtb
arch/arm64/boot/dts/qcom/sdm845-cheza-r3.dtb
cpufreq@17d43000: interrupts-extended: [[235, 0], [236, 0]] is too short
arch/arm64/boot/dts/qcom/sdm850-lenovo-yoga-c630.dtb
cpufreq@17d43000: interrupts-extended: [[256, 0], [257, 0]] is too short
arch/arm64/boot/dts/qcom/sdm845-db845c.dtb
cpufreq@17d91000: compatible: 'oneOf' conditional failed, one must be fixed:
arch/arm64/boot/dts/qcom/sm8450-hdk.dtb
arch/arm64/boot/dts/qcom/sm8450-qrd.dtb
arch/arm64/boot/dts/qcom/sm8450-sony-xperia-nagara-pdx223.dtb
cpufreq@18591000: compatible: 'oneOf' conditional failed, one must be fixed:
arch/arm64/boot/dts/qcom/sa8295p-adp.dtb
arch/arm64/boot/dts/qcom/sc7280-crd-r3.dtb
arch/arm64/boot/dts/qcom/sc7280-herobrine-crd.dtb
arch/arm64/boot/dts/qcom/sc7280-herobrine-evoker-r0.dtb
arch/arm64/boot/dts/qcom/sc7280-herobrine-herobrine-r1.dtb
arch/arm64/boot/dts/qcom/sc7280-herobrine-villager-r0.dtb
arch/arm64/boot/dts/qcom/sc7280-herobrine-villager-r1.dtb
arch/arm64/boot/dts/qcom/sc7280-herobrine-villager-r1-lte.dtb
arch/arm64/boot/dts/qcom/sc7280-idp2.dtb
arch/arm64/boot/dts/qcom/sc7280-idp.dtb
arch/arm64/boot/dts/qcom/sc8280xp-crd.dtb
arch/arm64/boot/dts/qcom/sc8280xp-lenovo-thinkpad-x13s.dtb
arch/arm64/boot/dts/qcom/sm8350-hdk.dtb
arch/arm64/boot/dts/qcom/sm8350-microsoft-surface-duo2.dtb
arch/arm64/boot/dts/qcom/sm8350-mtp.dtb
arch/arm64/boot/dts/qcom/sm8350-sony-xperia-sagami-pdx214.dtb
arch/arm64/boot/dts/qcom/sm8350-sony-xperia-sagami-pdx215.dtb
^ permalink raw reply [flat|nested] 3+ messages in thread