[PATCH v2 2/2] hwrng: ba431-rng: add support for BA431 hwrng

Silex insight BA431 is an IP designed to generate random numbers that can be integrated in various FPGA. This driver adds support for it through the hwrng interface. This driver is used in Silex Insight Viper OEM boards. Signed-off-by: Olivier Sobrie Signed-off-by: Waleed Ziad Acked-by: Arnd

[PATCH v2 1/2] dt-bindings: rng: document Silex Insight BA431 hwrng

This patch documents the device tree bindings of the BA431 hardware random number generator. This IP is for instance present in the Viper OEM boards sold by Silex Insight. Signed-off-by: Olivier Sobrie Reviewed-by: Rob Herring --- .../bindings/rng/silex-insight,ba431-rng.yaml | 36

[PATCH v2 0/2] hwrng: add support for Silex Insight BA431

BA431 hwrng driver. Olivier Sobrie (2): dt-bindings: rng: document Silex Insight BA431 hwrng hwrng: ba431-rng: add support for BA431 hwrng .../bindings/rng/silex-insight,ba431-rng.yaml | 36 +++ drivers/char/hw_random/Kconfig| 12 + drivers/char/hw_random/Makefile

Re: [PATCH 3/3] hwrng: ba431-rng: add support for BA431 hwrng

On Fri, May 29, 2020 at 11:12:31AM -0600, Rob Herring wrote: > On Mon, May 25, 2020 at 10:28:46PM +0200, Arnd Bergmann wrote: > > On Mon, May 25, 2020 at 10:07 PM Olivier Sobrie > > wrote: > > > > > > Silex insight BA431 is an IP designed to generate random number

Re: [PATCH 3/3] hwrng: ba431-rng: add support for BA431 hwrng

On Mon, May 25, 2020 at 10:28:46PM +0200, Arnd Bergmann wrote: > On Mon, May 25, 2020 at 10:07 PM Olivier Sobrie > wrote: > > > > Silex insight BA431 is an IP designed to generate random numbers that > > can be integrated in various FPGA. > > This driver adds su

[PATCH 3/3] hwrng: ba431-rng: add support for BA431 hwrng

Silex insight BA431 is an IP designed to generate random numbers that can be integrated in various FPGA. This driver adds support for it through the hwrng interface. This driver is used in Silex Insight Viper OEM boards. Signed-off-by: Olivier Sobrie Signed-off-by: Waleed Ziad --- drivers

[PATCH 1/3] dt-bindings: vendor-prefixes: Add Silex Insight vendor prefix

Silex Insight is a microelectronic company whose headquarter is located in Belgium. Web site of the company: https://www.silexinsight.com/ Signed-off-by: Olivier Sobrie --- Documentation/devicetree/bindings/vendor-prefixes.yaml | 2 ++ 1 file changed, 2 insertions(+) diff --git a/Documentation

[PATCH 2/3] dt-bindings: rng: document Silex Insight BA431 hwrng

This patch documents the device tree bindings of the BA431 hardware random number generator. This IP is for instance present in the Viper OEM boards sold by Silex Insight. Signed-off-by: Olivier Sobrie --- .../bindings/rng/silex-insight,ba431-rng.yaml | 36 +++ 1 file changed

[PATCH 0/3] hwrng: add support for Silex Insight BA431

contains the BA431 hwrng driver. Olivier Sobrie (3): dt-bindings: vendor-prefixes: Add Silex Insight vendor prefix dt-bindings: rng: document Silex Insight BA431 hwrng hwrng: ba431-rng: add support for BA431 hwrng .../bindings/rng/silex-insight,ba431-rng.yaml | 33 +++ .../devicetree/bindings

[PATCH] gpu: drm: i915: intel_hotplug: avoid NULL pointer dereference

not NULL. The same issue may happen in intel_hpd_irq_storm_reenable_work(). This patch should fix these issues. Signed-off-by: Olivier Sobrie --- drivers/gpu/drm/i915/intel_hotplug.c | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/drivers/gpu/drm/i915/intel_hotplug.c b/dr

Re: [PATCH] hso: fix refcnt leak in recent patch.

kref_put(&serial->parent->ref, hso_serial_ref_free); > > > > after hso_serial_ref_free has been called, which dereferences a freed > > pointer. > > > > This patch adds the missing kref_get(). > > > > Fixes: commit 29bd3bc1194c624ce863cab2a7d

Re: [PATCH] hso: fix refcnt leak in recent patch.

in calls to > >kref_put(&serial->parent->ref, hso_serial_ref_free); > > > > after hso_serial_ref_free has been called, which dereferences a freed > > pointer. > > > > This patch adds the missing kref_get(). > > > > Fixes: commit 29bd3bc1194c6

[PATCH] input: misc: pwm-beeper: remove useless call to pwm_config()

Calling pwm_config() with a period equal to zero always results in error (-EINVAL) and pwm chip config method is never called. Signed-off-by: Olivier Sobrie --- drivers/input/misc/pwm-beeper.c | 1 - 1 file changed, 1 deletion(-) diff --git a/drivers/input/misc/pwm-beeper.c b/drivers/input

Re: [PATCH v2 11/11] hso: fix rfkill name conflicts

Hello Dan, On Fri, Jan 30, 2015 at 09:47:59AM -0600, Dan Williams wrote: > On Fri, 2015-01-30 at 13:22 +0100, Olivier Sobrie wrote: > > By using only the usb interface number for the rfkill name, we might > > have a name conflicts in case two similar hso devices are connected. &

[PATCH v2 03/11] hso: fix memory leak when device disconnects

In the disconnect path, tx_buffer should freed like tx_data to avoid a memory leak when the device disconnects. Signed-off-by: Olivier Sobrie --- drivers/net/usb/hso.c | 1 + 1 file changed, 1 insertion(+) diff --git a/drivers/net/usb/hso.c b/drivers/net/usb/hso.c index 3a6c630..470ef9e 100644

[PATCH v2 01/11] hso: remove useless header file timer.h

No timer related function is used in this driver. Signed-off-by: Olivier Sobrie --- drivers/net/usb/hso.c | 1 - 1 file changed, 1 deletion(-) diff --git a/drivers/net/usb/hso.c b/drivers/net/usb/hso.c index babda7d..cb0bcc1 100644 --- a/drivers/net/usb/hso.c +++ b/drivers/net/usb/hso.c

[PATCH v2 00/11] hso: fix some problems in the disconnect path

fix the issue. See http://marc.info/?l=linux-usb&m=142186699418489 for more info. - Added an extra patch avoiding name conflicts for the rfkill interface. Olivier Sobrie (11): hso: remove useless header file timer.h hso: fix crash when device disappears while serial port is open

[PATCH v2 04/11] hso: fix memory leak in hso_create_rfkill()

When the rfkill interface was created, a buffer containing the name of the rfkill node was allocated. This buffer was never freed when the device disappears. To fix the problem, we put the name given to rfkill_alloc() in the hso_net structure. Signed-off-by: Olivier Sobrie --- drivers/net/usb

[PATCH v2 08/11] hso: move tty_unregister outside hso_serial_common_free()

x3d8/0x560) [] (worker_thread) from [] (kthread+0xc0/0xcc) [] (kthread) from [] (ret_from_fork+0x14/0x24) ---[ end trace cb88537fdc8fa208 ]--- Signed-off-by: Olivier Sobrie --- drivers/net/usb/hso.c | 12 +--- 1 file changed, 9 insertions(+), 3 deletions(-) diff --git a/drivers/net/usb/hso.c

[PATCH v2 07/11] hso: replace reset_device work by usb_queue_reset_device()

There is no need for a dedicated reset work in the hso driver since there is already a reset work foreseen in usb_interface that does the same. Signed-off-by: Olivier Sobrie --- drivers/net/usb/hso.c | 25 + 1 file changed, 1 insertion(+), 24 deletions(-) diff --git a

[PATCH v2 05/11] hso: fix small indentation error

Simply remove the useless extra tab. Signed-off-by: Olivier Sobrie --- drivers/net/usb/hso.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/net/usb/hso.c b/drivers/net/usb/hso.c index a49ac2e..2f2343d 100644 --- a/drivers/net/usb/hso.c +++ b/drivers/net/usb

[PATCH v2 02/11] hso: fix crash when device disappears while serial port is open

is called at the end of the hangup process and we drop the last kref in this function when all the ports have been closed, when tty_port is no more needed and when it is safe to free the structure containing the tty_port structure. Signed-off-by: Olivier Sobrie --- drivers/net/usb/hso.c | 17

[PATCH v2 10/11] hso: add missing cancel_work_sync in disconnect()

For hso serial devices, two cancel_work_sync were missing in the disconnect method. Signed-off-by: Olivier Sobrie --- drivers/net/usb/hso.c | 2 ++ 1 file changed, 2 insertions(+) diff --git a/drivers/net/usb/hso.c b/drivers/net/usb/hso.c index c916ab5..c14fc80 100644 --- a/drivers/net/usb

[PATCH v2 06/11] hso: rename hso_dev into serial in hso_free_interface()

In other functions of the driver, variables of type "struct hso_serial" are denoted by "serial" and variables of type "struct hso_device" are denoted by "hso_dev". This patch makes the hso_free_interface() consistent with these notations. Signed-off-by: Ol

[PATCH v2 09/11] hso: update serial_table in usb disconnect method

for serial port linked to usb interface "x" and possibly crash because the tty_port structure contained in the hso_device structure has been freed. Signed-off-by: Olivier Sobrie --- drivers/net/usb/hso.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/net/us

[PATCH v2 11/11] hso: fix rfkill name conflicts

: Dan Williams Signed-off-by: Olivier Sobrie --- drivers/net/usb/hso.c | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) diff --git a/drivers/net/usb/hso.c b/drivers/net/usb/hso.c index c14fc80..d31a165 100644 --- a/drivers/net/usb/hso.c +++ b/drivers/net/usb/hso.c @@ -153,7 +153,7

Re: [PATCH 11/11] usb: core: fix a race with usb_queue_reset_device()

Hello Alan, On Tue, Jan 20, 2015 at 10:26:30AM -0500, Alan Stern wrote: > On Tue, 20 Jan 2015, Olivier Sobrie wrote: > > > When usb_queue_reset() is called it schedules a work in view of > > resetting the usb interface. When the reset work is running, it > > can be sche

Re: [PATCH 11/11] usb: core: fix a race with usb_queue_reset_device()

On Tue, Jan 20, 2015 at 10:26:30AM -0500, Alan Stern wrote: > On Tue, 20 Jan 2015, Olivier Sobrie wrote: > > > When usb_queue_reset() is called it schedules a work in view of > > resetting the usb interface. When the reset work is running, it > > can be scheduled again (e.

Re: [PATCH 04/11] hso: fix memory leak in hso_create_rfkill()

On Tue, Jan 20, 2015 at 02:13:17PM +0100, Oliver Neukum wrote: > On Tue, 2015-01-20 at 13:29 +0100, Olivier Sobrie wrote: > > When the rfkill interface was created, a buffer containing the name > > of the rfkill node was allocated. This buffer was never freed when the > &

Re: [PATCH 11/11] usb: core: fix a race with usb_queue_reset_device()

Hi Oliver, On Tue, Jan 20, 2015 at 02:48:37PM +0100, Oliver Neukum wrote: > On Tue, 2015-01-20 at 13:29 +0100, Olivier Sobrie wrote: > > When usb_queue_reset() is called it schedules a work in view of > > resetting the usb interface. When the reset work is running, it > > c

[PATCH 02/11] hso: fix crash when device disappears while serial port is open

is called at the end of the hangup process and we drop the last kref in this function when all the ports have been closed, when tty_port is no more needed and when it is safe to free the structure containing the tty_port structure. Signed-off-by: Olivier Sobrie --- drivers/net/usb/hso.c | 17

[PATCH 04/11] hso: fix memory leak in hso_create_rfkill()

When the rfkill interface was created, a buffer containing the name of the rfkill node was allocated. This buffer was never freed when the device disappears. To fix the problem, we put the name given to rfkill_alloc() in the hso_net structure. Signed-off-by: Olivier Sobrie --- drivers/net/usb

[PATCH 03/11] hso: fix memory leak when device disconnects

In the disconnect path, tx_buffer should freed like tx_data to avoid a memory leak when the device disconnects. Signed-off-by: Olivier Sobrie --- drivers/net/usb/hso.c | 1 + 1 file changed, 1 insertion(+) diff --git a/drivers/net/usb/hso.c b/drivers/net/usb/hso.c index 3a6c630..470ef9e 100644

[PATCH 00/11] hso: fix some problems with reset/disconnect path

queued on an usb interface. This serie of patches is based on v3.18. Olivier Sobrie (11): hso: remove useless header file timer.h hso: fix crash when device disappears while serial port is open hso: fix memory leak when device disconnects hso: fix memory leak in hso_create_rfkill() hso: fix

[PATCH 01/11] hso: remove useless header file timer.h

No timer related function is used in this driver. Signed-off-by: Olivier Sobrie --- drivers/net/usb/hso.c | 1 - 1 file changed, 1 deletion(-) diff --git a/drivers/net/usb/hso.c b/drivers/net/usb/hso.c index babda7d..cb0bcc1 100644 --- a/drivers/net/usb/hso.c +++ b/drivers/net/usb/hso.c

[PATCH 07/11] hso: replace reset_device work by usb_queue_reset_device()

There is no need for a dedicated reset work in the hso driver since there is already a reset work foreseen in usb_interface that does the same. Signed-off-by: Olivier Sobrie --- drivers/net/usb/hso.c | 25 + 1 file changed, 1 insertion(+), 24 deletions(-) diff --git a

[PATCH 08/11] hso: move tty_unregister outside hso_serial_common_free()

x3d8/0x560) [] (worker_thread) from [] (kthread+0xc0/0xcc) [] (kthread) from [] (ret_from_fork+0x14/0x24) ---[ end trace cb88537fdc8fa208 ]--- Signed-off-by: Olivier Sobrie --- drivers/net/usb/hso.c | 12 +--- 1 file changed, 9 insertions(+), 3 deletions(-) diff --git a/drivers/net/usb/hso.c

[PATCH 06/11] hso: rename hso_dev into serial in hso_free_interface()

In other functions of the driver, variables of type "struct hso_serial" are denoted by "serial" and variables of type "struct hso_device" are denoted by "hso_dev". This patch makes the hso_free_interface() consistent with these notations. Signed-off-by: Ol

[PATCH 11/11] usb: core: fix a race with usb_queue_reset_device()

work that should be run To avoid this problem, we use a delayed work so that if the reset work is currently run, we can avoid further call to __usb_queue_reset_device() work by using cancel_delayed_work(). Unfortunately it increases the size of the usb_interface structure... Signed-off-by: O

[PATCH 10/11] hso: add missing cancel_work_sync in disconnect()

For hso serial devices, two cancel_work_sync were missing in the disconnect method. Signed-off-by: Olivier Sobrie --- drivers/net/usb/hso.c | 2 ++ 1 file changed, 2 insertions(+) diff --git a/drivers/net/usb/hso.c b/drivers/net/usb/hso.c index c916ab5..c14fc80 100644 --- a/drivers/net/usb

[PATCH 09/11] hso: update serial_table in usb disconnect method

for serial port linked to usb interface "x" and possibly crash because the tty_port structure contained in the hso_device structure has been freed. Signed-off-by: Olivier Sobrie --- drivers/net/usb/hso.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/net/us

[PATCH 05/11] hso: fix small indentation error

Simply remove the useless extra tab. Signed-off-by: Olivier Sobrie --- drivers/net/usb/hso.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/net/usb/hso.c b/drivers/net/usb/hso.c index a49ac2e..2f2343d 100644 --- a/drivers/net/usb/hso.c +++ b/drivers/net/usb

Re: [PATCH v4 3/4] can: kvaser_usb: Add support for the Usbcan-II family

e based on the following: > * - Kvaser linux leaf driver (version 4.78) > * - CAN driver for esd CAN-USB/2 > + * - Kvaser linux usbcanII driver (version 5.3) > * > * Copyright (C) 2002-2006 KVASER AB, Sweden. All rights reserved. > * Copyright (C) 2010 Matthias Fuchs

Re: [PATCH v4 4/4] can: kvaser_usb: Retry first bulk transfer on -ETIMEDOUT

Hello, On Mon, Jan 12, 2015 at 05:14:07AM -0500, Ahmed S. Darwish wrote: > On Sun, Jan 11, 2015 at 09:51:10PM +0100, Marc Kleine-Budde wrote: > > On 01/11/2015 09:45 PM, Ahmed S. Darwish wrote: > > > From: Ahmed S. Darwish > > > > > > (This is a draft patch, I'm not sure if this fixes the USB >

Re: [PATCH] can: kvaser_usb: Add support for the Usbcan-II family

On Tue, Dec 30, 2014 at 10:33:26AM -0500, Ahmed S. Darwish wrote: > On Sun, Dec 28, 2014 at 10:51:34PM +0100, Olivier Sobrie wrote: > [...] > > > > > > > > > > + if (LEAF_PRODUCT_ID(id->idProduct)) { > > > > > + dev->family

Re: [PATCH v2 3/4] can: kvaser_usb: Don't send a RESET_CHIP for non-existing channels

orah Signed-off-by: Olivier Sobrie Kr, Olivier > > Suggested-by: Olivier Sobrie > Signed-off-by: Ahmed S. Darwish > --- > drivers/net/can/usb/kvaser_usb.c | 7 --- > 1 file changed, 4 insertions(+), 3 deletions(-) > > diff --git a/drivers/net/can/usb/kvaser_usb.c

Re: [PATCH v2 1/4] can: kvaser_usb: Don't free packets when tight on URBs

> is a driver bug in and out of itself: it means that our start/stop > queue flow control is broken. > > This patch only fixes the (buggy) error handling code; the root > cause shall be fixed in a later commit. > > Signed-off-by: Ahmed S. Darwish Acked-by: Olivier Sobrie &

Re: [PATCH] can: kvaser_usb: Add support for the Usbcan-II family

Hello Ahmed, On Wed, Dec 24, 2014 at 05:04:17PM +0200, Ahmed S. Darwish wrote: > Hi Olivier, > > On Wed, Dec 24, 2014 at 01:36:27PM +0100, Olivier Sobrie wrote: > > Hello Ahmed, > > > > On Tue, Dec 23, 2014 at 05:53:11PM +0200, Ahmed S. Darwish wrote: &

Re: [PATCH] can: kvaser_usb: Add support for the Usbcan-II family

Kvaser linux leaf driver (version 4.78) > * - CAN driver for esd CAN-USB/2 > + * - Kvaser linux usbcanII driver (version 5.3) > * > * Copyright (C) 2002-2006 KVASER AB, Sweden. All rights reserved. > * Copyright (C) 2010 Matthias Fuchs , esd gmbh > * Copyright (C) 2012 Olivier Sobrie >

Re: [PATCH] can: kvaser_usb: Don't free packets when tight on URBs

Hello Ahmed, On Tue, Dec 23, 2014 at 05:46:54PM +0200, Ahmed S. Darwish wrote: > From: Ahmed S. Darwish > > Flooding the Kvaser CAN to USB dongle with multiple reads and > writes in high frequency caused seemingly-random panics in the > kernel. > > On further inspection, it seems the driver erron

[PATCH v2 2/2] hso: fix deadlock when receiving bursts of data

oom in the tty buffer, discards all the data received in the URB. Cc: David Miller Cc: David Laight Cc: One Thousand Gnomes Cc: Dan Williams Cc: Jan Dumon Signed-off-by: Olivier Sobrie --- Changes in v2: - remove the unthrottle timer added in the previous patch version - drop entire rx u

[PATCH v2 1/2] hso: remove unused workqueue

The workqueue "retry_unthrottle_workqueue" is not scheduled anywhere in the code. So, remove it. Signed-off-by: Olivier Sobrie --- drivers/net/usb/hso.c | 12 1 file changed, 12 deletions(-) diff --git a/drivers/net/usb/hso.c b/drivers/net/usb/hso.c index a3a0586..9ca2

[PATCH v2 2/2] hso: fix deadlock when receiving bursts of data

oom in the tty buffer, discards all the data received in the URB. Cc: David Miller Cc: David Laight Cc: One Thousand Gnomes Cc: Dan Williams Cc: Jan Dumon Signed-off-by: Olivier Sobrie --- Changes in v2: - remove the unthrottle timer added in the previous patch version - drop entire rx u

Re: [PATCH 2/2] hso: fix deadlock when receiving bursts of data

On Fri, Jul 11, 2014 at 09:28:47AM +, David Laight wrote: > From: Olivier Sobrie Olivier Sobrie > > Hi Alan and Davids, > > > > On Thu, Jul 10, 2014 at 04:50:03PM +0100, One Thousand Gnomes wrote: > > > On Thu, 10 Jul 2014 14:37:37 + > > > David Lai

Re: [PATCH 2/2] hso: fix deadlock when receiving bursts of data

Hi Alan and Davids, On Thu, Jul 10, 2014 at 04:50:03PM +0100, One Thousand Gnomes wrote: > On Thu, 10 Jul 2014 14:37:37 + > David Laight wrote: > > > From: Olivier Sobrie > > ... > > > The function put_rxbuf_data() is called from the urb completion handler. >

Re: [PATCH 2/2] hso: fix deadlock when receiving bursts of data

Hi David, On Tue, Jul 08, 2014 at 04:16:33PM -0700, David Miller wrote: > From: Olivier Sobrie > Date: Mon, 7 Jul 2014 11:06:07 +0200 > > > When the module sends bursts of data, sometimes a deadlock happens in > > the hso driver when the tty buffer doesn't ge

Re: [PATCH 2/2] hso: fix deadlock when receiving bursts of data

Hi Dan, On Mon, Jul 07, 2014 at 11:41:20AM -0500, Dan Williams wrote: > On Mon, 2014-07-07 at 11:06 +0200, Olivier Sobrie wrote: > > When the module sends bursts of data, sometimes a deadlock happens in > > the hso driver when the tty buffer doesn't get the chance to be flushe

Re: [PATCH 2/2] hso: fix deadlock when receiving bursts of data

On Mon, Jul 07, 2014 at 12:55:44PM +, David Laight wrote: > From: Olivier Sobrie > > Hi David, > > > > On Mon, Jul 07, 2014 at 09:13:53AM +, David Laight wrote: > > > From: Olivier Sobrie > > > > When the module sends bursts of data, sometimes a

Re: [PATCH 2/2] hso: fix deadlock when receiving bursts of data

Hi David, On Mon, Jul 07, 2014 at 09:13:53AM +, David Laight wrote: > From: Olivier Sobrie > > When the module sends bursts of data, sometimes a deadlock happens in > > the hso driver when the tty buffer doesn't get the chance to be flushed > > quickly enough. >

[PATCH 1/2] hso: remove unused workqueue

The workqueue "retry_unthrottle_workqueue" is not scheduled anywhere in the code. So, remove it. Signed-off-by: Olivier Sobrie --- drivers/net/usb/hso.c | 12 1 file changed, 12 deletions(-) diff --git a/drivers/net/usb/hso.c b/drivers/net/usb/hso.c index a3a0586..9ca2

[PATCH 2/2] hso: fix deadlock when receiving bursts of data

re is no room anymore in the tty buffer, we set up a timer of 100 msecs to give a chance to the upper layer to flush the tty buffer and make room for new data. Signed-off-by: Olivier Sobrie --- drivers/net/usb/hso.c | 51 + 1 file change