Bug 103046

Summary:	Netcell's "Revolution" RAID card support patch should be included
Product:	[openSUSE] SUSE LINUX 10.0	Reporter:	Vince Negri <vnegri>
Component:	Kernel	Assignee:	Hubert Mantel <mantel>
Status:	VERIFIED FIXED	QA Contact:	E-mail List <qa-bugs>
Severity:	Enhancement
Priority:	P5 - None
Version:	Beta 1
Target Milestone:	---
Hardware:	i386
OS:	SUSE Other
URL:	http://lkml.org/lkml/2005/7/16/45
Whiteboard:
Found By:	Beta-Customer	Services Priority:
Business Priority:		Blocker:	---
Marketing QA Status:	---	IT Deployment:	---
Attachments:	Cleaned-up patch as taken from LKML post Patch with whitespace issues fixed

Description Vince Negri 2005-08-09 14:47:25 UTC

NetCell's Matt Gillette submitted a small patch against 2.6.13-rc3 to LKML on
the 16th July 2005:

http://lkml.org/lkml/2005/7/16/45

This patch was included in the Fedora core rawhide builds as of early this month.

I have inspected the SUSE 10 beta 1 kernel sources and (unless I am mistaken)
the patch has not yet made it into the SUSE tree.

Since the patch merely adds PCI ID recognition for a device that the kernel can
already support, there is very low risk in including this patch. It would be
very useful to myself and some colleagues to have "out of the box" support for
the NetCell cards in SuSE 10.

Comment 1 Hubert Mantel 2005-08-09 15:09:01 UTC

Please submit the patch as attachment in raw, usable format. The html-ized
version from your link is unusable for us and I do not intent to type the patch
line by line.

Comment 2 Vince Negri 2005-08-09 15:18:40 UTC

Created attachment 45346 [details]
Cleaned-up patch as taken from LKML post

My apologies - I wasn't thinking.

Comment 3 Vince Negri 2005-08-10 14:08:31 UTC

The patch attached to the previous comment is against 2.6.13-rc3. 

Is this the desired base point, or should the patch be made against 2.6.12
(which is the base tarball used in the 10 beta 1 src RPM?)

Comment 4 Vince Negri 2005-08-10 14:37:21 UTC

Created attachment 45542 [details]
Patch with whitespace issues fixed

The previous patch, as taken off the list, did not apply cleanly without -l
option. I have edited the patch to fix this.

Comment 5 Hubert Mantel 2005-09-07 15:36:44 UTC

With our update to 2.6.13, we already have this ;)

Comment 6 Vince Negri 2005-09-07 15:52:32 UTC

Great! So it should be in the release candidate for me to test? :)

Comment 7 Vince Negri 2005-10-09 19:03:00 UTC

Tested and working with RC1