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

[Claudiu Beznea]

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.

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