Re: [RFT 0/6] rtl8187se: Initial introduction of driver

["John W. Linville" <linville@tuxdriver.com>]

On Tue, Nov 10, 2009 at 10:25:03AM +0100, Marcel Holtmann wrote:
> Hi Larry,
> 
> > Patches to introduce a driver for the RTL8187SE PCI-based device.
> > 
> > Note: This driver is only compile-tested and will require more work
> > for it to be useful. I'm posting now to let others know what I have done.
> > 
> > I have applied for an account at kernel.org. If my request is approved, I
> > plan to establish an rtl8187se git tree.
> > 
> > Patch 1/6: Create file rtl8187se.h
> > Patch 2/6: Create file rtl8187se_dev.c
> > Patch 3/6: Create file rtl8187se_rtl8225.c
> > Patch 4/6: Create file rtl8187se_rtl8225.h
> > Patch 5/6: Modify file rtl818x.h
> > Patch 6/6: Modify Kconfig and Makefile for rtl8187se
> 
> so personally I don't think the split in a per file commit is helpful at
> all. It adds no real value to the tree. For review and working around
> the size limit of vger.kernel.org it is fine, but for the final merge I
> prefer that all files are added in one commit.

Yes, I agree.

-- 
John W. Linville		Someday the world will need a hero, and you
linville@tuxdriver.com			might be all we have.  Be ready.