Re: [PATCH v2 04/10] drm: Add Gamma correction structure

Emil Velikov
2015-06-18T15:00:50Z

On 14 June 2015 at 10:02, Sharma, Shashank <shashank.sharma@intel.com> wrote:
> Hi, Emil Velikov
>
> The reason behind a zero sized array is that we want to use the same variable for various color correction possible across various driver .
> Due to current blob implementation, it doesn’t look very efficient to have another pointer in the structure, so we are left with this option only.
>
Can you elaborate (to suggest any reading material) about those inefficiencies ?

> I guess as long as we are using gcc (which is for all Linux distributions), we are good. The size of the zero sized array will be zero, so no alignment errors as such.
>
Note that most of the DRM subsystem code is dual-licensed. As such it
is used in other OSes - Solaris, *BSD, not to mention the work (in
progress) about using clang/LLVM to build the kernel. In the former
case not everyone uses GCC.

Thanks
Emil