From: Tony Lindgren <tony@atomide.com>
To: Nishanth Menon <nm@ti.com>
Cc: Kristian Otnes <kotnes@cisco.com>,
Peter Ujfalusi <peter.ujfalusi@ti.com>,
Santosh Shilimkar <santosh.shilimkar@ti.com>,
linux-omap@vger.kernel.org, linux-arm-kernel@lists.infradead.org,
linux-kernel@vger.kernel.org
Subject: Re: [PATCH] bus: omap_l3_noc: Fix connID for OMAP4
Date: Thu, 11 Sep 2014 10:44:28 -0700 [thread overview]
Message-ID: <20140911174428.GC18849@atomide.com> (raw)
In-Reply-To: <1410344427-26006-1-git-send-email-nm@ti.com>
* Nishanth Menon <nm@ti.com> [140910 03:21]:
> Commit d4d8819e205854c ("bus: omap_l3_noc: fix masterid detection")
> did the right thing in dropping the LSB 2 bits which is not part
> of the ConnID for NTTP master address. However, as part of that
> change, we should also have ensured that existing list of OMAP4 connID
> codes are also shifted by 2 bits to ensure that connIDs map to "Table
> 13-18. ConnID Values" as provided in Technical Reference Manuals for
> OMAP4430(Rev AP, April 2014, SWPU220AP) and OMAP4460(Rev AB, April
> 2014, SWPU234AB)
>
> Fixes: d4d8819e205854c ("bus: omap_l3_noc: fix masterid detection")
> Reported-by: Kristian Otnes <kotnes@cisco.com>
> Signed-off-by: Nishanth Menon <nm@ti.com>
> ---
>
> Based on v3.17-rc1 - but might be 3.17 rc material
Yes as it's a regression. Applying into omap-for-v3.17/fixes-v3.
Tony
> Quick links (seems to work on firefox):
> 4430:
> http://www.ti.com/lit/ug/swpu231ap/swpu231ap.pdf#page=3030&zoom=auto,0,699.5
> 4460:
> http://www.ti.com/lit/ug/swpu235ab/swpu235ab.pdf#page=3073&zoom=auto,0,699.5
>
> NOTE: there are a few more cleanup patches that can follow these..
> will leave those out for now.
>
> drivers/bus/omap_l3_noc.h | 50 ++++++++++++++++++++++-----------------------
> 1 file changed, 25 insertions(+), 25 deletions(-)
>
> diff --git a/drivers/bus/omap_l3_noc.h b/drivers/bus/omap_l3_noc.h
> index 551e010..9525458 100644
> --- a/drivers/bus/omap_l3_noc.h
> +++ b/drivers/bus/omap_l3_noc.h
> @@ -188,31 +188,31 @@ static struct l3_flagmux_data omap_l3_flagmux_clk3 = {
> };
>
> static struct l3_masters_data omap_l3_masters[] = {
> - { 0x0 , "MPU"},
> - { 0x10, "CS_ADP"},
> - { 0x14, "xxx"},
> - { 0x20, "DSP"},
> - { 0x30, "IVAHD"},
> - { 0x40, "ISS"},
> - { 0x44, "DucatiM3"},
> - { 0x48, "FaceDetect"},
> - { 0x50, "SDMA_Rd"},
> - { 0x54, "SDMA_Wr"},
> - { 0x58, "xxx"},
> - { 0x5C, "xxx"},
> - { 0x60, "SGX"},
> - { 0x70, "DSS"},
> - { 0x80, "C2C"},
> - { 0x88, "xxx"},
> - { 0x8C, "xxx"},
> - { 0x90, "HSI"},
> - { 0xA0, "MMC1"},
> - { 0xA4, "MMC2"},
> - { 0xA8, "MMC6"},
> - { 0xB0, "UNIPRO1"},
> - { 0xC0, "USBHOSTHS"},
> - { 0xC4, "USBOTGHS"},
> - { 0xC8, "USBHOSTFS"}
> + { 0x00, "MPU"},
> + { 0x04, "CS_ADP"},
> + { 0x05, "xxx"},
> + { 0x08, "DSP"},
> + { 0x0C, "IVAHD"},
> + { 0x10, "ISS"},
> + { 0x11, "DucatiM3"},
> + { 0x12, "FaceDetect"},
> + { 0x14, "SDMA_Rd"},
> + { 0x15, "SDMA_Wr"},
> + { 0x16, "xxx"},
> + { 0x17, "xxx"},
> + { 0x18, "SGX"},
> + { 0x1C, "DSS"},
> + { 0x20, "C2C"},
> + { 0x22, "xxx"},
> + { 0x23, "xxx"},
> + { 0x24, "HSI"},
> + { 0x28, "MMC1"},
> + { 0x29, "MMC2"},
> + { 0x2A, "MMC6"},
> + { 0x2C, "UNIPRO1"},
> + { 0x30, "USBHOSTHS"},
> + { 0x31, "USBOTGHS"},
> + { 0x32, "USBHOSTFS"}
> };
>
> static struct l3_flagmux_data *omap_l3_flagmux[] = {
> --
> 1.7.9.5
>
WARNING: multiple messages have this Message-ID (diff)
From: tony@atomide.com (Tony Lindgren)
To: linux-arm-kernel@lists.infradead.org
Subject: [PATCH] bus: omap_l3_noc: Fix connID for OMAP4
Date: Thu, 11 Sep 2014 10:44:28 -0700 [thread overview]
Message-ID: <20140911174428.GC18849@atomide.com> (raw)
In-Reply-To: <1410344427-26006-1-git-send-email-nm@ti.com>
* Nishanth Menon <nm@ti.com> [140910 03:21]:
> Commit d4d8819e205854c ("bus: omap_l3_noc: fix masterid detection")
> did the right thing in dropping the LSB 2 bits which is not part
> of the ConnID for NTTP master address. However, as part of that
> change, we should also have ensured that existing list of OMAP4 connID
> codes are also shifted by 2 bits to ensure that connIDs map to "Table
> 13-18. ConnID Values" as provided in Technical Reference Manuals for
> OMAP4430(Rev AP, April 2014, SWPU220AP) and OMAP4460(Rev AB, April
> 2014, SWPU234AB)
>
> Fixes: d4d8819e205854c ("bus: omap_l3_noc: fix masterid detection")
> Reported-by: Kristian Otnes <kotnes@cisco.com>
> Signed-off-by: Nishanth Menon <nm@ti.com>
> ---
>
> Based on v3.17-rc1 - but might be 3.17 rc material
Yes as it's a regression. Applying into omap-for-v3.17/fixes-v3.
Tony
> Quick links (seems to work on firefox):
> 4430:
> http://www.ti.com/lit/ug/swpu231ap/swpu231ap.pdf#page=3030&zoom=auto,0,699.5
> 4460:
> http://www.ti.com/lit/ug/swpu235ab/swpu235ab.pdf#page=3073&zoom=auto,0,699.5
>
> NOTE: there are a few more cleanup patches that can follow these..
> will leave those out for now.
>
> drivers/bus/omap_l3_noc.h | 50 ++++++++++++++++++++++-----------------------
> 1 file changed, 25 insertions(+), 25 deletions(-)
>
> diff --git a/drivers/bus/omap_l3_noc.h b/drivers/bus/omap_l3_noc.h
> index 551e010..9525458 100644
> --- a/drivers/bus/omap_l3_noc.h
> +++ b/drivers/bus/omap_l3_noc.h
> @@ -188,31 +188,31 @@ static struct l3_flagmux_data omap_l3_flagmux_clk3 = {
> };
>
> static struct l3_masters_data omap_l3_masters[] = {
> - { 0x0 , "MPU"},
> - { 0x10, "CS_ADP"},
> - { 0x14, "xxx"},
> - { 0x20, "DSP"},
> - { 0x30, "IVAHD"},
> - { 0x40, "ISS"},
> - { 0x44, "DucatiM3"},
> - { 0x48, "FaceDetect"},
> - { 0x50, "SDMA_Rd"},
> - { 0x54, "SDMA_Wr"},
> - { 0x58, "xxx"},
> - { 0x5C, "xxx"},
> - { 0x60, "SGX"},
> - { 0x70, "DSS"},
> - { 0x80, "C2C"},
> - { 0x88, "xxx"},
> - { 0x8C, "xxx"},
> - { 0x90, "HSI"},
> - { 0xA0, "MMC1"},
> - { 0xA4, "MMC2"},
> - { 0xA8, "MMC6"},
> - { 0xB0, "UNIPRO1"},
> - { 0xC0, "USBHOSTHS"},
> - { 0xC4, "USBOTGHS"},
> - { 0xC8, "USBHOSTFS"}
> + { 0x00, "MPU"},
> + { 0x04, "CS_ADP"},
> + { 0x05, "xxx"},
> + { 0x08, "DSP"},
> + { 0x0C, "IVAHD"},
> + { 0x10, "ISS"},
> + { 0x11, "DucatiM3"},
> + { 0x12, "FaceDetect"},
> + { 0x14, "SDMA_Rd"},
> + { 0x15, "SDMA_Wr"},
> + { 0x16, "xxx"},
> + { 0x17, "xxx"},
> + { 0x18, "SGX"},
> + { 0x1C, "DSS"},
> + { 0x20, "C2C"},
> + { 0x22, "xxx"},
> + { 0x23, "xxx"},
> + { 0x24, "HSI"},
> + { 0x28, "MMC1"},
> + { 0x29, "MMC2"},
> + { 0x2A, "MMC6"},
> + { 0x2C, "UNIPRO1"},
> + { 0x30, "USBHOSTHS"},
> + { 0x31, "USBOTGHS"},
> + { 0x32, "USBHOSTFS"}
> };
>
> static struct l3_flagmux_data *omap_l3_flagmux[] = {
> --
> 1.7.9.5
>
next prev parent reply other threads:[~2014-09-11 17:44 UTC|newest]
Thread overview: 5+ messages / expand[flat|nested] mbox.gz Atom feed top
2014-09-10 10:20 [PATCH] bus: omap_l3_noc: Fix connID for OMAP4 Nishanth Menon
2014-09-10 10:20 ` Nishanth Menon
2014-09-10 10:20 ` Nishanth Menon
2014-09-11 17:44 ` Tony Lindgren [this message]
2014-09-11 17:44 ` Tony Lindgren
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20140911174428.GC18849@atomide.com \
--to=tony@atomide.com \
--cc=kotnes@cisco.com \
--cc=linux-arm-kernel@lists.infradead.org \
--cc=linux-kernel@vger.kernel.org \
--cc=linux-omap@vger.kernel.org \
--cc=nm@ti.com \
--cc=peter.ujfalusi@ti.com \
--cc=santosh.shilimkar@ti.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.