[RFC][PATCH V2 1/2] drm: add drm_edid_to_eld helper extracting SADs from EDID
Rafał Miłecki
zajec5 at gmail.com
Mon Apr 8 04:49:33 PDT 2013
2013/4/8 Ville Syrjälä <ville.syrjala at linux.intel.com>:
>> + *sads = kzalloc(count * sizeof(*sads), GFP_KERNEL);
>
> Still looks a bit wrong.
>
> kcalloc(count, sizeof(**sads), GFP_KERNEL);
>
> Also a minor nit, but the scope of some variables is needlessly large.
> db and dbl are only needed inside the outer loop, and count is only
> needed inside the 'if (... == AUDIO_BLOCK)' block. But this is not a big
> deal, so feel free to ignore me on this point if you're feeling lazy :)
>> +#define SAD_FORMAT_WMA_PRO 0x0e
>
> We already have names for these in include/linux/hdmi.h. No need to
> duplicate them here.
Thanks for your comments Ville! I appreciate that :)
--
Rafał
More information about the dri-devel
mailing list