RE: [PATCH 2/2] ARM: dts: microchip: sam9x75_curiosity: Add power monitor support

[Mihai.Sain]

> Hi, Mihai,
> 
> On 02.12.2024 10:32, Mihai.Sain@xxxxxxxxxxxxx wrote:
> > Hi Claudiu,
> >
> >>> +             channel@2 {
> >>> +                     reg = <0x2>;
> >>> +                     shunt-resistor-micro-ohms = <10000>;
> >>> +                     label = "VDDOUT4";
> >>
> >> On the schematics at [1] this seems to be named DCDC4. Is there a mistake on
> >> [1] or this patch needs adjustments? If so, I can do it while applying.
> >
> > I just wanted to use a more friendly name for buck4, since this regulator is
> not used.
> 
> For consistency with the rest of the channel names and the fact that we
> describe the HW though device tree, I would keep the schematics naming
> ("DCDC4" in this case). No need to re-send, I can adjust while applying.
> Just let me know if all good with you, too.

Yes.
I agree we should use the schematics naming for buck4.

> 
> Thank you,
> Claudiu
> 
> >
> >>
> >> Thank you,
> >> Claudiu
> >>
> >> [1]
> >>
> https://ww1.microchip.com/downloads/aemDocuments/documents/MPU32/ProductDocumen
> >> ts/UserGuides/SAM9X75-Curiosity-User-Guide-DS60001859.pdf
> >>
> >
> > Regards,
> > Mihai

Regards,
Mihai