Hi Sekhar, On Monday 10 September 2012 02:57 PM, Sekhar Nori wrote: > Hi Prabhakar, > > On 9/7/2012 11:04 PM, Prabhakar Lad wrote: >> On Fri, Sep 7, 2012 at 10:48 PM, Sekhar Nori <[email protected]> wrote: > >>>> +static struct vpif_subdev_info da850_vpif_capture_sdev_info[] = { >>>> +#if defined(CONFIG_DA850_UI_SD_VIDEO_PORT) >>> >>> You agreed to get rid of these multiple ifdefs but you have not fixed that. >>> >> I have merged the top one, at later stage when we go ahead and add sensor >> support then we would have require to have define two >> da850_vpif_capture_sdev_info >> so to avoid it I have kept this one. > > Lets get to that when we actually add sensor support. It might make > sense then. Right now, there is no reason to have them. > Ok I'll fix it for v3.
Regards, --Prabhakar > Thanks, > Sekhar > _______________________________________________ Davinci-linux-open-source mailing list [email protected] http://linux.davincidsp.com/mailman/listinfo/davinci-linux-open-source
