Re: [PATCH v16 4/4] drm/bridge: dw-hdmi: fix bus formats negotiation for 8 bit modes

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Hi Neil,

> Am 03.03.2022 um 12:45 schrieb H. Nikolaus Schaller <hns@xxxxxxxxxxxxx>:
> 
> Hi Neil,
> 
>> Am 03.03.2022 um 12:42 schrieb Neil Armstrong <narmstrong@xxxxxxxxxxxx>:
>> 
>>> So how can we get that merged? IMHO your proposal should be before we add ingenic-dw-hdmi.
>>> If you have a version with proper commit message I can add it to the beginning of my
>>> seried and include it in a v17. Or if you get yours merged to drm-misc/drm-misc-next I
>>> can build on top.
>> 
>> You can add it in your v17 patchset with my authorship and my Signed-off-by tag + yours.
>> 
>> As commit message something like :
>> ====================
>> drm/bridge: dw-hdmi: handle unusable or non-configured CSC module
>> 
>> The dw-hdmi integrates an optional Color Space Conversion feature used
>> to handle color-space conversions.
>> 
>> On some platforms, the CSC isn't built-in or non-functional.
>> 
>> This adds the necessary code to disable the CSC functionality
>> and limit the bus format negotiation to force using the same
>> input bus format as the output bus format.
>> ====================
> 
> Fine! Will do.

I was a little too early.

While preparing the patches I found that I still had the hack to force
sink_is_hdmi = false in my test branch. Sorry for that.

Removing this made the panel go black again, even with your latest
proposal.

So I looked deeper into your patch and it seems to influence the
input formats only in dw_hdmi_bridge_atomic_get_input_bus_fmts()?

While the problem I see is with output formats and we had worked on
modifying dw_hdmi_bridge_atomic_get_output_bus_fmts().

BR and thanks,
Nikolaus





[Index of Archives]     [Linux DRI Users]     [Linux Intel Graphics]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]     [XFree86]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [XFree86]
  Powered by Linux