2009-11-09 23:27:34

by Larry Finger

[permalink] [raw]
Subject: [RFT 0/6] rtl8187se: Initial introduction of driver

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

Signed-off-by: Larry Finger <[email protected]>
---



2009-11-10 15:39:31

by Larry Finger

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

On 11/10/2009 07:34 AM, John W. Linville wrote:
> 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.

No problem. In fact, there is one gigantic patch on my system that I split
before the send. I will keep them split while the review/testing phase is
happening, then combine them for final submission.

Larry


2009-11-10 09:25:05

by Marcel Holtmann

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

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.

Regards

Marcel



2009-11-10 13:45:16

by John W. Linville

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

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
[email protected] might be all we have. Be ready.