[PATCH v2 3/4] hwmon: (ltc2945): add support for sense resistor

2020-11-11 Thread Alexandru Ardelean
harder to make the application code portable from one board to another. This change implements support for the sense resistor to be configured from DT/ACPI and used in current calculations. Signed-off-by: Alexandru Ardelean --- drivers/hwmon/ltc2945.c | 53

[PATCH v2 1/4] hwmon: (ltc2945): wrap regmap into an ltc2945_state struct

2020-11-11 Thread Alexandru Ardelean
The intent is to add pass the value of the sense resistor in the driver. This change wraps a 'struct ltc2945_state', and moves the regmap reference on that object. Then we can add the value of the sense resistor, or other information that would be useful for the driver. Signed-off-by: Alexandru

[PATCH v2 2/4] docs: hwmon: (ltc2945): change type of val to ULL in ltc2945_val_to_reg()

2020-11-11 Thread Alexandru Ardelean
In order to account for any potential overflows that could occur. Signed-off-by: Alexandru Ardelean --- drivers/hwmon/ltc2945.c | 12 ++-- 1 file changed, 6 insertions(+), 6 deletions(-) diff --git a/drivers/hwmon/ltc2945.c b/drivers/hwmon/ltc2945.c index 1cea710df668..6d4569a25471

[PATCH v2 4/4] dt-bindings: hwmon: ltc2945: add device tree doc for ltc2945

2020-11-11 Thread Alexandru Ardelean
This change adds a device-tree binding documentation for the Linear Technology (now Analog Devices) LTC2945 Wide Range I2C Power Monitor. Signed-off-by: Alexandru Ardelean --- .../bindings/hwmon/adi,ltc2945.yaml | 49 +++ 1 file changed, 49 insertions(+) create mode

[PATCH v2 0/4] hwmon: (ltc2945): add support for sense resistor

2020-11-11 Thread Alexandru Ardelean
iver. Changelog v1 -> v2: * https://lore.kernel.org/linux-hwmon/20201106101825.30960-1-alexandru.ardel...@analog.com/ * reverted kstrotoull() -> kstrtoul() * added sanity check for resistor, to prevent crashing when DT provides zero value * add DT binding doc for ltc2945 Alexandru A

[PATCH 6/6] Input: adp5589: use devm_add_action_or_reset() for register clear

2020-11-11 Thread Alexandru Ardelean
The driver clears the general configuration register during the remove() hook. This should also be done in case the driver exits on error. This change move the clear of that register to the devm_add_action_or_reset() hook. Signed-off-by: Alexandru Ardelean --- drivers/input/keyboard/adp5589

[PATCH 3/6] Input: adp5589: use device-managed function in adp5589_keypad_add()

2020-11-11 Thread Alexandru Ardelean
move() hook, but doesn't look like it is. This change may also also fix some potential leaks that were probably omitted earlier. Signed-off-by: Alexandru Ardelean --- drivers/input/keyboard/adp5589-keys.c | 47 +++ 1 file changed, 11 insertions(+), 36 deletions(-) diff -

[PATCH 2/6] Input: adp5589: use devm_kzalloc() to allocate the kpad object

2020-11-11 Thread Alexandru Ardelean
This removes the need to manually free the kpad object and cleans up some exit/error paths. The error path cleanup should reduce the risk of any memory leaks with this object. Signed-off-by: Alexandru Ardelean --- drivers/input/keyboard/adp5589-keys.c | 9 +++-- 1 file changed, 3 insertions

[PATCH 5/6] Input: adp5589: use devm_gpiochip_add_data() for gpios

2020-11-11 Thread Alexandru Ardelean
This change makes use of the devm_gpiochip_add_data() function. With this the gpiochip_remove() function can be removed, and the adp5589_gpio_remove() function as well. The kpad->export_gpio variable is also redundant now, and has been removed. Signed-off-by: Alexandru Ardelean --- driv

[PATCH 4/6] Input: adp5589: remove setup/teardown hooks for gpios

2020-11-11 Thread Alexandru Ardelean
This is currently just dead code. It's from around a time when platform-data was used, and a board could hook it's own special callback for setup/teardown, and a private object (via 'context'). This change removes it, as there are no more users in mainline for this. Signed-off-by: Alexandru

[PATCH 0/6] Input: adp5589: cleanup and use device-managed functions

2020-11-11 Thread Alexandru Ardelean
This change-set does a cleanup of the driver to use device-managed functions. The error & exit paths are simplified, and some potential leaks should be removed. Alexandru Ardelean (6): Input: adp5589: use a single variable for error in probe Input: adp5589: use devm_kzalloc() to allo

[PATCH 1/6] Input: adp5589: use a single variable for error in probe

2020-11-11 Thread Alexandru Ardelean
The 'error' & 'ret' variables are used. This is a bit of duplication. This change replaces the use of error with the 'ret' variable since the name is a bit more generic. Signed-off-by: Alexandru Ardelean --- drivers/input/keyboard/adp5589-keys.c | 20 +--- 1 file change

[PATCH] ASoc: adi: Kconfig: Remove depends on for ADI reference designs

2020-11-10 Thread Alexandru Ardelean
the usage of this driver for the systems described above. Signed-off-by: Bogdan Togorean Signed-off-by: Alexandru Ardelean --- sound/soc/adi/Kconfig | 1 - 1 file changed, 1 deletion(-) diff --git a/sound/soc/adi/Kconfig b/sound/soc/adi/Kconfig index e321e3b672da..0236dc5b4e9f 100644

[PATCH v2] dt-bindings: adau1977: convert text binding to yaml format

2020-11-10 Thread Alexandru Ardelean
This change converts the old device-tree binding for ADAU1977 from text format to the new yaml format. Signed-off-by: Alexandru Ardelean --- Changelog v1 -> v2: * updated libraries to catch newer schema errors/warnings * fixed examples, added i2c & spi nodes .../bindings/sound/adi,a

Re: [PATCH 3/3] hwmon: (ltc2945): add support for sense resistor

2020-11-08 Thread Alexandru Ardelean
On Fri, Nov 6, 2020 at 3:17 PM Guenter Roeck wrote: > > On Fri, Nov 06, 2020 at 12:18:25PM +0200, Alexandru Ardelean wrote: > > The sense resistor is a parameter of the board. It should be configured in > > the driver via a device-tree / ACPI property, so that the proper curren

Re: [PATCH 2/3] docs: hwmon: (ltc2945): change type of val to ULL in ltc2945_val_to_reg()

2020-11-08 Thread Alexandru Ardelean
On Fri, Nov 6, 2020 at 3:14 PM Guenter Roeck wrote: > > On Fri, Nov 06, 2020 at 12:18:24PM +0200, Alexandru Ardelean wrote: > > In order to account for any potential overflows that could occur. > > > > Signed-off-by: Alexandru Ardelean > > --- >

[PATCH] dt-bindings: adau1977: convert text binding to yaml format

2020-11-06 Thread Alexandru Ardelean
This change converts the old device-tree binding for ADAU1977 from text format to the new yaml format. Signed-off-by: Alexandru Ardelean --- .../bindings/sound/adi,adau1977.txt | 61 --- .../bindings/sound/adi,adau1977.yaml | 77 +++ 2 files

[PATCH 1/3] hwmon: (ltc2945): wrap regmap into an ltc2945_state struct

2020-11-06 Thread Alexandru Ardelean
The intent is to add pass the value of the sense resistor in the driver. This change wraps a 'struct ltc2945_state', and moves the regmap reference on that object. Then we can add the value of the sense resistor, or other information that would be useful for the driver. Signed-off-by: Alexandru

[PATCH 2/3] docs: hwmon: (ltc2945): change type of val to ULL in ltc2945_val_to_reg()

2020-11-06 Thread Alexandru Ardelean
In order to account for any potential overflows that could occur. Signed-off-by: Alexandru Ardelean --- drivers/hwmon/ltc2945.c | 14 +++--- 1 file changed, 7 insertions(+), 7 deletions(-) diff --git a/drivers/hwmon/ltc2945.c b/drivers/hwmon/ltc2945.c index 1cea710df668..75d997d31e01

[PATCH 3/3] hwmon: (ltc2945): add support for sense resistor

2020-11-06 Thread Alexandru Ardelean
harder to make the application code portable from one board to another. This change implements support for the sense resistor to be configured from DT/ACPI and used in current calculations. Signed-off-by: Alexandru Ardelean --- drivers/hwmon/ltc2945.c | 48

[PATCH net-next v2 1/2][RESEND] net: phy: adin: disable diag clock & disable standby mode in config_aneg

2020-11-02 Thread Alexandru Ardelean
retry value and the ADIN1300_LINKING_EN are in the same register. And the ADIN1300_DIAG_CLK_EN bit (13) is cleared, to disable the diagnostics clock. Reviewed-by: Andrew Lunn Signed-off-by: Alexandru Ardelean --- This is a re-send of: https://lore.kernel.org/netdev/20201022074551.11520-1

[PATCH net-next v2 2/2][RESEND] net: phy: adin: implement cable-test support

2020-11-02 Thread Alexandru Ardelean
implements support for this using phylib's cable-test support. Reviewed-by: Andrew Lunn Signed-off-by: Alexandru Ardelean --- drivers/net/phy/adin.c | 138 + 1 file changed, 138 insertions(+) diff --git a/drivers/net/phy/adin.c b/drivers/net/phy/adin.c index

Re: [PATCH] iio: core: centralize ioctl() calls to the main chardev

2020-11-01 Thread Alexandru Ardelean
On Fri, Sep 25, 2020 at 3:38 PM Jonathan Cameron wrote: > > On Thu, 24 Sep 2020 11:41:55 +0300 > Alexandru Ardelean wrote: > > > The aim of this is to improve a bit the organization of ioctl() calls in > > IIO core. Currently the chardev is split across IIO core sub-mod

Re: [PATCH v2 1/2] net: phy: adin: disable diag clock & disable standby mode in config_aneg

2020-10-31 Thread Alexandru Ardelean
On Sat, Oct 24, 2020 at 1:43 AM Andrew Lunn wrote: > > On Thu, Oct 22, 2020 at 10:45:50AM +0300, Alexandru Ardelean wrote: > > When the PHY powers up, the diagnostics clock isn't enabled (bit 2 in > > register PHY_CTRL_1 (0x0012)). > > Also, the PHY is not in standby mode,

Re: [PATCH v2 2/2] net: phy: adin: implement cable-test support

2020-10-23 Thread Alexandru Ardelean
On Fri, Oct 23, 2020 at 3:02 AM Jakub Kicinski wrote: > > On Thu, 22 Oct 2020 10:45:51 +0300 Alexandru Ardelean wrote: > > The ADIN1300/ADIN1200 support cable diagnostics using TDR. > > > > The cable fault detection is automatically run on all four pairs looking at >

[PATCH v2 2/2] net: phy: adin: implement cable-test support

2020-10-22 Thread Alexandru Ardelean
implements support for this using phylib's cable-test support. Signed-off-by: Alexandru Ardelean --- drivers/net/phy/adin.c | 138 + 1 file changed, 138 insertions(+) diff --git a/drivers/net/phy/adin.c b/drivers/net/phy/adin.c index 619d36685b5d..3e66f97c7611

[PATCH v2 1/2] net: phy: adin: disable diag clock & disable standby mode in config_aneg

2020-10-22 Thread Alexandru Ardelean
retry value and the ADIN1300_LINKING_EN are in the same register. And the ADIN1300_DIAG_CLK_EN bit (13) is cleared, to disable the diagnostics clock. Signed-off-by: Alexandru Ardelean --- Changelog v1 -> v2: * updated title; updated description to better describe the diagnostics cl

Re: [PATCH 2/2] net: phy: adin: implement cable-test support

2020-10-21 Thread Alexandru Ardelean
On Wed, Oct 21, 2020 at 5:28 PM Andrew Lunn wrote: > > > Actually, I'd also be interested [for this PHY], to report a > > "significance impedance" detection, which is similar to the > > short-detection that is already done. > > You can add that as just another element of the enum. > > > At first,

Re: [PATCH 1/2] net: phy: adin: clear the diag clock and set LINKING_EN during autoneg

2020-10-21 Thread Alexandru Ardelean
On Wed, Oct 21, 2020 at 5:13 PM Andrew Lunn wrote: > > > The frame-generator is an interesting feature of the PHY, that's not > > useful for the current phylib; the PHY can send packages [like a > > signal generator], and then these can be looped back, or sent over the > > wire. > removed my

Re: [PATCH 2/2] net: phy: adin: implement cable-test support

2020-10-21 Thread Alexandru Ardelean
On Wed, Oct 21, 2020 at 5:09 PM Andrew Lunn wrote: > > > Signed-off-by: Alexandru Ardelean > removed my typo-ed email > Hi Alexandru > > Overall, this looks good. > > > +static int adin_cable_test_report_trans(int result) > > +{ > >

Re: [PATCH 1/2] net: phy: adin: clear the diag clock and set LINKING_EN during autoneg

2020-10-21 Thread Alexandru Ardelean
On Wed, Oct 21, 2020 at 4:58 PM Andrew Lunn wrote: > > On Wed, Oct 21, 2020 at 04:51:39PM +0300, Alexandru Ardelean wrote: > > The LINKING_EN bit is always cleared during reset. Initially it was set > > during the downshift setup, because it's in the same register as the > &g

[PATCH 1/2] net: phy: adin: clear the diag clock and set LINKING_EN during autoneg

2020-10-21 Thread Alexandru Ardelean
). During this call, the diagnostics clock should be disabled, and the LINKING_EN bit set in the PHY_CTRL1 register. Signed-off-by: Alexandru Ardelean --- drivers/net/phy/adin.c | 12 ++-- 1 file changed, 10 insertions(+), 2 deletions(-) diff --git a/drivers/net/phy/adin.c b/drivers/net

[PATCH 2/2] net: phy: adin: implement cable-test support

2020-10-21 Thread Alexandru Ardelean
implements support for this using phylib's cable-test support. Signed-off-by: Alexandru Ardelean --- drivers/net/phy/adin.c | 138 + 1 file changed, 138 insertions(+) diff --git a/drivers/net/phy/adin.c b/drivers/net/phy/adin.c index 619d36685b5d..3e66f97c7611

[PATCH 2/2] clk: axi-clkgen: move the OF table at the bottom of the file

2020-10-19 Thread Alexandru Ardelean
The change is mostly cosmetic. No functional changes. Since the driver now uses device_get_match_data() to obtain the driver specific info, there is no need to define the OF table before the probe function. Signed-off-by: Alexandru Ardelean --- drivers/clk/clk-axi-clkgen.c | 18

[PATCH 1/2] clk: axi-clkgen: wrap limits in a struct and keep copy on the state object

2020-10-19 Thread Alexandru Ardelean
on the device-tree OF table, so that we can adjust them via the compatible string. Signed-off-by: Alexandru Ardelean --- drivers/clk/clk-axi-clkgen.c | 48 +++- 1 file changed, 31 insertions(+), 17 deletions(-) diff --git a/drivers/clk/clk-axi-clkgen.c b/drivers/clk/clk

[PATCH] ASoC: adau1977: remove platform data and move micbias bindings include

2020-10-19 Thread Alexandru Ardelean
can understand them. The driver then, also needs to include the new 'dt-bindings/sound/adi,adau1977.h' file. Signed-off-by: Alexandru Ardelean --- include/dt-bindings/sound/adi,adau1977.h | 15 include/linux/platform_data/adau1977.h | 44 sound/soc/codecs

[PATCH v2] dt-bindings: clock: adi,axi-clkgen: convert old binding to yaml format

2020-10-13 Thread Alexandru Ardelean
usen Signed-off-by: Alexandru Ardelean --- Changelog v1 -> v2: * add 'additionalProperties: false' * changed 'clock@...' -> 'clock-controller@...' in example * added Acked-by for Michael & Lars on the re-licensing * updated description for 'clocks' property .../bindings/clock/adi,a

[PATCH] docs: hwmon: (ltc2945): update datasheet link

2020-10-05 Thread Alexandru Ardelean
Old one isn't working anymore. Update to the latest datasheet link. Signed-off-by: Alexandru Ardelean --- Documentation/hwmon/ltc2945.rst | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/Documentation/hwmon/ltc2945.rst b/Documentation/hwmon/ltc2945.rst index 20c884985367

[PATCH] hwmon: (pmbus/max34440) Fix/add missing break statements

2020-10-05 Thread Alexandru Ardelean
This one was caught by the compiler. Adds missing break statements, which look omitted by accident. Fixes: 599a5ac5f96c ("hwmon: (pmbus/max34440) Fix/add missing break statements") Signed-off-by: Alexandru Ardelean --- Maybe the lkp bot also reported this [by now]. I just bumped int

[PATCH v2] iio: adc: ad7887: invert/rework external ref logic

2020-10-02 Thread Alexandru Ardelean
-by: Alexandru Ardelean --- Changelog v1 -> v2: * remove omitted '!pdata->use_onchip_ref' check; the field was removed from the platform data, but was still used drivers/iio/adc/ad7887.c | 12 include/linux/platform_data/ad7887.h | 4 2 files chan

Re: [PATCH] iio: adc: ad7887: invert/rework external ref logic

2020-10-02 Thread Alexandru Ardelean
ed to the wrong git tree, kindly drop us a note. > And when submitting patch, we suggest to use '--base' as documented in > https://git-scm.com/docs/git-format-patch] > > url: > https://github.com/0day-ci/linux/commits/Alexandru-Ardelean/iio-adc-ad7887-invert-rework-external-re

[PATCH] iio: adc: ad7887: invert/rework external ref logic

2020-10-01 Thread Alexandru Ardelean
-by: Alexandru Ardelean --- drivers/iio/adc/ad7887.c | 10 +++--- include/linux/platform_data/ad7887.h | 4 2 files changed, 7 insertions(+), 7 deletions(-) diff --git a/drivers/iio/adc/ad7887.c b/drivers/iio/adc/ad7887.c index 037bcb47693c..9b32b1f43c34 100644 --- a/drivers

[PATCH] iio: adc: ad7298: rework external ref setup & remove platform data

2020-10-01 Thread Alexandru Ardelean
as before. Signed-off-by: Alexandru Ardelean --- drivers/iio/adc/ad7298.c | 17 + include/linux/platform_data/ad7298.h | 19 --- 2 files changed, 9 insertions(+), 27 deletions(-) delete mode 100644 include/linux/platform_data/ad7298.h diff --git

[PATCH] iio: dac: ad7303: remove platform data header

2020-10-01 Thread Alexandru Ardelean
The information in the ad7303 platform_data header is unused, so it's dead code. This change removes it and it's inclusion from the driver. Signed-off-by: Alexandru Ardelean --- drivers/iio/dac/ad7303.c | 2 -- include/linux/platform_data/ad7303.h | 20 2 files

[PATCH 2/2] clk: axi-clkgen: Set power bits for fractional mode

2020-10-01 Thread Alexandru Ardelean
mode. Signed-off-by: Lars-Peter Clausen Signed-off-by: Alexandru Ardelean --- drivers/clk/clk-axi-clkgen.c | 7 +++ 1 file changed, 7 insertions(+) diff --git a/drivers/clk/clk-axi-clkgen.c b/drivers/clk/clk-axi-clkgen.c index 1df03cc6d089..14d803e6af62 100644 --- a/drivers/clk/clk-axi

[PATCH 1/2] clk: axi-clkgen: Add support for fractional dividers

2020-10-01 Thread Alexandru Ardelean
to be the recommended mode of operation. Signed-off-by: Lars-Peter Clausen Signed-off-by: Alexandru Ardelean --- Series split away from [1]: https://lore.kernel.org/linux-clk/20200929144417.89816-9-alexandru.ardel...@analog.com/T/#t After a review on that, it was concluded that dt-binding

[PATCH] dt-bindings: clock: adi,axi-clkgen: convert old binding to yaml format

2020-10-01 Thread Alexandru Ardelean
This change converts the old binding for the AXI clkgen driver to a yaml format. As maintainers, added: - Lars-Peter Clausen - as original author of driver & binding - Michael Hennerich - as supporter of Analog Devices drivers Signed-off-by: Alexandru Ardelean --- .../bindings/c

Re: [PATCH v4 5/7] clk: axi-clkgen: Respect ZYNQMP PFD/VCO frequency limits

2020-10-01 Thread Alexandru Ardelean
On Thu, Oct 1, 2020 at 8:18 AM Alexandru Ardelean wrote: > > On Wed, Sep 30, 2020 at 8:16 PM Moritz Fischer wrote: > > > > On Wed, Sep 30, 2020 at 08:22:23AM +0300, Alexandru Ardelean wrote: > > > On Tue, Sep 29, 2020 at 6:30 PM Moritz Fischer wrote: &

Re: [PATCH v4 5/7] clk: axi-clkgen: Respect ZYNQMP PFD/VCO frequency limits

2020-09-30 Thread Alexandru Ardelean
On Wed, Sep 30, 2020 at 8:16 PM Moritz Fischer wrote: > > On Wed, Sep 30, 2020 at 08:22:23AM +0300, Alexandru Ardelean wrote: > > On Tue, Sep 29, 2020 at 6:30 PM Moritz Fischer wrote: > > > > > > Hi Alexandru, > > > > > > On Tue, Sep 29, 20

[PATCH v4 3/4] iio: adc: at91_adc: add Kconfig dep on the OF symbol and remove of_match_ptr()

2020-09-30 Thread Alexandru Ardelean
-by: kernel test robot Reviewed-by: Alexandre Belloni Signed-off-by: Alexandru Ardelean --- drivers/iio/adc/Kconfig| 2 +- drivers/iio/adc/at91_adc.c | 2 +- 2 files changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/iio/adc/Kconfig b/drivers/iio/adc/Kconfig index 91ae90514af

[PATCH v4 4/4] iio: adc: at91_adc: remove platform data and move defs in driver file

2020-09-30 Thread Alexandru Ardelean
include] have been moved in the driver, since they are needed in the driver. Signed-off-by: Alexandru Ardelean --- drivers/iio/adc/at91_adc.c | 60 ++ include/linux/platform_data/at91_adc.h | 49 - 2 files changed, 22 insertions(+), 87 deletions

[PATCH v4 2/4] iio: adc: at91_adc: const-ify some driver data

2020-09-30 Thread Alexandru Ardelean
), but that cast is not touched in this patch, since the old platform_data should be removed/cleaned-away. Also, that cast deals with converting a kernel_ulong_t type to a pointer. So, updating that cast doesn't yield any benefit. Reviewed-by: Alexandre Belloni Signed-off-by: Alexandru Ardelean

[PATCH v4 0/4] iio: adc: at91: misc driver cleanups

2020-09-30 Thread Alexandru Ardelean
o: adc: at91_adc: use of_device_get_match_data() helper' changed description; it's just tidy-up patch, not a fix - added 2 more patches: - iio: adc: at91_adc: add Kconfig dependency on the OF symbol - iio: adc: at91_adc: remove of_match_ptr() usage Alexandru Ardelean (4): iio: adc: at91_adc: use of_

[PATCH v4 1/4] iio: adc: at91_adc: use of_device_get_match_data() helper

2020-09-30 Thread Alexandru Ardelean
This is a small tidy-up. The of_device_get_match_data() helper retrieves the driver data from the OF table, without needed to explicitly know the table variable (since it can retrieve it from the driver object). Reviewed-by: Alexandre Belloni Signed-off-by: Alexandru Ardelean --- drivers/iio

[PATCH v3 0/4] iio: adc: at91: misc driver cleanups

2020-09-30 Thread Alexandru Ardelean
_match_ptr() usage Alexandru Ardelean (4): iio: adc: at91_adc: use of_device_get_match_data() helper iio: adc: at91_adc: const-ify some driver data iio: adc: at91_adc: add Kconfig dep on the OF symbol and remove of_match_ptr() iio: adc: at91_adc: remove platform data and move defs in driver

[PATCH v3 4/4] iio: adc: at91_adc: remove platform data and move defs in driver file

2020-09-30 Thread Alexandru Ardelean
include] have been to be moved in the driver, since they are needed in the driver. Signed-off-by: Alexandru Ardelean --- drivers/iio/adc/at91_adc.c | 51 ++ include/linux/platform_data/at91_adc.h | 49 - 2 files changed, 20 insertions(+), 80

[PATCH v3 1/4] iio: adc: at91_adc: use of_device_get_match_data() helper

2020-09-30 Thread Alexandru Ardelean
This is a small tidy-up. The of_device_get_match_data() helper retrieves the driver data from the OF table, without needed to explicitly know the table variable (since it can retrieve it from the driver object). Reviewed-by: Alexandre Belloni Signed-off-by: Alexandru Ardelean --- drivers/iio

[PATCH v3 2/4] iio: adc: at91_adc: const-ify some driver data

2020-09-30 Thread Alexandru Ardelean
), but that cast is not touched in this patch, since the old platform_data should be removed/cleaned-away. Also, that cast deals with converting a kernel_ulong_t type to a pointer. So, updating that cast doesn't yield any benefit. Reviewed-by: Alexandre Belloni Signed-off-by: Alexandru Ardelean

[PATCH v3 3/4] iio: adc: at91_adc: add Kconfig dep on the OF symbol and remove of_match_ptr()

2020-09-30 Thread Alexandru Ardelean
-by: kernel test robot Signed-off-by: Alexandru Ardelean --- drivers/iio/adc/Kconfig| 2 +- drivers/iio/adc/at91_adc.c | 2 +- 2 files changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/iio/adc/Kconfig b/drivers/iio/adc/Kconfig index 91ae90514aff..17e9ceb9c6c4 100644 --- a/drivers

Re: [PATCH v2 3/4] iio: adc: at91_adc: add Kconfig dependency on the OF symbol

2020-09-30 Thread Alexandru Ardelean
On Wed, Sep 30, 2020 at 10:39 AM Alexandre Belloni wrote: > > Hi, > > On 30/09/2020 09:00:07+0300, Alexandru Ardelean wrote: > > This tries to solve a warning reported by the lkp bot: > > > > >> drivers/iio/adc/at91_adc.c:1439:34: warning: unused variable >

Re: [PATCH v2 4/4] iio: adc: at91_adc: remove of_match_ptr() usage

2020-09-30 Thread Alexandru Ardelean
On Wed, Sep 30, 2020 at 10:48 AM Alexandre Belloni wrote: > > Hi, > > On 30/09/2020 09:00:08+0300, Alexandru Ardelean wrote: > > Since the driver should be allowed to build without OF support, the > > of_match_ptr() is redundant. > > > > This can probably

Re: [PATCH 4/5] iio: adc: vf610_adc: Replace indio_dev->mlock with own device lock

2020-09-29 Thread Alexandru Ardelean
On Tue, Sep 29, 2020 at 7:15 PM Jonathan Cameron wrote: > > On Mon, 28 Sep 2020 16:13:32 +0300 > Mircea Caprioru wrote: > > > From: Sergiu Cuciurean > > > > As part of the general cleanup of indio_dev->mlock, this change replaces > > it with a local lock on the device's state structure. > > > >

[PATCH v2 2/4] iio: adc: at91_adc: const-ify some driver data

2020-09-29 Thread Alexandru Ardelean
), but that cast is not touched in this patch, since the old platform_data should be removed/cleaned-away. Also, that cast deals with converting a kernel_ulong_t type to a pointer. So, updating that cast doesn't yield any benefit. Signed-off-by: Alexandru Ardelean --- drivers/iio/adc/at91_adc.c | 8

[PATCH v2 3/4] iio: adc: at91_adc: add Kconfig dependency on the OF symbol

2020-09-29 Thread Alexandru Ardelean
E_TEST dependency to driver"), which put this driver on the lkp's bot radar. Fixes: 4027860dcc4c ("iio: Kconfig: at91_adc: add COMPILE_TEST dependency to driver") Reported-by: kernel test robot Signed-off-by: Alexandru Ardelean --- drivers/iio/adc/Kconfig | 2 +- 1 file changed, 1 i

[PATCH v2 0/4] iio: adc: at91: misc driver cleanups

2020-09-29 Thread Alexandru Ardelean
ail.gmail.com/T/#m7c0efef4dc623776fe8bafdb5f734b0eaca50f82 - for patch 'iio: adc: at91_adc: use of_device_get_match_data() helper' changed description; it's just tidy-up patch, not a fix - added 2 more patches: - iio: adc: at91_adc: add Kconfig dependency on the OF symbol - iio: adc: at91_adc: remove of_match_ptr() u

[PATCH v2 4/4] iio: adc: at91_adc: remove of_match_ptr() usage

2020-09-29 Thread Alexandru Ardelean
Since the driver should be allowed to build without OF support, the of_match_ptr() is redundant. Signed-off-by: Alexandru Ardelean --- drivers/iio/adc/at91_adc.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/iio/adc/at91_adc.c b/drivers/iio/adc/at91_adc.c index

[PATCH v2 1/4] iio: adc: at91_adc: use of_device_get_match_data() helper

2020-09-29 Thread Alexandru Ardelean
This is a small tidy-up. The of_device_get_match_data() helper retrieves the driver data from the OF table, without needed to explicitly know the table variable (since it can retrieve it from the driver object). Signed-off-by: Alexandru Ardelean --- drivers/iio/adc/at91_adc.c | 5 + 1 file

Re: [PATCH 1/2] iio: adc: at91_adc: use of_device_get_match_data() helper

2020-09-29 Thread Alexandru Ardelean
On Tue, Sep 29, 2020 at 6:55 PM Jonathan Cameron wrote: > > On Mon, 28 Sep 2020 15:54:23 +0300 > Alexandru Ardelean wrote: > > > This tries to solve a warning reported by the lkp bot: > > > > >> drivers/iio/adc/at91_adc.c:1439:34: warning: unused variable >

Re: [PATCH 1/2] iio: adc: at91_adc: use of_device_get_match_data() helper

2020-09-29 Thread Alexandru Ardelean
On Tue, Sep 29, 2020 at 6:55 PM Jonathan Cameron wrote: > > On Mon, 28 Sep 2020 15:54:23 +0300 > Alexandru Ardelean wrote: > > > This tries to solve a warning reported by the lkp bot: > > > > >> drivers/iio/adc/at91_adc.c:1439:34: warning: unused variable >

Re: [PATCH v4 5/7] clk: axi-clkgen: Respect ZYNQMP PFD/VCO frequency limits

2020-09-29 Thread Alexandru Ardelean
On Tue, Sep 29, 2020 at 6:30 PM Moritz Fischer wrote: > > Hi Alexandru, > > On Tue, Sep 29, 2020 at 05:44:15PM +0300, Alexandru Ardelean wrote: > > From: Mathias Tausen > > > > Since axi-clkgen is now supported on ZYNQMP, make sure the max/min > > frequenc

[PATCH v4 7/7] clk: axi-clkgen: Add support for FPGA info

2020-09-29 Thread Alexandru Ardelean
, which are stored in the 'include/linux/fpga/adi-axi-common.h' file as they are common to all ADI FPGA cores. Signed-off-by: Mircea Caprioru Signed-off-by: Alexandru Ardelean --- drivers/clk/clk-axi-clkgen.c | 52 +++- 1 file changed, 51 insertions(+), 1 deletion

[PATCH v4 2/7] clk: axi-clkgen: Set power bits for fractional mode

2020-09-29 Thread Alexandru Ardelean
mode. Signed-off-by: Lars-Peter Clausen Signed-off-by: Alexandru Ardelean --- drivers/clk/clk-axi-clkgen.c | 7 +++ 1 file changed, 7 insertions(+) diff --git a/drivers/clk/clk-axi-clkgen.c b/drivers/clk/clk-axi-clkgen.c index 1df03cc6d089..14d803e6af62 100644 --- a/drivers/clk/clk-axi

[PATCH v4 6/7] include: fpga: adi-axi-common.h: add definitions for supported FPGAs

2020-09-29 Thread Alexandru Ardelean
-by: Mircea Caprioru Signed-off-by: Alexandru Ardelean --- include/linux/fpga/adi-axi-common.h | 103 1 file changed, 103 insertions(+) diff --git a/include/linux/fpga/adi-axi-common.h b/include/linux/fpga/adi-axi-common.h index 141ac3f251e6..1a7f18e3a384 100644

[PATCH v4 4/7] clk: axi-clkgen: wrap limits in a struct and keep copy on the state object

2020-09-29 Thread Alexandru Ardelean
to be adjusted for each instance of the AXI clock. This change wraps the current constants in 'axi_clkgen_limits' struct and the 'axi_clkgen' instance keeps a copy of these limits, which is initialized at probe from the default limits. Signed-off-by: Alexandru Ardelean --- drivers/clk/clk-axi

[PATCH v4 5/7] clk: axi-clkgen: Respect ZYNQMP PFD/VCO frequency limits

2020-09-29 Thread Alexandru Ardelean
From: Mathias Tausen Since axi-clkgen is now supported on ZYNQMP, make sure the max/min frequencies of the PFD and VCO are respected. Signed-off-by: Mathias Tausen Signed-off-by: Alexandru Ardelean --- drivers/clk/clk-axi-clkgen.c | 9 + 1 file changed, 9 insertions(+) diff --git

[PATCH v4 0/7] clk: axi-clk-gen: misc updates to the driver

2020-09-29 Thread Alexandru Ardelean
ithub.com/analogdevicesinc/hdl/blob/4e438261aa319b1dda4c593c155218a93b1d869b/library/scripts/adi_intel_device_info_enc.tcl https://github.com/analogdevicesinc/hdl/blob/4e438261aa319b1dda4c593c155218a93b1d869b/library/scripts/adi_xilinx_device_info_enc.tcl Alexandru Ardelean (1): clk: axi-clkgen: wr

[PATCH v4 1/7] clk: axi-clkgen: Add support for fractional dividers

2020-09-29 Thread Alexandru Ardelean
to be the recommended mode of operation. Signed-off-by: Lars-Peter Clausen Signed-off-by: Alexandru Ardelean --- drivers/clk/clk-axi-clkgen.c | 180 +-- 1 file changed, 129 insertions(+), 51 deletions(-) diff --git a/drivers/clk/clk-axi-clkgen.c b/drivers/clk/clk

[PATCH v4 7/7] clk: axi-clkgen: Add support for FPGA info

2020-09-29 Thread Alexandru Ardelean
, which are stored in the 'include/linux/fpga/adi-axi-common.h' file as they are common to all ADI FPGA cores. Signed-off-by: Mircea Caprioru Signed-off-by: Alexandru Ardelean --- drivers/clk/clk-axi-clkgen.c | 52 +++- 1 file changed, 51 insertions(+), 1 deletion

[PATCH v4 5/7] clk: axi-clkgen: Respect ZYNQMP PFD/VCO frequency limits

2020-09-29 Thread Alexandru Ardelean
From: Mathias Tausen Since axi-clkgen is now supported on ZYNQMP, make sure the max/min frequencies of the PFD and VCO are respected. Signed-off-by: Mathias Tausen Signed-off-by: Alexandru Ardelean --- drivers/clk/clk-axi-clkgen.c | 9 + 1 file changed, 9 insertions(+) diff --git

[PATCH v4 6/7] include: fpga: adi-axi-common.h: add definitions for supported FPGAs

2020-09-29 Thread Alexandru Ardelean
-by: Mircea Caprioru Signed-off-by: Alexandru Ardelean --- include/linux/fpga/adi-axi-common.h | 103 1 file changed, 103 insertions(+) diff --git a/include/linux/fpga/adi-axi-common.h b/include/linux/fpga/adi-axi-common.h index 141ac3f251e6..1a7f18e3a384 100644

[PATCH v4 3/7] clk: axi-clkgen: add support for ZynqMP (UltraScale)

2020-09-29 Thread Alexandru Ardelean
From: Dragos Bogdan This IP core also works and is supported on the Xilinx ZynqMP (UltraScale) FPGA boards. This patch enables the driver to be available on these platforms as well. Signed-off-by: Dragos Bogdan Signed-off-by: Alexandru Ardelean --- drivers/clk/Kconfig | 2 +- 1 file changed

[PATCH v4 3/7] clk: axi-clkgen: add support for ZynqMP (UltraScale)

2020-09-29 Thread Alexandru Ardelean
From: Dragos Bogdan This IP core also works and is supported on the Xilinx ZynqMP (UltraScale) FPGA boards. This patch enables the driver to be available on these platforms as well. Signed-off-by: Dragos Bogdan Signed-off-by: Alexandru Ardelean --- drivers/clk/Kconfig | 2 +- 1 file changed

[PATCH v4 2/7] clk: axi-clkgen: Set power bits for fractional mode

2020-09-29 Thread Alexandru Ardelean
mode. Signed-off-by: Lars-Peter Clausen Signed-off-by: Alexandru Ardelean --- drivers/clk/clk-axi-clkgen.c | 7 +++ 1 file changed, 7 insertions(+) diff --git a/drivers/clk/clk-axi-clkgen.c b/drivers/clk/clk-axi-clkgen.c index 1df03cc6d089..14d803e6af62 100644 --- a/drivers/clk/clk-axi

[PATCH v4 1/7] clk: axi-clkgen: Add support for fractional dividers

2020-09-29 Thread Alexandru Ardelean
to be the recommended mode of operation. Signed-off-by: Lars-Peter Clausen Signed-off-by: Alexandru Ardelean --- drivers/clk/clk-axi-clkgen.c | 180 +-- 1 file changed, 129 insertions(+), 51 deletions(-) diff --git a/drivers/clk/clk-axi-clkgen.c b/drivers/clk/clk

[PATCH v4 4/7] clk: axi-clkgen: wrap limits in a struct and keep copy on the state object

2020-09-29 Thread Alexandru Ardelean
to be adjusted for each instance of the AXI clock. This change wraps the current constants in 'axi_clkgen_limits' struct and the 'axi_clkgen' instance keeps a copy of these limits, which is initialized at probe from the default limits. Signed-off-by: Alexandru Ardelean --- drivers/clk/clk-axi

[PATCH v4 0/7] clk: axi-clk-gen: misc updates to the driver

2020-09-29 Thread Alexandru Ardelean
ithub.com/analogdevicesinc/hdl/blob/4e438261aa319b1dda4c593c155218a93b1d869b/library/scripts/adi_intel_device_info_enc.tcl https://github.com/analogdevicesinc/hdl/blob/4e438261aa319b1dda4c593c155218a93b1d869b/library/scripts/adi_xilinx_device_info_enc.tcl Alexandru Ardelean (1): clk: axi-clkgen: wr

Re: [PATCH v3 7/9] iio: cros_ec: use devm_iio_triggered_buffer_setup_ext()

2020-09-29 Thread Alexandru Ardelean
On Tue, Sep 29, 2020 at 4:09 PM Andy Shevchenko wrote: > > On Tue, Sep 29, 2020 at 3:55 PM Alexandru Ardelean > wrote: > > > This change switches to the new devm_iio_triggered_buffer_setup_ext() > > function and removes the iio_buffer_set_attrs() call, for assigning th

[PATCH v3 6/9] iio: adc: at91-sama5d2_adc: use devm_iio_triggered_buffer_setup_ext()

2020-09-29 Thread Alexandru Ardelean
This change switches to the new devm_iio_triggered_buffer_setup_ext() function and removes the iio_buffer_set_attrs() call, for assigning the HW FIFO attributes to the buffer. Signed-off-by: Alexandru Ardelean --- drivers/iio/adc/at91-sama5d2_adc.c | 12 1 file changed, 8

[PATCH v3 5/9] iio: accel: bmc150: use iio_triggered_buffer_setup_ext()

2020-09-29 Thread Alexandru Ardelean
This change switches to the new iio_triggered_buffer_setup_ext() function and removes the iio_buffer_set_attrs() call, for assigning the HW FIFO attributes to the buffer. Signed-off-by: Alexandru Ardelean --- drivers/iio/accel/bmc150-accel-core.c | 25 ++--- 1 file changed

[PATCH v3 2/9] iio: adc: at91-sama5d2_adc: merge buffer & trigger init into a function

2020-09-29 Thread Alexandru Ardelean
to the parent device vs the IIO device object. Also, the new at91_adc_buffer_and_trigger_init() function was moved after the definition of the 'at91_adc_fifo_attributes'. Signed-off-by: Alexandru Ardelean --- drivers/iio/adc/at91-sama5d2_adc.c | 78 ++ 1 file changed, 36

[PATCH v3 4/9] iio: accel: adxl372: use devm_iio_triggered_buffer_setup_ext()

2020-09-29 Thread Alexandru Ardelean
This change switches to the new devm_iio_triggered_buffer_setup_ext() function and removes the iio_buffer_set_attrs() call, for assigning the HW FIFO attributes to the buffer. Signed-off-by: Alexandru Ardelean --- drivers/iio/accel/adxl372.c | 11 +-- 1 file changed, 5 insertions(+), 6

[PATCH v3 3/9] iio: triggered-buffer: add {devm_}iio_triggered_buffer_setup_ext variants

2020-09-29 Thread Alexandru Ardelean
is what this change does. At this moment, buffers allocated with {devm_}iio_triggered_buffer_setup() are the only ones in mainline IIO to call iio_buffer_set_attrs(). Signed-off-by: Alexandru Ardelean --- .../buffer/industrialio-triggered-buffer.c| 31 --- include/li

[PATCH v3 7/9] iio: cros_ec: use devm_iio_triggered_buffer_setup_ext()

2020-09-29 Thread Alexandru Ardelean
This change switches to the new devm_iio_triggered_buffer_setup_ext() function and removes the iio_buffer_set_attrs() call, for assigning the HW FIFO attributes to the buffer. Signed-off-by: Alexandru Ardelean --- .../common/cros_ec_sensors/cros_ec_sensors_core.c | 15 +-- 1 file

[PATCH v3 1/9] iio: buffer: dmaengine: unwrap the use of iio_buffer_set_attrs()

2020-09-29 Thread Alexandru Ardelean
The iio_buffer_set_attrs() helper will be removed in this series. So, just assign the attributes of the DMAEngine buffer logic directly. This is IIO buffer core context, so there is direct access to the buffer->attrs object. Signed-off-by: Alexandru Ardelean --- Changelog v2 -> v3: *

[PATCH v3 8/9] iio: hid-sensors: use iio_triggered_buffer_setup_ext()

2020-09-29 Thread Alexandru Ardelean
This change switches to the new iio_triggered_buffer_setup_ext() function and removes the iio_buffer_set_attrs() call, for assigning the HW FIFO attributes to the buffer. Signed-off-by: Alexandru Ardelean --- .../common/hid-sensors/hid-sensor-trigger.c | 22 --- 1 file changed

[PATCH v3 9/9] iio: buffer: remove iio_buffer_set_attrs() helper

2020-09-29 Thread Alexandru Ardelean
The iio_buffer_set_attrs() is no longer used in the drivers, so it can be removed now. Signed-off-by: Alexandru Ardelean --- drivers/iio/industrialio-buffer.c | 12 include/linux/iio/buffer.h| 3 --- 2 files changed, 15 deletions(-) diff --git a/drivers/iio/industrialio

[PATCH v3 0/9] iio: remove iio_buffer_set_attrs() and assign buffer attrs during alloc

2020-09-29 Thread Alexandru Ardelean
atch * add 'at91-sama5d2_adc: merge buffer & trigger' patch in this series, so that the removal of iio_buffer_set_attrs() is easier to view Alexandru Ardelean (9): iio: buffer: dmaengine: unwrap the use of iio_buffer_set_attrs() iio: adc: at91-sama5d2_adc: merge buffer & trigger init i

Re: [PATCH v2 1/9] iio: buffer: dmaengine: unwrap the use of iio_buffer_set_attrs()

2020-09-29 Thread Alexandru Ardelean
On Tue, Sep 29, 2020 at 10:49 AM Andy Shevchenko wrote: > > On Tue, Sep 29, 2020 at 10:09 AM Alexandru Ardelean > wrote: > > > > The iio_buffer_set_attrs() helper will be removed in this series. So, just > > assign the attributes of the DMAEngine buffer logic directly.

[PATCH v2 0/9] iio: remove iio_buffer_set_attrs() and assign buffer attrs during alloc

2020-09-29 Thread Alexandru Ardelean
asier to view Alexandru Ardelean (9): iio: buffer: dmaengine: unwrap the use of iio_buffer_set_attrs() iio: adc: at91-sama5d2_adc: merge buffer & trigger init into a function iio: triggered-buffer: add {devm_}iio_triggered_buffer_setup_ext variants iio: ac

[PATCH v2 4/9] iio: accel: adxl372: use devm_iio_triggered_buffer_setup_ext()

2020-09-29 Thread Alexandru Ardelean
This change switches to the new devm_iio_triggered_buffer_setup_ext() function and removes the iio_buffer_set_attrs() call, for assigning the HW FIFO attributes to the buffer. Signed-off-by: Alexandru Ardelean --- drivers/iio/accel/adxl372.c | 11 +-- 1 file changed, 5 insertions(+), 6

<    1   2   3   4   5   6   7   8   9   10   >