-by: Lucas Stankus
---
Changes in v2:
Previously this change set was composed of two patches, but it lead to
regressions between the two. Merging them avoids the unwanted behaviour
without having to deal with the special cases.
drivers/staging/iio/cdc/ad7746.c | 42
Add device tree binding documentation for AD7746 cdc in YAML format.
Signed-off-by: Lucas Stankus
---
A minor note about the adi,excitation-vdd-permille property. Jonathan
suggested the name to be adi,excitation-vdd-milicent, but I was unsure of
the milicent naming. With a quick search I found
two patches to avoid special handling of remaining
platform_data properties.
Lucas Stankus (2):
dt-bindings: staging: iio: cdc: ad7746: add binding documentation for
AD7746
staging: iio: cdc: ad7746: use dt for capacitive channel setup.
.../bindings/iio/cdc/adi,ad7746.yaml |
Set device excitation level using properties from device tree binding
instead of using platform_data.
As this replaces the last instance where the platform_data struct was
used, remove ad7746.h header file since it's no longer needed.
Signed-off-by: Lucas Stankus
---
drivers/staging/iio/cdc
Add device tree binding documentation for AD7746 cdc in YAML format.
Signed-off-by: Lucas Stankus
---
.../bindings/iio/cdc/adi,ad7746.yaml | 79 +++
1 file changed, 79 insertions(+)
create mode 100644 Documentation/devicetree/bindings/iio/cdc/adi,ad7746.yaml
diff
Ditch platform_data fields in favor of device tree properties for
configuring EXCA and EXCB output.
This also removes the fields from the platform_data struct, since they're
not used anymore.
Signed-off-by: Lucas Stankus
---
drivers/staging/iio/cdc/ad7746.c | 33
entirely.
Lucas Stankus (3):
dt-bindings: staging: iio: cdc: ad7746: add binding documentation for
AD7746
staging: iio: cdc: ad7746: use dt bindings to set the EXCx pins output
staging: iio: cdc: ad7746: use dt binding to set the excitation level
.../bindings/iio/cdc/adi,ad7746.yaml
Registers the device using the devm variant.
This is the final step of converting the ad7923 to only use devm routines,
meaning that the ad7923_remove() function is no longer needed to release
resources on device detach.
Signed-off-by: Lucas Stankus
---
drivers/iio/adc/ad7923.c | 12
Converts the iio_triggered_buffer_setup() call to its device-managed
counterpart.
With this, the error handling routine in the ad7923_probe() function
becomes unnecessary as well as the iio_triggered_buffer_cleanup() call.
Signed-off-by: Lucas Stankus
---
drivers/iio/adc/ad7923.c | 15
Adds a device-managed action to handle disabling the driver's regulator on
device detach.
This slightly simplifies deinitialization and enables further conversion of
the driver to device-managed routines without breaking the init order.
Signed-off-by: Lucas Stankus
---
drivers/iio/adc/ad7923.c
was no longer necessary and could
be entirely removed from the driver.
Lucas Stankus (3):
iio: adc: ad7923: use devm_add_action_or_reset for regulator disable
iio: adc: ad7923: use device-managed function for triggered buffer
iio: adc: ad7923: register device with devm_iio_device_register
Tidy up code by fixing the following checkpatch warnings:
CHECK: Alignment should match open parenthesis
CHECK: Lines should not end with a '('
Signed-off-by: Lucas Stankus
Reviewed-by: Brendan Higgins
---
Changelog v2 -> v3:
Reword subject line
lib/kunit/assert.c |
On Wed, Mar 03, 2021 at 12:56:05PM -0800, Brendan Higgins wrote:
> Did you change anything other than fixing the Signed-off-by that Shuah
> requested?
No, I only fixed the Signed-off-by warning.
> Generally when you make a small change after receiving a Reviewed-by
> (especially one so small as
Tidy up code by fixing the following checkpatch warnings:
CHECK: Alignment should match open parenthesis
CHECK: Lines should not end with a '('
Signed-off-by: Lucas Stankus
---
Change log v1 -> v2
fixed signed-off-by tag
lib/kunit/assert.c | 31 ---
1 file chan
14 matches
Mail list logo
