Re: [PATCH] Marvell 6440 SAS/SATA driver

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Ke Wei wrote:
Added support for hotplug and wide port.


Signed-off-by: Ke Wei <kewei@xxxxxxxxxxx>
---
 drivers/scsi/mvsas.c |  445 ++++++++++++++++++++++++++++++++++++++------------
 1 files changed, 339 insertions(+), 106 deletions(-)

Technically speaking, everything is looking great so far.

We need to correct one process problem, and we should be able to get your work into the 'mvsas' branch of scsi-misc-2.6.git... and soon thereafter hopefully into the official upstream kernel.

The process problem is... when making revisions, we need to get the full patch each time. In this case, each patch should be diff'd against the current version in the 'mvsas' branch: version 0.1.

The succession of emails would look like this:

Email #1:

	Subject: [PATCH] mvsas: make it work

	Convert the skeleton mvsas driver into a real, working
	driver.  Currently, the following works:

	SAS, SAS expanders, SAS wide ports
	SATA devices, SATAPI

	Signed-off-by: Ke Wei <kewei@xxxxxxxxxxx>


And then James, myself, other reviewers reply. Or, you add some new features like hotplugging.

Each time, you must regenerate a full patch against the most git repository revision:

	If a previous patch of yours, version "0.2", has been applied
	to git, then you would create your patch against version 0.2.

	If a previous patch of yours has NOT yet been applied to git,
	then you would create your patch against version 0.1.

Thus, in this case, version 0.1 is in 'mvsas' branch, so your second email would then be

Email #2:

	Subject: [PATCH v2] mvsas: make it work

	Convert the skeleton mvsas driver into a real, working
	driver.  Currently, the following works:

	SAS, SAS expanders, SAS wide ports
	SATA devices, SATAPI

	Signed-off-by: Ke Wei <kewei@xxxxxxxxxxx>
	---
	Changes since last posting (version 0.2):
	- fix coding problem


And then you continue your work, and add another revision while everyone else is sleeping, the third email would look like:

Email #3:

	Subject: [PATCH v3] mvsas: make it work

	Convert the skeleton mvsas driver into a real, working
	driver.  Currently, the following works:

	SAS, SAS expanders, SAS wide ports
	SATA devices, SATAPI

	Signed-off-by: Ke Wei <kewei@xxxxxxxxxxx>
	---
	Changes since last posting (version 0.3):
	- add hotplug support
	- add wide port support

	Changes since last posting (version 0.2):
	- fix coding problem


Thus, you always create a patch against the most recent source code in the git repository.

It is common for patches to go through a few revisions on the mailing list, before it is applied to the git repository.

So anyway... send a patch against the latest #mvsas (version 0.1), and that patch should go in rapidly.

Thanks!  And keep up the good work,

	Jeff



-
To unsubscribe from this list: send the line "unsubscribe linux-scsi" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[Index of Archives]     [SCSI Target Devel]     [Linux SCSI Target Infrastructure]     [Kernel Newbies]     [IDE]     [Security]     [Git]     [Netfilter]     [Bugtraq]     [Yosemite News]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Linux ATA RAID]     [Linux IIO]     [Samba]     [Device Mapper]
  Powered by Linux