Re: [PATCH 3/3] ARM: dts: add dedicated dtsi for hi6421

[Mark Brown]

On Tue, Jun 04, 2013 at 10:28:43PM +0800, Guodong Xu wrote:

> +	ldo1: ldo@21 {
> +		compatible = "hisilicon,hi6421-ldo";
> +		regulator-name = "LDO1";
> +		regulator-min-microvolt = <1700000>;
> +		regulator-max-microvolt = <2000000>;

This all looks very board specific.  There's no point in names like
"LDO1" really - the goal for names is to provide something which matches
the schematics to aid diagnostics.

Attachment: signature.asc
Description: Digital signature