Hi Marek,
The genmask of regsiter SSCR, BPCR & others were setted accordly to the
chipset stm32f4.
You can see more details on page 493 of reference manual RM0090:
https://www.st.com/resource/en/reference_manual/DM00031020-.pdf
With future hardware, all of these registers will aligned on 16bits.
I would like to know if you use a display which resolution exceed 2048.
Best regards
Yannick Fertré
On 10/14/22 14:17, Yannick FERTRE wrote:
Hi Marek,
thanks for the patch.
Reviewed-by: Yannick Fertre <yannick.fer...@foss.st.com>
On 10/12/22 01:10, Marek Vasut wrote:
STM32MP15xx RM0436 Rev 6 "35.7.3 LTDC synchronization size configuration
register (LTDC_SSCR)" on page 1784 and onward indicates VSH and similar
bits are all [11:0] instead of [10:0] wide. Fix this.
[1] https://www.st.com/resource/en/reference_manual/DM00327659-.pdf
Fixes: b759012c5fa7 ("drm/stm: Add STM32 LTDC driver")
Signed-off-by: Marek Vasut <ma...@denx.de>
---
Cc: Alexandre Torgue <alexandre.tor...@foss.st.com>
Cc: Antonio Borneo <antonio.bor...@foss.st.com>
Cc: Benjamin Gaignard <benjamin.gaign...@foss.st.com>
Cc: Maxime Coquelin <mcoquelin.st...@gmail.com>
Cc: Philippe Cornu <philippe.co...@foss.st.com>
Cc: Sam Ravnborg <s...@ravnborg.org>
Cc: Vincent Abriou <vincent.abr...@foss.st.com>
Cc: Yannick Fertre <yannick.fer...@foss.st.com>
Cc: linux-arm-ker...@lists.infradead.org
Cc: linux-st...@st-md-mailman.stormreply.com
To: dri-devel@lists.freedesktop.org
---
drivers/gpu/drm/stm/ltdc.c | 8 ++++----
1 file changed, 4 insertions(+), 4 deletions(-)
diff --git a/drivers/gpu/drm/stm/ltdc.c b/drivers/gpu/drm/stm/ltdc.c
index 03c6becda795c..639ed00b44a57 100644
--- a/drivers/gpu/drm/stm/ltdc.c
+++ b/drivers/gpu/drm/stm/ltdc.c
@@ -111,16 +111,16 @@
#define LTDC_L1FPF1R (ldev->caps.layer_regs[24]) /* L1
Flexible Pixel Format 1 */
/* Bit definitions */
-#define SSCR_VSH GENMASK(10, 0) /* Vertical Synchronization
Height */
+#define SSCR_VSH GENMASK(11, 0) /* Vertical Synchronization
Height */
#define SSCR_HSW GENMASK(27, 16) /* Horizontal
Synchronization Width */
-#define BPCR_AVBP GENMASK(10, 0) /* Accumulated Vertical
Back Porch */
+#define BPCR_AVBP GENMASK(11, 0) /* Accumulated Vertical Back
Porch */
#define BPCR_AHBP GENMASK(27, 16) /* Accumulated Horizontal
Back Porch */
-#define AWCR_AAH GENMASK(10, 0) /* Accumulated Active Height */
+#define AWCR_AAH GENMASK(11, 0) /* Accumulated Active Height */
#define AWCR_AAW GENMASK(27, 16) /* Accumulated Active Width */
-#define TWCR_TOTALH GENMASK(10, 0) /* TOTAL Height */
+#define TWCR_TOTALH GENMASK(11, 0) /* TOTAL Height */
#define TWCR_TOTALW GENMASK(27, 16) /* TOTAL Width */
#define GCR_LTDCEN BIT(0) /* LTDC ENable */
