From mboxrd@z Thu Jan 1 00:00:00 1970 From: tony@atomide.com (Tony Lindgren) Date: Mon, 4 May 2015 09:31:14 -0700 Subject: [PATCH] bus: omap_l3_noc: Fix master id address decoding for OMAP5 In-Reply-To: <553A8C68.5080903@ti.com> References: <1429898060-51944-1-git-send-email-s-anna@ti.com> <553A8C68.5080903@ti.com> Message-ID: <20150504163113.GP24469@atomide.com> To: linux-arm-kernel@lists.infradead.org List-Id: linux-arm-kernel.lists.infradead.org * Nishanth Menon [150424 11:34]: > On 04/24/2015 12:54 PM, Suman Anna wrote: > > The L3 Error handling on OMAP5 for the most part is very similar > > to that of OMAP4, and had leveraged common data structures and > > register layout definitions so far. Upon closer inspection, there > > are a few minor differences causing an incorrect decoding and > > reporting of the master NIU upon an error: > > > > 1. The L3_TARG_STDERRLOG_MSTADDR.STDERRLOG_MSTADDR occupies > > 11 bits on OMAP5 as against 8 bits on OMAP4, with the master > > NIU connID encoded in the 6 MSBs of the STDERRLOG_MSTADDR > > field. > > 2. The CLK3 FlagMux component has 1 input source on OMAP4 and 3 > > input sources on OMAP5. The common DEBUGSS source is at a > > different input on each SoC. > > > > Fix the above issues by using a OMAP5-specific compatible property > > and using SoC-specific data where there are differences. > > > > Cc: Nishanth Menon > > Signed-off-by: Suman Anna ... > If tony does not mind dts+driver patch, then except for the above comment: > > Acked-by: Nishanth Menon Looks OK to me for a fix. In general we want to have the .dts changes separated from the driver changes though. Applying into omap-for-v4.1/fixes. Regards, Tony