On 9/2/26 8:33 AM, Julien Stephan wrote:
> Le mer. 26 août 2026 à 23:22, David Lechner <[email protected]> a écrit :
>>
>> Add a driver for the Startek KD070FHFID078 7" DSI panel. This panel is
>> used on the MediaTek Genio 360 EVK board.
>>
>> The display mode and DSI mode flags follow the upstream Linux driver
>> panel-himax-hx8279.c, which is what the "himax,hx8279" fallback
>> compatible in the device tree binds to.
>>
>> Signed-off-by: David Lechner <[email protected]>
>> ---
>> Add a driver for the Startek KD070FHFID078 7" DSI panel.
>>
>> This depends on the mipi_dsi {generic,dcs}_write_seq helper macros series
>> for mipi_dsi_generic_write_seq().
>>
>> Note: no defconfig enables this driver yet, since there is no MediaTek DSI
>> host driver in U-Boot at this point. It will be enabled together with the
>> display pipeline support.
>> ---
>> Changes in v2:
>> - Added missing BACKLIGHT and DM_REGULATOR Kconfig dependencies.
>> - Use the display mode and MIPI_DSI_CLOCK_NON_CONTINUOUS flag from the
>> upstream Linux panel-himax-hx8279.c driver instead of the MediaTek
>> downstream driver.
>> - enable-gpios is now optional, matching the binding.
>> - A missing backlight is now an error instead of a warning.
>> - Do not request reset/enable GPIOs as active, so they are not asserted
>> before the regulators are enabled.
>> - Use mipi_dsi_dcs_exit_sleep_mode()/mipi_dsi_dcs_set_display_on() instead
>> of raw DCS opcodes and check their return values.
>> - Dropped stale InforceComputing/Linaro copyright and author lines.
>> - Dropped the unused dev field and the unreachable mode check.
>> - Link to v1:
>> https://patch.msgid.link/20260814-mtk-video-panel-startek-078-v1-1-5f84a5724...@baylibre.com
>> ---
>> MAINTAINERS | 6 +
>> drivers/video/Kconfig | 10 +
>> drivers/video/Makefile | 1 +
>> drivers/video/panel-startek-kd070fhfid078.c | 283
>> ++++++++++++++++++++++++++++
>> 4 files changed, 300 insertions(+)
>>
>> diff --git a/MAINTAINERS b/MAINTAINERS
>> index fabcb8b2968..b3bf5b1f8c7 100644
>> --- a/MAINTAINERS
>> +++ b/MAINTAINERS
>> @@ -1723,6 +1723,12 @@ R: GSS_MTK_Uboot_upstream
>> <[email protected]>
>> S: Maintained
>> F: drivers/video/panel-startek-kd070fhfid015.c
>>
>> +STARTEK KD070FHFID078 PANEL
>> +M: David Lechner <[email protected]>
>> +R: GSS_MTK_Uboot_upstream <[email protected]>
>> +S: Maintained
>> +F: drivers/video/panel-startek-kd070fhfid078.c
>> +
>> TARGET_BCMNS3
>> M: Rayagonda Kokatanur <[email protected]>
>> S: Maintained
>> diff --git a/drivers/video/Kconfig b/drivers/video/Kconfig
>> index 37ab17f0ab2..495e4382a22 100644
>> --- a/drivers/video/Kconfig
>> +++ b/drivers/video/Kconfig
>> @@ -330,6 +330,16 @@ config PANEL_STARTEK_KD070FHFID015
>> 1200 x 1920 pixels. It provides a MIPI DSI interface to the host, a
>> built-in LED backlight and touch controller.
>>
>> +config PANEL_STARTEK_KD070FHFID078
>> + bool "STARTEK KD070FHFID078 panel"
>> + depends on PANEL && BACKLIGHT && DM_GPIO && DM_REGULATOR
>> + select VIDEO_MIPI_DSI
>> + help
>> + Say Y here if you want to enable support for STARTEK KD070FHFID078
>> DSI panel.
>> + The panel is a 7-inch TFT LCD display with a resolution of 1200 x
>> 1920
>> + pixels. It provides a MIPI DSI interface to the host, a built-in
>> LED
>> + backlight and touch controller.
>
> This looks like a copy/paste from PANEL_STARTEK_KD070FHFID015. The
> built-in LED backlight is wrong for this panel. drop it.
>
Not sure what you mean about the backlight. The panel certainly has one.
Maybe you meant that the backlight is controlled differently? Not sure we
need to change the text here though.
