Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1753510AbaKLVYg (ORCPT ); Wed, 12 Nov 2014 16:24:36 -0500 Received: from mho-02-ewr.mailhop.org ([204.13.248.72]:58554 "EHLO mho-02-ewr.mailhop.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1753378AbaKLVYe (ORCPT ); Wed, 12 Nov 2014 16:24:34 -0500 X-Mail-Handler: Dyn Standard SMTP by Dyn X-Originating-IP: 104.193.169.186 X-Report-Abuse-To: abuse@dyndns.com (see http://www.dyndns.com/services/sendlabs/outbound_abuse.html for abuse reporting information) X-MHO-User: U2FsdGVkX19CTTfWMlE21pTr5Lvprp13 Date: Wed, 12 Nov 2014 13:23:41 -0800 From: Tony Lindgren To: Belisko Marek Cc: Rob Herring , Pawel Moll , Mark Rutland , "ijc+devicetree@hellion.org.uk" , Kumar Gala , Benoit Cousson , Russell King - ARM Linux , Tomi Valkeinen , Jean-Christophe PLAGNIOL-VILLARD , Grant Likely , "devicetree@vger.kernel.org" , LKML , "linux-omap@vger.kernel.org" , linux-arm-kernel , linux-fbdev@vger.kernel.org, "Dr. H. Nikolaus Schaller" Subject: Re: [PATCH 4/4] arm: dts: omap3-gta04: Add static configuration for devconf1 register Message-ID: <20141112212340.GL26481@atomide.com> References: <1415051968-4878-1-git-send-email-marek@goldelico.com> <1415051968-4878-5-git-send-email-marek@goldelico.com> <20141110233630.GD31454@atomide.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: User-Agent: Mutt/1.5.23 (2014-03-12) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org * Belisko Marek [141112 13:04]: > Hi Tony, > > On Tue, Nov 11, 2014 at 12:36 AM, Tony Lindgren wrote: > > * Marek Belisko [141103 14:01]: > >> gta04 board need for tvout enabled 2 bits in devconf1 register (tvbypass and acbias). > >> Add single pinmux entry and enable it. > >> > >> Signed-off-by: Marek Belisko > >> --- > >> arch/arm/boot/dts/omap3-gta04.dtsi | 22 ++++++++++++++++++++++ > >> 1 file changed, 22 insertions(+) > >> > >> diff --git a/arch/arm/boot/dts/omap3-gta04.dtsi b/arch/arm/boot/dts/omap3-gta04.dtsi > >> index e4d05f0..a456d37 100644 > >> --- a/arch/arm/boot/dts/omap3-gta04.dtsi > >> +++ b/arch/arm/boot/dts/omap3-gta04.dtsi > >> @@ -118,6 +118,17 @@ > >> }; > >> }; > >> }; > >> + > >> + /* pinmux for devconf1 */ > >> + control_devconf1: pinmux@480022d8 { > >> + compatible = "pinctrl-single"; > >> + reg = <0x480022d8 4>; /* single register */ > >> + #address-cells = <1>; > >> + #size-cells = <0>; > >> + pinctrl-single,bit-per-mux; > >> + pinctrl-single,register-width = <32>; > >> + pinctrl-single,function-mask = <0xfc0bd5>; > >> + }; > >> }; > >> > > > > The pinctrl-single entry should be in omap3.dtsi as the mux register is there > > for all the omap3 devices, can you please update the patch for that? > Ok I'll do. Just one question. I checked TRM for omap3430 and omap3630 and > reserved bits in devconf1 are different. So keep function-mask for > omap3430 in omap3.dtsi > and redefine in omap36xx.dtsi (not sure if this will work in this > way)? Or exist other way how to deal with that? Thanks. Oh OK. Yes if they are different you should have the common entry in omap3.dtsi and the SoC specific fields in omap34xx.dtsi and omap36xx.dtsi. And please leave out the bit for configuring the MPUFORCEWRNP. Regards, Tony -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/