[PATCH] drm/panel: simple: Fix Innolux G101ICE-L01 bus flags

Marek Vasut marex at denx.de
Fri Nov 17 17:07:00 UTC 2023


On 11/17/23 09:40, Maxime Ripard wrote:
> On Thu, Nov 16, 2023 at 10:15:31PM +0100, Marek Vasut wrote:
>> On 10/9/23 10:58, Neil Armstrong wrote:
>>> On 09/10/2023 00:33, Marek Vasut wrote:
>>>> Add missing .bus_flags = DRM_BUS_FLAG_DE_HIGH to this panel description,
>>>> ones which match both the datasheet and the panel display_timing flags .
>>>>
>>>> Fixes: 1e29b840af9f ("drm/panel: simple: Add Innolux G101ICE-L01 panel")
>>>> Signed-off-by: Marek Vasut <marex at denx.de>
>>>> ---
>>>> Cc: Daniel Vetter <daniel at ffwll.ch>
>>>> Cc: David Airlie <airlied at gmail.com>
>>>> Cc: Jessica Zhang <quic_jesszhan at quicinc.com>
>>>> Cc: Maarten Lankhorst <maarten.lankhorst at linux.intel.com>
>>>> Cc: Maxime Ripard <mripard at kernel.org>
>>>> Cc: Neil Armstrong <neil.armstrong at linaro.org>
>>>> Cc: Sam Ravnborg <sam at ravnborg.org>
>>>> Cc: Thomas Zimmermann <tzimmermann at suse.de>
>>>> Cc: dri-devel at lists.freedesktop.org
>>>> ---
>>>>    drivers/gpu/drm/panel/panel-simple.c | 1 +
>>>>    1 file changed, 1 insertion(+)
>>>>
>>>> diff --git a/drivers/gpu/drm/panel/panel-simple.c
>>>> b/drivers/gpu/drm/panel/panel-simple.c
>>>> index 44c11c418cd56..8e4ea15f0e1e5 100644
>>>> --- a/drivers/gpu/drm/panel/panel-simple.c
>>>> +++ b/drivers/gpu/drm/panel/panel-simple.c
>>>> @@ -2318,6 +2318,7 @@ static const struct panel_desc
>>>> innolux_g101ice_l01 = {
>>>>            .disable = 200,
>>>>        },
>>>>        .bus_format = MEDIA_BUS_FMT_RGB888_1X7X4_SPWG,
>>>> +    .bus_flags = DRM_BUS_FLAG_DE_HIGH,
>>>>        .connector_type = DRM_MODE_CONNECTOR_LVDS,
>>>>    };
>>>
>>> Reviewed-by: Neil Armstrong <neil.armstrong at linaro.org>
>>
>> Maybe it is time to apply ?
> 
> The expectation is that you would kind of apply it yourself. Do you have
> a drm-misc committer account? If not, you should apply for one, you
> definitely qualify.

I do, I'm just not a big fan of applying my own patches, but since there 
is a RB, I will do that in a bit.


More information about the dri-devel mailing list