Re: [PATCH v1 10/12] ARM: dts: microchip: sam9x60: Add clock properties to OTPC

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

 



Hello Claudiu,

Am Sat, Aug 24, 2024 at 06:57:23PM +0300 schrieb claudiu beznea:
> 
> 
> On 21.08.2024 13:59, Alexander Dahl wrote:
> > This will allow to enable the main RC Oscillator from
> > nvmem_microchip_otpc driver.
> > 
> > Signed-off-by: Alexander Dahl <ada@xxxxxxxxxxx>
> > ---
> > 
> > Notes:
> >     This requires some DT bindings update, right?
> 
> Yes

Okay, I suspected that already.

This raises the question if the main rc osc clock should be mandatory
for SAMA7G5 too, or if SAM9X60 and SAMA7G5 should be handled
differently with regard to that clock?  Affects both binding and
driver then, see NOTE in patch 11, which is directly related to that
question.

> 
> > 
> >  arch/arm/boot/dts/microchip/sam9x60.dtsi | 2 ++
> >  1 file changed, 2 insertions(+)
> > 
> > diff --git a/arch/arm/boot/dts/microchip/sam9x60.dtsi b/arch/arm/boot/dts/microchip/sam9x60.dtsi
> > index 2159a6817f44..4f0651b8cb60 100644
> > --- a/arch/arm/boot/dts/microchip/sam9x60.dtsi
> > +++ b/arch/arm/boot/dts/microchip/sam9x60.dtsi
> > @@ -162,6 +162,8 @@ otpc: efuse@eff00000 {
> >  			reg = <0xeff00000 0xec>;
> >  			#address-cells = <1>;
> >  			#size-cells = <1>;
> > +			clocks = <&pmc PMC_TYPE_CORE 11>;
> > +			clock-names = "main_rc_osc";
> 
> And this should be squashed w/ patch 05/12.

Okay, this may require patch reordering, we'll see.

Greets
Alex

> 
> >  		};
> >  
> >  		apb {




[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