[v3,0/2] Modify the method of sending "exit sleep

Message

Zhaoxiong Lv Aug. 7, 2024, 10:04 a.m. UTC

This "exit sleep mode" and "set display on" command needs to 
be sent in LP mode, so move "exit sleep mode" and "set display 
on" command to the init() function.

Modify the Melfas panel init code to satisfy the gamma value of 2.2.

Changes between V3 and V2:
- PATCH 1/2: Modify the commit message and subject.
- PATCH 2/2: No changes.
- Link to v2: https://lore.kernel.org/all/[e-mail]/

Changes between V2 and V1:
- PATCH 1/2: Modify the commit message and subject.
- PATCH 2/2: No changes.
- Link to v1: https://lore.kernel.org/all/[e-mail]/

Zhaoxiong Lv (2):
  drm/panel: jd9365da: Move "exit sleep mode" and "set display on" cmds
  drm/panel: jd9365da: Modify the init code of Melfas

 .../gpu/drm/panel/panel-jadard-jd9365da-h3.c  | 137 +++++++++---------
 1 file changed, 71 insertions(+), 66 deletions(-)

Comments

Doug Anderson Aug. 7, 2024, 9:56 p.m. UTC | #1

Hi,

On Wed, Aug 7, 2024 at 3:04 AM Zhaoxiong Lv
<[e-mail]> wrote:
>
> This "exit sleep mode" and "set display on" command needs to
> be sent in LP mode, so move "exit sleep mode" and "set display
> on" command to the init() function.
>
> Modify the Melfas panel init code to satisfy the gamma value of 2.2.
>
> Changes between V3 and V2:
> - PATCH 1/2: Modify the commit message and subject.
> - PATCH 2/2: No changes.
> - Link to v2: https://lore.kernel.org/all/[e-mail]/
>
> Changes between V2 and V1:
> - PATCH 1/2: Modify the commit message and subject.
> - PATCH 2/2: No changes.
> - Link to v1: https://lore.kernel.org/all/[e-mail]/
>
> Zhaoxiong Lv (2):
>   drm/panel: jd9365da: Move "exit sleep mode" and "set display on" cmds
>   drm/panel: jd9365da: Modify the init code of Melfas
>
>  .../gpu/drm/panel/panel-jadard-jd9365da-h3.c  | 137 +++++++++---------
>  1 file changed, 71 insertions(+), 66 deletions(-)

As per my response in v2 [1], I feel like patch #1 would be best
reviewed by someone with more MIPI DSI experience. My current plan for
this series.

* Snooze for 2 weeks. If someone else has reviewed / landed in the
meantime then great!
* After two weeks, send a warning that I'll apply the series soon
unless someone yells.
* After a few more days, apply the series.

Please yell if you disagree with any of the above.

[2] https://lore.kernel.org/r/CAD=FV=WrMxyxkuCYEbd=aYFaTJKNqGqXr6Re+V=[e-mail]

Neil Armstrong Aug. 19, 2024, 4:03 p.m. UTC | #2

Hi,

On Wed, 07 Aug 2024 18:04:27 +0800, Zhaoxiong Lv wrote:
> This "exit sleep mode" and "set display on" command needs to
> be sent in LP mode, so move "exit sleep mode" and "set display
> on" command to the init() function.
> 
> Modify the Melfas panel init code to satisfy the gamma value of 2.2.
> 
> Changes between V3 and V2:
> - PATCH 1/2: Modify the commit message and subject.
> - PATCH 2/2: No changes.
> - Link to v2: https://lore.kernel.org/all/[e-mail]/
> 
> [...]

Thanks, Applied to https://gitlab.freedesktop.org/drm/misc/kernel.git (drm-misc-next)

[1/2] drm/panel: jd9365da: Move "exit sleep mode" and "set display on" cmds
      https://gitlab.freedesktop.org/drm/misc/kernel/-/commit/97d1f449c359207b2fb5bc62eaefb7e21ad619ae
[2/2] drm/panel: jd9365da: Modify the init code of Melfas
      https://gitlab.freedesktop.org/drm/misc/kernel/-/commit/64ddf5123eff2edf47202e08744c3c14a9d28f59