[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID:
<PH8PR11MB68044686611DE17930DFD77082362@PH8PR11MB6804.namprd11.prod.outlook.com>
Date: Tue, 3 Dec 2024 08:47:02 +0000
From: <Mihai.Sain@...rochip.com>
To: <claudiu.beznea@...on.dev>, <robh@...nel.org>, <krzk+dt@...nel.org>,
<conor+dt@...nel.org>, <Nicolas.Ferre@...rochip.com>,
<alexandre.belloni@...tlin.com>, <devicetree@...r.kernel.org>,
<linux-arm-kernel@...ts.infradead.org>, <linux-kernel@...r.kernel.org>
Subject: RE: [PATCH 2/2] ARM: dts: microchip: sam9x75_curiosity: Add power
monitor support
> Hi, Mihai,
>
> On 02.12.2024 10:32, Mihai.Sain@...rochip.com 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
Powered by blists - more mailing lists