On Tue, Aug 18, 2020 at 03:34:59PM +0300, Andy Shevchenko wrote: > On Mon, Aug 03, 2020 at 04:50:31PM +0300, Andy Shevchenko wrote: > > On Mon, Aug 03, 2020 at 03:33:18PM +0300, Mika Westerberg wrote: > > > On Wed, Jul 29, 2020 at 02:57:06PM +0300, Andy Shevchenko wrote: > > > > intel_pinctrl_get_soc_data() helper can be used in few driver instead of > > > > open-coded variants. Thus, extract it as a standalone API. > > > > ... > > > > > > +const struct intel_pinctrl_soc_data *intel_pinctrl_get_soc_data(struct platform_device *pdev) > > > > > > Can we make this take const parameter as well? > > > > You mean > > > > const struct intel_pinctrl_soc_data * > > intel_pinctrl_get_soc_data(const struct platform_device *pdev) > > > > ? > > > > Sure, I can do it for v2. > > Actually I can't. device_match_data() requires struct device * pointer and > compiler is unable to compile with const qualifier. OK > Are you good with current version then? Yes :)