RE: [PATCH v3 1/4] dt-bindings: clock: versaclock3: Document clock-output-names

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

 



Hi Stephen Boyd,

> Subject: Re: [PATCH v3 1/4] dt-bindings: clock: versaclock3: Document
> clock-output-names
> 
> Quoting Biju Das (2023-08-17 07:22:08)
> > Document clock-output-names property and fix the "assigned-clock-rates"
> > for each clock output in the example based on Table 3. ("Output
> > Source") in the 5P35023 datasheet(ie: {REF,SE1,SE2,SE3,DIFF1,DIFF2}).
> >
> > While at it, replace clocks phandle in the example from x1_x2->x1 as
> > X2 is a different 32768 kHz crystal.
> >
> > Suggested-by: Geert Uytterhoeven <geert+renesas@xxxxxxxxx>
> > Closes:
> > Fixes: a03d23f860eb ("dt-bindings: clock: Add Renesas versa3 clock
> > generator bindings")
> > Signed-off-by: Biju Das <biju.das.jz@xxxxxxxxxxxxxx>
> > ---
> 
> Applied to clk-next

As per Geert's suggestion [1], I am about to send a patch
for dropping "clock-output-names", as there is no validation for it and people can get it wrong.

Is it ok,if I send a patch dropping clock-output-names? Please let me know your opinion on this.

[1] https://patchwork.kernel.org/project/linux-renesas-soc/patch/20230817090810.203900-2-biju.das.jz@xxxxxxxxxxxxxx/

Cheers,
Biju




[Index of Archives]     [Linux Samsung SOC]     [Linux Wireless]     [Linux Kernel]     [ATH6KL]     [Linux Bluetooth]     [Linux Netdev]     [Kernel Newbies]     [IDE]     [Security]     [Git]     [Netfilter]     [Bugtraq]     [Yosemite News]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Linux ATA RAID]     [Samba]     [Device Mapper]

  Powered by Linux