lkml.org 
[lkml]   [2023]   [Feb]   [5]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
    /
    From
    Date
    SubjectRe: [PATCH v1 RFC] video/hdmi: Fix HDMI_VENDOR_INFOFRAME_SIZE
    Hello Jani, Hello Ville,

    On Tue, Jan 10, 2023 at 7:20 PM Jani Nikula <jani.nikula@linux.intel.com> wrote:
    [...]
    > > I'm not an expert on this topic and I'm not sure if the size still
    > > depends on that if condition from long time ago. So please share your
    > > thoughts.
    >
    > I tried to look at this quickly, but it makes my brain hurt. I don't
    > think simply changing the size here is right either.
    I think I see what you're saying here: hdmi_vendor_infoframe_length()
    has logic to determine the "correct" size.

    My idea here is to use the maximum possible size for
    HDMI_VENDOR_INFOFRAME_SIZE so it can be used with the
    HDMI_INFOFRAME_SIZE macro (just like the other _SIZE definitions right
    above the vendor infoframe one).
    If you have suggestions on my patch then please let me know.


    Best regards,
    Martin

    \
     
     \ /
      Last update: 2023-03-27 00:10    [W:3.869 / U:0.104 seconds]
    ©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site