On Monday 08 of July 2013 16:12:56 Sylwester Nawrocki wrote:
> Hi,
>
> On 07/06/2013 01:26 AM, Tomasz Figa wrote:
> > On Monday 01 of July 2013 17:22:34 Sylwester Nawrocki wrote:
> >> From: Andrzej Hajda <a.hajda@xxxxxxxxxxx>
> >>
> >> Add MAX8998 LDO12 and fixed voltage regulator nodes. While at it,
> >> all fixed voltage regulator nodes are grouped in a 'regulators' node.
> >>
> >> Signed-off-by: Andrzej Hajda <a.hajda@xxxxxxxxxxx>
> >> Signed-off-by: Sylwester Nawrocki <s.nawrocki@xxxxxxxxxxx>
> >> Signed-off-by: Kyungmin Park <kyungmin.park@xxxxxxxxxxx>
> >> ---
> >>
> >> arch/arm/boot/dts/exynos4210-trats.dts | 80
> >>
> >> +++++++++++++++++++++++++------- 1 file changed, 64 insertions(+), 16
> >> deletions(-)
> >>
> >> diff --git a/arch/arm/boot/dts/exynos4210-trats.dts
> >> b/arch/arm/boot/dts/exynos4210-trats.dts index 6b1568e..f62e299 100644
> >> --- a/arch/arm/boot/dts/exynos4210-trats.dts
> >> +++ b/arch/arm/boot/dts/exynos4210-trats.dts
> >> @@ -30,13 +30,64 @@
> >>
> >> bootargs = "console=ttySAC2,115200N8 root=/dev/mmcblk0p5
> >
> > rootwait
> >
> >> earlyprintk panic=5"; };
> >>
> >> - vemmc_reg: voltage-regulator@0 {
> >> - compatible = "regulator-fixed";
> >> - regulator-name = "VMEM_VDD_2.8V";
> >> - regulator-min-microvolt = <2800000>;
> >> - regulator-max-microvolt = <2800000>;
> >> - gpio = <&gpk0 2 0>;
> >> - enable-active-high;
> >> + regulators {
> >> + compatible = "simple-bus";
> >> + #address-cells = <1>;
> >> + #size-cells = <0>;
> >
> > I don't think any addressing is needed for these regulators, so I'd
> > suggest removing those #properties and replacing @N with -N suffix.
>
> Originally there were also 'reg' properties in the individual regulator
> nodes, but these were unused and I've removed them before posting. Just
> missed to get rid of #size/address-cells as well. Please note you
> similarly use such properties in patch [1].
Oh, you got me here. I must have forgotten to remove them as well.
As we already noticed some time ago, mistakes propagate much faster than
correct solutions. ;)
> I suppose it is correct to have something like:
>
> regulators {
> compatible = "simple-bus";
> regulator-0 {
> ...
> };
>
> regulator-1 {
> ...
> };
> ...
> };
>
> rather than:
>
> regulators {
> compatible = "simple-bus";
> #address-cells = <1>;
> #size-cells = <0>;
>
> regulator@0 {
> reg = <...>;
> ...
> };
>
> regulator@1 {
> reg = <...>;
> ...
> };
> };
>
> Both patterns seem to be used in existing *.dts files.
Both patterns are correct, I guess. I'm not sure if it makes sense to specify
address of something that is not addressable and so approach 1 makes more
sense to me.
> I'm going to use the first option in the next iteration, unless
> someone suggest otherwise.
OK.
Best regards,
Tomasz
--
To unsubscribe from this list: send the line "unsubscribe linux-samsung-soc" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
