Re: [PATCH 1/2] iio: magnetometer: add support for Infineon TLV493D 3D Magentic sensor

[Andy Shevchenko]

On Tue, Jul 29, 2025 at 5:26 AM Dixit Parmar <dixitparmar19@xxxxxxxxx> wrote:
> On Sat, Jul 26, 2025 at 03:44:03PM -0500, David Lechner wrote:
> > On 7/26/25 4:37 AM, Dixit Parmar wrote:

...

> > >   config SI7210
> > >       To compile this driver as a module, choose M here: the module
> > >       will be called si7210.
> > >
> > > +config TLV493D
> > > +   tristate "Infineon TLV493D Low-Power 3D Magnetic Sensor"
> > > +   depends on I2C
> > > +   select REGMAP_I2C
> > > +   select IIO_BUFFER
> > > +   select IIO_TRIGGERED_BUFFER
> > > +   help
> > > +     Say Y here to add support for the Infineon TLV493D-A1B6 Low-
> > > +     Power 3D Megnetic Sensor.
> > > +
> > > +     This driver can also be compiled as a module.
> > > +     To compile this driver as a module, choose M here: the module
> > > +     will be called tlv493d.
> > > +
> > >  config TI_TMAG5273
> > >     tristate "TI TMAG5273 Low-Power Linear 3D Hall-Effect Sensor"
> > >     depends on I2C

...

> > > @@ -35,4 +35,6 @@ obj-$(CONFIG_SI7210)                      += si7210.o
> > >
> > >  obj-$(CONFIG_TI_TMAG5273)          += tmag5273.o
> > >
> > > +obj-$(CONFIG_TLV493D)      += tlv493d.o
> >
> > We try to keep these in alphabetical order.
> >
> Ofcourse, I considered TI_TMAG5273 as whole. Will move it above that.

David, Jonathan, I remember I have asked Jonathan once already about
these cases and unfortunately I forgot what was the conclusion about
this. The filename has no vendor prefix, and  I think we prefer the
order done by filename.

> > >  obj-$(CONFIG_YAMAHA_YAS530)                += yamaha-yas530.o


-- 
With Best Regards,
Andy Shevchenko