On 20/05/16 16:14, Jyri Sarha wrote:
> On 05/20/16 15:51, Tomi Valkeinen wrote:
>>> +u32 dispc_mgr_gamma_size(enum omap_channel channel)
>>>> +{
>>>> +	const struct dispc_gamma_desc *gdesc = &mgr_desc[channel].gamma;
>>>> +
>>>> +	if (!dss_has_feature(FEAT_GAMMA_TABLE))
>>>> +		return 0;
>>>> +
>> This should probably check the availability of LCD2/LCD3?
>>
> 
> That would be a double check (the function should never be called with
> channels that are not there), but I am happy to add it.
> 

True, I think we can trust omapdrm here =).

 Tomi