On Tue, 2022-05-10 at 19:19 +0800, Krzysztof Kozlowski wrote: > 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 Hello Krzysztof, Thanks for your suggestion. We will use hdr-engine to name this node. Thanks! BRs, Rex
