Hi Mark, > -----Original Message----- > From: Mark Brown [mailto:broonie@xxxxxxxxxx] > Sent: Tuesday, December 17, 2013 8:38 PM > To: Yang, Wenyou > Cc: lgirdwood@xxxxxxxxx; linux-kernel@xxxxxxxxxxxxxxx; > grant.likely@xxxxxxxxxx; rob.herring@xxxxxxxxxxx; linux- > doc@xxxxxxxxxxxxxxx; vpalatin@xxxxxxxxxxxx; Ferre, Nicolas; > plagnioj@xxxxxxxxxxxx; linux-arm-kernel@xxxxxxxxxxxxxxxxxxx; > devicetree@xxxxxxxxxxxxxxx > Subject: Re: [PATCH 3/3 v2] ARM: dts: sama5d3xcm: add the regulator > device node > > On Tue, Dec 17, 2013 at 01:36:37PM +0800, Wenyou Yang wrote: > > > + vcc_1v8_reg: DCDC_REG1 { > > + regulator-name = "DCDC_REG1"; > > The whole point of naming the regulators is to help users read kernel > diagnostic output so the should normally be named after the supply in > the schematic. If you're using the name of the supply on the PMIC you > can just omit it since that's the default anyway and it's not adding > extra information. Use something like VCC_1V8. Thanks a lot. Change regulator-name with VCC_1V8, VCC_1V2 ... which named after the supply in the schematic. Best Regards, Wenyou Yang -- To unsubscribe from this list: send the line "unsubscribe linux-doc" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
