Re: [PATCH v2 1/2] [resend] radio-isa: PnP support for the new ISA radio framework

2012-03-30 Thread Hans Verkuil
, Hans On Thursday, March 22, 2012 19:53:01 Ondrej Zary wrote: Add PnP support to the new ISA radio framework. Signed-off-by: Ondrej Zary li...@rainbow-software.org diff --git a/drivers/media/radio/radio-isa.c b/drivers/media/radio/radio-isa.c index 02bcead..ed9039f 100644

[PATCH v2 1/2] [resend] radio-isa: PnP support for the new ISA radio framework

2012-03-22 Thread Ondrej Zary
Add PnP support to the new ISA radio framework. Signed-off-by: Ondrej Zary li...@rainbow-software.org diff --git a/drivers/media/radio/radio-isa.c b/drivers/media/radio/radio-isa.c index 02bcead..ed9039f 100644 --- a/drivers/media/radio/radio-isa.c +++ b/drivers/media/radio/radio-isa.c @@ -26,6

[PATCH 1/2] radio-isa: PnP support for the new ISA radio framework

2012-03-05 Thread Ondrej Zary
Add PnP support to the new ISA radio framework. Signed-off-by: Ondrej Zary li...@rainbow-software.org diff --git a/drivers/media/radio/radio-isa.c b/drivers/media/radio/radio-isa.c index 02bcead..b0c0d7a 100644 --- a/drivers/media/radio/radio-isa.c +++ b/drivers/media/radio/radio-isa.c @@ -26,6

[PATCH v2 1/2] radio-isa: PnP support for the new ISA radio framework

2012-03-05 Thread Ondrej Zary
Add PnP support to the new ISA radio framework. Signed-off-by: Ondrej Zary li...@rainbow-software.org diff --git a/drivers/media/radio/radio-isa.c b/drivers/media/radio/radio-isa.c index 02bcead..ed9039f 100644 --- a/drivers/media/radio/radio-isa.c +++ b/drivers/media/radio/radio-isa.c @@ -26,6

Re: [PATCH 1/2] radio-isa: PnP support for the new ISA radio framework

2012-03-05 Thread Ondrej Zary
On Monday 05 March 2012 21:30:51 Ondrej Zary wrote: Add PnP support to the new ISA radio framework. [...] +int radio_isa_pnp_remove(struct pnp_dev *dev) Please ignore this patch, it's broken (this function should return void and radio-gemtek fails to compile without CONFIG_PNP). I've sent

Re: [RFC PATCH] PnP support for the new ISA radio framework

2012-03-02 Thread Hans Verkuil
Hi Ondrej! Thanks for working on this. It looks good, I just have a few minor points: On Thursday, March 01, 2012 20:25:05 Ondrej Zary wrote: Hello, this is the first attempt to add PnP support to the new ISA radio framework. I don't like the region_size function parameter - it's needed

[RFC PATCH] PnP support for the new ISA radio framework

2012-03-01 Thread Ondrej Zary
Hello, this is the first attempt to add PnP support to the new ISA radio framework. I don't like the region_size function parameter - it's needed because PnP reports longer port range than drv-region_size. There is a small patch to radio-gemtek at the end that uses this PnP support for AOpen FX

[RFC PATCH] PnP support for the new ISA radio framework

2012-02-28 Thread Ondrej Zary
Hello, this is the first attempt to add PnP support to the new ISA radio framework. I don't like the region_size function parameter - it's needed because PnP reports longer port range than drv-region_size. There is a small patch to radio-gemtek at the end that uses this PnP support for AOpen FX

Re: PnP support for the new ISA radio framework?

2012-02-24 Thread Hans Verkuil
On Wednesday, February 22, 2012 21:33:29 Ondrej Zary wrote: On Saturday 18 February 2012 17:33:32 Ondrej Zary wrote: Hello, there are some ISA radio cards with PnP support (e.g. SF16-FMI) but the new ISA radio framework has no PnP support. I got AOpen FX-3D/Pro Radio card which

Re: PnP support for the new ISA radio framework?

2012-02-22 Thread Ondrej Zary
On Saturday 18 February 2012 17:33:32 Ondrej Zary wrote: Hello, there are some ISA radio cards with PnP support (e.g. SF16-FMI) but the new ISA radio framework has no PnP support. I got AOpen FX-3D/Pro Radio card which is AD1816 with Gemtek radio - and with PnP. But radio-gemtek fails

PnP support for the new ISA radio framework?

2012-02-18 Thread Ondrej Zary
Hello, there are some ISA radio cards with PnP support (e.g. SF16-FMI) but the new ISA radio framework has no PnP support. I got AOpen FX-3D/Pro Radio card which is AD1816 with Gemtek radio - and with PnP. But radio-gemtek fails to load because the radio I/O port is not enabled (and the driver