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

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <CAHp75Vf8nj7xg5xadCTXQhaXzaCL0y5T5qsUHK_dcoQ3hozDXw@mail.gmail.com>
Date: Tue, 29 Jul 2025 09:49:45 +0200
From: Andy Shevchenko <andy.shevchenko@...il.com>
To: Dixit Parmar <dixitparmar19@...il.com>
Cc: David Lechner <dlechner@...libre.com>, Jonathan Cameron <jic23@...nel.org>, 
	Nuno Sá <nuno.sa@...log.com>, 
	Andy Shevchenko <andy@...nel.org>, Rob Herring <robh@...nel.org>, 
	Krzysztof Kozlowski <krzk+dt@...nel.org>, Conor Dooley <conor+dt@...nel.org>, linux-kernel@...r.kernel.org, 
	linux-iio@...r.kernel.org, devicetree@...r.kernel.org
Subject: Re: [PATCH 1/2] iio: magnetometer: add support for Infineon TLV493D
 3D Magentic sensor

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