On 9/20/23 12:31, Lucas Stach wrote:
The comment regarding AXI bust size configuration is a bit hard
to read. Improve the wording somewhat.
Signed-off-by: Lucas Stach <l.stach@xxxxxxxxxxxxxx>
---
drivers/gpu/drm/mxsfb/lcdif_kms.c | 8 ++++----
1 file changed, 4 insertions(+), 4 deletions(-)
diff --git a/drivers/gpu/drm/mxsfb/lcdif_kms.c b/drivers/gpu/drm/mxsfb/lcdif_kms.c
index 2541d2de4e45..f5bfe8b52920 100644
--- a/drivers/gpu/drm/mxsfb/lcdif_kms.c
+++ b/drivers/gpu/drm/mxsfb/lcdif_kms.c
@@ -329,12 +329,12 @@ static void lcdif_set_mode(struct lcdif_drm_private *lcdif, u32 bus_flags)
lcdif->base + LCDC_V8_CTRLDESCL0_1);
/*
- * Undocumented P_SIZE and T_SIZE register but those written in the
- * downstream kernel those registers control the AXI burst size. As of
- * now there are two known values:
+ * Undocumented P_SIZE and T_SIZE bitfields written in the downstream
s@written@documented only@ or so, since yeah, those fields are only used
in the downstream driver and miss in the TRM .
+ * driver. Those bitfields control the AXI burst size. As of now there
+ * are two known values:
* 1 - 128Byte
* 2 - 256Byte
- * Downstream set it to 256B burst size to improve the memory
+ * Downstream sets this to 256B burst size to improve the memory access
* efficiency so set it here too.
With that fixed:
Reviewed-by: Marek Vasut <marex@xxxxxxx>
