RE: [PATCH 1/2] arm64: dts: renesas: r9a07g044: Fix external clk node names

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

 



Hi Krzysztof Kozlowski,

Thanks for the feedback.

> Subject: Re: [PATCH 1/2] arm64: dts: renesas: r9a07g044: Fix external clk
> node names
> 
> On 23/04/2022 16:06, Biju Das wrote:
> > Fix audio clk node names with "_" -> "-" and add suffix '-clk' for can
> > and extal clks.
> >
> > Signed-off-by: Biju Das <biju.das.jz@xxxxxxxxxxxxxx>
> > ---
> >  arch/arm64/boot/dts/renesas/r9a07g044.dtsi | 8 ++++----
> >  1 file changed, 4 insertions(+), 4 deletions(-)
> >
> > diff --git a/arch/arm64/boot/dts/renesas/r9a07g044.dtsi
> > b/arch/arm64/boot/dts/renesas/r9a07g044.dtsi
> > index 19287cccb1f0..4f9a84d7af08 100644
> > --- a/arch/arm64/boot/dts/renesas/r9a07g044.dtsi
> > +++ b/arch/arm64/boot/dts/renesas/r9a07g044.dtsi
> > @@ -13,14 +13,14 @@ / {
> >  	#address-cells = <2>;
> >  	#size-cells = <2>;
> >
> > -	audio_clk1: audio_clk1 {
> > +	audio_clk1: audio-clk1 {
> 
> How about in such case keeping suffix "clk" everywhere and moving the index
> (1/2) to the first part? This way one day maybe schema will match the
> clocks.

Just a question,

Your suggestion is "audio_clk1: audio-clk1" -> "audio_clk1: audio-clk"

In that case, If you plan to drop the label in future, how will you differentiate between
audio-clk1 and audio-clk2 with just node names?

Cheers,
Biju






[Index of Archives]     [Device Tree Compilter]     [Device Tree Spec]     [Linux Driver Backports]     [Video for Linux]     [Linux USB Devel]     [Linux PCI Devel]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [XFree86]     [Yosemite Backpacking]


  Powered by Linux