Re: [PATCH] [CM4000,CM4040] Add device class bits to enable udevdevice creation

From: Andrew Morton
Date: Tue Feb 14 2006 - 03:15:27 EST


Harald Welte <laforge@xxxxxxxxxxxx> wrote:
>
> On Tue, Jan 31, 2006 at 06:09:27PM -0800, Andrew Morton wrote:
> > Harald Welte <laforge@xxxxxxxxxxxx> wrote:
> > >
> > > Please apply this fix to the cm4000/4040 drivers, thanks!
> > >
> > > [CM4000,CM4040] Add device class bits to enable udev device creation
> > >
> > > Using this patch, Omnikey CardMan 4000 and 4040 devices automatically
> > > get their device nodes created by udev.
> >
> > Dominik has made quite widespread changes to these drivers - enough that
> > I'm not confident to fix the rejects, make it compile and hope that it
> > still works.
>
> sorry for that. I honestly don't have the time to track two trees, and
> I do all my development work against Linus' main tree, therefore my
> patches are against that tree, too.

Dominik maintains the pcmcia development tree, and these are
pcmcia drivers...

> > So can you please sort things out with Dominik? I guess a tested patch
> > against -mm4 would be ideal.
>
> The question is: Why wouldn't my patch directly go mainline but rather
> via -mm? It is a very special-purpose device, the number of users are
> small, it clearly fixes the bug that no device nodes are created, and
> the fix came from the original maintainer.

pcmcia patches should preferably go via the pcmcia development tree, and
get integrated and reviewed by the maintainer and all that good stuff.

So it wouldn't go via -mm at all - pcmcia only goes into -mm for testing.


The more specific point is that your patch conflicted with Dominik's
current development tree in a big way. That needs to get sorted out.
-
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/