On Mi, 2018-12-12 at 17:31 +0300, Dan Carpenter wrote:
> On Wed, Dec 12, 2018 at 01:15:31PM +0100, Christian Gromm wrote:
> >
> > diff --git a/drivers/staging/most/Documentation/driver_usage.txt
> > b/drivers/staging/most/Documentation/driver_usage.txt
> > index bb9b4e8..da7a8f4 100644
> > --- a/d
Dobrý den,
moderní příspěvek na stravování v podobě obědové karty, kterou lze použít ve
více než 15 000 restauracích a gastronomických zařízení po celé zemi, je
nástroj, který Vám pomůže poskytnout svým zaměstnancům nejžádanější benefit v
ČR.
Výzkum Světové zdravotnické organizace WHO naznaču
Dobrý den,
moderní příspěvek na stravování v podobě obědové karty, kterou lze použít ve
více než 15 000 restauracích a gastronomických zařízení po celé zemi, je
nástroj, který Vám pomůže poskytnout svým zaměstnancům nejžádanější benefit v
ČR.
Výzkum Světové zdravotnické organizace WHO naznaču
Dobrý den,
moderní příspěvek na stravování v podobě obědové karty, kterou lze použít ve
více než 15 000 restauracích a gastronomických zařízení po celé zemi, je
nástroj, který Vám pomůže poskytnout svým zaměstnancům nejžádanější benefit v
ČR.
Výzkum Světové zdravotnické organizace WHO naznaču
On 2018/12/13 9:51, Gao Xiang wrote:
> Hi Sungkyung,
>
> On 2018/12/12 23:50, Sungkyung Kim wrote:
>> Fix a warning from checkpathch.pl: 'Missing a blank line after
>> declarations'
>>
>> Signed-off-by: Sungkyung Kim
>> ---
>> drivers/staging/erofs/inode.c | 1 +
>> 1 file changed, 1 insertion(+
Hi Michael,
On 12/12/2018 05:00, Michael Kelley wrote:
> From: Marc Zyngier Sent: Friday, December 7, 2018
> 6:43 AM
>
>>> Add ARM64-specific code to enable Hyper-V. This code includes:
>>> * Detecting Hyper-V and initializing the guest/Hyper-V interface
>>> * Setting up Hyper-V's synthetic cl
I'm not really complaining about breaking userspace, I'm complaining
that I had to discover it by reading the code. It should have been
mentioned in the commit message. We should probably just fold
patch 1 & 6 together.
I'm also not really complaining about this patch in particular when it
comes
On Wed, Dec 12, 2018 at 01:15:31PM +0100, Christian Gromm wrote:
> This patch updates driver_usage.txt file to reflect the latest changes
> that this patch set introduces.
>
> Signed-off-by: Christian Gromm
> ---
> drivers/staging/most/Documentation/driver_usage.txt | 16 +---
> 1 fi
On Wed, Dec 12, 2018 at 03:31:13PM +, christian.gr...@microchip.com wrote:
> On Mi, 2018-12-12 at 17:21 +0300, Dan Carpenter wrote:
> > On Wed, Dec 12, 2018 at 01:15:26PM +0100, Christian Gromm wrote:
> > >
> > > This patch avoids that a sound card is created and registered with
> > > ALSA
> >
On Wed, Dec 12, 2018 at 01:15:26PM +0100, Christian Gromm wrote:
> @@ -571,6 +600,40 @@ static int audio_probe_channel(struct most_interface
> *iface, int channel_id,
> return -EINVAL;
> }
>
> + ret = split_arg_list(arg_list, &card_name, &ch_num, &sample_res,
> +
On Thu, Dec 13, 2018 at 02:58:00PM +0300, Dan Carpenter wrote:
> On Wed, Dec 12, 2018 at 01:15:31PM +0100, Christian Gromm wrote:
> > This patch updates driver_usage.txt file to reflect the latest changes
> > that this patch set introduces.
> >
> > Signed-off-by: Christian Gromm
> > ---
> > driv
On Wed, Dec 12, 2018 at 03:31:13PM +, christian.gr...@microchip.com wrote:
> An additional field is added to the configuration parameter,
> which is provided by user space.
> This seemed to be less painful than adding a new sysfs
> file and make the configuration even more complicated.
I thin
On Thu, Dec 13, 2018 at 03:38:03PM +0300, Dan Carpenter wrote:
> On Wed, Dec 12, 2018 at 03:31:13PM +, christian.gr...@microchip.com wrote:
> > An additional field is added to the configuration parameter,
> > which is provided by user space.
> > This seemed to be less painful than adding a new
This patch replaces the license text at the top of ad7606 driver files
and instead adds SPDX GPL-2.0+ license identifier.
Signed-off-by: Stefan Popa
---
drivers/staging/iio/adc/ad7606.c | 5 ++---
drivers/staging/iio/adc/ad7606.h | 3 +--
drivers/staging/iio/adc/ad7606_par.c | 5 ++---
d
The driver does not have a struct of_device_id table, but supported
devices are registered via Device Trees. This patch adds and OF device
ID table.
Signed-off-by: Stefan Popa
---
drivers/staging/iio/adc/ad7606_par.c | 10 ++
drivers/staging/iio/adc/ad7606_spi.c | 10 ++
2 files
This patch replaces the use of wait_event_interruptible() with
wait_for_completion_timeout() when reading the result of a single
conversion. In this way, if the interrupt never occurs, the program will
not remain blocked.
Signed-off-by: Stefan Popa
---
drivers/staging/iio/adc/ad7606.c | 14 +
Hi
I would like to make use of an ISA card which generates IRQ 5
preferably using user-space code only, if possible.
Does anybody know of any example code using uio_pdrv_genirq
which could be adapted for this purpose?
Or, does anyone have a simple example kernel module?
Thanks!
Andrew
Switch to devm version of request_irq, iio_triggered_buffer_setup,
iio_device_register. To avoid potential ordering issues in probe,
devm_add_action_or_reset() is used for the regulator_disable(). This
simplifies the code and decreases the chance of bugs.
Signed-off-by: Stefan Popa
---
drivers/s
The 'adi' vendor prefix needs to be added to conversion-start, range,
first-data and oversampling-ratio properties.
Signed-off-by: Stefan Popa
---
drivers/staging/iio/adc/ad7606.c | 10 ++
1 file changed, 6 insertions(+), 4 deletions(-)
diff --git a/drivers/staging/iio/adc/ad7606.c b/dr
There is no point in having three menu entries that can be selected
individually. Instead, the SPI and parallel interfaces should select
AD7606.
Signed-off-by: Stefan Popa
---
drivers/staging/iio/adc/Kconfig | 24 +---
1 file changed, 9 insertions(+), 15 deletions(-)
diff --
When looking for the available scale or oversampling ratio, it is better
to use the find_closest() macro. This simplifies the code and also does
not require an exact value to be entered from the user space.
Signed-off-by: Stefan Popa
---
drivers/staging/iio/adc/ad7606.c | 58 +++-
This patch replaces the use of a polling ring buffer with a threaded
interrupt.
Enabling the buffer sets the CONVST signal to high. When the rising edge
of the CONVST is applied, BUSY signal goes logic high and transitions low
at the end of the entire conversion process. The falling edge of the BU
This series attempts to clean up the driver according to the feedback
received during review and finally moves it out of staging.
Stefan Popa (11):
staging: iio: adc: ad7606: Simplify the Kconfing menu
staging: iio: adc: ad7606: Use SPDX identifier
staging: iio: adc: ad7606: Use wait-for-com
* Placed includes in alphabetical order
* Added brackets around num and mask through out for AD760X_CHANNEL
* Used single line comments where needed
* Removed extra lines and spaces
Signed-off-by: Stefan Popa
---
drivers/staging/iio/adc/ad7606.c | 27 ---
drivers/stag
Hi Steve,
On Tue, 2018-10-16 at 17:00 -0700, Steve Longerbeam wrote:
> The function ipu_csi_init_interface() was inverting the F-bit for
> NTSC case, in the CCIR_CODE_1/2 registers. The result being that
> for NTSC bottom-top field order, the CSI would swap fields and
> capture in top-bottom order
Move ad7606 ADC driver out of staging and into the mainline.
Signed-off-by: Stefan Popa
---
MAINTAINERS | 7 +
drivers/iio/adc/Kconfig | 28 ++
drivers/iio/adc/Makefile | 3 +
drivers/iio/adc/ad7606.c | 588 ++
Document support for AD7606 Analog to Digital Converter.
Signed-off-by: Stefan Popa
Reviewed-by: Rob Herring
---
.../devicetree/bindings/iio/adc/adi,ad7606.txt | 65 ++
MAINTAINERS| 1 +
2 files changed, 66 insertions(+)
create m
On Thu, Dec 13, 2018 at 02:46:14PM +0200, Stefan Popa wrote:
> This patch replaces the license text at the top of ad7606 driver files
> and instead adds SPDX GPL-2.0+ license identifier.
>
> Signed-off-by: Stefan Popa
> ---
> drivers/staging/iio/adc/ad7606.c | 5 ++---
> drivers/staging/iio/
On Thu, Dec 13, 2018 at 02:46:17PM +0200, Stefan Popa wrote:
> +static int ad7606_buffer_predisable(struct iio_dev *indio_dev)
> +{
> + struct ad7606_state *st = iio_priv(indio_dev);
> + int ret;
> +
> + reinit_completion(&st->completion);
> + gpiod_set_value(st->gpio_convst, 1);
>
Hello, Linux Driver Project.
My name is Andrii.
I`m interesting in wireless driver projects i.e 802.11n/ac/ maybe ax.
If there are any projects, related with it (new drivers, legacy
support), I will be happy to join.
___
devel mailing list
de...@linuxdr
On Do, 2018-12-13 at 15:16 +0300, Dan Carpenter wrote:
> On Wed, Dec 12, 2018 at 01:15:26PM +0100, Christian Gromm wrote:
> >
> > @@ -571,6 +600,40 @@ static int audio_probe_channel(struct
> > most_interface *iface, int channel_id,
> > return -EINVAL;
> > }
> >
> > + ret = spli
Fix some "bigger" issues reported by checkpatch.pl:
- Block comments use * on subsequent lines
- Blank lines aren't necessary before a close brace
- Block comments use a trailing */ on a separate line
- Alignment should match open parenthesis
- void function return statements are not generally usef
On Do, 2018-12-13 at 15:38 +0300, Dan Carpenter wrote:
> On Wed, Dec 12, 2018 at 03:31:13PM +, Christian.Gromm@microchip.c
> om wrote:
> >
> > An additional field is added to the configuration parameter,
> > which is provided by user space.
> > This seemed to be less painful than adding a new
On Thu, Dec 13, 2018 at 8:18 AM Stefan Popa wrote:
>
> There is no point in having three menu entries that can be selected
> individually. Instead, the SPI and parallel interfaces should select
> AD7606.
>
> Signed-off-by: Stefan Popa
> ---
> drivers/staging/iio/adc/Kconfig | 24 +---
On Do, 2018-12-13 at 14:54 +0300, Dan Carpenter wrote:
> I'm not really complaining about breaking userspace, I'm complaining
> that I had to discover it by reading the code. It should have been
> mentioned in the commit message. We should probably just fold
> patch 1 & 6 together.
>
Ok, I proba
On Do, 2018-12-13 at 13:32 +0100, Greg KH wrote:
> On Thu, Dec 13, 2018 at 02:58:00PM +0300, Dan Carpenter wrote:
> >
> > On Wed, Dec 12, 2018 at 01:15:31PM +0100, Christian Gromm wrote:
> > >
> > > This patch updates driver_usage.txt file to reflect the latest
> > > changes
> > > that this patch
Before 98f4c651762c, we returned zeros for unopened channels.
With 98f4c651762c, we started to return random on-stack values.
We'd better return -EINVAL instead.
Fixes: 98f4c651762c ("hv: move ringbuffer bus attributes to dev_groups")
Cc: sta...@vger.kernel.org
Cc: K. Y. Srinivasan
Cc: Haiyang
Hi Christian,
Thank you for the patch! Perhaps something to improve:
[auto build test WARNING on staging/staging-testing]
[also build test WARNING on v4.20-rc6 next-20181212]
[if your patch is applied to the wrong git tree, please drop us a note to help
improve the system]
url:
https://gith
From: Amir Mahdi Ghorbanian
Fixed a coding style issue.
Signed-off-by: Amir Mahdi Ghorbanian
---
drivers/staging/comedi/drivers/8255.h | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/staging/comedi/drivers/8255.h
b/drivers/staging/comedi/drivers/8255.h
index 6cd133
On Thu, Dec 13, 2018 at 04:32:25PM +, christian.gr...@microchip.com wrote:
> On Do, 2018-12-13 at 13:32 +0100, Greg KH wrote:
> > On Thu, Dec 13, 2018 at 02:58:00PM +0300, Dan Carpenter wrote:
> > >
> > > On Wed, Dec 12, 2018 at 01:15:31PM +0100, Christian Gromm wrote:
> > > >
> > > > This pa
On Wed, Dec 12, 2018 at 01:51:27PM +0100, Christian Brauner wrote:
> > > Cc: Martijn Coenen
> > > Cc: Todd Kjos
> > > Cc: Greg Kroah-Hartman
> > > Signed-off-by: Christian Brauner
>
> Do we plan to bring this into mergeable shape before Christmas? I'm
> happy to do it. :)
I haven't had the ch
Fixed a coding style issue.
Signed-off-by: Amir Mahdi Ghorbanian
---
drivers/staging/comedi/drivers/cb_pcidas.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/staging/comedi/drivers/cb_pcidas.c
b/drivers/staging/comedi/drivers/cb_pcidas.c
index 8429d57..02ae00c 1006
ADT7316 driver no more uses platform data and hence use device tree
data instead of platform data for assigning irq_type field and
implement this in adt7316_irq_setup function.
Switch case figures out the type of irq and if it's the default case
then assign the default value to the irq_type i.e.
ir
On Thu, Dec 13, 2018 at 04:35:43PM +, Dexuan Cui wrote:
Before 98f4c651762c, we returned zeros for unopened channels.
With 98f4c651762c, we started to return random on-stack values.
We'd better return -EINVAL instead.
Fixes: 98f4c651762c ("hv: move ringbuffer bus attributes to dev_groups")
On Thu, Dec 13, 2018 at 06:56:26PM +0100, Greg Kroah-Hartman wrote:
> On Wed, Dec 12, 2018 at 01:51:27PM +0100, Christian Brauner wrote:
> > > > Cc: Martijn Coenen
> > > > Cc: Todd Kjos
> > > > Cc: Greg Kroah-Hartman
> > > > Signed-off-by: Christian Brauner
> >
> > Do we plan to bring this int
44d8047f1d8 ("binder: use standard functions to allocate fds")
exposed a pre-existing issue in the binder driver.
fdget() is used in ksys_ioctl() as a performance optimization.
One of the rules associated with fdget() is that ksys_close() must
not be called between the fdget() and the fdput(). The
On Thu, Dec 13, 2018 at 09:52:03PM +0100, Christian Brauner wrote:
> On Thu, Dec 13, 2018 at 06:56:26PM +0100, Greg Kroah-Hartman wrote:
> > On Wed, Dec 12, 2018 at 01:51:27PM +0100, Christian Brauner wrote:
> > > > > Cc: Martijn Coenen
> > > > > Cc: Todd Kjos
> > > > > Cc: Greg Kroah-Hartman
>
On Thu, Dec 13, 2018 at 02:15:59PM +, christian.gr...@microchip.com wrote:
> Sounds reasonable. But, problem here is that we want the
> process of how channels are linked to be independent from the
> component that is being used. That's when things start to
> become complicated.
I'm not goin
As discussed at Linux Plumbers Conference 2018 in Vancouver [1] this is the
implementation of binderfs.
/* Abstract */
binderfs is a backwards-compatible filesystem for Android's binder ipc
mechanism. Each ipc namespace will mount a new binderfs instance. Mounting
binderfs multiple times at differ
On Wed, Dec 12, 2018 at 11:23:16AM +0300, Dan Carpenter wrote:
> On Tue, Dec 11, 2018 at 05:54:56PM -0700, Jeremy Fertic wrote:
> > @@ -651,10 +649,12 @@ static ssize_t
> > adt7316_store_da_high_resolution(struct device *dev,
> > u8 config3;
> > int ret;
> >
> > + if (chip->id == ID_AD
On Wed, Dec 12, 2018 at 11:19:49AM +0300, Dan Carpenter wrote:
> On Tue, Dec 11, 2018 at 05:54:54PM -0700, Jeremy Fertic wrote:
> > ADT7316_DA_EN_VIA_DAC_LDCA is set when the dac and ldac registers are being
> > used to update the dacs instead of the ldac pin. ADT7516_SEL_AIN3 is an adc
> > input t
Quoting Jonathan Cameron (2018-12-08 07:29:54)
> On Thu, 6 Dec 2018 11:10:51 +0200
> Mircea Caprioru wrote:
>
> > This patch adds a clock to the state structure of ad7192 for getting the
> > external clock frequency. This modifications is in accordance with clock
> > framework dt bindings documen
Hi,
How are you? I must confess that you're a nice looking gentle man on your
profile.Are you married?, Can we be friends?
Susan.
___
devel mailing list
de...@linuxdriverproject.org
http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-
On Thu, Dec 13, 2018 at 03:06:29PM -0700, Jeremy Fertic wrote:
> On Wed, Dec 12, 2018 at 11:19:49AM +0300, Dan Carpenter wrote:
> > On Tue, Dec 11, 2018 at 05:54:54PM -0700, Jeremy Fertic wrote:
> > > ADT7316_DA_EN_VIA_DAC_LDCA is set when the dac and ldac registers are
> > > being
> > > used to u
On Thu, Dec 13, 2018 at 03:01:46PM -0700, Jeremy Fertic wrote:
> On Wed, Dec 12, 2018 at 11:23:16AM +0300, Dan Carpenter wrote:
> > On Tue, Dec 11, 2018 at 05:54:56PM -0700, Jeremy Fertic wrote:
> > > @@ -651,10 +649,12 @@ static ssize_t
> > > adt7316_store_da_high_resolution(struct device *dev,
>
On Thu, Dec 13, 2018 at 10:59:11PM +0100, Christian Brauner wrote:
> +/**
> + * binderfs_new_inode - allocate inode from super block of a binderfs mount
> + * @ref_inode: inode from wich the super block will be taken
> + * @userp: buffer to copy information about new device for userspace to
> +
56 matches
Mail list logo
