Re: [PATCH V11 1/3] ARM: dts: Add pmu sysreg node to exynos5250 and exynos5420 dtsi files

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Leela Krishna,

On Mon, Dec 2, 2013 at 11:49 AM, Tomasz Figa <tomasz.figa@xxxxxxxxx> wrote:
> On Monday 02 of December 2013 10:50:14 Olof Johansson wrote:
>> Hi,
>>
>> On Wed, Nov 27, 2013 at 8:34 PM, Leela Krishna Amudala
>> <l.krishna@xxxxxxxxxxx> wrote:
>> > This patch adds pmusysreg node to exynos5250 and exynos5420 dtsi files to
>> > handle PMU register accesses in a centralized way using syscon driver
>> >
>> > Signed-off-by: Leela Krishna Amudala <l.krishna@xxxxxxxxxxx>
>> > Reviewed-by: Tomasz Figa <t.figa@xxxxxxxxxxx>
>> > Reviewed-by: Doug Anderson <dianders@xxxxxxxxxxxx>
>> > Tested-by: Doug Anderson <dianders@xxxxxxxxxxxx>
>> > ---
>> >  Documentation/devicetree/bindings/arm/samsung/pmu.txt |   16 ++++++++++++++++
>> >  arch/arm/boot/dts/exynos5250.dtsi                     |    5 +++++
>> >  arch/arm/boot/dts/exynos5420.dtsi                     |    5 +++++
>> >  3 files changed, 26 insertions(+)
>> >  create mode 100644 Documentation/devicetree/bindings/arm/samsung/pmu.txt
>> >
>> > diff --git a/Documentation/devicetree/bindings/arm/samsung/pmu.txt b/Documentation/devicetree/bindings/arm/samsung/pmu.txt
>> > new file mode 100644
>> > index 0000000..307e727
>> > --- /dev/null
>> > +++ b/Documentation/devicetree/bindings/arm/samsung/pmu.txt
>> > @@ -0,0 +1,16 @@
>> > +SAMSUNG Exynos SoC series PMU Registers
>> > +
>> > +Properties:
>> > + - name : should be 'syscon';
>>
>> It's common to use a non-abbreviated name, such as 'system-controller'
>> here. Or, given that it's referring to "pmusysreg" then maybe
>> something like "pmu-system-registers".
>
> Hmm, it's two syscons vs two system-controllers in existing device trees.
> I agree that system-controller sounds much better as a name for this class
> of devices. I don't remember why I initially suggested syscon, though.
> Possibly based on those two existing device trees using this name.

I'd vote for using "pmu-system-registers".  We end up using the
"syscon" subsystem but really we're describing pmu registers.

I'd even say that you don't need to formally specify the "name" in the
bindings (though I'm not up with all the latest device tree
requirements).  ...still you'd want to use "pmu-system-registers" in
the DTS changes.

Can you spin up a v12?

Thanks!

-Doug
--
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




[Index of Archives]     [Linux SoC Development]     [Linux Rockchip Development]     [Linux USB Development]     [Video for Linux]     [Linux Audio Users]     [Linux SCSI]     [Yosemite News]

  Powered by Linux