[PATCH v2 4/8] davinci: vpfe: add support for CCDC hardware for dm365

Sylwester Nawrocki snjw23 at gmail.com
Fri Sep 9 11:39:38 CDT 2011


Hi Manju,

On 09/09/2011 03:30 PM, Hadli, Manjunath wrote:
> Thank you for these comments too.
> 
> My responses inlined.
> -Manju

Thanks for addressing my comments.

> 
> On Fri, Sep 02, 2011 at 02:53:08, Sylwester Nawrocki wrote:
...
>>> +/**
>>> + * ccdc float type S8Q8/U8Q8
>>> + */
>>> +struct ccdc_float_8 {
>>> +	/* 8 bit integer part */
>>> +	unsigned char integer;
>>> +	/* 8 bit decimal part */
>>> +	unsigned char decimal;
>>> +};
>>
>> Isn't it better to use explicit width type, like u8, u16, etc. ?
>> Then we could just have:
>>
>> +struct ccdc_float_8 {
>> +	u8 integer;
>> +	u8 decimal;
>> +};
>>
> This is an interface header which is also used by apps. So we have
> kept it as unsigned char. Any suggestions on that?

OK, sorry, I haven't noticed that. Anyway, I think you could use the
double underscore prefixed types (__[u,s][8,32,16]), from linux/types.h.
They are widely used in in linux/videodev2.h for instance.


--
Regards,
Sylwester


More information about the Davinci-linux-open-source mailing list