On Thu, Apr 09, 2026 at 09:17:14PM +0200, Linus Walleij wrote: > This makes the driver usable on contemporary Linux, maybe it will > even work, I can't test it. Don't think so... > @@ -742,6 +745,7 @@ static const struct snd_soc_component_driver soc_component_dev_uda1380 = { > static int uda1380_i2c_probe(struct i2c_client *i2c) > { > struct uda1380_platform_data *pdata = i2c->dev.platform_data; > + struct device *dev = &i2c->dev; > struct uda1380_priv *uda1380; > int ret; We still have the platform data above, and just below this we have: if (!pdata) return -EINVAL; so I'm not sure how the driver is going to instantiate, there's mandatory platform data still. Not that the driver would've instantiated before either with anyhing in mainline...