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's >> opinion on this (CC'ed). > IIUC: I'd suggest to leave it as it is. > > There are people working on removing the quirky fallback to > i2c_device_id in case of_compatible is not provided. If that gets > mainline, you don't have i2c_device_id anymore with DT matching. So all > binding methods should be self-contained. ok Should we leave as it is, redundant if-else code block also, or fix it as posted in patch v2 > > > > _______________________________________________ > dri-devel mailing list > dri-devel at lists.freedesktop.org > https://lists.freedesktop.org/mailman/listinfo/dri-devel -------------- next part -------------- An HTML attachment was scrubbed... URL: <https://lists.freedesktop.org/archives/dri-devel/attachments/20161020/16049e45/attachment.html>
