linux-toradex.git/drivers/misc/ad525x_dpot.c, branch v6.2-rc3

misc: ad525x_dpot: Make ad_dpot_remove() return void

2021-10-13T12:35:37+00:00

Up to now ad_dpot_remove() returns zero unconditionally. Make it return
void instead which makes it easier to see in the callers that there is
no error to handle.

Also the return value of i2c and spi remove callbacks is ignored anyway.

Signed-off-by: Uwe Kleine-König 
Link: https://lore.kernel.org/r/20211012153945.2651412-12-u.kleine-koenig@pengutronix.de
Acked-by: Michael Hennerich 
Signed-off-by: Greg Kroah-Hartman

drivers: misc: ad525x_dpot: Add missing check in dpot_read_spi

2021-03-10T08:21:02+00:00

The use of dpot_read_r8d8() after checking dpot->uid is similar.
However, we check the return value and return an error code only
in one path, which is odd.

Link: https://lore.kernel.org/r/20210301125057.28819-1-dinghao.liu@zju.edu.cn
Signed-off-by: Dinghao Liu 
Signed-off-by: Greg Kroah-Hartman

docs: misc-devices/pci-endpoint-test.txt: convert to ReST

2020-06-19T20:05:49+00:00

- use title markups;
- mark literal blocks.

Signed-off-by: Mauro Carvalho Chehab 
Link: https://lore.kernel.org/r/1b1f4e5e57fd2065828cecc9d07afbd247349e94.1592203650.git.mchehab+huawei@kernel.org
Signed-off-by: Jonathan Corbet

treewide: Replace GPLv2 boilerplate/reference with SPDX - rule 149

2019-05-30T18:25:18+00:00

Based on 1 normalized pattern(s):

  licensed under the gpl 2 or later

extracted by the scancode license scanner the SPDX license identifier

  GPL-2.0-or-later

has been chosen to replace the boilerplate/reference in 82 file(s).

Signed-off-by: Thomas Gleixner 
Reviewed-by: Allison Randal 
Reviewed-by: Kate Stewart 
Reviewed-by: Richard Fontana 
Cc: linux-spdx@vger.kernel.org
Link: https://lkml.kernel.org/r/20190524100845.150836982@linutronix.de
Signed-off-by: Greg Kroah-Hartman

drivers: misc: ad525x_dpot: clean indentation issue, remove tabs

2019-01-18T15:34:05+00:00

There is a hunk of code in a case statement that is indented one level
too deeply, fix this by removing extra tabs. Also remove one empty line.

Signed-off-by: Colin Ian King 
Signed-off-by: Greg Kroah-Hartman

drivers: misc: ad525x_dpot: Update MODULE AUTHOR email address

2018-09-14T13:36:20+00:00

no functional changes

Signed-off-by: Michael Hennerich 
Signed-off-by: Greg Kroah-Hartman

misc: ad525x_dpot: macros should not use a trailing semicolon

2017-12-18T15:02:26+00:00

Resolved all the macros should not use a trailing semicolon
checkpatch warnings. Issue found by checkpatch.

Signed-off-by: Dhaval Shah 
Signed-off-by: Greg Kroah-Hartman

misc: ad525x_dpot: please, no space before tabs

2017-12-18T15:02:26+00:00

Resolved the please, no space beofore tabs checkpatch
warning. Issue found by checkpatch.

Signed-off-by: Dhaval Shah 
Signed-off-by: Greg Kroah-Hartman

misc: ad525x_dpot: Prefer 'unsigned int' to bare use of 'unsigned'

2017-12-18T15:02:26+00:00

Resolved all the Prefer 'unsigned int' to bare use of 'unsigned'
checkpatch warnings. Issue found by checkpatch.

Signed-off-by: Dhaval Shah 
Signed-off-by: Greg Kroah-Hartman

misc: ad525x_dpot: Fix the enabling of the "otpXen" attributes

2016-03-02T00:55:13+00:00

Currently writing the attributes with "echo" will result in comparing:
"enabled\n" with "enabled\0" and attribute is always set to false.

Use the sysfs_streq() instead because it treats both NUL and
new-line-then-NUL as equivalent string terminations.

Signed-off-by: Dan Bogdan Nechita 
Signed-off-by: Greg Kroah-Hartman