Re: [PATCH v5 1/2] dt-bindings: iio: adc: add maxim,max40080

[Siratul Islam <siratul.islam@linux.dev>]

On Sun, 2026-07-26 at 00:31 +0100, Jonathan Cameron wrote:
> On Thu, 23 Jul 2026 22:23:39 +0600
> Siratul Islam <siratul.islam@linux.dev> wrote:
> 
> > On Thu, 2026-07-23 at 09:50 +0300, Stefan Popa wrote:
> > > Add device tree bindings for the Maxim MAX40080 bidirectional
> > > current-sense amplifier with a 12-bit ADC and an I2C/SMBus interface.
> > > 
> > > Signed-off-by: Stefan Popa <stefan.popa@analog.com>
> > > ---
> > >   
> > ...
> > > +
> > > +title: Maxim MAX40080 bidirectional current-sense amplifier
> > > +
> > > +maintainers:
> > > +  - Ciprian Hegbeli <ciprian.hegbeli@analog.com>
> > > +  - Stefan Popa <stefan.popa@analog.com>
> > > +
> > > +description: |
> > > +  The MAX40080 is a high-precision, bidirectional current-sense amplifier with
> > > +  an integrated 12-bit ADC and an I2C/SMBus interface. It measures the voltage
> > > +  across an external shunt resistor and the input bus voltage, and stores the
> > > +  results in an internal FIFO.
> > > +
> > > +  Datasheet:
> > > +    https://www.analog.com/en/products/max40080.html  
> > This URL links to a product page rather than a datasheet as the tag suggests.
> > Would be difficult to fetch using CLI tools.
> > Were you supposed to use something like
> > "https://www.analog.com/media/en/technical-documentation/data-sheets/MAX40080.pdf"?
> 
> I think the argument for using product pages is that they are stable and datasheet
> paths sometimes change.
> 
> For ADI there is a neater form as well that is supposed to be stable (and
> has been for last few years at least)
> https://www.analog.com/max40080
> Works for any product and takes you to the equilvalent product page to the
> link above.
> 
> So whilst you make a good point I think we will have to let Analog decide
> on a standard way to link to their product info / datasheets.  That is
> on the perhaps optimistic basis they have more insight into stable choices
> than we do!
> 
I was just nitpicking because it said "Datasheet" to point to a link which was not a 
datasheet.

But yes, a standard way to link all their products (like https://www.analog.com/max40080)
makes more sense. I just checked and currently some adi devices use product page links,
some use PDF links and some don't include any links at all.
> > 
> > > +
> > >   
> > ...
...
> > > 
> > 
--
Best regards,
Sirat