> -Original Message-
> From: Moritz Fischer
> Sent: Wednesday, January 27, 2021 4:39 AM
> To: Ardelean, Alexandru
> Cc: linux-...@vger.kernel.org; devicet...@vger.kernel.org; linux-
> ker...@vger.kernel.org; mturque...@baylibre.com; sb...@kernel.org;
>
> -Original Message-
> From: Tom Rix
> Sent: Thursday, December 24, 2020 4:03 PM
> To: Ardelean, Alexandru ; linux-
> c...@vger.kernel.org; devicet...@vger.kernel.org; linux-kernel@vger.kernel.org
> Cc: mturque...@baylibre.com; sb...@kernel.org; robh...@kernel.org;
> -Original Message-
> From: Andy Shevchenko
> Sent: Monday, January 4, 2021 3:51 PM
> To: Ardelean, Alexandru
> Cc: linux-spi ; Linux Kernel Mailing List ker...@vger.kernel.org>; Mark Brown ; Stephen Rothwell
>
> Subject: Re: [PATCH] spi: stm32: update dev_db
> -Original Message-
> From: Stephen Rothwell
> Sent: Monday, January 4, 2021 2:07 AM
> To: Mark Brown
> Cc: Ardelean, Alexandru ; Andy Shevchenko
> ; Linux Kernel Mailing List ker...@vger.kernel.org>; Linux Next Mailing List
> Subject: linux-next: b
> -Original Message-
> From: Andy Shevchenko
> Sent: Monday, December 21, 2020 4:37 PM
> To: Ardelean, Alexandru
> Cc: linux-spi ; devicetree
> ; Linux Kernel Mailing List ker...@vger.kernel.org>; Bogdan, Dragos ;
> Mark Brown ; Rob Herring
> Subject:
> -Original Message-
> From: Alexandru Ardelean
> Sent: Friday, November 27, 2020 1:14 PM
> To: linux-in...@vger.kernel.org; linux-kernel@vger.kernel.org;
> devicet...@vger.kernel.org
> Cc: l...@metafoo.de; dmitry.torok...@gmail.com; robh...@kernel.org;
> Ardelean,
> -Original Message-
> From: Andrew Lunn
> Sent: Thursday, December 3, 2020 4:16 PM
> To: Ardelean, Alexandru
> Cc: net...@vger.kernel.org; linux-kernel@vger.kernel.org;
> hkallwe...@gmail.com; li...@armlinux.org.uk; da...@davemloft.net;
> k...@kernel.or
> -Original Message-
> From: Andy Shevchenko
> Sent: Friday, November 27, 2020 4:26 PM
> To: Ardelean, Alexandru
> Cc: linux-spi ; devicetree
> ; Linux Kernel Mailing List ker...@vger.kernel.org>; Rob Herring ; Mark Brown
> ; Bogdan, Dragos
> Subject:
> -Original Message-
> From: Andy Shevchenko
> Sent: Friday, November 27, 2020 4:24 PM
> To: Ardelean, Alexandru
> Cc: linux-spi ; devicetree
> ; Linux Kernel Mailing List ker...@vger.kernel.org>; Rob Herring ; Mark Brown
> ; Bogdan, Dragos
> Subject:
> -Original Message-
> From: Andy Shevchenko
> Sent: Friday, November 27, 2020 4:13 PM
> To: Ardelean, Alexandru
> Cc: linux-spi ; devicetree
> ; Linux Kernel Mailing List ker...@vger.kernel.org>; Rob Herring ; Mark Brown
> ; Bogdan, Dragos
> Subject: R
> -Original Message-
> From: Lars-Peter Clausen
> Sent: Tuesday, November 24, 2020 10:43 AM
> To: Ardelean, Alexandru ; linux-
> in...@vger.kernel.org; linux-kernel@vger.kernel.org
> Cc: dmitry.torok...@gmail.com
> Subject: Re: [PATCH v2 3/3] Input: adp5589-keys -
> -Original Message-
> From: Andy Shevchenko
> Sent: Tuesday, November 24, 2020 1:50 PM
> To: kernel test robot
> Cc: Ardelean, Alexandru ; linux-spi s...@vger.kernel.org>; devicetree ; Linux Kernel
> Mailing List ; kbuild-...@lists.01.org; Mark
> Brow
> -Original Message-
> From: Alexandru Ardelean
> Sent: Monday, November 23, 2020 4:35 PM
> To: linux-kernel@vger.kernel.org; k...@vger.kernel.org
> Cc: m...@redhat.com; gre...@linuxfoundation.org; Ardelean, Alexandru
>
> Subject: [PATCH v2] uio/uio_pci_gen
> -Original Message-
> From: Greg KH
> Sent: Friday, November 20, 2020 5:46 PM
> To: Ardelean, Alexandru
> Cc: linux-kernel@vger.kernel.org
> Subject: Re: [PATCH] uio/uio_pci_generic: remove unneeded pci_set_drvdata()
>
> [External]
>
> On Thu, No
> -Original Message-
> From: Dmitry Torokhov
> Sent: Thursday, November 19, 2020 9:25 AM
> To: linux-in...@vger.kernel.org
> Cc: Ardelean, Alexandru ; linux-
> ker...@vger.kernel.org
> Subject: [PATCH] Input: adp5589-keys - use BIT()
>
> [External]
>
&
> -Original Message-
> From: Dmitry Torokhov
> Sent: Thursday, November 19, 2020 9:24 AM
> To: linux-in...@vger.kernel.org
> Cc: Ardelean, Alexandru ; linux-
> ker...@vger.kernel.org
> Subject: [PATCH] Input: adp5589-keys - mark suspend and resume method
> -Original Message-
> From: Mark Brown
> Sent: Monday, November 16, 2020 11:19 PM
> To: Ardelean, Alexandru
> Cc: alsa-de...@alsa-project.org; devicet...@vger.kernel.org; linux-
> ker...@vger.kernel.org; l...@metafoo.de; robh...@kernel.org;
> lgirdw...@gmail.com
> -Original Message-
> From: Ioana Ciornei
> Sent: Friday, November 13, 2020 6:52 PM
> To: Andrew Lunn ; Heiner Kallweit ;
> Russell King ; Florian Fainelli ;
> Jakub Kicinski ; net...@vger.kernel.org; linux-
> ker...@vger.kernel.org
> Cc: Ioana Ciorn
> -Original Message-
> From: Ioana Ciornei
> Sent: Friday, November 13, 2020 6:52 PM
> To: Andrew Lunn ; Heiner Kallweit ;
> Russell King ; Florian Fainelli ;
> Jakub Kicinski ; net...@vger.kernel.org; linux-
> ker...@vger.kernel.org
> Cc: Ioana Ciorn
> -Original Message-
> From: Ardelean, Alexandru
> Sent: Thursday, November 12, 2020 8:40 AM
> To: Dmitry Torokhov
> Cc: linux-in...@vger.kernel.org; linux-kernel@vger.kernel.org
> Subject: RE: [PATCH 1/6] Input: adp5589: use a single variable for
> -Original Message-
> From: Dmitry Torokhov
> Sent: Thursday, November 12, 2020 2:38 AM
> To: Ardelean, Alexandru
> Cc: linux-in...@vger.kernel.org; linux-kernel@vger.kernel.org
> Subject: Re: [PATCH 1/6] Input: adp5589: use a single variable for error in
>
> -Original Message-
> From: Oleksij Rempel
> Sent: Wednesday, October 21, 2020 12:05 PM
> To: Dmitry Torokhov ; Ardelean, Alexandru
>
> Cc: Oleksij Rempel ; ker...@pengutronix.de; linux-
> ker...@vger.kernel.org; linux-in...@vger.kernel.org; David Jander
&g
> -Original Message-
> From: Mark Brown
> Sent: Monday, October 19, 2020 4:04 PM
> To: Ardelean, Alexandru
> Cc: linux-kernel@vger.kernel.org; alsa-de...@alsa-project.org;
> lgirdw...@gmail.com; l...@metafoo.de; Sa, Nuno ;
> pe...@perex.cz; ti...@suse.com
> S
> -Original Message-
> From: Rob Herring
> Sent: Tuesday, October 6, 2020 11:25 PM
> To: Ardelean, Alexandru
> Cc: linux-...@vger.kernel.org; devicet...@vger.kernel.org; linux-
> ker...@vger.kernel.org; Hennerich, Michael
> ; l...@metafoo.de; sb...@
ust 29, 2020 5:58 PM
To: Jonathan Cameron
Cc: Andy Shevchenko ; Linux Kernel Mailing List
; Hennerich, Michael
; linux-iio ;
Ardelean, Alexandru
Subject: Re: [PATCH] MAINTAINERS: Remove bouncing email of Beniamin Bia
On Sat, 29 Aug 2020 at 16:54, Jonathan Cameron wrote:
(...)
> > >
-Original Message-
From: Tom Rix
Sent: Wednesday, August 5, 2020 7:02 PM
To: Ardelean, Alexandru ;
linux-...@vger.kernel.org; linux-f...@vger.kernel.org;
linux-kernel@vger.kernel.org
Cc: mturque...@baylibre.com; sb...@kernel.org; m...@kernel.org; Caprioru,
Mircea
Subject: Re: [PATCH
.
-Original Message-
From: Lars-Peter Clausen
Sent: Saturday, July 18, 2020 7:37 PM
To: Jonathan Cameron ; Ardelean, Alexandru
Cc: linux-...@vger.kernel.org; linux-kernel@vger.kernel.org
Subject: Re: [PATCH] iio: trigger: Staticise stub functions
[External]
On 7/18/20 6:25 PM, Jonathan Cameron
On Fri, 2020-05-22 at 13:46 +0300, Alexandru Ardelean wrote:
> From: Lars-Peter Clausen
>
> All devices using a triggered buffer need to attach and detach the
> trigger
> to the device in order to properly work. Instead of doing this in each
> and
> every driver by hand move this into the core.
On Wed, 2020-04-15 at 08:58 +0300, Alexandru Ardelean wrote:
> The change is more cosmetic. There is no need to reference this table in
> the probe function since 'of_device_get_match_data' is used, which
> obtains
> this information from the driver object.
This looks like it could be applied
On Sun, 2020-07-12 at 13:02 +0100, Jonathan Cameron wrote:
> On Thu, 9 Jul 2020 17:21:56 +0200
> Artur Rojek wrote:
>
> > Introduce error checks for the clk_enable calls used in this driver.
> > As part of the changes, move clk_enable/clk_disable calls out of
> > ingenic_adc_set_config and into
On Mon, 2020-07-06 at 14:02 +0300, Alexandru Ardelean wrote:
> [External]
>
> There are 2 exit paths where the lock isn't held, but try to unlock the
> mutex when exiting. In these places we should just return from the
> function.
>
> A neater approach would be to cleanup the ad5592r_read_raw(),
On Wed, 2020-07-01 at 19:42 +0100, Jonathan Cameron wrote:
> [External]
>
> On Tue, 30 Jun 2020 04:58:06 +
> "Ardelean, Alexandru" wrote:
>
> > On Tue, 2020-06-30 at 07:57 +0300, Alexandru Ardelean wrote:
> > > This change moves all iio_dev debugfs
On Mon, 2020-06-29 at 16:41 -0700, Douglas Anderson wrote:
> [External]
>
> On some SPI controllers (like spi-geni-qcom) setting the chip select
> is a heavy operation. For instance on spi-geni-qcom, with the current
> code, is was measured as taking upwards of 20 us. Even on SPI
> controllers
On Tue, 2020-06-30 at 07:57 +0300, Alexandru Ardelean wrote:
> This change moves all iio_dev debugfs fields to the iio_dev_priv object.
> It's not the biggest advantage yet (to the whole thing of
> abstractization)
> but it's a start.
>
> The iio_get_debugfs_dentry() function (which is moved in
>
On Sat, 2020-06-27 at 17:40 +0100, Jonathan Cameron wrote:
> On Sun, 21 Jun 2020 15:33:40 +0300
> Alexandru Ardelean wrote:
>
> > There are plenty of bad designs we want to discourage or not have to
> > review
> > manually usually about accessing private (marked as [INTERN]) fields of
> >
On Sun, 2020-05-31 at 15:42 +0100, Jonathan Cameron wrote:
> [External]
>
> On Mon, 25 May 2020 13:25:13 +0300
> Alexandru Ardelean wrote:
>
> > We may want to get rid of the iio_priv_to_dev() helper. The reason is
> > that
> > we will hide some of the members of the iio_dev structure (to
On Sun, 2020-05-31 at 15:45 +0100, Jonathan Cameron wrote:
> [External]
>
> On Mon, 25 May 2020 11:26:48 +0300
> Alexandru Ardelean wrote:
>
> > We may want to get rid of the iio_priv_to_dev() helper. The reason is
> > that
> > we will hide some of the members of the iio_dev structure (to
On Thu, 2020-06-18 at 13:01 +, eugen.hris...@microchip.com wrote:
> On 17.06.2020 16:52, Ardelean, Alexandru wrote:
> > On Wed, 2020-06-17 at 13:37 +, eugen.hris...@microchip.com wrote:
> > > [External]
> > >
> > > On 02.06.2020 11:54, Jonathan Cameron
On Thu, 2020-06-18 at 12:47 +, eugen.hris...@microchip.com wrote:
> [External]
>
> On 17.06.2020 17:02, Ardelean, Alexandru wrote:
> > On Wed, 2020-06-17 at 13:25 +, eugen.hris...@microchip.com wrote:
> > > On 31.05.2020 17:39, Jonathan Cameron wrote:
> > >
On Wed, 2020-06-17 at 13:25 +, eugen.hris...@microchip.com wrote:
> On 31.05.2020 17:39, Jonathan Cameron wrote:
>
> > On Mon, 25 May 2020 13:53:41 +0300
> > Alexandru Ardelean wrote:
> >
> > > We may want to get rid of the iio_priv_to_dev() helper. The reason is that
> > > we will hide
On Wed, 2020-06-17 at 13:37 +, eugen.hris...@microchip.com wrote:
> [External]
>
> On 02.06.2020 11:54, Jonathan Cameron wrote:
> > On Tue, 2 Jun 2020 07:50:23 +0000
> > "Ardelean, Alexandru" wrote:
> >
> > > On Sun, 2020-05-31 at 16:40 +0100,
On Sat, 2020-06-06 at 17:05 +0100, Jonathan Cameron wrote:
> [External]
>
> On Wed, 3 Jun 2020 14:40:23 +0300
> Alexandru Ardelean wrote:
>
> > These were found by doing some shell magic:
> >
> > for file in $(git grep -w devm_iio_device_alloc | cut -d: -f1 | sort | uniq)
> > ; do
On Wed, 2020-06-03 at 14:40 +0300, Alexandru Ardelean wrote:
> This patch updates the {devm_}iio_device_alloc() functions to automatically
> assign the parent device on allocation.
> For iio_device_alloc() this means a new parameter.
> For devm_iio_device_alloc() this means a new behavior; the
On Wed, 2020-06-03 at 17:26 +0800, Chuhong Yuan wrote:
> Since if there is no GPIO, nothing happens, replace devm_gpiod_get()
> with devm_gpiod_get_optional().
> Also add IS_ERR() to fix the missing-check bug.
>
Acked-by: Alexandru Ardelean
> Fixes: cee211f4e5a0 ("iio: amplifiers: ad8366: Add
On Sun, 2020-05-31 at 15:54 +0100, Jonathan Cameron wrote:
> [External]
>
> On Fri, 29 May 2020 15:45:33 +0200
> Johan Hovold wrote:
>
> > [ Trimming CC to something more reasonable... ]
> >
> > On Fri, May 29, 2020 at 11:08:38AM +, Ardelean, Alexandru wrot
On Sun, 2020-05-31 at 16:40 +0100, Jonathan Cameron wrote:
> On Mon, 25 May 2020 14:38:55 +0300
> Alexandru Ardelean wrote:
>
> > From: Lars-Peter Clausen
> >
> > This patch should be squashed into the first one, as the first one is
> > breaking the build (intentionally) to make the IIO core
On Thu, 2020-05-28 at 14:40 +0800, Chuhong Yuan wrote:
> [External]
>
> ad8366_probe() forgets to check the return value of devm_gpiod_get().
> Add the missed check to fix it.
>
> Fixes: cee211f4e5a0 ("iio: amplifiers: ad8366: Add support for the
> ADA4961 DGA")
> Signed-off-by: Chuhong Yuan
>
On Tue, 2020-05-26 at 23:02 +0200, Rikard Falkeborn wrote:
> [External]
>
> These are never modified and can be made const to allow the compiler to
> put it in read-only memory.
>
> Before:
>textdata bss dec hex filename
>6642 12608 64 193144b72
On Tue, 2020-05-26 at 23:02 +0200, Rikard Falkeborn wrote:
> [External]
>
> ad5592r_ext_info is not modified and can be made const to allow the
> compiler to put it in read-only memory.
>
> Before:
>textdata bss dec hex filename
> 132932088 256 156373d15
On Tue, 2020-05-26 at 23:02 +0200, Rikard Falkeborn wrote:
> [External]
>
> ad5380_ext_info is not modified and can be made const to allow the
> compiler to put it in read-only memory.
>
> Before:
>textdata bss dec hex filename
> 120603280 192 155323cac
On Mon, 2020-05-25 at 18:27 +0200, Fabrice Gasnier wrote:
> [External]
>
> On 5/25/20 11:07 AM, Alexandru Ardelean wrote:
> > We may want to get rid of the iio_priv_to_dev() helper. The reason is
> > that
> > we will hide some of the members of the iio_dev structure (to prevent
> > drivers from
On Sun, 2020-05-24 at 14:38 +0100, Jonathan Cameron wrote:
> [External]
>
> On Fri, 22 May 2020 13:46:32 +0300
> Alexandru Ardelean wrote:
>
> > From: Lars-Peter Clausen
> >
> > This patch should be squashed into the first one, as the first one is
> > breaking the build (intentionally) to
On Sun, 2020-05-24 at 17:49 +0100, Jonathan Cameron wrote:
> [External]
>
> On Fri, 8 May 2020 16:53:43 +0300
> Alexandru Ardelean wrote:
>
> > Currently the iio_buffer_{alloc,free}_sysfs_and_mask() take 'indio_dev' as
> > primary argument. This change converts to take an IIO buffer as a
On Sun, 2020-05-24 at 17:47 +0100, Jonathan Cameron wrote:
> [External]
>
> On Fri, 8 May 2020 16:53:42 +0300
> Alexandru Ardelean wrote:
>
> > This change makes use of the new centralized ioctl() mechanism. The event
> > interface registers it's ioctl() handler to IIO device.
> > Both the
On Sun, 2020-05-24 at 17:45 +0100, Jonathan Cameron wrote:
> [External]
>
> On Fri, 8 May 2020 16:53:41 +0300
> Alexandru Ardelean wrote:
>
> > The aim of this is to reduce the organization violation of ioctl() calls in
> > IIO core. Currently, since the chardev is split across files, event
On Sun, 2020-05-24 at 15:39 +0100, Jonathan Cameron wrote:
> [External]
>
> On Fri, 22 May 2020 09:56:16 +0300
> Alexandru Ardelean wrote:
>
> > We may want to get rid of the iio_priv_to_dev() helper. That's a bit
> > uncertain at this point. The reason is that we will hide some of the
> >
On Thu, 2020-05-14 at 16:17 +0300, Alexandru Ardelean wrote:
> This change starts to hide some internal fields of the IIO device into
> the framework.
>
> Because the iio_priv_to_dev() will be hidden some pre-work is done to
> try to remove it from some interrupt handlers.
> iio_priv_to_dev()
On Wed, 2020-05-20 at 09:18 +0300, Sergiu Cuciurean wrote:
> [External]
>
> As part of the general cleanup of indio_dev->mlock, this change replaces
> it with a local lock on the device's state structure.
>
This requires a V2.
It produces a warning:
159 | struct iio_dev *iio_dev =
On Sat, 2020-05-16 at 18:17 +0100, Jonathan Cameron wrote:
> [External]
>
> On Thu, 14 May 2020 16:17:05 +0300
> Alexandru Ardelean wrote:
>
> > Since there will be some changes to how iio_priv_to_dev() is implemented,
> > it could be that the helper becomes a bit slower, as it will be hidden
On Sat, 2020-05-16 at 17:29 +0100, Jonathan Cameron wrote:
> [External]
>
> On Tue, 12 May 2020 11:26:08 +
> "Ardelean, Alexandru" wrote:
>
> > On Mon, 2020-05-11 at 18:42 +0100, Jonathan Cameron wrote:
> > > [External]
> > >
> &g
On Sat, 2020-05-16 at 17:24 +0100, Jonathan Cameron wrote:
> [External]
>
> On Sat, 16 May 2020 13:08:46 +
> "Ardelean, Alexandru" wrote:
>
> > On Tue, 2020-05-12 at 06:26 +, Ardelean, Alexandru wrote:
> > > [External]
> > >
> >
On Tue, 2020-05-12 at 06:26 +, Ardelean, Alexandru wrote:
> [External]
>
> On Mon, 2020-05-11 at 21:56 +0200, Lars-Peter Clausen wrote:
> > [External]
> >
> > On 5/11/20 4:56 PM, Ardelean, Alexandru wrote:
> > > On Mon, 2020-05-11 at 15:58 +0200, Lars-Pe
..@st.com;
> > mcoquelin.st...@gmail.com; a...@it-klinger.de; Ardelean, Alexandru
> >
> > Subject: [PATCH v2 7/8] iio: core: simplify alloc alignment code
> >
> > There was a recent discussion about this code:
> > https://urldefense.com/v3/__https://
On Mon, 2020-05-11 at 18:42 +0100, Jonathan Cameron wrote:
> [External]
>
> On Mon, 11 May 2020 09:16:32 +
> "Ardelean, Alexandru" wrote:
>
> > On Fri, 2020-05-08 at 16:44 +0100, Jonathan Cameron wrote:
> > > [External]
> > >
> > >
On Mon, 2020-05-11 at 21:56 +0200, Lars-Peter Clausen wrote:
> [External]
>
> On 5/11/20 4:56 PM, Ardelean, Alexandru wrote:
> > On Mon, 2020-05-11 at 15:58 +0200, Lars-Peter Clausen wrote:
> > > [External]
> > >
> > > On 5/11/20 3:24 PM, Ardelean, A
On Mon, 2020-05-11 at 15:58 +0200, Lars-Peter Clausen wrote:
> [External]
>
> On 5/11/20 3:24 PM, Ardelean, Alexandru wrote:
> > On Mon, 2020-05-11 at 13:03 +0000, Ardelean, Alexandru wrote:
> > > [External]
> > >
> > > On Mon, 2020-05-1
On Mon, 2020-05-11 at 13:03 +, Ardelean, Alexandru wrote:
> [External]
>
> On Mon, 2020-05-11 at 12:37 +0200, Lars-Peter Clausen wrote:
> > [External]
> >
> > On 5/11/20 12:33 PM, Ardelean, Alexandru wrote:
> > > On Sun, 2020-05-10 at 11:09 +0100, Jonat
On Mon, 2020-05-11 at 12:37 +0200, Lars-Peter Clausen wrote:
> [External]
>
> On 5/11/20 12:33 PM, Ardelean, Alexandru wrote:
> > On Sun, 2020-05-10 at 11:09 +0100, Jonathan Cameron wrote:
> > > [External]
> > >
> > > On Sat, 9 May 2020 10:5
On Sun, 2020-05-10 at 11:09 +0100, Jonathan Cameron wrote:
> [External]
>
> On Sat, 9 May 2020 10:52:14 +0200
> Lars-Peter Clausen wrote:
>
> > On 5/8/20 3:53 PM, Alexandru Ardelean wrote:
> > > [...]
> > > What I don't like, is that iio:device3 has iio:buffer3:0 (to 3).
> > > This is because
On Thu, 2020-04-16 at 17:31 +0300, Alexandru Ardelean wrote:
> [External]
>
> The kernel provides a facility for attribute groups to specify the stat
> flags of a sysfs file with an 'is_visible()' hook. This mechanism is
> usually more flexible than assigning read-only attributes at construction
On Fri, 2020-05-08 at 16:44 +0100, Jonathan Cameron wrote:
> [External]
>
> On Fri, 8 May 2020 16:40:15 +0100
> Jonathan Cameron wrote:
>
> > On Fri, 8 May 2020 17:13:04 +0300
> > Alexandru Ardelean wrote:
> >
> > > There are plenty of bad designs we want to discourage or not have to
> > >
On Sun, 2020-05-03 at 16:51 +0100, Jonathan Cameron wrote:
> [External]
>
> On Mon, 27 Apr 2020 16:10:54 +0300
> Alexandru Ardelean wrote:
>
> > The main intent is to be able to add more chardevs per IIO device, one
> > for each buffer. To get there, some rework is needed.
> > This tries to
On Tue, 2020-05-05 at 16:23 +0200, Arnd Bergmann wrote:
> [External]
>
> Checking the pointer value of st->chip_info->convst_channel is pointless
> since this this an array inside of a struct: even if st->chip_info is NULL,
> the pointer is non-zero. Clang warns about this:
>
>
On Sun, 2020-05-03 at 16:39 +0100, Jonathan Cameron wrote:
> [External]
>
> On Mon, 27 Apr 2020 16:10:57 +0300
> Alexandru Ardelean wrote:
>
> > The final intent is to localize all buffer ops into the
> > industrialio-buffer.c file, to be able to add support for multiple buffers
> > per IIO
On Sun, 2020-05-03 at 13:51 +0100, Jonathan Cameron wrote:
> On Wed, 29 Apr 2020 18:17:39 +0300
> Alexandru Ardelean wrote:
>
> > From: Lars-Peter Clausen
> >
> > Let the core handle the buffer scan mask management including allocation
> > and channel selection. Having this handled in a
On Sun, 2020-05-03 at 12:37 +0100, Jonathan Cameron wrote:
> [External]
>
> On Wed, 29 Apr 2020 10:21:29 +0300
> Alexandru Ardelean wrote:
>
> > From: Dragos Bogdan
> >
> > If the serial interface is used, the 8-bit address should be latched using
> > the rising edge of the WR/FSYNC signal.
>
On Sat, 2020-05-02 at 19:25 +0100, Jonathan Cameron wrote:
> On Tue, 28 Apr 2020 12:31:28 +0300
> Alexandru Ardelean wrote:
>
> > This change cleans up the driver's probe function to use only devm_
> > function variants. This also gets rid of the remove function and moves the
> > clock &
On Thu, 2020-04-30 at 11:52 +0300, Alexandru Ardelean wrote:
> Not used anywhere.
> Looks like it was forgotten in iio.h
>
Actually, disregard this.
I've found places where 'indio_dev->debugfs_dentry' is accessed directly and
that should have used this instead.
Apologies for the spam.
>
On Thu, 2020-04-30 at 07:30 +, Ardelean, Alexandru wrote:
> On Mon, 2020-04-27 at 13:00 +0000, Ardelean, Alexandru wrote:
> > [External]
> >
> > On Mon, 2020-04-27 at 12:20 +, eugen.hris...@microchip.com wrote:
> > > [External]
> > >
> > >
On Mon, 2020-04-27 at 13:00 +, Ardelean, Alexandru wrote:
> [External]
>
> On Mon, 2020-04-27 at 12:20 +, eugen.hris...@microchip.com wrote:
> > [External]
> >
> > On 15.04.2020 09:33, Ardelean, Alexandru wrote:
> >
> > > On Tue, 2020-04
On Wed, 2019-10-16 at 22:25 +0800, Chuhong Yuan wrote:
> ad9834_probe misses a check for devm_clk_get and may cause problems.
> Add a check like what ad9832 does to fix it.
>
This could also use a Fixes tag, but not a big deal.
Reviewed-by: Alexandru Ardelean
> Signed-off-by: Chuhong Yuan
>
On Wed, 2019-10-16 at 10:38 +0530, Vinod Koul wrote:
> [External]
>
> On 15-10-19, 23:06, Lars-Peter Clausen wrote:
>
> > > > This DMA controller is a bit special.
> > > > It gets synthesized in FPGA, so the configuration is fixed and
> > > > cannot be
> > > > changed at runtime. Maybe later we
On Mon, 2019-10-14 at 12:31 +0530, Vinod Koul wrote:
> [External]
>
Hey,
> On 13-09-19, 17:54, Alexandru Ardelean wrote:
> > From: Rodrigo Alencar
> >
> > dmaengine_slave_config is called by dmaengine_pcm_hw_params when using
> > axi-i2s with axi-dmac. If device_config is NULL, -ENOSYS is
On Wed, 2019-10-09 at 15:15 +0530, Manivannan Sadhasivam wrote:
> [External]
>
> Hi Ardelean,
>
> Thanks for the quick review!
>
> On Tue, Oct 08, 2019 at 06:52:50AM +, Ardelean, Alexandru wrote:
> > On Mon, 2019-10-07 at 15:40 +0530, Manivannan Sadhas
On Tue, 2019-10-08 at 08:47 +, Ardelean, Alexandru wrote:
> [External]
>
> On Tue, 2019-10-08 at 06:54 +, Ardelean, Alexandru wrote:
> > [External]
> >
> > On Mon, 2019-10-07 at 22:16 +0100, Jonathan Cameron wrote:
> > > On Sun, 6 Oct 2019 10:12:01
On Tue, 2019-10-08 at 06:54 +, Ardelean, Alexandru wrote:
> [External]
>
> On Mon, 2019-10-07 at 22:16 +0100, Jonathan Cameron wrote:
> > On Sun, 6 Oct 2019 10:12:01 +0100
> > Jonathan Cameron wrote:
> >
> > > On Thu, 26 Sep 2019 14:18:04 +
On Mon, 2019-10-07 at 22:16 +0100, Jonathan Cameron wrote:
> On Sun, 6 Oct 2019 10:12:01 +0100
> Jonathan Cameron wrote:
>
> > On Thu, 26 Sep 2019 14:18:04 +0300
> > Alexandru Ardelean wrote:
> >
> > > This will allow more flexible control to group reads & writes into a
> > > single
> > > lock
On Mon, 2019-10-07 at 15:40 +0530, Manivannan Sadhasivam wrote:
> [External]
>
Hey,
Comments inline.
I thought I sent an initial review, but seems to have gotten lost [maybe in
my email client].
Oh well. I managed to re-do it anyway.
I tried to group them this time.
The more prominent part
On Mon, 2019-10-07 at 18:10 +0530, Manivannan Sadhasivam wrote:
> [External]
>
> Hi Ardelean,
>
> On 7 October 2019 3:51:16 PM IST, "Ardelean, Alexandru" <
> alexandru.ardel...@analog.com> wrote:
> > On Mon, 2019-10-07 at 15:40 +0530, Mani
On Mon, 2019-10-07 at 15:40 +0530, Manivannan Sadhasivam wrote:
> [External]
>
> Add devicetree binding for Analog Devices ADUX1020 Photometric
> sensor.
>
Hey,
Thanks for the patches.
This dt-binding docs is in text format.
dt-binding docs now need to be in YAML format.
Also, patches for
On Sun, 2019-10-06 at 10:06 +0100, Jonathan Cameron wrote:
> [External]
>
> On Sun, 6 Oct 2019 09:53:33 +0100
> Jonathan Cameron wrote:
>
> > On Thu, 26 Sep 2019 14:18:03 +0300
> > Alexandru Ardelean wrote:
> >
> > > The lock can be extended a bit to protect other elements that are not
> > >
On Thu, 2019-09-26 at 11:10 +0300, Dan Carpenter wrote:
> [External]
>
> The "t" variable is unsigned so it can't be less than zero. We really
> are just trying to prevent divide by zero bugs so just checking against
> zero is sufficient.
>
> Signed-off-by: Dan Carpenter
> ---
>
On Wed, 2019-09-25 at 10:14 -0500, Adam Ford wrote:
> [External]
>
> On Fri, Sep 20, 2019 at 2:06 AM Philipp Puschmann
> wrote:
> > Hi Andy,
> >
> > Am 20.09.19 um 05:42 schrieb Andy Duan:
> > > From: Philipp Puschmann Sent: Thursday,
> > > September 19, 2019 10:51 PM
> > > > Using only 4 DMA
On Thu, 2019-09-19 at 10:56 -0500, Navid Emamdoost wrote:
> In adis_update_scan_mode_burst, if adis->buffer allocation fails release
> the adis->xfer.
>
> v2: set adis->xfer = NULL to avoid any potential double free.
>
Reviewed-by: Alexandru Ardelean
> Signed-off-by: Navid Emamdoost
> ---
>
On Thu, 2019-09-19 at 10:50 -0500, Navid Emamdoost wrote:
> In adis_update_scan_mode, if allocation for adis->buffer fails,
> previously allocated adis->xfer needs to be released.
>
> v2: added adis->xfer = NULL to avoid any potential double free.
Reviewed-by: Alexandru Ardelean
>
On Wed, 2019-09-18 at 12:03 -0500, Navid Emamdoost wrote:
> [External]
>
Hey,
Thanks for this patch as well.
Comments inline here as well.
> In adis_update_scan_mode_burst, if adis->buffer allocation fails release
> the adis->xfer.
>
> Signed-off-by: Navid Emamdoost
> ---
>
On Wed, 2019-09-18 at 11:57 -0500, Navid Emamdoost wrote:
> [External]
>
Hey,
Good catch.
One comment inline.
> In adis_update_scan_mode, if allocation for adis->buffer fails,
> previously allocated adis->xfer needs to be released.
>
> Signed-off-by: Navid Emamdoost
> ---
>
On Tue, 2019-09-17 at 14:41 +0200, Andrew Lunn wrote:
> [External]
>
> On Tue, Sep 17, 2019 at 01:30:52PM +0300, Alexandru Ardelean wrote:
> > The 'mac-mode' property is similar to 'phy-mode' and 'phy-connection-type',
> > which are enums of mode strings.
> >
> > The 'dwmac' driver supports
On Mon, 2019-09-16 at 12:49 +0300, Alexandru Ardelean wrote:
> On Fri, 2019-09-13 at 15:36 +0100, Rob Herring wrote:
> > [External]
> >
> > On Fri, Sep 06, 2019 at 04:02:56PM +0300, Alexandru Ardelean wrote:
> > > This change documents the 'mac-mode' property that was introduced in
> > > the
> >
On Mon, 2019-09-16 at 14:43 +0100, Mark Brown wrote:
> [External]
>
> On Mon, Sep 16, 2019 at 01:04:42PM +, Ardelean, Alexandru wrote:
> > On Mon, 2019-09-16 at 13:47 +0100, Mark Brown wrote:
> > > That v3 seems to be a small subset of this series?
> > Ack.
>
1 - 100 of 233 matches
Mail list logo