Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]


Groups > linux.kernel > #1442166

Re: [PATCH v2 2/2] spi: add DT binding for clps711x SPI

From Alexander Shiyan <shc_work@mail.ru>
Newsgroups linux.kernel
Subject Re: [PATCH v2 2/2] spi: add DT binding for clps711x SPI
Date 2016-07-13 10:30 +0200
Message-ID <rUnIl-rS-15@gated-at.bofh.it> (permalink)
References <rSe6u-59g-5@gated-at.bofh.it> <rUcWE-1C4-79@gated-at.bofh.it> <rUnyG-oj-19@gated-at.bofh.it>
Organization linux.* mail to news gateway

Show all headers | View raw


>Среда, 13 июля 2016, 11:13 +03:00 от Arnd Bergmann <arnd@arndb.de>:
>
>On Tuesday, July 12, 2016 3:49:47 PM CEST Rob Herring wrote:
>> On Thu, Jul 07, 2016 at 11:49:46AM +0200, Arnd Bergmann wrote:
>> > +
>> > +An additional register is present in the system controller,
>> > +which is assumed to be in the same device tree, with and marked
>> > +as compatible with "cirrus,ep7209-syscon3".
>> > +
>> > +Example:
>> > +
>> > +spi@80000500 {
>> > +     #address-cells = <1>;
>> > +     #size-cells = <0>;
>> > +     compatible = "cirrus,ep7209-spi";
>> > +     reg = <0x80000500 0x4>;
>> > +     interrupts = <15>;
>> > +     clocks = <&clks CLPS711X_CLK_SPI>;
>> > +     status = "disabled";
>> > +};
>> > +
>> > +syscon3: syscon@80002200 {
>> 
>> This looks like it shouldn't be here.
>
>The label, or the entire node?

I think that the whole syscon3 node is redundant.

---

Back to linux.kernel | Previous | NextPrevious in thread | Next in thread | Find similar | Unroll thread


Thread

Re: [PATCH v2 2/2] spi: add DT binding for clps711x SPI Rob Herring <robh@kernel.org> - 2016-07-12 23:00 +0200
  Re: [PATCH v2 2/2] spi: add DT binding for clps711x SPI Arnd Bergmann <arnd@arndb.de> - 2016-07-13 10:20 +0200
    Re: [PATCH v2 2/2] spi: add DT binding for clps711x SPI Alexander Shiyan <shc_work@mail.ru> - 2016-07-13 10:30 +0200
      Re: [PATCH v2 2/2] spi: add DT binding for clps711x SPI Arnd Bergmann <arnd@arndb.de> - 2016-07-13 10:40 +0200

csiph-web