netdev - Re: [PATCH v2 1/2] dt-bindings: net/nfc: ti,trf7970a: Add ti,rx-gain-reduction option

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: <ddb9a1a9-7070-416c-848e-00d151846999@lunn.ch>
Date: Thu, 26 Jun 2025 10:57:18 +0200
From: Andrew Lunn <andrew@...n.ch>
To: Paul Geurts <paul.geurts@...drive-technologies.com>
Cc: mgreer@...malcreek.com, krzk@...nel.org, andrew+netdev@...n.ch,
	davem@...emloft.net, edumazet@...gle.com, kuba@...nel.org,
	pabeni@...hat.com, robh@...nel.org, conor+dt@...nel.org,
	linux-wireless@...r.kernel.org, netdev@...r.kernel.org,
	devicetree@...r.kernel.org, linux-kernel@...r.kernel.org,
	martijn.de.gouw@...drive-technologies.com
Subject: Re: [PATCH v2 1/2] dt-bindings: net/nfc: ti,trf7970a: Add
 ti,rx-gain-reduction option

> > You should include the units, "ti,rx-gain-reduction-db"
> 
> Well, Currently it's not really a dB value (see below).
> 
> > 
> > > +    $ref: /schemas/types.yaml#/definitions/uint32
> > > +    description: |
> > > +      Specify a RX gain reduction to reduce antenna sensitivity with 5dB per
> > > +      increment, with a maximum of 15dB.
> > 
> > Given that description i think you can provide a list of values, [0,
> > 5, 10, 15] and the tools will validate values in .dts files.
> > 
> > > +
> > >  required:
> > >    - compatible
> > >    - interrupts
> > > @@ -95,5 +101,6 @@ examples:
> > >              irq-status-read-quirk;
> > >              en2-rf-quirk;
> > >              clock-frequency = <27120000>;
> > > +            ti,rx-gain-reduction = <3>;
> > 
> > Err, how does 3 fit into 5dB increments?
> 
> I implemented it in a way that the value of ti,rx-gain-reduction is programmed
> directly into the RX_GAIN reduction register, and there it means 5 dB/LSB. My
> description probably was not clear enough about that. So a value of 3 here actually
> means 15dB.
> So I could either improve the description here that this is the case, or make the
> value in here in actual dB, and do some calculations in the driver. What has your
> preference?

DT should use SI units, Volts, Amps, degrees C, meters, etc. The
driver then should do whatever conversion is needed to convert to
hardware register values.

Less important, but i'm also wondering if this should be negative,
ti,rx-gain-db, with a value of -15. You say this receiver is overly
sensitive, so you need to reduce the gain. But are there TI devices
where you can actually increase the gain? Ideally the property should
be generic and be able to cover that use case as well.

    Andrew

---
pw-bot: cr