[...]

> +     {
> +             .encoding = DRM_COLOR_YCBCR_BT709,
> +             .range = DRM_COLOR_YCBCR_FULL_RANGE,
> +             .n_colors = 4,
> +             .colors = {
> +                     {"white", {0xff, 0x80, 0x80}, {0x0000, 0xffff, 0xffff, 
> 0xffff}},
> +                     {"gray",  {0x80, 0x80, 0x80}, {0x0000, 0x8000, 0x8000, 
> 0x8000}},
> +                     {"black", {0x00, 0x80, 0x80}, {0x0000, 0x0000, 0x0000, 
> 0x0000}},
> +                     {"red",   {0x35, 0x63, 0xff}, {0x0000, 0xffff, 0x0000, 
> 0x0000}},
> +                     {"green", {0xb6, 0x1e, 0x0c}, {0x0000, 0x0000, 0xffff, 
> 0x0000}},
> +                     {"blue",  {0x12, 0xff, 0x74}, {0x0000, 0x0000, 0x0000, 
> 0xffff}},
> +             },
> +     },
> +     {
> +             .encoding = DRM_COLOR_YCBCR_BT709,
> +             .range = DRM_COLOR_YCBCR_LIMITED_RANGE,
> +             .n_colors = 4,

I think there is a mistake in n_colors here, if I understand correctly it 
must be the size of .colors, so here it should probably be 6?

> +             .colors = {
> +                     {"white", {0xeb, 0x80, 0x80}, {0x0000, 0xffff, 0xffff, 
> 0xffff}},
> +                     {"gray",  {0x7e, 0x80, 0x80}, {0x0000, 0x8000, 0x8000, 
> 0x8000}},
> +                     {"black", {0x10, 0x80, 0x80}, {0x0000, 0x0000, 0x0000, 
> 0x0000}},
> +                     {"red",   {0x3f, 0x66, 0xf0}, {0x0000, 0xffff, 0x0000, 
> 0x0000}},
> +                     {"green", {0xad, 0x2a, 0x1a}, {0x0000, 0x0000, 0xffff, 
> 0x0000}},
> +                     {"blue",  {0x20, 0xf0, 0x76}, {0x0000, 0x0000, 0x0000, 
> 0xffff}},
> +             },
> +     },
> +     {
> +             .encoding = DRM_COLOR_YCBCR_BT2020,
> +             .range = DRM_COLOR_YCBCR_FULL_RANGE,
> +             .n_colors = 4,

Same here.

> +             .colors = {
> +                     {"white", {0xff, 0x80, 0x80}, {0x0000, 0xffff, 0xffff, 
> 0xffff}},
> +                     {"gray",  {0x80, 0x80, 0x80}, {0x0000, 0x8000, 0x8000, 
> 0x8000}},
> +                     {"black", {0x00, 0x80, 0x80}, {0x0000, 0x0000, 0x0000, 
> 0x0000}},
> +                     {"red",   {0x43, 0x5c, 0xff}, {0x0000, 0xffff, 0x0000, 
> 0x0000}},
> +                     {"green", {0xad, 0x24, 0x0b}, {0x0000, 0x0000, 0xffff, 
> 0x0000}},
> +                     {"blue",  {0x0f, 0xff, 0x76}, {0x0000, 0x0000, 0x0000, 
> 0xffff}},
> +             },
> +     },
> +     {
> +             .encoding = DRM_COLOR_YCBCR_BT2020,
> +             .range = DRM_COLOR_YCBCR_LIMITED_RANGE,
> +             .n_colors = 4,

Same here.

> +             .colors = {
> +                     {"white", {0xeb, 0x80, 0x80}, {0x0000, 0xffff, 0xffff, 
> 0xffff}},
> +                     {"gray",  {0x7e, 0x80, 0x80}, {0x0000, 0x8000, 0x8000, 
> 0x8000}},
> +                     {"black", {0x10, 0x80, 0x80}, {0x0000, 0x0000, 0x0000, 
> 0x0000}},
> +                     {"red",   {0x4a, 0x61, 0xf0}, {0x0000, 0xffff, 0x0000, 
> 0x0000}},
> +                     {"green", {0xa4, 0x2f, 0x19}, {0x0000, 0x0000, 0xffff, 
> 0x0000}},
> +                     {"blue",  {0x1d, 0xf0, 0x77}, {0x0000, 0x0000, 0x0000, 
> 0xffff}},
> +             },
> +     },
> +};

[...]

-- 
Louis Chauvet, Bootlin
Embedded Linux and Kernel engineering
https://bootlin.com

Reply via email to