Linux-Hwmon Archive on lore.kernel.org
 help / color / Atom feed
* nct7802.c bug : in4_input index is not 17 but 16
@ 2019-07-26  9:16 Gilles Buloz
  0 siblings, 0 replies; only message in thread
From: Gilles Buloz @ 2019-07-26  9:16 UTC (permalink / raw)
  To: linux; +Cc: linux-hwmon

Hi,

Bug description :
The "in4" related entries in structure nct7802_in_attrs start at index 16, not 17 (the comment is wrong here)
Also nct7802_in_is_visible() should use 16 instead of 17 for the same reason (two occurences)

Impact of this bug :
- when RTD3_MD=11 (VSEN3 present), everything is as expected (no bug)
- but if we have RTD3_MD!=11 (no VSEN3), we unexpectedly have a in4_input file under /sys and the command "sensors" displays the 
in4_input. But as expected, we have no in4_min, in4_max, in4_alarm, in4_beep.

Best regards

Gilles BULOZ
Senior software(/hardware) engineer | R&D
Kontron Modular Computers S.A.S.

^ permalink raw reply	[flat|nested] only message in thread

only message in thread, back to index

Thread overview: (only message) (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2019-07-26  9:16 nct7802.c bug : in4_input index is not 17 but 16 Gilles Buloz

Linux-Hwmon Archive on lore.kernel.org

Archives are clonable:
	git clone --mirror https://lore.kernel.org/linux-hwmon/0 linux-hwmon/git/0.git

	# If you have public-inbox 1.1+ installed, you may
	# initialize and index your mirror using the following commands:
	public-inbox-init -V2 linux-hwmon linux-hwmon/ https://lore.kernel.org/linux-hwmon \
		linux-hwmon@vger.kernel.org linux-hwmon@archiver.kernel.org
	public-inbox-index linux-hwmon


Newsgroup available over NNTP:
	nntp://nntp.lore.kernel.org/org.kernel.vger.linux-hwmon


AGPL code for this site: git clone https://public-inbox.org/ public-inbox