[PATCH] media: radio: Critical interrupt bugfix for si470x over i2c

2018-01-18 Thread Douglas Fischer
Fixed si470x_start() disabling the interrupt signal, causing tune operations to never complete. This does not affect USB radios because they poll the registers instead of using the IRQ line. Signed-off-by: Douglas Fischer --- diff -uprN linux.orig/drivers/media/radio/si470x/radio-si470x

[PATCH] media: radio: Tuning bugfix for si470x over i2c

2018-01-18 Thread Douglas Fischer
Fixed si470x_set_channel() trying to tune before chip is turned on, which causes warnings in dmesg and when probing, makes driver wait for 3s for tuning timeout. This issue did not affect USB devices because they have a different probing sequence. Signed-off-by: Douglas Fischer --- diff -uprN

[PATCH] media: radio: Critical v4l2 registration bugfix for si470x over i2c

2018-01-18 Thread Douglas Fischer
Added the call to v4l2_device_register() required to add a new radio device. Without this patch, it is impossible for the driver to load. This does not affect USB devices. Signed-off-by: Douglas Fischer --- diff -uprN linux.orig/drivers/media/radio/si470x/radio-si470x-i2c.c linux/drivers/media

[PATCH] media: radio: Critical interrupt bugfix for si470x over i2c

2018-01-20 Thread Douglas Fischer
Fixed si470x_start() disabling the interrupt signal, causing tune operations to never complete. This does not affect USB radios because they poll the registers instead of using the IRQ line. Signed-off-by: Douglas Fischer --- diff -uprN linux.orig/drivers/media/radio/si470x/radio-si470x

[PATCH] media: radio: Tuning bugfix for si470x over i2c

2018-01-20 Thread Douglas Fischer
Fixed si470x_set_channel() trying to tune before chip is turned on, which causes warnings in dmesg and when probing, makes driver wait for 3s for tuning timeout. This issue did not affect USB devices because they have a different probing sequence. Signed-off-by: Douglas Fischer --- diff -uprN

[PATCH] media: radio: Critical v4l2 registration bugfix for si470x over i2c

2018-01-20 Thread Douglas Fischer
Added the call to v4l2_device_register() required to add a new radio device. Without this patch, it is impossible for the driver to load. This does not affect USB devices. Signed-off-by: Douglas Fischer --- diff -uprN linux.orig/drivers/media/radio/si470x/radio-si470x-i2c.c linux/drivers/media

[PATCH] media: radio: Critical interrupt bugfix for si470x over i2c

2018-01-26 Thread Douglas Fischer
Fixed si470x_start() disabling the interrupt signal, causing tune operations to never complete. This does not affect USB radios because they poll the registers instead of using the IRQ line. Signed-off-by: Douglas Fischer --- diff -uprN linux.orig/drivers/media/radio/si470x/radio-si470x

Re: [PATCH] media: radio: Critical interrupt bugfix for si470x over i2c

2018-02-24 Thread Douglas Fischer
18 00:42, Douglas Fischer wrote: > > Fixed si470x_start() disabling the interrupt signal, causing tune > > operations to never complete. This does not affect USB radios > > because they poll the registers instead of using the IRQ line. > > > > Signed-off-by: Dougla

[PATCH v3] media: radio: Tuning bugfix for si470x over i2c

2018-02-25 Thread Douglas Fischer
power is off instead of putting the rest of the function in the conditional. Signed-off-by: Douglas Fischer --- diff -uprN linux.orig/drivers/media/radio/si470x/radio-si470x-common.c linux/drivers/media/radio/si470x/radio-si470x-common.c --- linux.orig/drivers/media/radio/si470x/radio-si470x

[PATCH v3] media: radio: Critical v4l2 registration bugfix for si470x over i2c

2018-02-25 Thread Douglas Fischer
Added the call to v4l2_device_register() required to add a new radio device. Without this patch, it is impossible for the driver to load. This does not affect USB devices. Fixed cleanup order from v2. Signed-off-by: Douglas Fischer --- diff -uprN linux.orig/drivers/media/radio/si470x/radio

[PATCH v3] media: radio: Critical interrupt bugfix for si470x over i2c

2018-02-25 Thread Douglas Fischer
Fixed si470x_start() disabling the interrupt signal, causing tune operations to never complete. This does not affect USB radios because they poll the registers instead of using the IRQ line. Stylistic and comment changes from v2. Signed-off-by: Douglas Fischer --- diff -uprN linux.orig/drivers

[PATCH v3] media: radio: Critical v4l2 registration bugfix for si470x over i2c

2018-02-26 Thread Douglas Fischer
Added the call to v4l2_device_register() required to add a new radio device. Without this patch, it is impossible for the driver to load. This does not affect USB devices. Fixed cleanup order from v2. Signed-off-by: Douglas Fischer --- diff -uprN linux.orig/drivers/media/radio/si470x/radio

Re: [PATCH v3] media: radio: Critical interrupt bugfix for si470x over i2c

2018-02-26 Thread Douglas Fischer
Hans, See comments below. Thank you for the quick response on this and all your patience and help in general with these patches. On Mon, 26 Feb 2018 12:57:26 +0100 Hans Verkuil wrote: > On 02/26/2018 03:27 AM, Douglas Fischer wrote: > > Fixed si470x_start() disabling the interru

[PATCH v4] media: radio: Critical interrupt bugfix for si470x over i2c

2018-03-02 Thread Douglas Fischer
Fixed si470x_start() disabling the interrupt signal, causing tune operations to never complete. This does not affect USB radios because they poll the registers instead of using the IRQ line. Stylistic and comment changes from v3. Signed-off-by: Douglas Fischer --- diff -uprN linux.orig/drivers