Re: [PATCH 1/2] Staging: comedi: add dyna_pci10xx driver

From: Prashant Shah
Date: Thu Jun 09 2011 - 07:41:22 EST


Hi Ian,

On Thu, Jun 9, 2011 at 4:34 PM, Ian Abbott <abbotti@xxxxxxxxx> wrote:
> On 09/06/11 10:24, Prashant P. Shah wrote:

> It looks mostly okay to me as long as you got the PCI device ID
> allocated by PLX (which they'll happily do for their customers if you
> tell them what product the chip is is part of), and didn't just pull it
> out of a hat!

I will forward that to the vendor...

> I have a few more specific comments on the code (not many!):
> Best regards,
> Ian.

fixed and sent patch for the same.

Regards.
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/