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

Andy Shevchenko
2025-07-29T07:50:24Z

On Tue, Jul 29, 2025 at 5:26 AM Dixit Parmar <dixitparmar19@gmail.com> 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