From mboxrd@z Thu Jan 1 00:00:00 1970 From: Tony Lindgren Subject: Re: [PATCH] ARM: OMAP: Revert "ARM: OMAP: ctrl: Fix CONTROL_DSIPHY register fields" Date: Mon, 9 Apr 2012 16:02:34 -0700 Message-ID: <20120409230234.GD6487@atomide.com> References: <1333955479-2675-1-git-send-email-archit@ti.com> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Return-path: Received: from mho-02-ewr.mailhop.org ([204.13.248.72]:37307 "EHLO mho-02-ewr.mailhop.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751109Ab2DIXCl (ORCPT ); Mon, 9 Apr 2012 19:02:41 -0400 Content-Disposition: inline In-Reply-To: <1333955479-2675-1-git-send-email-archit@ti.com> Sender: linux-omap-owner@vger.kernel.org List-Id: linux-omap@vger.kernel.org To: Archit Taneja Cc: tomi.valkeinen@ti.com, linux@arm.linux.org.uk, linux-omap@vger.kernel.org, santosh.shilimkar@ti.com, paul@pwsan.com, b-cousson@ti.com * Archit Taneja [120409 00:15]: > This reverts commit 46f8c3c7e95c0d30d95911e7975ddc4f93b3e237. > > The commit above swapped the DSI1_PPID and DSI2_PPID register fields in > CONTROL_DSIPHY to be in sync with the newer public OMAP TRMs(after version V). > > With this commit, contention errors were reported on DSI lanes some OMAP4 SDPs. > After probing the DSI lanes on OMAP4 SDP, it was seen that setting bits in the > DSI2_PPID field was pulling up voltage on DSI1 lanes, and DSI1_PPID field was > pulling up voltage on DSI2 lanes. > > This proves that the current version of OMAP4 TRM is incorrect, swap the > position of register fields according to the older TRM versions as they were > correct. > > Signed-off-by: Archit Taneja Tomi, care to ack this one? Tony > --- > .../include/mach/ctrl_module_pad_core_44xx.h | 8 ++++---- > 1 files changed, 4 insertions(+), 4 deletions(-) > > diff --git a/arch/arm/mach-omap2/include/mach/ctrl_module_pad_core_44xx.h b/arch/arm/mach-omap2/include/mach/ctrl_module_pad_core_44xx.h > index 1e2d332..c88420d 100644 > --- a/arch/arm/mach-omap2/include/mach/ctrl_module_pad_core_44xx.h > +++ b/arch/arm/mach-omap2/include/mach/ctrl_module_pad_core_44xx.h > @@ -941,10 +941,10 @@ > #define OMAP4_DSI2_LANEENABLE_MASK (0x7 << 29) > #define OMAP4_DSI1_LANEENABLE_SHIFT 24 > #define OMAP4_DSI1_LANEENABLE_MASK (0x1f << 24) > -#define OMAP4_DSI2_PIPD_SHIFT 19 > -#define OMAP4_DSI2_PIPD_MASK (0x1f << 19) > -#define OMAP4_DSI1_PIPD_SHIFT 14 > -#define OMAP4_DSI1_PIPD_MASK (0x1f << 14) > +#define OMAP4_DSI1_PIPD_SHIFT 19 > +#define OMAP4_DSI1_PIPD_MASK (0x1f << 19) > +#define OMAP4_DSI2_PIPD_SHIFT 14 > +#define OMAP4_DSI2_PIPD_MASK (0x1f << 14) > > /* CONTROL_MCBSPLP */ > #define OMAP4_ALBCTRLRX_FSX_SHIFT 31 > -- > 1.7.5.4 >