[PATCH v4 3/5] drm/panel: samsung-s6e88a0-ams427ap24: Add initial driver

[Linus Walleij]

On Mon, Oct 28, 2024 at 12:42 AM Jakob Hauser <jahau at rocketmail.com> wrote:

> This initial part of the panel driver was mostly generated by the
> "linux-mdss-dsi-panel-driver-generator" tool [1], reading downstream
> Android kernel file "dsi_panel_S6E88A0_AMS427AP24_qhd_octa_video.dtsi" [2].
>
> On top of the generic output of the tool, there were a couple of changes
> applied:
> - Added mipi_dsi_dcs_set_display_on() to function s6e88a0_ams427ap24_on(),
>   otherwise the display does not show up.
> - In functions s6e88a0_ams427ap24_on() and s6e88a0_ams427ap24_off()
>   changed DSI commands to multi context and used "accum_err" returns.
> - In functions s6e88a0_ams427ap24_on() and s6e88a0_ams427ap24_off() replaced
>   msleep() by mipi_dsi_msleep().
> - The function s6e88a0_ams427ap24_get_modes() was changed to make use of
>   drm_connector_helper_get_modes_fixed(). This also required to include
>   drm/drm_probe_helper.h.
> - In function s6e88a0_ams427ap24_probe() registring the regulators was changed
>   to devm_regulator_bulk_get_const(). This required to change supplies in struct
>   s6e88a0_ams427ap24 to a pointer.
> - Removed bool "prepared" from struct s6e88a0_ams427ap24 and according parts in
>   functions s6e88a0_ams427ap24_prepare() and s6e88a0_ams427ap24_unprepare().
> - Removed include <linux/of.h>, it's not needed.
> - Added comments to the mipi_dsi_dcs_write_seq_multi() lines in function
>   s6e88a0_ams427ap24_on().
>
> [1] https://github.com/msm8916-mainline/linux-mdss-dsi-panel-driver-generator
> [2] https://github.com/msm8916-mainline/linux-downstream/blob/GT-I9195I/drivers/video/msm/mdss/samsung/S6E88A0_AMS427AP24/dsi_panel_S6E88A0_AMS427AP24_qhd_octa_video.dtsi
>
> Signed-off-by: Jakob Hauser <jahau at rocketmail.com>

Comments instead of #defines?

OK then, it's readable too.

Reviewed-by: Linus Walleij <linus.walleij at linaro.org>

Yours,
Linus Walleij