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=-15.8 required=3.0 tests=BAYES_00,DKIM_SIGNED, DKIM_VALID,DKIM_VALID_AU,HEADER_FROM_DIFFERENT_DOMAINS,INCLUDES_CR_TRAILER, INCLUDES_PATCH,MAILING_LIST_MULTI,SPF_HELO_NONE,SPF_PASS,URIBL_BLOCKED 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 59A1EC433B4 for ; Wed, 19 May 2021 01:18:14 +0000 (UTC) Received: from vger.kernel.org (vger.kernel.org [23.128.96.18]) by mail.kernel.org (Postfix) with ESMTP id 2A576613AA for ; Wed, 19 May 2021 01:18:14 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S231791AbhESBTb (ORCPT ); Tue, 18 May 2021 21:19:31 -0400 Received: from perceval.ideasonboard.com ([213.167.242.64]:33202 "EHLO perceval.ideasonboard.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S231641AbhESBTa (ORCPT ); Tue, 18 May 2021 21:19:30 -0400 Received: from pendragon.ideasonboard.com (62-78-145-57.bb.dnainternet.fi [62.78.145.57]) by perceval.ideasonboard.com (Postfix) with ESMTPSA id 6F13A45E; Wed, 19 May 2021 03:18:10 +0200 (CEST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=ideasonboard.com; s=mail; t=1621387090; bh=w3br3R9b5YuogXvJ+4Md07rNPhvvQST+awL6ZItkg4w=; h=Date:From:To:Cc:Subject:References:In-Reply-To:From; b=lb07uRx60fMPu5ms9sIytoW0ORUUV5fheREpobjjSIJe0OrFKSgOQBhEd17nudWhe QuwvuALY/bQxeEXjeSC/xnfH3Lt+w9VGRcoEUXp8MHdkm2iMtCwlPcTRomOrpgI6J2 bjyifFCmqde8Uyf/vENl5mUahqgtnjlT41e0Wio4= Date: Wed, 19 May 2021 04:18:09 +0300 From: Laurent Pinchart To: Rob Herring Cc: devicetree@vger.kernel.org, Peter Rosin , Wolfram Sang , linux-kernel@vger.kernel.org, Alexandre Belloni , Jacopo Mondi , Kieran Bingham , Kishon Vijay Abraham I , Lee Jones , Niklas =?utf-8?Q?S=C3=B6derlund?= , Roger Quadros , Jonathan Cameron Subject: Re: [PATCH 6/6] dt-bindings: i2c: maxim,max9286: Use the i2c-mux.yaml schema Message-ID: References: <20210518232858.1535403-1-robh@kernel.org> <20210518232858.1535403-7-robh@kernel.org> MIME-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Disposition: inline Content-Transfer-Encoding: 8bit In-Reply-To: <20210518232858.1535403-7-robh@kernel.org> Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Hi Rob, Thank you for the patch. On Tue, May 18, 2021 at 06:28:58PM -0500, Rob Herring wrote: > Use the i2c-mux.yaml schema in the maxim,max9286 binding schema. With this, > several properties can be dropped as they are defined in i2c-mux.yaml > already. > > Cc: Jacopo Mondi > Cc: Kieran Bingham > Cc: Laurent Pinchart > Cc: Niklas Söderlund > Signed-off-by: Rob Herring Reviewed-by: Laurent Pinchart > --- > .../bindings/media/i2c/maxim,max9286.yaml | 23 +++---------------- > 1 file changed, 3 insertions(+), 20 deletions(-) > > diff --git a/Documentation/devicetree/bindings/media/i2c/maxim,max9286.yaml b/Documentation/devicetree/bindings/media/i2c/maxim,max9286.yaml > index ee16102fdfe7..02f656e78700 100644 > --- a/Documentation/devicetree/bindings/media/i2c/maxim,max9286.yaml > +++ b/Documentation/devicetree/bindings/media/i2c/maxim,max9286.yaml > @@ -111,17 +111,10 @@ properties: > > i2c-mux: > type: object > + $ref: /schemas/i2c/i2c-mux.yaml# > + unevaluatedProperties: false > description: | > - Each GMSL link is modelled as a child bus of an i2c bus > - multiplexer/switch, in accordance with bindings described in > - Documentation/devicetree/bindings/i2c/i2c-mux.txt. > - > - properties: > - '#address-cells': > - const: 1 > - > - '#size-cells': > - const: 0 > + Each GMSL link is modelled as a child bus of an i2c bus multiplexer/switch. > > patternProperties: > "^i2c@[0-3]$": > @@ -133,12 +126,6 @@ properties: > channels. > > properties: > - '#address-cells': > - const: 1 > - > - '#size-cells': > - const: 0 > - > reg: > description: The index of the GMSL channel. > maxItems: 1 > @@ -173,10 +160,6 @@ properties: > > additionalProperties: false > > - additionalProperties: false > - > - additionalProperties: false > - > required: > - compatible > - reg -- Regards, Laurent Pinchart