[PATCH v3 0/3] drm/panel: ilitek-ili9805: convert to *_multi API

Michail Tatas posted 3 patches 3 days, 6 hours ago
There is a newer version of this series
drivers/gpu/drm/panel/panel-ilitek-ili9805.c | 211 ++++++++-----------
1 file changed, 87 insertions(+), 124 deletions(-)
[PATCH v3 0/3] drm/panel: ilitek-ili9805: convert to *_multi API
Posted by Michail Tatas 3 days, 6 hours ago
This series converts panel-ilitek-ili9805 to the
mipi_dsi_*_multi helpers and cleans up the resulting code.

Patch 1: Convert the individual DSI calls to their
*_multi variants

Patch 2: Replace the ILI9805_INSTR table with per
panel init functions using the *_multi API, dropping
struct ili9805_instr

Patch 3: makes ili9805_power_off() and ili9805_deactivate()
return void, since no one checks their return values.

Michail Tatas (3):
  drm/panel: ilitek-ili9805: Use _multi variants
  drm/panel: ilitek-ili9805: Use dsi_multi in init
  drm/panel: ilitek-ili9805: make cleanup funcs void

 drivers/gpu/drm/panel/panel-ilitek-ili9805.c | 211 ++++++++-----------
 1 file changed, 87 insertions(+), 124 deletions(-)

-- 
Changes in v3:
Addresses review feedback from Doug on v2
- Convert hex to lowercase
- Align to match open parenthesis
- Change lines to not cross 100 chars

Changes in v2:
Addresses review feedback from Doug on v1
- Patch 1 remains the same as v1
- Added patch 2 dropping struct ili9805_instr and used
  per-panel init functions
- Added patch 3 making the cleanup functions return void

Revisions
- v1: https://lore.kernel.org/all/allWpYxgHfpKughy@michalis-linux/
- v2: https://lore.kernel.org/all/20260719190358.438699-1-michail.tatas@gmail.com/

The changes are compile tested only. Also I run checkpatch.pl --strict, 
sparse, smatch, gcc W=1, clang W=1 and coccinelle.
2.43.0
Re: [PATCH v3 0/3] drm/panel: ilitek-ili9805: convert to *_multi API
Posted by Doug Anderson 3 days, 3 hours ago
Hi,

On Tue, Jul 21, 2026 at 10:27 AM Michail Tatas <michail.tatas@gmail.com> wrote:
>
> This series converts panel-ilitek-ili9805 to the
> mipi_dsi_*_multi helpers and cleans up the resulting code.
>
> Patch 1: Convert the individual DSI calls to their
> *_multi variants
>
> Patch 2: Replace the ILI9805_INSTR table with per
> panel init functions using the *_multi API, dropping
> struct ili9805_instr
>
> Patch 3: makes ili9805_power_off() and ili9805_deactivate()
> return void, since no one checks their return values.
>
> Michail Tatas (3):
>   drm/panel: ilitek-ili9805: Use _multi variants
>   drm/panel: ilitek-ili9805: Use dsi_multi in init
>   drm/panel: ilitek-ili9805: make cleanup funcs void
>
>  drivers/gpu/drm/panel/panel-ilitek-ili9805.c | 211 ++++++++-----------
>  1 file changed, 87 insertions(+), 124 deletions(-)

This looks great now, thanks! Unless there is more feedback on the
lists, I'll plan to apply to drm-misc-next in ~1 week.

-Doug