From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1752096AbdJMQ4h (ORCPT ); Fri, 13 Oct 2017 12:56:37 -0400 Received: from fllnx209.ext.ti.com ([198.47.19.16]:19849 "EHLO fllnx209.ext.ti.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751057AbdJMQ4f (ORCPT ); Fri, 13 Oct 2017 12:56:35 -0400 Date: Fri, 13 Oct 2017 11:55:56 -0500 From: Benoit Parrot To: Tony Lindgren CC: Tero Kristo , Rob Herring , , , , Subject: Re: [Patch 1/6] ARM: dts: DRA72: Add CAL dtsi node Message-ID: <20171013165555.GF25400@ti.com> References: <20171012192719.15193-1-bparrot@ti.com> <20171012192719.15193-2-bparrot@ti.com> <20171013165408.GJ4394@atomide.com> MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Disposition: inline In-Reply-To: <20171013165408.GJ4394@atomide.com> User-Agent: Mutt/1.5.23 (2014-03-12) X-EXCLAIMER-MD-CONFIG: e1e8a2fd-e40a-4ac6-ac9b-f7e9cc9ee180 Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Tony Lindgren wrote on Fri [2017-Oct-13 09:54:09 -0700]: > * Benoit Parrot [171012 12:29]: > > This patch adds the required dtsi node to support the Camera > > Adaptation Layer (CAL) for the DRA72 family of devices. > > > > - Added CAL entry in dra72x.dtsi. > > > > Signed-off-by: Benoit Parrot > > --- > > arch/arm/boot/dts/dra72x.dtsi | 31 +++++++++++++++++++++++++++++++ > > 1 file changed, 31 insertions(+) > > > > diff --git a/arch/arm/boot/dts/dra72x.dtsi b/arch/arm/boot/dts/dra72x.dtsi > > index 67107605fb4c..d0ba4f238084 100644 > > --- a/arch/arm/boot/dts/dra72x.dtsi > > +++ b/arch/arm/boot/dts/dra72x.dtsi > > @@ -17,6 +17,37 @@ > > interrupt-parent = <&wakeupgen>; > > interrupts = ; > > }; > > + > > + ocp { > > + cal: cal@4845b000 { > > + compatible = "ti,dra72-cal"; > > + ti,hwmods = "cal"; > > + reg = <0x4845B000 0x400>, > > + <0x4845B800 0x40>, > > + <0x4845B900 0x40>, > > + <0x4A002e94 0x4>; > > Care to fix the cAmelcasing here? All lower case hex is pretty much the > standard with linux, so might as well lower case them all while at it. Sure thing, I'll fix it. Benoit > > Regards, > > Tony