Re: [PATCH v2 3/3] decode-vaio: Scan more i2c buses

[Jean Delvare <jdelvare@suse.de>]

On Mon, 31 Aug 2020 11:48:55 +0200, Wolfram Sang wrote:
> > -for (my $i = 0, $found=0; $i <= 4 && !$found; $i++)
> > +for (my $i = 0, $found=0; $i <= 31 && !$found; $i++)  
> 
> To get rid of the problem entirely, can't we do something like this
> with shell globs? Pseudo code:
> 
> 	foreach (/sys/bus/i2c/devices/*-0057/eeprom)
> 		$found += <found device>
> 
> 	if (!$found)
> 		foreach (/proc/sys/dev/sensors/eeprom-i2c-*-57)
> 		check_old_interface
> 		$found += <found device>

You have hit my perl knowledge limits ;-)

I know that arbitrary limits are bad, but the thing is, I'm not sure if
there is any actual user left for that script. I adjusted it for the
at24 driver because *if* there are users left then things should keep
working. But I wrote all this in 2001, laptops from then are definitely
no longer in use, and at this point I have no idea if recent VAIO
laptops still have this EEPROM.

So until someone comes to me with a recent VAIO laptop that needs this,
I am reluctant to invest more time into this.

-- 
Jean Delvare
SUSE L3 Support