On Thu, Sep 04, 2025 at 10:01:55PM +0200, Marek Vasut wrote:
> Add configuration for the 5" Raspberry Pi 720x1280 DSI panel
> based on ili9881. This uses 10px longer horizontal sync pulse
> and 10px shorter HBP to avoid very short hsync pulse.
>
> Signed-off-by: Marek Vasut <marek.vasut+renesas@xxxxxxxxxxx>
> ---
> Cc: Conor Dooley <conor+dt@xxxxxxxxxx>
> Cc: Dave Stevenson <dave.stevenson@xxxxxxxxxxxxxxx>
> Cc: David Airlie <airlied@xxxxxxxxx>
> Cc: Jessica Zhang <jessica.zhang@xxxxxxxxxxxxxxxx>
> Cc: Krzysztof Kozlowski <krzk+dt@xxxxxxxxxx>
> Cc: Maarten Lankhorst <maarten.lankhorst@xxxxxxxxxxxxxxx>
> Cc: Maxime Ripard <mripard@xxxxxxxxxx>
> Cc: Neil Armstrong <neil.armstrong@xxxxxxxxxx>
> Cc: Rob Herring <robh@xxxxxxxxxx>
> Cc: Simona Vetter <simona@xxxxxxxx>
> Cc: Thomas Zimmermann <tzimmermann@xxxxxxx>
> Cc: devicetree@xxxxxxxxxxxxxxx
> Cc: dri-devel@xxxxxxxxxxxxxxxxxxxxx
> Cc: linux-renesas-soc@xxxxxxxxxxxxxxx
> ---
> drivers/gpu/drm/panel/panel-ilitek-ili9881c.c | 214 ++++++++++++++++++
> 1 file changed, 214 insertions(+)
>
> diff --git a/drivers/gpu/drm/panel/panel-ilitek-ili9881c.c b/drivers/gpu/drm/panel/panel-ilitek-ili9881c.c
> index 39929c6ebdad9..98c6b28691e42 100644
> --- a/drivers/gpu/drm/panel/panel-ilitek-ili9881c.c
> +++ b/drivers/gpu/drm/panel/panel-ilitek-ili9881c.c
> @@ -1224,6 +1224,194 @@ static const struct ili9881c_instr am8001280g_init[] = {
> ILI9881C_COMMAND_INSTR(MIPI_DCS_WRITE_POWER_SAVE, 0x00),
> };
>
> +static const struct ili9881c_instr rpi_5inch_init[] = {
> + ILI9881C_SWITCH_PAGE_INSTR(3),
> + ILI9881C_COMMAND_INSTR(0x01, 0x00),
> + ILI9881C_COMMAND_INSTR(0x02, 0x00),
> + ILI9881C_COMMAND_INSTR(0x03, 0x73),
> + ILI9881C_COMMAND_INSTR(0x04, 0x73),
> + ILI9881C_COMMAND_INSTR(0x05, 0x00),
> + ILI9881C_COMMAND_INSTR(0x06, 0x06),
> + ILI9881C_COMMAND_INSTR(0x07, 0x02),
> + ILI9881C_COMMAND_INSTR(0x08, 0x00),
> + ILI9881C_COMMAND_INSTR(0x09, 0x01),
> + ILI9881C_COMMAND_INSTR(0x0a, 0x01),
> + ILI9881C_COMMAND_INSTR(0x0b, 0x01),
> + ILI9881C_COMMAND_INSTR(0x0c, 0x01),
> + ILI9881C_COMMAND_INSTR(0x0d, 0x01),
> + ILI9881C_COMMAND_INSTR(0x0e, 0x01),
> + ILI9881C_COMMAND_INSTR(0x0f, 0x01),
> + ILI9881C_COMMAND_INSTR(0x10, 0x01),
> + ILI9881C_COMMAND_INSTR(0x11, 0x00),
> + ILI9881C_COMMAND_INSTR(0x12, 0x00),
> + ILI9881C_COMMAND_INSTR(0x13, 0x01),
> + ILI9881C_COMMAND_INSTR(0x14, 0x00),
> + ILI9881C_COMMAND_INSTR(0x15, 0x00),
> + ILI9881C_COMMAND_INSTR(0x16, 0x00),
> + ILI9881C_COMMAND_INSTR(0x17, 0x00),
> + ILI9881C_COMMAND_INSTR(0x18, 0x00),
> + ILI9881C_COMMAND_INSTR(0x19, 0x00),
> + ILI9881C_COMMAND_INSTR(0x1a, 0x00),
> + ILI9881C_COMMAND_INSTR(0x1b, 0x00),
> + ILI9881C_COMMAND_INSTR(0x1c, 0x00),
> + ILI9881C_COMMAND_INSTR(0x1d, 0x00),
> + ILI9881C_COMMAND_INSTR(0x1e, 0xc0),
> + ILI9881C_COMMAND_INSTR(0x1f, 0x80),
> + ILI9881C_COMMAND_INSTR(0x20, 0x04),
> + ILI9881C_COMMAND_INSTR(0x21, 0x03),
> + ILI9881C_COMMAND_INSTR(0x22, 0x00),
> + ILI9881C_COMMAND_INSTR(0x23, 0x00),
> + ILI9881C_COMMAND_INSTR(0x24, 0x00),
> + ILI9881C_COMMAND_INSTR(0x25, 0x00),
> + ILI9881C_COMMAND_INSTR(0x26, 0x00),
> + ILI9881C_COMMAND_INSTR(0x27, 0x00),
> + ILI9881C_COMMAND_INSTR(0x28, 0x33),
> + ILI9881C_COMMAND_INSTR(0x29, 0x03),
> + ILI9881C_COMMAND_INSTR(0x2a, 0x00),
> + ILI9881C_COMMAND_INSTR(0x2b, 0x00),
> + ILI9881C_COMMAND_INSTR(0x2c, 0x00),
> + ILI9881C_COMMAND_INSTR(0x2d, 0x00),
> + ILI9881C_COMMAND_INSTR(0x2e, 0x00),
> + ILI9881C_COMMAND_INSTR(0x2f, 0x00),
> + ILI9881C_COMMAND_INSTR(0x30, 0x00),
> + ILI9881C_COMMAND_INSTR(0x31, 0x00),
> + ILI9881C_COMMAND_INSTR(0x32, 0x00),
> + ILI9881C_COMMAND_INSTR(0x33, 0x00),
> + ILI9881C_COMMAND_INSTR(0x34, 0x03),
> + ILI9881C_COMMAND_INSTR(0x35, 0x00),
> + ILI9881C_COMMAND_INSTR(0x36, 0x03),
> + ILI9881C_COMMAND_INSTR(0x37, 0x00),
> + ILI9881C_COMMAND_INSTR(0x38, 0x00),
> + ILI9881C_COMMAND_INSTR(0x39, 0x00),
> + ILI9881C_COMMAND_INSTR(0x3a, 0x00),
> + ILI9881C_COMMAND_INSTR(0x3b, 0x00),
> + ILI9881C_COMMAND_INSTR(0x3c, 0x00),
> + ILI9881C_COMMAND_INSTR(0x3d, 0x00),
> + ILI9881C_COMMAND_INSTR(0x3e, 0x00),
> + ILI9881C_COMMAND_INSTR(0x3f, 0x00),
> + ILI9881C_COMMAND_INSTR(0x40, 0x00),
> + ILI9881C_COMMAND_INSTR(0x41, 0x00),
> + ILI9881C_COMMAND_INSTR(0x42, 0x00),
> + ILI9881C_COMMAND_INSTR(0x43, 0x00),
> + ILI9881C_COMMAND_INSTR(0x44, 0x00),
> + ILI9881C_COMMAND_INSTR(0x50, 0x01),
> + ILI9881C_COMMAND_INSTR(0x51, 0x23),
> + ILI9881C_COMMAND_INSTR(0x52, 0x45),
> + ILI9881C_COMMAND_INSTR(0x53, 0x67),
> + ILI9881C_COMMAND_INSTR(0x54, 0x89),
> + ILI9881C_COMMAND_INSTR(0x55, 0xab),
> + ILI9881C_COMMAND_INSTR(0x56, 0x01),
> + ILI9881C_COMMAND_INSTR(0x57, 0x23),
> + ILI9881C_COMMAND_INSTR(0x58, 0x45),
> + ILI9881C_COMMAND_INSTR(0x59, 0x67),
> + ILI9881C_COMMAND_INSTR(0x5a, 0x89),
> + ILI9881C_COMMAND_INSTR(0x5b, 0xab),
> + ILI9881C_COMMAND_INSTR(0x5c, 0xcd),
> + ILI9881C_COMMAND_INSTR(0x5d, 0xef),
> + ILI9881C_COMMAND_INSTR(0x5e, 0x10),
> + ILI9881C_COMMAND_INSTR(0x5f, 0x09),
> + ILI9881C_COMMAND_INSTR(0x60, 0x08),
> + ILI9881C_COMMAND_INSTR(0x61, 0x0f),
> + ILI9881C_COMMAND_INSTR(0x62, 0x0e),
> + ILI9881C_COMMAND_INSTR(0x63, 0x0d),
> + ILI9881C_COMMAND_INSTR(0x64, 0x0c),
> + ILI9881C_COMMAND_INSTR(0x65, 0x02),
> + ILI9881C_COMMAND_INSTR(0x66, 0x02),
> + ILI9881C_COMMAND_INSTR(0x67, 0x02),
> + ILI9881C_COMMAND_INSTR(0x68, 0x02),
> + ILI9881C_COMMAND_INSTR(0x69, 0x02),
> + ILI9881C_COMMAND_INSTR(0x6a, 0x02),
> + ILI9881C_COMMAND_INSTR(0x6b, 0x02),
> + ILI9881C_COMMAND_INSTR(0x6c, 0x02),
> + ILI9881C_COMMAND_INSTR(0x6d, 0x02),
> + ILI9881C_COMMAND_INSTR(0x6e, 0x02),
> + ILI9881C_COMMAND_INSTR(0x6f, 0x02),
> + ILI9881C_COMMAND_INSTR(0x70, 0x02),
> + ILI9881C_COMMAND_INSTR(0x71, 0x06),
> + ILI9881C_COMMAND_INSTR(0x72, 0x07),
> + ILI9881C_COMMAND_INSTR(0x73, 0x02),
> + ILI9881C_COMMAND_INSTR(0x74, 0x02),
> + ILI9881C_COMMAND_INSTR(0x75, 0x06),
> + ILI9881C_COMMAND_INSTR(0x76, 0x07),
> + ILI9881C_COMMAND_INSTR(0x77, 0x0e),
> + ILI9881C_COMMAND_INSTR(0x78, 0x0f),
> + ILI9881C_COMMAND_INSTR(0x79, 0x0c),
> + ILI9881C_COMMAND_INSTR(0x7a, 0x0d),
> + ILI9881C_COMMAND_INSTR(0x7b, 0x02),
> + ILI9881C_COMMAND_INSTR(0x7c, 0x02),
> + ILI9881C_COMMAND_INSTR(0x7d, 0x02),
> + ILI9881C_COMMAND_INSTR(0x7e, 0x02),
> + ILI9881C_COMMAND_INSTR(0x7f, 0x02),
> + ILI9881C_COMMAND_INSTR(0x80, 0x02),
> + ILI9881C_COMMAND_INSTR(0x81, 0x02),
> + ILI9881C_COMMAND_INSTR(0x82, 0x02),
> + ILI9881C_COMMAND_INSTR(0x83, 0x02),
> + ILI9881C_COMMAND_INSTR(0x84, 0x02),
> + ILI9881C_COMMAND_INSTR(0x85, 0x02),
> + ILI9881C_COMMAND_INSTR(0x86, 0x02),
> + ILI9881C_COMMAND_INSTR(0x87, 0x09),
> + ILI9881C_COMMAND_INSTR(0x88, 0x08),
> + ILI9881C_COMMAND_INSTR(0x89, 0x02),
> + ILI9881C_COMMAND_INSTR(0x8A, 0x02),
> + ILI9881C_SWITCH_PAGE_INSTR(4),
> + ILI9881C_COMMAND_INSTR(0x6C, 0x15),
> + ILI9881C_COMMAND_INSTR(0x6E, 0x2a),
> + ILI9881C_COMMAND_INSTR(0x6F, 0x57),
> + ILI9881C_COMMAND_INSTR(0x3A, 0xa4),
> + ILI9881C_COMMAND_INSTR(0x8D, 0x1a),
> + ILI9881C_COMMAND_INSTR(0x87, 0xba),
> + ILI9881C_COMMAND_INSTR(0x26, 0x76),
> + ILI9881C_COMMAND_INSTR(0xB2, 0xd1),
> + ILI9881C_SWITCH_PAGE_INSTR(1),
> + ILI9881C_COMMAND_INSTR(0x22, 0x0A),
> + ILI9881C_COMMAND_INSTR(0x31, 0x00),
> + ILI9881C_COMMAND_INSTR(0x53, 0x35),
> + ILI9881C_COMMAND_INSTR(0x55, 0x50),
> + ILI9881C_COMMAND_INSTR(0x50, 0xaf),
> + ILI9881C_COMMAND_INSTR(0x51, 0xaf),
> + ILI9881C_COMMAND_INSTR(0x60, 0x14),
> + ILI9881C_COMMAND_INSTR(0xA0, 0x08),
> + ILI9881C_COMMAND_INSTR(0xA1, 0x1d),
> + ILI9881C_COMMAND_INSTR(0xA2, 0x2c),
> + ILI9881C_COMMAND_INSTR(0xA3, 0x14),
> + ILI9881C_COMMAND_INSTR(0xA4, 0x19),
> + ILI9881C_COMMAND_INSTR(0xA5, 0x2e),
> + ILI9881C_COMMAND_INSTR(0xA6, 0x22),
> + ILI9881C_COMMAND_INSTR(0xA7, 0x23),
> + ILI9881C_COMMAND_INSTR(0xA8, 0x97),
> + ILI9881C_COMMAND_INSTR(0xA9, 0x1e),
> + ILI9881C_COMMAND_INSTR(0xAA, 0x29),
> + ILI9881C_COMMAND_INSTR(0xAB, 0x7b),
> + ILI9881C_COMMAND_INSTR(0xAC, 0x18),
> + ILI9881C_COMMAND_INSTR(0xAD, 0x17),
> + ILI9881C_COMMAND_INSTR(0xAE, 0x4b),
> + ILI9881C_COMMAND_INSTR(0xAF, 0x1f),
> + ILI9881C_COMMAND_INSTR(0xB0, 0x27),
> + ILI9881C_COMMAND_INSTR(0xB1, 0x52),
> + ILI9881C_COMMAND_INSTR(0xB2, 0x63),
> + ILI9881C_COMMAND_INSTR(0xB3, 0x39),
> + ILI9881C_COMMAND_INSTR(0xC0, 0x08),
> + ILI9881C_COMMAND_INSTR(0xC1, 0x1d),
> + ILI9881C_COMMAND_INSTR(0xC2, 0x2c),
> + ILI9881C_COMMAND_INSTR(0xC3, 0x14),
> + ILI9881C_COMMAND_INSTR(0xC4, 0x19),
> + ILI9881C_COMMAND_INSTR(0xC5, 0x2e),
> + ILI9881C_COMMAND_INSTR(0xC6, 0x22),
> + ILI9881C_COMMAND_INSTR(0xC7, 0x23),
> + ILI9881C_COMMAND_INSTR(0xC8, 0x97),
> + ILI9881C_COMMAND_INSTR(0xC9, 0x1e),
> + ILI9881C_COMMAND_INSTR(0xCA, 0x29),
> + ILI9881C_COMMAND_INSTR(0xCB, 0x7b),
> + ILI9881C_COMMAND_INSTR(0xCC, 0x18),
> + ILI9881C_COMMAND_INSTR(0xCD, 0x17),
> + ILI9881C_COMMAND_INSTR(0xCE, 0x4b),
> + ILI9881C_COMMAND_INSTR(0xCF, 0x1f),
> + ILI9881C_COMMAND_INSTR(0xD0, 0x27),
> + ILI9881C_COMMAND_INSTR(0xD1, 0x52),
> + ILI9881C_COMMAND_INSTR(0xD2, 0x63),
> + ILI9881C_COMMAND_INSTR(0xD3, 0x39),
> +};
Should the hex be lower-cased? Other than that LGTM.
--
With best wishes
Dmitry
