Re: [PATCH v1 1/2] drm/panel: jd9365da: Move the sending location of the 11/29 command

From: Jani Nikula
Date: Fri Jul 26 2024 - 04:07:09 EST

On Fri, 26 Jul 2024, zhaoxiong lv <lvzhaoxiong@xxxxxxxxxxxxxxxxxxxxxxxxxxxxxx> wrote:
> On Thu, Jul 25, 2024 at 4:41 PM Jani Nikula <jani.nikula@xxxxxxxxxxxxxxx> wrote:
>>
>> On Thu, 25 Jul 2024, Zhaoxiong Lv <lvzhaoxiong@xxxxxxxxxxxxxxxxxxxxxxxxxxxxxx> wrote:
>> > Move the 11/29 command from enable() to init() function
>>
>> OOC, what is the "11/29" command?
>>
>> BR,
>> Jani.
>
> hi Jani
> Sorry, maybe I didn't describe it clearly. The 11/29 commands are sent
> by these two functions.
>
> mipi_dsi_dcs_exit_sleep_mode_multi(&dsi_ctx);
> mipi_dsi_dcs_set_display_on_multi(&dsi_ctx);
>
> MIPI_DCS_EXIT_SLEEP_MODE = 0x11,
> MIPI_DCS_SET_DISPLAY_ON= 0x29,

Maybe refer to the commands with their names then? Exit sleep mode and
set display on.

BR,
Jani.



>
> BR,
>>
>> >
>> > As mentioned in the patch:
>> > https://lore.kernel.org/all/20240624141926.5250-2-lvzhaoxiong@xxxxxxxxxxxxxxxxxxxxxxxxxxxxxx/
>> >
>> > Our DSI host has different modes in prepare() and enable()
>> > functions. prepare() is in LP mode and enable() is in HS mode.
>> > Since the 11/29 command must also be sent in LP mode,
>> > so we also move 11/29 command to the init() function.
>> >
>> > After moving the 11/29 command to the init() function,
>> > we no longer need additional delay judgment, so we delete
>> > variables "exit_sleep_to_display_on_delay_ms" and
>> > "display_on_delay_ms".
>> >
>> > Signed-off-by: Zhaoxiong Lv <lvzhaoxiong@xxxxxxxxxxxxxxxxxxxxxxxxxxxxxx>
>> > ---
>> > .../gpu/drm/panel/panel-jadard-jd9365da-h3.c | 59 ++++++++++---------
>> > 1 file changed, 32 insertions(+), 27 deletions(-)
>> >
>> > diff --git a/drivers/gpu/drm/panel/panel-jadard-jd9365da-h3.c b/drivers/gpu/drm/panel/panel-jadard-jd9365da-h3.c
>> > index 04d315d96bff..ce73e8cb1db5 100644
>> > --- a/drivers/gpu/drm/panel/panel-jadard-jd9365da-h3.c
>> > +++ b/drivers/gpu/drm/panel/panel-jadard-jd9365da-h3.c
>> > @@ -31,8 +31,6 @@ struct jadard_panel_desc {
>> > bool reset_before_power_off_vcioo;
>> > unsigned int vcioo_to_lp11_delay_ms;
>> > unsigned int lp11_to_reset_delay_ms;
>> > - unsigned int exit_sleep_to_display_on_delay_ms;
>> > - unsigned int display_on_delay_ms;
>> > unsigned int backlight_off_to_display_off_delay_ms;
>> > unsigned int display_off_to_enter_sleep_delay_ms;
>> > unsigned int enter_sleep_to_reset_down_delay_ms;
>> > @@ -66,26 +64,6 @@ static inline struct jadard *panel_to_jadard(struct drm_panel *panel)
>> > return container_of(panel, struct jadard, panel);
>> > }
>> >
>> > -static int jadard_enable(struct drm_panel *panel)
>> > -{
>> > - struct jadard *jadard = panel_to_jadard(panel);
>> > - struct mipi_dsi_multi_context dsi_ctx = { .dsi = jadard->dsi };
>> > -
>> > - msleep(120);
>> > -
>> > - mipi_dsi_dcs_exit_sleep_mode_multi(&dsi_ctx);
>> > -
>> > - if (jadard->desc->exit_sleep_to_display_on_delay_ms)
>> > - mipi_dsi_msleep(&dsi_ctx, jadard->desc->exit_sleep_to_display_on_delay_ms);
>> > -
>> > - mipi_dsi_dcs_set_display_on_multi(&dsi_ctx);
>> > -
>> > - if (jadard->desc->display_on_delay_ms)
>> > - mipi_dsi_msleep(&dsi_ctx, jadard->desc->display_on_delay_ms);
>> > -
>> > - return dsi_ctx.accum_err;
>> > -}
>> > -
>> > static int jadard_disable(struct drm_panel *panel)
>> > {
>> > struct jadard *jadard = panel_to_jadard(panel);
>> > @@ -202,7 +180,6 @@ static const struct drm_panel_funcs jadard_funcs = {
>> > .disable = jadard_disable,
>> > .unprepare = jadard_unprepare,
>> > .prepare = jadard_prepare,
>> > - .enable = jadard_enable,
>> > .get_modes = jadard_get_modes,
>> > .get_orientation = jadard_panel_get_orientation,
>> > };
>> > @@ -382,6 +359,12 @@ static int radxa_display_8hd_ad002_init_cmds(struct jadard *jadard)
>> >
>> > jd9365da_switch_page(&dsi_ctx, 0x00);
>> >
>> > + mipi_dsi_msleep(&dsi_ctx, 120);
>> > +
>> > + mipi_dsi_dcs_exit_sleep_mode_multi(&dsi_ctx);
>> > +
>> > + mipi_dsi_dcs_set_display_on_multi(&dsi_ctx);
>> > +
>> > return dsi_ctx.accum_err;
>> > };
>> >
>> > @@ -608,6 +591,12 @@ static int cz101b4001_init_cmds(struct jadard *jadard)
>> > mipi_dsi_dcs_write_seq_multi(&dsi_ctx, 0xE6, 0x02);
>> > mipi_dsi_dcs_write_seq_multi(&dsi_ctx, 0xE7, 0x0C);
>> >
>> > + mipi_dsi_msleep(&dsi_ctx, 120);
>> > +
>> > + mipi_dsi_dcs_exit_sleep_mode_multi(&dsi_ctx);
>> > +
>> > + mipi_dsi_dcs_set_display_on_multi(&dsi_ctx);
>> > +
>> > return dsi_ctx.accum_err;
>> > };
>> >
>> > @@ -831,6 +820,16 @@ static int kingdisplay_kd101ne3_init_cmds(struct jadard *jadard)
>> >
>> > jd9365da_switch_page(&dsi_ctx, 0x00);
>> >
>> > + mipi_dsi_msleep(&dsi_ctx, 120);
>> > +
>> > + mipi_dsi_dcs_exit_sleep_mode_multi(&dsi_ctx);
>> > +
>> > + mipi_dsi_msleep(&dsi_ctx, 120);
>> > +
>> > + mipi_dsi_dcs_set_display_on_multi(&dsi_ctx);
>> > +
>> > + mipi_dsi_msleep(&dsi_ctx, 20);
>> > +
>> > return dsi_ctx.accum_err;
>> > };
>> >
>> > @@ -859,8 +858,6 @@ static const struct jadard_panel_desc kingdisplay_kd101ne3_40ti_desc = {
>> > .reset_before_power_off_vcioo = true,
>> > .vcioo_to_lp11_delay_ms = 5,
>> > .lp11_to_reset_delay_ms = 10,
>> > - .exit_sleep_to_display_on_delay_ms = 120,
>> > - .display_on_delay_ms = 20,
>> > .backlight_off_to_display_off_delay_ms = 100,
>> > .display_off_to_enter_sleep_delay_ms = 50,
>> > .enter_sleep_to_reset_down_delay_ms = 100,
>> > @@ -1074,6 +1071,16 @@ static int melfas_lmfbx101117480_init_cmds(struct jadard *jadard)
>> > mipi_dsi_dcs_write_seq_multi(&dsi_ctx, 0xe6, 0x02);
>> > mipi_dsi_dcs_write_seq_multi(&dsi_ctx, 0xe7, 0x06);
>> >
>> > + mipi_dsi_msleep(&dsi_ctx, 120);
>> > +
>> > + mipi_dsi_dcs_exit_sleep_mode_multi(&dsi_ctx);
>> > +
>> > + mipi_dsi_msleep(&dsi_ctx, 120);
>> > +
>> > + mipi_dsi_dcs_set_display_on_multi(&dsi_ctx);
>> > +
>> > + mipi_dsi_msleep(&dsi_ctx, 20);
>> > +
>> > return dsi_ctx.accum_err;
>> > };
>> >
>> > @@ -1102,8 +1109,6 @@ static const struct jadard_panel_desc melfas_lmfbx101117480_desc = {
>> > .reset_before_power_off_vcioo = true,
>> > .vcioo_to_lp11_delay_ms = 5,
>> > .lp11_to_reset_delay_ms = 10,
>> > - .exit_sleep_to_display_on_delay_ms = 120,
>> > - .display_on_delay_ms = 20,
>> > .backlight_off_to_display_off_delay_ms = 100,
>> > .display_off_to_enter_sleep_delay_ms = 50,
>> > .enter_sleep_to_reset_down_delay_ms = 100,
>>
>> --
>> Jani Nikula, Intel

--
Jani Nikula, Intel