[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <20130604164555.GF31367@sirena.org.uk>
Date: Tue, 4 Jun 2013 17:45:55 +0100
From: Mark Brown <broonie@...nel.org>
To: Guodong Xu <guodong.xu@...aro.org>
Cc: sameo@...ux.intel.com, lgirdwood@...il.com,
linux-kernel@...r.kernel.org, patches@...aro.org
Subject: Re: [PATCH 3/3] ARM: dts: add dedicated dtsi for hi6421
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.
Download attachment "signature.asc" of type "application/pgp-signature" (837 bytes)
Powered by blists - more mailing lists