lkml.org 
[lkml]   [2022]   [May]   [10]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
    /
    Date
    SubjectRe: [PATCH v2 3/3] dt-bindings: mediatek: add ethdr definition for mt8195
    From
    On 10/05/2022 03:46, Rex-BC Chen wrote:
    >>
    >>
    >> That's mediatek-drm, and this refers to the HDR block in the display
    >> IP...
    >>
    >> Though, I have no idea of what "ET" stands for in "ETHDR", so, it
    >> would be
    >> definitely nice if MediaTek can write the meaning in the description,
    >> like
    >>
    >> description:
    >> ETHDR (E??? T??? High Dynamic Range) is designed for HDR video and
    >> ...blah
    >>
    >> Cheers,
    >> Angelo
    >
    > Hello Krzysztof and Angelo,
    >
    > "ET" is actually meaningless.
    > ET is one of mediatek departments and it's where the engine from.
    > Therefore, I think we will add description like this:
    >> ETHDR (ET High Dynamic Range) is a MediaTek internal HDR engine and
    >> designed for HDR video...

    Sure, sounds good, but then the node name should not have it. Please try
    to find some more generic name (DT spec gives examples). Could be
    display-controller, "hdr-engine", "isp".


    Best regards,
    Krzysztof

    \
     
     \ /
      Last update: 2022-05-10 13:21    [W:2.827 / U:0.032 seconds]
    ©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site