Re: [PATCH 7/7] arm64: dts: qcom: sm8250: add watchdog device

[Sai Prakash Ranjan <saiprakash.ranjan@codeaurora.org>]

On 2020-06-04 17:05, Manivannan Sadhasivam wrote:
> On Thu, Jun 04, 2020 at 04:37:01PM +0530, Sai Prakash Ranjan wrote:
>> On 2020-06-04 06:13, Dmitry Baryshkov wrote:
>> > Add on-SoC watchdog device node.
>> >
>> > Signed-off-by: Dmitry Baryshkov <dmitry.baryshkov@linaro.org>
>> > ---
>> >  arch/arm64/boot/dts/qcom/sm8250.dtsi | 6 ++++++
>> >  1 file changed, 6 insertions(+)
>> >
>> > diff --git a/arch/arm64/boot/dts/qcom/sm8250.dtsi
>> > b/arch/arm64/boot/dts/qcom/sm8250.dtsi
>> > index 972d8e04c8a2..f1641c6fe203 100644
>> > --- a/arch/arm64/boot/dts/qcom/sm8250.dtsi
>> > +++ b/arch/arm64/boot/dts/qcom/sm8250.dtsi
>> > @@ -1662,6 +1662,12 @@ config {
>> >  			};
>> >  		};
>> >
>> > +		watchdog@17c10000 {
>> > +			compatible = "qcom,apss-wdt-sm8250", "qcom,kpss-wdt";
>> 
>> Need to add this compatible to bindings.
>> 
> 
> I did look into this but the binding says,
> "compatible : shall contain only one of the following"
> 
> So clearly the fallback is not going to work and there is no need to 
> add a
> dedicated compatible in the driver. The binding is not in the YAML 
> format to
> be validated but still... Other option is to convert the binding to 
> YAML and
> make the compatibles listed as 'enum' and 'const' elements 
> appropriately.
> 

I already converted the bindings to YAML and added the missing 
compatibles for
other SoCs[1] . I thought it was already merged since Rob has already 
reviewed
them, but seems like it was missed. Bjorn, can you please take it?

[1] https://lore.kernel.org/patchwork/cover/1192757/

Thanks,
Sai

-- 
QUALCOMM INDIA, on behalf of Qualcomm Innovation Center, Inc. is a 
member
of Code Aurora Forum, hosted by The Linux Foundation