Greg Kroah-Hartman
b580b17d06
Merge tag 'w1-drv-6.14' of ssh://gitolite.kernel.org/pub/scm/linux/kernel/git/krzk/linux-w1 into char-misc-next
...
Krzysztof writes:
1-Wire bus drivers for v6.14
1. ds2482: Add support for handling the VCC regulator supply and three
more minor improvements/cleanups.
2. Constify 'struct bin_attribute' in all drivers.
3. W1 core: use sysfs_emit() instead of sprintf(), as preferred coding
style.
* tag 'w1-drv-6.14' of ssh://gitolite.kernel.org/pub/scm/linux/kernel/git/krzk/linux-w1:
w1: core: use sysfs_emit() instead of sprintf()
w1: ds28e04: Constify 'struct bin_attribute'
w1: ds2805: Constify 'struct bin_attribute'
w1: ds2781: Constify 'struct bin_attribute'
w1: ds2780: Constify 'struct bin_attribute'
w1: ds2438: Constify 'struct bin_attribute'
w1: ds2433: Constify 'struct bin_attribute'
w1: ds2431: Constify 'struct bin_attribute'
w1: ds2430: Constify 'struct bin_attribute'
w1: ds2413: Constify 'struct bin_attribute'
w1: ds2408: Constify 'struct bin_attribute'
w1: ds2406: Constify 'struct bin_attribute'
w1: Constify 'struct bin_attribute'
w1: ds2482: Fix datasheet URL
w1: ds2482: Add regulator support
w1: ds2482: switch to devm_kzalloc() from kzalloc()
dt-bindings: w1: ds2482: Add vcc-supply property
2025-01-09 10:54:19 +01:00
zhangheng
419a40cc2b
w1: core: use sysfs_emit() instead of sprintf()
...
Follow the advice in Documentation/filesystems/sysfs.rst:
show() should only use sysfs_emit() or sysfs_emit_at() when formatting
the value to be returned to user space.
Signed-off-by: zhangheng <zhangheng@kylinos.cn >
Link: https://lore.kernel.org/r/20241220102337.295864-1-zhangheng@kylinos.cn
Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org >
2024-12-22 15:42:15 +01:00
Thomas Weißschuh
0f28374e99
w1: ds28e04: Constify 'struct bin_attribute'
...
The sysfs core now allows instances of 'struct bin_attribute' to be
moved into read-only memory. Make use of that to protect them against
accidental or malicious modifications.
Signed-off-by: Thomas Weißschuh <linux@weissschuh.net >
Link: https://lore.kernel.org/r/20241211-sysfs-const-bin_attr-w1-v1-12-c4befd2aa7cc@weissschuh.net
Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org >
2024-12-14 11:06:34 +01:00
Thomas Weißschuh
0ef2a9b243
w1: ds2805: Constify 'struct bin_attribute'
...
The sysfs core now allows instances of 'struct bin_attribute' to be
moved into read-only memory. Make use of that to protect them against
accidental or malicious modifications.
Signed-off-by: Thomas Weißschuh <linux@weissschuh.net >
Link: https://lore.kernel.org/r/20241211-sysfs-const-bin_attr-w1-v1-11-c4befd2aa7cc@weissschuh.net
Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org >
2024-12-14 11:06:33 +01:00
Thomas Weißschuh
c797bbdac5
w1: ds2781: Constify 'struct bin_attribute'
...
The sysfs core now allows instances of 'struct bin_attribute' to be
moved into read-only memory. Make use of that to protect them against
accidental or malicious modifications.
Signed-off-by: Thomas Weißschuh <linux@weissschuh.net >
Link: https://lore.kernel.org/r/20241211-sysfs-const-bin_attr-w1-v1-10-c4befd2aa7cc@weissschuh.net
Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org >
2024-12-14 11:06:33 +01:00
Thomas Weißschuh
4a68c8530f
w1: ds2780: Constify 'struct bin_attribute'
...
The sysfs core now allows instances of 'struct bin_attribute' to be
moved into read-only memory. Make use of that to protect them against
accidental or malicious modifications.
Signed-off-by: Thomas Weißschuh <linux@weissschuh.net >
Link: https://lore.kernel.org/r/20241211-sysfs-const-bin_attr-w1-v1-9-c4befd2aa7cc@weissschuh.net
Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org >
2024-12-14 11:06:33 +01:00
Thomas Weißschuh
83544525d1
w1: ds2438: Constify 'struct bin_attribute'
...
The sysfs core now allows instances of 'struct bin_attribute' to be
moved into read-only memory. Make use of that to protect them against
accidental or malicious modifications.
Signed-off-by: Thomas Weißschuh <linux@weissschuh.net >
Link: https://lore.kernel.org/r/20241211-sysfs-const-bin_attr-w1-v1-8-c4befd2aa7cc@weissschuh.net
Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org >
2024-12-14 11:06:32 +01:00
Thomas Weißschuh
1398800d82
w1: ds2433: Constify 'struct bin_attribute'
...
The sysfs core now allows instances of 'struct bin_attribute' to be
moved into read-only memory. Make use of that to protect them against
accidental or malicious modifications.
Signed-off-by: Thomas Weißschuh <linux@weissschuh.net >
Link: https://lore.kernel.org/r/20241211-sysfs-const-bin_attr-w1-v1-7-c4befd2aa7cc@weissschuh.net
Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org >
2024-12-14 11:06:27 +01:00
Thomas Weißschuh
86b04e4dcf
w1: ds2431: Constify 'struct bin_attribute'
...
The sysfs core now allows instances of 'struct bin_attribute' to be
moved into read-only memory. Make use of that to protect them against
accidental or malicious modifications.
Signed-off-by: Thomas Weißschuh <linux@weissschuh.net >
Link: https://lore.kernel.org/r/20241211-sysfs-const-bin_attr-w1-v1-6-c4befd2aa7cc@weissschuh.net
Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org >
2024-12-14 11:06:26 +01:00
Thomas Weißschuh
be0d277fd3
w1: ds2430: Constify 'struct bin_attribute'
...
The sysfs core now allows instances of 'struct bin_attribute' to be
moved into read-only memory. Make use of that to protect them against
accidental or malicious modifications.
Signed-off-by: Thomas Weißschuh <linux@weissschuh.net >
Link: https://lore.kernel.org/r/20241211-sysfs-const-bin_attr-w1-v1-5-c4befd2aa7cc@weissschuh.net
Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org >
2024-12-14 11:06:26 +01:00
Thomas Weißschuh
f597a4ce8c
w1: ds2413: Constify 'struct bin_attribute'
...
The sysfs core now allows instances of 'struct bin_attribute' to be
moved into read-only memory. Make use of that to protect them against
accidental or malicious modifications.
Signed-off-by: Thomas Weißschuh <linux@weissschuh.net >
Link: https://lore.kernel.org/r/20241211-sysfs-const-bin_attr-w1-v1-4-c4befd2aa7cc@weissschuh.net
Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org >
2024-12-14 11:06:25 +01:00
Thomas Weißschuh
492772838d
w1: ds2408: Constify 'struct bin_attribute'
...
The sysfs core now allows instances of 'struct bin_attribute' to be
moved into read-only memory. Make use of that to protect them against
accidental or malicious modifications.
Signed-off-by: Thomas Weißschuh <linux@weissschuh.net >
Link: https://lore.kernel.org/r/20241211-sysfs-const-bin_attr-w1-v1-3-c4befd2aa7cc@weissschuh.net
Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org >
2024-12-14 11:06:22 +01:00
Thomas Weißschuh
edc52050f8
w1: ds2406: Constify 'struct bin_attribute'
...
The sysfs core now allows instances of 'struct bin_attribute' to be
moved into read-only memory. Make use of that to protect them against
accidental or malicious modifications.
Signed-off-by: Thomas Weißschuh <linux@weissschuh.net >
Link: https://lore.kernel.org/r/20241211-sysfs-const-bin_attr-w1-v1-2-c4befd2aa7cc@weissschuh.net
Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org >
2024-12-14 11:06:15 +01:00
Thomas Weißschuh
699e5f2f28
w1: Constify 'struct bin_attribute'
...
The sysfs core now allows instances of 'struct bin_attribute' to be
moved into read-only memory. Make use of that to protect them against
accidental or malicious modifications.
Signed-off-by: Thomas Weißschuh <linux@weissschuh.net >
Link: https://lore.kernel.org/r/20241211-sysfs-const-bin_attr-w1-v1-1-c4befd2aa7cc@weissschuh.net
Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org >
2024-12-14 11:05:56 +01:00
Kryštof Černý
5f69c091a6
w1: ds2482: Fix datasheet URL
...
Current link does redirect to wrong place.
Signed-off-by: Kryštof Černý <cleverline1mc@gmail.com >
Link: https://lore.kernel.org/r/20241129-ds2482-add-reg-v6-4-bd95ad171e19@gmail.com
Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org >
2024-12-03 15:53:29 +01:00
Kryštof Černý
6e0bb206c6
w1: ds2482: Add regulator support
...
Adds a support for attaching a supply regulator.
Signed-off-by: Kryštof Černý <cleverline1mc@gmail.com >
Link: https://lore.kernel.org/r/20241129-ds2482-add-reg-v6-3-bd95ad171e19@gmail.com
Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org >
2024-12-03 15:53:29 +01:00
Kryštof Černý
19c6d8bd88
w1: ds2482: switch to devm_kzalloc() from kzalloc()
...
Refactored the driver to devm_kzalloc() from kzalloc(), so the future
driver edits are easier and less error-prone.
Signed-off-by: Kryštof Černý <cleverline1mc@gmail.com >
Link: https://lore.kernel.org/r/20241129-ds2482-add-reg-v6-2-bd95ad171e19@gmail.com
Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org >
2024-12-03 15:53:28 +01:00
Linus Torvalds
e70140ba0d
Get rid of 'remove_new' relic from platform driver struct
...
The continual trickle of small conversion patches is grating on me, and
is really not helping. Just get rid of the 'remove_new' member
function, which is just an alias for the plain 'remove', and had a
comment to that effect:
/*
* .remove_new() is a relic from a prototype conversion of .remove().
* New drivers are supposed to implement .remove(). Once all drivers are
* converted to not use .remove_new any more, it will be dropped.
*/
This was just a tree-wide 'sed' script that replaced '.remove_new' with
'.remove', with some care taken to turn a subsequent tab into two tabs
to make things line up.
I did do some minimal manual whitespace adjustment for places that used
spaces to line things up.
Then I just removed the old (sic) .remove_new member function, and this
is the end result. No more unnecessary conversion noise.
Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org >
2024-12-01 15:12:43 -08:00
Uwe Kleine-König
b39121354f
w1: ds2482: Drop explicit initialization of struct i2c_device_id::driver_data to 0
...
This driver doesn't use the driver_data member of struct i2c_device_id,
so don't explicitly initialize this member.
This prepares putting driver_data in an anonymous union which requires
either no initialization or named designators. But it's also a nice
cleanup on its own.
Signed-off-by: Uwe Kleine-König <u.kleine-koenig@baylibre.com >
Reviewed-by: Wolfram Sang <wsa+renesas@sang-engineering.com >
Link: https://lore.kernel.org/r/20240725160909.326143-2-u.kleine-koenig@baylibre.com
Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org >
Link: https://lore.kernel.org/r/20240906141135.72080-1-krzysztof.kozlowski@linaro.org
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org >
2024-09-06 19:18:32 +02:00
Thorsten Blum
26bf5fc860
w1: Drop allocation error message
...
Drop the custom error message because kzalloc() already prints
allocation failures.
Signed-off-by: Thorsten Blum <thorsten.blum@toblux.com >
Link: https://lore.kernel.org/r/20240527134946.338398-2-thorsten.blum@toblux.com
Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org >
2024-05-28 17:45:02 +02:00
Thorsten Blum
56813b244e
w1: Add missing newline and fix typos in w1_bus_master comment
...
- Add missing newline before @return
- s/bytes/byte/
- s/handles/handle/
- s/exists/exist/ in dev_info() message
Signed-off-by: Thorsten Blum <thorsten.blum@toblux.com >
Link: https://lore.kernel.org/r/20240527092746.263038-2-thorsten.blum@toblux.com
[krzysztof: squash "w1: Fix typo in dev_info() message"]
Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org >
2024-05-27 13:51:29 +02:00
Andy Shevchenko
cde37a5bdb
w1: gpio: Don't use "proxy" headers
...
Update header inclusions to follow IWYU (Include What You Use)
principle.
Signed-off-by: Andy Shevchenko <andriy.shevchenko@linux.intel.com >
Link: https://lore.kernel.org/r/20240307143644.3787260-6-andriy.shevchenko@linux.intel.com
Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org >
2024-03-25 12:06:37 +01:00
Andy Shevchenko
540d3f15c0
w1: gpio: Remove duplicate NULL checks
...
gpiod_set_value() is NULL-aware, no need to check that in the caller.
Signed-off-by: Andy Shevchenko <andriy.shevchenko@linux.intel.com >
Acked-by: Uwe Kleine-König <u.kleine-koenig@pengutronix.de >
Link: https://lore.kernel.org/r/20240307143644.3787260-5-andriy.shevchenko@linux.intel.com
Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org >
2024-03-25 12:06:37 +01:00
Andy Shevchenko
ef2b810e11
w1: gpio: Use sizeof(*pointer) instead of sizeof(type)
...
It is preferred to use sizeof(*pointer) instead of sizeof(type).
The type of the variable can change and one needs not change
the former (unlike the latter). No functional change intended.
Signed-off-by: Andy Shevchenko <andriy.shevchenko@linux.intel.com >
Acked-by: Uwe Kleine-König <u.kleine-koenig@pengutronix.de >
Link: https://lore.kernel.org/r/20240307143644.3787260-4-andriy.shevchenko@linux.intel.com
Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org >
2024-03-25 12:06:37 +01:00
Andy Shevchenko
9e085c0458
w1: gpio: Switch to use dev_err_probe()
...
Switch to use dev_err_probe() to simplify the error path and
unify a message template.
Signed-off-by: Andy Shevchenko <andriy.shevchenko@linux.intel.com >
Acked-by: Uwe Kleine-König <u.kleine-koenig@pengutronix.de >
Link: https://lore.kernel.org/r/20240307143644.3787260-3-andriy.shevchenko@linux.intel.com
Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org >
2024-03-25 12:06:36 +01:00