On Wed, Nov 20, 2024 at 05:59:21PM +0100, Krzysztof Kozlowski wrote:
> On 19/11/2024 11:23, Xin Liu wrote:
> > Add watchdog clock on the Qualcomm QCS8300 Ride platform.
> >
> > Signed-off-by: Xin Liu <quic_liuxin@xxxxxxxxxxx>
> > ---
> > arch/arm64/boot/dts/qcom/qcs8300-ride.dts | 4 ++++
> > 1 file changed, 4 insertions(+)
> >
> > diff --git a/arch/arm64/boot/dts/qcom/qcs8300-ride.dts b/arch/arm64/boot/dts/qcom/qcs8300-ride.dts
> > index 7eed19a694c3..3024338bcfbc 100644
> > --- a/arch/arm64/boot/dts/qcom/qcs8300-ride.dts
> > +++ b/arch/arm64/boot/dts/qcom/qcs8300-ride.dts
> > @@ -265,3 +265,7 @@ &ufs_mem_phy {
> > vdda-pll-supply = <&vreg_l5a>;
> > status = "okay";
> > };
> > +
> > +&watchdog {
> > + clocks = <&sleep_clk>;
> > +};
> > \ No newline at end of file
>
> Look, your patches have errors...
But it's not an error, it is a part of the patch format, telling that
there was no \n on the last line. We might prefer having files with tail
\n, but I'd not call it an error.
Nevertheless, I agree, it should be fixed.
--
With best wishes
Dmitry
