Re: [PATCH 2/7] dt-bindings: i2c: add support for 'i2c-arb' subnode

Rob Herring
2016-07-29T21:24:52Z

On Wed, Jul 27, 2016 at 10:43:25AM +0200, Peter Rosin wrote:
> This gets rid of the need for a pointless 'reg' property for i2c
> arbitrators.
>
> I.e. this new and more compact style
>
> some-arbitrator {
> i2c-mux {

You mean i2c-arb here?

> #address-cells = <1>;
> #size-cells = <0>;
>
> some-i2c-device@50 {
> reg = <0x50>;
> };
> };
> };
>
> instead of the old
>
> some-arbitrator {
> #address-cells = <1>;
> #size-cells = <0>;
>
> i2c@0 {
> reg = <0>;
>
> #address-cells = <1>;
> #size-cells = <0>;
>
> some-i2c-device@50 {
> reg = <0x50>;
> };
> };
> };
>
> Signed-off-by: Peter Rosin <peda@axentia.se>
> ---
> .../bindings/i2c/i2c-arb-gpio-challenge.txt | 8 ++---
> Documentation/devicetree/bindings/i2c/i2c-arb.txt | 35 ++++++++++++++++++++++
> MAINTAINERS | 1 +
> 3 files changed, 38 insertions(+), 6 deletions(-)
> create mode 100644 Documentation/devicetree/bindings/i2c/i2c-arb.txt

Otherwise,

Acked-by: Rob Herring <robh@kernel.org>