| Message ID | 1457712748-1408-1-git-send-email-nm@ti.com (mailing list archive) |
|---|---|
| State | New, archived |
| Headers | show |
Hi Nishanth, Thank you for the patch. On Friday 11 March 2016 10:12:28 Nishanth Menon wrote: > The following commits: > commit 3fa609755c11 ("ARM: omap2: restore OMAP4 barrier behaviour") > commit f746929ffdc8 ("Revert "ARM: OMAP4: remove dead kconfig option > OMAP4_ERRATA_I688"") and > commit ea827ad5ffbb ("ARM: DRA7: Provide proper IO map table") > came in around the same time, unfortunately this seem to have missed > initializing the barrier for DRA7 platforms - omap5_map_io was reused > for dra7 till it was split out by the last patch. barrier_init > needs to be hence carried forward as it is valid for DRA7 family of > processors as they are for OMAP5. > > Fixes: ea827ad5ffbb7 ("ARM: DRA7: Provide proper IO map table") > Reported-by: Laurent Pinchart <laurent.pinchart@ideasonboard.com> > Reported-by: Tomi Valkeinen <tomi.valkeinen@ti.com> > Cc: Russell King <rmk+kernel@arm.linux.org.uk> > Signed-off-by: Nishanth Menon <nm@ti.com> Reviewed-by: Laurent Pinchart <laurent.pinchart@ideasonboard.com> > --- > > Original thread of report: > http://www.spinics.net/lists/dri-devel/msg101246.html > > arch/arm/mach-omap2/io.c | 1 + > 1 file changed, 1 insertion(+) > > diff --git a/arch/arm/mach-omap2/io.c b/arch/arm/mach-omap2/io.c > index 3c87e40650cf..9821be6dfd5e 100644 > --- a/arch/arm/mach-omap2/io.c > +++ b/arch/arm/mach-omap2/io.c > @@ -368,6 +368,7 @@ void __init omap5_map_io(void) > void __init dra7xx_map_io(void) > { > iotable_init(dra7xx_io_desc, ARRAY_SIZE(dra7xx_io_desc)); > + omap_barriers_init(); > } > #endif > /*
* Laurent Pinchart <laurent.pinchart@ideasonboard.com> [160311 08:46]: > Hi Nishanth, > > Thank you for the patch. > > On Friday 11 March 2016 10:12:28 Nishanth Menon wrote: > > The following commits: > > commit 3fa609755c11 ("ARM: omap2: restore OMAP4 barrier behaviour") > > commit f746929ffdc8 ("Revert "ARM: OMAP4: remove dead kconfig option > > OMAP4_ERRATA_I688"") and > > commit ea827ad5ffbb ("ARM: DRA7: Provide proper IO map table") > > came in around the same time, unfortunately this seem to have missed > > initializing the barrier for DRA7 platforms - omap5_map_io was reused > > for dra7 till it was split out by the last patch. barrier_init > > needs to be hence carried forward as it is valid for DRA7 family of > > processors as they are for OMAP5. > > > > Fixes: ea827ad5ffbb7 ("ARM: DRA7: Provide proper IO map table") > > Reported-by: Laurent Pinchart <laurent.pinchart@ideasonboard.com> > > Reported-by: Tomi Valkeinen <tomi.valkeinen@ti.com> > > Cc: Russell King <rmk+kernel@arm.linux.org.uk> > > Signed-off-by: Nishanth Menon <nm@ti.com> > > Reviewed-by: Laurent Pinchart <laurent.pinchart@ideasonboard.com> Applying into omap-for-v4.6/fixes with Cc stable thanks. Tony -- To unsubscribe from this list: send the line "unsubscribe linux-omap" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
diff --git a/arch/arm/mach-omap2/io.c b/arch/arm/mach-omap2/io.c index 3c87e40650cf..9821be6dfd5e 100644 --- a/arch/arm/mach-omap2/io.c +++ b/arch/arm/mach-omap2/io.c @@ -368,6 +368,7 @@ void __init omap5_map_io(void) void __init dra7xx_map_io(void) { iotable_init(dra7xx_io_desc, ARRAY_SIZE(dra7xx_io_desc)); + omap_barriers_init(); } #endif /*
The following commits: commit 3fa609755c11 ("ARM: omap2: restore OMAP4 barrier behaviour") commit f746929ffdc8 ("Revert "ARM: OMAP4: remove dead kconfig option OMAP4_ERRATA_I688"") and commit ea827ad5ffbb ("ARM: DRA7: Provide proper IO map table") came in around the same time, unfortunately this seem to have missed initializing the barrier for DRA7 platforms - omap5_map_io was reused for dra7 till it was split out by the last patch. barrier_init needs to be hence carried forward as it is valid for DRA7 family of processors as they are for OMAP5. Fixes: ea827ad5ffbb7 ("ARM: DRA7: Provide proper IO map table") Reported-by: Laurent Pinchart <laurent.pinchart@ideasonboard.com> Reported-by: Tomi Valkeinen <tomi.valkeinen@ti.com> Cc: Russell King <rmk+kernel@arm.linux.org.uk> Signed-off-by: Nishanth Menon <nm@ti.com> --- Original thread of report: http://www.spinics.net/lists/dri-devel/msg101246.html arch/arm/mach-omap2/io.c | 1 + 1 file changed, 1 insertion(+)