On Mon, Jan 16, 2023 at 10:44 PM Marijn Suijten <marijn.suijten@xxxxxxxxxxxxxx> wrote: > > On 2022-08-06 19:20:48, Jonathan Cameron wrote: > > On Fri, 15 Jul 2022 14:29:00 +0200 > > Nuno Sá <nuno.sa@xxxxxxxxxx> wrote: > > > > > Make the conversion to firmware agnostic device properties. As part of > > > the conversion the IIO inkern interface 'of_xlate()' is also converted to > > > 'fwnode_xlate()'. The goal is to completely drop 'of_xlate' and hence OF > > > dependencies from IIO. > > > > > > Signed-off-by: Nuno Sá <nuno.sa@xxxxxxxxxx> > > > Acked-by: Linus Walleij <linus.walleij@xxxxxxxxxx> > > > Reviewed-by: Andy Shevchenko <andy.shevchenko@xxxxxxxxx> > > +CC Marijin who happend to post a patch for this driver that I just accepted > > and hence probably has hardware access. Any chance of a test for this series? > > > > If not, no problem as this is fairly mechanical and we have testing on some of > > the other drivers using the new code. > > > > I'll probably queue this up in the meantime but it won't end up upstream > > for a few weeks yet. > > Jonathan, > > This CC just surfaced in my inbox while searching for our current > discussion around missing labels in qcom-spmi-vadc - and on the side a > userspace @xx label name ABI break (in qcom-spmi-adc5) caused by this > patch's fwnode_get_name change - we could've caught it if I had not > accidentally marked it as read and/or forgot about it. My apologies. Does the following addition to the top of the adc5_get_fw_channel_data() fix the issue? + name = devm_kasprintf(adc->dev, GFP_KERNEL, "%pfwP", fwnode); + if (!name) + return -ENOMEM; -- With Best Regards, Andy Shevchenko