Re: [PATCH] arm64: dts: qcom: pm8916: Fix pwm declaration

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On 23/08/2022 15:12, Krzysztof Kozlowski wrote:
> On 22/08/2022 15:03, Bryan O'Donoghue wrote:
>> We need to define pwm@bc000 to stop dtbs_check from making the following
>> complaint, text pruned.
>>
>> pmic@1: 'pwm' does not match any of the regexes:  'pwm@[0-9a-f]+$'
>>
>> Fixes: e79a1385ab74 ("arm64: dts: qcom: Add LPG to pm8916, pm8994, pmi8994 and pmi8998")
>> Signed-off-by: Bryan O'Donoghue <bryan.odonoghue@xxxxxxxxxx>
>> ---
>>  arch/arm64/boot/dts/qcom/pm8916.dtsi | 2 +-
>>  1 file changed, 1 insertion(+), 1 deletion(-)
>>
>> diff --git a/arch/arm64/boot/dts/qcom/pm8916.dtsi b/arch/arm64/boot/dts/qcom/pm8916.dtsi
>> index 606c2a6d1f0fc..d6922379729cb 100644
>> --- a/arch/arm64/boot/dts/qcom/pm8916.dtsi
>> +++ b/arch/arm64/boot/dts/qcom/pm8916.dtsi
>> @@ -124,7 +124,7 @@ pm8916_1: pmic@1 {
>>  		#address-cells = <1>;
>>  		#size-cells = <0>;
>>  
>> -		pm8916_pwm: pwm {
>> +		pm8916_pwm: pwm@bc00 {
>>  			compatible = "qcom,pm8916-pwm";
> 
> This does not look like proper fix.
> 1. It requires a reg.
> 2. reg is not allowed by pwm bindings.
> 
> See also other wrong commit:
> https://lore.kernel.org/all/20220719205058.1004942-1-bhupesh.sharma@xxxxxxxxxx/
> 

and this:
https://lore.kernel.org/all/20220721195502.1525214-1-bhupesh.sharma@xxxxxxxxxx/


Best regards,
Krzysztof



[Index of Archives]     [Linux ARM Kernel]     [Linux ARM]     [Linux Omap]     [Fedora ARM]     [Linux for Sparc]     [IETF Annouce]     [Security]     [Bugtraq]     [Linux MIPS]     [ECOS]     [Asterisk Internet PBX]     [Linux API]

  Powered by Linux