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.