From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org X-Spam-Level: X-Spam-Status: No, score=-6.8 required=3.0 tests=HEADER_FROM_DIFFERENT_DOMAINS, INCLUDES_PATCH,MAILING_LIST_MULTI,SIGNED_OFF_BY,SPF_HELO_NONE,SPF_PASS autolearn=unavailable autolearn_force=no version=3.4.0 Received: from mail.kernel.org (mail.kernel.org [198.145.29.99]) by smtp.lore.kernel.org (Postfix) with ESMTP id 2B87EC433E1 for ; Mon, 18 May 2020 21:40:05 +0000 (UTC) Received: from vger.kernel.org (vger.kernel.org [23.128.96.18]) by mail.kernel.org (Postfix) with ESMTP id 1090F205CB for ; Mon, 18 May 2020 21:40:05 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1728176AbgERVkE (ORCPT ); Mon, 18 May 2020 17:40:04 -0400 Received: from mail.baikalelectronics.com ([87.245.175.226]:50314 "EHLO mail.baikalelectronics.ru" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1726502AbgERVkE (ORCPT ); Mon, 18 May 2020 17:40:04 -0400 Received: from localhost (unknown [127.0.0.1]) by mail.baikalelectronics.ru (Postfix) with ESMTP id 5B9488030875; Mon, 18 May 2020 21:40:01 +0000 (UTC) X-Virus-Scanned: amavisd-new at baikalelectronics.ru Received: from mail.baikalelectronics.ru ([127.0.0.1]) by localhost (mail.baikalelectronics.ru [127.0.0.1]) (amavisd-new, port 10024) with ESMTP id KgkFFm6deoXq; Tue, 19 May 2020 00:39:56 +0300 (MSK) Date: Tue, 19 May 2020 00:39:55 +0300 From: Serge Semin To: Rob Herring CC: Serge Semin , Jarkko Nikula , Alexey Malahov , Thomas Bogendoerfer , Paul Burton , Ralf Baechle , Andy Shevchenko , Mika Westerberg , Wolfram Sang , Frank Rowand , , , , Subject: Re: [PATCH v2 03/12] dt-bindings: i2c: dw: Add Baikal-T1 SoC I2C controller Message-ID: <20200518213955.wzw26wnvcr3qpnok@mobilestation> References: <20200306132001.1B875803087C@mail.baikalelectronics.ru> <20200510095019.20981-1-Sergey.Semin@baikalelectronics.ru> <20200510095019.20981-4-Sergey.Semin@baikalelectronics.ru> <20200518203319.GA14243@bogus> MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Disposition: inline In-Reply-To: <20200518203319.GA14243@bogus> X-ClientProxiedBy: MAIL.baikal.int (192.168.51.25) To mail (192.168.51.25) Sender: linux-kernel-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Mon, May 18, 2020 at 02:33:19PM -0600, Rob Herring wrote: > On Sun, May 10, 2020 at 12:50:09PM +0300, Serge Semin wrote: > > Add the "baikal,bt1-sys-i2c" compatible string to the DW I2C binding and > > make sure the reg property isn't required in this case because the > > controller is embedded into the Baikal-T1 System Controller. The rest of > > the DW APB I2C properties are compatible and can be freely used to describe > > the Baikal-T1 I2C controller dts-node. > > Is there not a sub-range of the system controller with the I2C > registers? I'd assume there is, so you can still have 'reg' even if > Linux doesn't use it (currently). Yes, there is a range. It's just three access registers. Is it wrong to make the reg property being optional in this case since it can be accessed over syscon regmap? Do you suggest to get back the reg property being required for our device? -Sergey > > > > > Signed-off-by: Serge Semin > > Cc: Alexey Malahov > > Cc: Thomas Bogendoerfer > > Cc: Paul Burton > > Cc: Ralf Baechle > > Cc: Andy Shevchenko > > Cc: Mika Westerberg > > Cc: Wolfram Sang > > Cc: Frank Rowand > > Cc: linux-mips@vger.kernel.org > > > > --- > > > > Rob, I had to remove your acked-by tag because of the changes introduced > > in v2 of the patch. > > > > Changelog v2: > > - Make the reg property being optional if it's Baikal-T1 System I2C DT > > node. > > --- > > .../devicetree/bindings/i2c/snps,designware-i2c.yaml | 12 +++++++++++- > > 1 file changed, 11 insertions(+), 1 deletion(-) > > > > diff --git a/Documentation/devicetree/bindings/i2c/snps,designware-i2c.yaml b/Documentation/devicetree/bindings/i2c/snps,designware-i2c.yaml > > index 8d4e5fccbd1c..579964098eb9 100644 > > --- a/Documentation/devicetree/bindings/i2c/snps,designware-i2c.yaml > > +++ b/Documentation/devicetree/bindings/i2c/snps,designware-i2c.yaml > > @@ -21,6 +21,15 @@ allOf: > > properties: > > reg: > > maxItems: 1 > > + - if: > > + properties: > > + compatible: > > + not: > > + contains: > > + const: baikal,bt1-sys-i2c > > + then: > > + required: > > + - reg > > > > properties: > > compatible: > > @@ -31,6 +40,8 @@ properties: > > items: > > - const: mscc,ocelot-i2c > > - const: snps,designware-i2c > > + - description: Baikal-T1 SoC System I2C controller > > + const: baikal,bt1-sys-i2c > > > > reg: > > minItems: 1 > > @@ -98,7 +109,6 @@ unevaluatedProperties: false > > > > required: > > - compatible > > - - reg > > - "#address-cells" > > - "#size-cells" > > - interrupts > > -- > > 2.25.1 > >