Re: [PATCH 05/09] sata_mv don't blindly enable IRQs

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

 



Mark Lord wrote:
These five small patches are on top of the four previously posted,
so I'm extending the numbering to be xx/09 now:

-----snip-----

Part one of simplifying/fixing handling of the main_irq_mask register
to resolve unexpected interrupt issues observed in 2.6.26-rc*.

Don't blindly enable port IRQs at host init time.
Instead, enable only the bits that we want,
which in this case is simply the PCI_ERR bit.

The per-port bits can wait until the ports are reset/probed for devices.

Signed-off-by: Mark Lord <mlord@xxxxxxxxx>

Two process notes for future use:

1) please include a colon (":") or other punctuation after "sata_mv" prefix in subject line.

2) when adding notes like "These five small patches...", we already have a method of doing so: the "---" separator. The format of the patch body should be:


	patch description for kernel changelog

	signed-off-by: ...
	---
	These five small patches are on top of the four previously
	posted, so I'm extending the numbering to be xx/09 now.


Both of these details are documented in Documentation/SubmittingPatches and http://linux.yyz.us/patch-format.html

OTOH, your numbering behavior -- extending the patch numbering as you've done here, when the first patchset is not yet applied to git -- is quite useful and should be emulated by others :)

Tejun sometimes simply posts a new patchseries, noting that that patchseries is dependent upon a previous series. That's fine, too.

Thanks,

	Jeff



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

[Index of Archives]     [Linux Filesystems]     [Linux SCSI]     [Linux RAID]     [Git]     [Kernel Newbies]     [Linux Newbie]     [Security]     [Netfilter]     [Bugtraq]     [Yosemite News]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Samba]     [Device Mapper]

  Powered by Linux