[PATCH v2] drm/bridge: adv7511: Remove unused code blocks

2016-10-20 Thread Sharma, Jitendra
Thanks for comments. On 10/20/2016 2:15 PM, Wolfram Sang wrote: >>> IMO it would be better to use id->driver_data unconditionally and save a >>> call to of_device_get_match_data() >>> What would you suggest to move ahead? >> I'd suggest using id->driver_data unconditionally, but I'd like Wolfram'

[PATCH v2] drm/bridge: adv7511: Remove unused code blocks

2016-10-19 Thread Sharma, Jitendra
Hi Laurent, On 10/19/2016 7:03 PM, Laurent Pinchart wrote: > Hi Jitendra, > > On Wednesday 19 Oct 2016 18:37:38 Sharma, Jitendra wrote: >> On 10/19/2016 5:21 PM, Laurent Pinchart wrote: >>> On Wednesday 19 Oct 2016 17:12:48 Jitendra Sharma wrote: >>>> Remove r

[PATCH v2] drm/bridge: adv7511: Remove unused code blocks

2016-10-19 Thread Sharma, Jitendra
Hi Laurent, On 10/19/2016 5:21 PM, Laurent Pinchart wrote: > Hi Jitendra, > > Thank you for the patch. > > On Wednesday 19 Oct 2016 17:12:48 Jitendra Sharma wrote: >> Remove redundant condition check >> Remove not necessary if-else block for checking DT entry because else >> part will never be pi

[PATCH] drm/bridge: adv7511: Remove unused code blocks

2016-10-19 Thread Sharma, Jitendra
Hi Laurent, Thanks for trying it out. On 10/18/2016 6:37 PM, Laurent Pinchart wrote: > Hi Jitendra, > > Thank you for the patch. > > On Tuesday 18 Oct 2016 15:39:58 Jitendra Sharma wrote: >> Remove unused id_table entries >> Currently this driver supports only device tree based configuration. >>