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

Claudiu Beznea claudiu.beznea at tuxon.dev
Tue Dec 3 00:37:37 PST 2024


Hi, Mihai,

On 02.12.2024 10:32, Mihai.Sain at microchip.com wrote:
> Hi Claudiu,
> 
>>> +             channel at 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



More information about the linux-arm-kernel mailing list