Hi hch, thanks for your feedback.
An updated version of the patch is at the bottom of this mail.
On Mon, Sep 05, 2005 at 08:06:35PM +0100, Christoph Hellwig wrote:
> > +#include
>
> I don't think you need this one.
done
> > +#include
>
> you shouldn't need this one.
done
> > +static
Hi hch, thanks for your feedback.
An updated version of the patch is at the bottom of this mail.
On Mon, Sep 05, 2005 at 08:06:35PM +0100, Christoph Hellwig wrote:
+#include linux/version.h
I don't think you need this one.
done
+#include pcmcia/version.h
you shouldn't need this one.
On Monday 05 September 2005 22:05, Jesper Juhl wrote:
> On Monday 05 September 2005 21:54, Harald Welte wrote:
> > Hi!
> >
> > I've now incorporated all the suggested changes (thanks once again on
> > the many comments received). The resulting driver has been tested and
> > works fine.
> >
> >
On Monday 05 September 2005 21:54, Harald Welte wrote:
> Hi!
>
> I've now incorporated all the suggested changes (thanks once again on
> the many comments received). The resulting driver has been tested and
> works fine.
>
> Please consider applying the driver to the mainline tree, thanks.
>
> +#include
I don't think you need this one.
> +#include
you shouldn't need this one.
> +static atomic_t cm4040_num_devices_open;
> +
> +#ifdef PCMCIA_DEBUG
> +static int pc_debug = PCMCIA_DEBUG;
> +module_param(pc_debug, int, 0600);
> +#define DEBUG(n, x, args...) do { if (pc_debug >= (n))
Hi!
I've now incorporated all the suggested changes (thanks once again on
the many comments received). The resulting driver has been tested and
works fine.
Please consider applying the driver to the mainline tree, thanks.
Hi!
I've now incorporated all the suggested changes (thanks once again on
the many comments received). The resulting driver has been tested and
works fine.
Please consider applying the driver to the mainline tree, thanks.
+#include linux/version.h
I don't think you need this one.
+#include pcmcia/version.h
you shouldn't need this one.
+static atomic_t cm4040_num_devices_open;
+
+#ifdef PCMCIA_DEBUG
+static int pc_debug = PCMCIA_DEBUG;
+module_param(pc_debug, int, 0600);
+#define DEBUG(n, x, args...) do
On Monday 05 September 2005 21:54, Harald Welte wrote:
Hi!
I've now incorporated all the suggested changes (thanks once again on
the many comments received). The resulting driver has been tested and
works fine.
Please consider applying the driver to the mainline tree, thanks.
What did
On Monday 05 September 2005 22:05, Jesper Juhl wrote:
On Monday 05 September 2005 21:54, Harald Welte wrote:
Hi!
I've now incorporated all the suggested changes (thanks once again on
the many comments received). The resulting driver has been tested and
works fine.
Please consider
10 matches
Mail list logo