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=DKIM_SIGNED,DKIM_VALID, DKIM_VALID_AU,HEADER_FROM_DIFFERENT_DOMAINS,INCLUDES_PATCH,MAILING_LIST_MULTI, SIGNED_OFF_BY,SPF_HELO_NONE,SPF_PASS,URIBL_BLOCKED autolearn=ham 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 3AD9EC433E0 for ; Thu, 14 May 2020 01:35:08 +0000 (UTC) Received: from vger.kernel.org (vger.kernel.org [23.128.96.18]) by mail.kernel.org (Postfix) with ESMTP id 239CA2054F for ; Thu, 14 May 2020 01:35:08 +0000 (UTC) Authentication-Results: mail.kernel.org; dkim=pass (1024-bit key) header.d=ideasonboard.com header.i=@ideasonboard.com header.b="t7PcZDVk" Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1725952AbgENBfH (ORCPT ); Wed, 13 May 2020 21:35:07 -0400 Received: from perceval.ideasonboard.com ([213.167.242.64]:51434 "EHLO perceval.ideasonboard.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1725943AbgENBfH (ORCPT ); Wed, 13 May 2020 21:35:07 -0400 Received: from pendragon.ideasonboard.com (81-175-216-236.bb.dnainternet.fi [81.175.216.236]) by perceval.ideasonboard.com (Postfix) with ESMTPSA id D62EA51F; Thu, 14 May 2020 03:35:04 +0200 (CEST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=ideasonboard.com; s=mail; t=1589420105; bh=d9OdqwwP8O7Oyrp19GkMYElKFQ2J18L8qVnB9k2Ji6s=; h=Date:From:To:Cc:Subject:References:In-Reply-To:From; b=t7PcZDVktgThjPZUZRI6UDyXAQz2s7gEeGYN7fgwgeUIGfBWCJF7HVz4Lx9GMLzph Gr5UaQUmcVEBVMll5MH+iZ6JmXcDlzjvR6Ak3k1BzbL+bNU1kUvtqfZmF8gvUU0xXI YghPmNZt5Yy1iDpNaKUFl5wVG9YilhCoySZ7fQXI= Date: Thu, 14 May 2020 04:34:58 +0300 From: Laurent Pinchart To: Ricardo =?utf-8?Q?Ca=C3=B1uelo?= Cc: kernel@collabora.com, devicetree@vger.kernel.org, linux-arm-kernel@lists.infradead.org, geert+renesas@glider.be, robh+dt@kernel.org, xuwei5@hisilicon.com Subject: Re: [PATCH v2 2/6] ARM: dts: renesas: make hdmi encoder nodes compliant with DT bindings Message-ID: <20200514013458.GB7425@pendragon.ideasonboard.com> References: <20200511110611.3142-1-ricardo.canuelo@collabora.com> <20200511110611.3142-3-ricardo.canuelo@collabora.com> MIME-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Disposition: inline Content-Transfer-Encoding: 8bit In-Reply-To: <20200511110611.3142-3-ricardo.canuelo@collabora.com> Sender: devicetree-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: devicetree@vger.kernel.org Hi Ricardo, Thank you for the patch. On Mon, May 11, 2020 at 01:06:07PM +0200, Ricardo Cañuelo wrote: > Small fixes to make these DTs compliant with the adi,adv7511w and > adiadv7513 bindings: > > r8a7745-iwg22d-sodimm-dbhd-ca.dts > r8a7790-lager.dts > r8a7790-stout.dts > r8a7791-koelsch.dts > r8a7791-porter.dts > r8a7792-blanche.dts > r8a7793-gose.dts > r8a7794-silk.dts: > Remove the adi,input-style and adi,input-justification properties > > r8a7792-wheat.dts: > Reorder the I2C slave addresses of hdmi@3d and hdmi@39 and remove the > adi,input-style and adi,input-justification properties. > > Signed-off-by: Ricardo Cañuelo Reviewed-by: Laurent Pinchart > --- > arch/arm/boot/dts/r8a7745-iwg22d-sodimm-dbhd-ca.dts | 2 -- > arch/arm/boot/dts/r8a7790-lager.dts | 2 -- > arch/arm/boot/dts/r8a7790-stout.dts | 2 -- > arch/arm/boot/dts/r8a7791-koelsch.dts | 2 -- > arch/arm/boot/dts/r8a7791-porter.dts | 2 -- > arch/arm/boot/dts/r8a7792-blanche.dts | 2 -- > arch/arm/boot/dts/r8a7792-wheat.dts | 12 ++++-------- > arch/arm/boot/dts/r8a7793-gose.dts | 2 -- > arch/arm/boot/dts/r8a7794-silk.dts | 2 -- > 9 files changed, 4 insertions(+), 24 deletions(-) > > diff --git a/arch/arm/boot/dts/r8a7745-iwg22d-sodimm-dbhd-ca.dts b/arch/arm/boot/dts/r8a7745-iwg22d-sodimm-dbhd-ca.dts > index 92aa26ba423c..b1f679da36b2 100644 > --- a/arch/arm/boot/dts/r8a7745-iwg22d-sodimm-dbhd-ca.dts > +++ b/arch/arm/boot/dts/r8a7745-iwg22d-sodimm-dbhd-ca.dts > @@ -84,8 +84,6 @@ > adi,input-depth = <8>; > adi,input-colorspace = "rgb"; > adi,input-clock = "1x"; > - adi,input-style = <1>; > - adi,input-justification = "evenly"; > > ports { > #address-cells = <1>; > diff --git a/arch/arm/boot/dts/r8a7790-lager.dts b/arch/arm/boot/dts/r8a7790-lager.dts > index 69745def44d4..bfe778c4c47b 100644 > --- a/arch/arm/boot/dts/r8a7790-lager.dts > +++ b/arch/arm/boot/dts/r8a7790-lager.dts > @@ -364,8 +364,6 @@ > adi,input-depth = <8>; > adi,input-colorspace = "rgb"; > adi,input-clock = "1x"; > - adi,input-style = <1>; > - adi,input-justification = "evenly"; > > ports { > #address-cells = <1>; > diff --git a/arch/arm/boot/dts/r8a7790-stout.dts b/arch/arm/boot/dts/r8a7790-stout.dts > index 4138efb2766d..6a457bc9280a 100644 > --- a/arch/arm/boot/dts/r8a7790-stout.dts > +++ b/arch/arm/boot/dts/r8a7790-stout.dts > @@ -297,8 +297,6 @@ > adi,input-depth = <8>; > adi,input-colorspace = "rgb"; > adi,input-clock = "1x"; > - adi,input-style = <1>; > - adi,input-justification = "evenly"; > > ports { > #address-cells = <1>; > diff --git a/arch/arm/boot/dts/r8a7791-koelsch.dts b/arch/arm/boot/dts/r8a7791-koelsch.dts > index 687167b70cb6..fc74c6cd6def 100644 > --- a/arch/arm/boot/dts/r8a7791-koelsch.dts > +++ b/arch/arm/boot/dts/r8a7791-koelsch.dts > @@ -387,8 +387,6 @@ > adi,input-depth = <8>; > adi,input-colorspace = "rgb"; > adi,input-clock = "1x"; > - adi,input-style = <1>; > - adi,input-justification = "evenly"; > > ports { > #address-cells = <1>; > diff --git a/arch/arm/boot/dts/r8a7791-porter.dts b/arch/arm/boot/dts/r8a7791-porter.dts > index a8e0335148a5..114bf1c4199b 100644 > --- a/arch/arm/boot/dts/r8a7791-porter.dts > +++ b/arch/arm/boot/dts/r8a7791-porter.dts > @@ -181,8 +181,6 @@ > adi,input-depth = <8>; > adi,input-colorspace = "rgb"; > adi,input-clock = "1x"; > - adi,input-style = <1>; > - adi,input-justification = "evenly"; > > ports { > #address-cells = <1>; > diff --git a/arch/arm/boot/dts/r8a7792-blanche.dts b/arch/arm/boot/dts/r8a7792-blanche.dts > index 248eb717eb35..9368ac2cf508 100644 > --- a/arch/arm/boot/dts/r8a7792-blanche.dts > +++ b/arch/arm/boot/dts/r8a7792-blanche.dts > @@ -289,8 +289,6 @@ > adi,input-depth = <8>; > adi,input-colorspace = "rgb"; > adi,input-clock = "1x"; > - adi,input-style = <1>; > - adi,input-justification = "evenly"; > > ports { > #address-cells = <1>; > diff --git a/arch/arm/boot/dts/r8a7792-wheat.dts b/arch/arm/boot/dts/r8a7792-wheat.dts > index bd2a63bdab3d..ba2d2a589012 100644 > --- a/arch/arm/boot/dts/r8a7792-wheat.dts > +++ b/arch/arm/boot/dts/r8a7792-wheat.dts > @@ -249,14 +249,12 @@ > */ > hdmi@3d { > compatible = "adi,adv7513"; > - reg = <0x3d>, <0x2d>, <0x4d>, <0x5d>; > - reg-names = "main", "cec", "edid", "packet"; > + reg = <0x3d>, <0x4d>, <0x2d>, <0x5d>; > + reg-names = "main", "edid", "cec", "packet"; > > adi,input-depth = <8>; > adi,input-colorspace = "rgb"; > adi,input-clock = "1x"; > - adi,input-style = <1>; > - adi,input-justification = "evenly"; > > ports { > #address-cells = <1>; > @@ -280,14 +278,12 @@ > > hdmi@39 { > compatible = "adi,adv7513"; > - reg = <0x39>, <0x29>, <0x49>, <0x59>; > - reg-names = "main", "cec", "edid", "packet"; > + reg = <0x39>, <0x49>, <0x29>, <0x59>; > + reg-names = "main", "edid", "cec", "packet"; > > adi,input-depth = <8>; > adi,input-colorspace = "rgb"; > adi,input-clock = "1x"; > - adi,input-style = <1>; > - adi,input-justification = "evenly"; > > ports { > #address-cells = <1>; > diff --git a/arch/arm/boot/dts/r8a7793-gose.dts b/arch/arm/boot/dts/r8a7793-gose.dts > index cfe06a74ce89..79baf06019f5 100644 > --- a/arch/arm/boot/dts/r8a7793-gose.dts > +++ b/arch/arm/boot/dts/r8a7793-gose.dts > @@ -366,8 +366,6 @@ > adi,input-depth = <8>; > adi,input-colorspace = "rgb"; > adi,input-clock = "1x"; > - adi,input-style = <1>; > - adi,input-justification = "evenly"; > > ports { > #address-cells = <1>; > diff --git a/arch/arm/boot/dts/r8a7794-silk.dts b/arch/arm/boot/dts/r8a7794-silk.dts > index 9aaa96ea9943..b8b0941f677c 100644 > --- a/arch/arm/boot/dts/r8a7794-silk.dts > +++ b/arch/arm/boot/dts/r8a7794-silk.dts > @@ -255,8 +255,6 @@ > adi,input-depth = <8>; > adi,input-colorspace = "rgb"; > adi,input-clock = "1x"; > - adi,input-style = <1>; > - adi,input-justification = "evenly"; > > ports { > #address-cells = <1>; -- Regards, Laurent Pinchart