From mboxrd@z Thu Jan 1 00:00:00 1970 From: Petr Cvek Date: Thu, 20 Aug 2015 20:07:45 +0000 Subject: Re: [PATCH v1, TRIVIAL] Fix pxafb nonstd field documentation and add errata info Message-Id: <55D63391.3080403@tul.cz> List-Id: References: <55C64864.6070405@tul.cz> <55D5B232.9090006@ti.com> In-Reply-To: <55D5B232.9090006@ti.com> MIME-Version: 1.0 Content-Type: text/plain; charset="iso-8859-1" Content-Transfer-Encoding: quoted-printable To: linux-arm-kernel@lists.infradead.org Dne 20.8.2015 v 12:55 Tomi Valkeinen napsal(a): >=20 >=20 > On 08/08/15 21:20, Petr Cvek wrote: >=20 >> diff --git a/drivers/video/fbdev/Kconfig b/drivers/video/fbdev/Kconfig >> index 2d98de5..8f485ec 100644 >> --- a/drivers/video/fbdev/Kconfig >> +++ b/drivers/video/fbdev/Kconfig >> @@ -1893,6 +1893,11 @@ config FB_PXA_OVERLAY >> bool "Support PXA27x/PXA3xx Overlay(s) as framebuffer" >> default n >> depends on FB_PXA && (PXA27x || PXA3xx) >> + ---help--- >> + Using the overlay 2 and YUV420 mode on a PXA27x C0 chip revision will >> + hang the LCD until the next SoC restart (errata E24). >=20 > Shouldn't the driver handle this? It should not allow the combination to > be used. >=20 > Tomi >=20 Probably, but I don't know which revisions are OK and which are not. Errata= document: Marvell=AE PXA270M Processor Specification Update MV-S900957-00 has only 3 revisions, but PXA Manual lists 6 (2.2.5.1 Processor ID Register= ) and one revision from errata is not shown in my PXA Manual. I can exclude= mine revision (from "system_rev" variable), but there should be some warni= ng somewhere in configuration. BTW There is software workaround, but I was not able to set registers right= (it requires to send some data through FIFO in different mode). Petr From mboxrd@z Thu Jan 1 00:00:00 1970 From: petr.cvek@tul.cz (Petr Cvek) Date: Thu, 20 Aug 2015 22:07:45 +0200 Subject: [PATCH v1, TRIVIAL] Fix pxafb nonstd field documentation and add errata info In-Reply-To: <55D5B232.9090006@ti.com> References: <55C64864.6070405@tul.cz> <55D5B232.9090006@ti.com> Message-ID: <55D63391.3080403@tul.cz> To: linux-arm-kernel@lists.infradead.org List-Id: linux-arm-kernel.lists.infradead.org Dne 20.8.2015 v 12:55 Tomi Valkeinen napsal(a): > > > On 08/08/15 21:20, Petr Cvek wrote: > >> diff --git a/drivers/video/fbdev/Kconfig b/drivers/video/fbdev/Kconfig >> index 2d98de5..8f485ec 100644 >> --- a/drivers/video/fbdev/Kconfig >> +++ b/drivers/video/fbdev/Kconfig >> @@ -1893,6 +1893,11 @@ config FB_PXA_OVERLAY >> bool "Support PXA27x/PXA3xx Overlay(s) as framebuffer" >> default n >> depends on FB_PXA && (PXA27x || PXA3xx) >> + ---help--- >> + Using the overlay 2 and YUV420 mode on a PXA27x C0 chip revision will >> + hang the LCD until the next SoC restart (errata E24). > > Shouldn't the driver handle this? It should not allow the combination to > be used. > > Tomi > Probably, but I don't know which revisions are OK and which are not. Errata document: Marvell? PXA270M Processor Specification Update MV-S900957-00 has only 3 revisions, but PXA Manual lists 6 (2.2.5.1 Processor ID Register) and one revision from errata is not shown in my PXA Manual. I can exclude mine revision (from "system_rev" variable), but there should be some warning somewhere in configuration. BTW There is software workaround, but I was not able to set registers right (it requires to send some data through FIFO in different mode). Petr