Re: [PATCH] hwmon: (ina2xx) drop unused platform data
From: Jonathan Cameron
Date: Thu Mar 26 2026 - 16:26:10 EST
On Thu, 26 Mar 2026 14:24:20 +0100
Bartosz Golaszewski <brgl@xxxxxxxxxx> wrote:
> On Thu, Mar 26, 2026 at 12:33 PM Guenter Roeck <linux@xxxxxxxxxxxx> wrote:
> >
> > On Thu, Mar 26, 2026 at 04:28:16AM -0700, Guenter Roeck wrote:
> > > Hi,
> > >
> > > On Thu, Mar 26, 2026 at 10:30:00AM +0100, Bartosz Golaszewski wrote:
> > > > Nobody defines struct ina2xx_platform_data. Remove platform data support
> > > > from the drivers which still have it (it's effectively dead code) and
> > > > remove the header.
> > > >
> > > > Signed-off-by: Bartosz Golaszewski <bartosz.golaszewski@xxxxxxxxxxxxxxxx>
> > > > Reviewed-by: Andy Shevchenko <andriy.shevchenko@xxxxxxxxx>
> > >
> > > Thanks a lot for your patch. Checkpatch complains:
> > >
> > > CHECK: Alignment should match open parenthesis
> > >
> > > No need to resend, I'll fix that up.
> > >
> > > Applied.
> > >
> >
> > Ah, no, I can not do that. You also touched drivers/iio/adc/ina2xx-adc.c,
> > meaning I'll either need an Ack from Jonathan or the patch will need to be
> > split.
> >
> > Guenter
>
> Yes, sorry I should have mentioned it. This should go through the
> hwmon tree with an Ack from iio maintainers.
Acked-by: Jonathan Cameron <jonathan.cameron@xxxxxxxxxx>
I have one patch on this driver queued on this driver, but the patch seems
to apply just fine on top of that so hopefully the merge will be automatic
/ trivial.
https://lore.kernel.org/all/20260315182304.613783-1-git@xxxxxxxxx/
>
> Bart