On Mon, 2018-08-20 at 17:09 +0300, Mika Westerberg wrote: > On Mon, Aug 20, 2018 at 04:35:08PM +0300, Andy Shevchenko wrote: > > +int intel_pinctrl_get_soc_data(struct platform_device *pdev, > > + const struct intel_pinctrl_soc_data > > **soc_data); > > int intel_pinctrl_probe(struct platform_device *pdev, > > const struct intel_pinctrl_soc_data *soc_data); > > Can we make intel_pinctrl_probe() to handle this instead? I'm not sure. There are basically two different ways how we get SoC data. For now, we have two (*) drivers that are using _UID, otherwise device_get_match_data() pretty much covers everything else. (*) I think it might come more in the future. -- Andy Shevchenko <andriy.shevchenko@xxxxxxxxxxxxxxx> Intel Finland Oy