On 01/08/2011 11:01 AM, Sergei Shtylyov wrote:
Changes from the previous version: - fixed several cases of a line over 80 chars that checkpatch.pl didn't report.
And for pata_hpt366 you also wrote:
Changes from the previous version: - fixed one case of a line over 80 chars that checkpatch.pl didn't report.
These will need to be on top of your patches applied last night (and just sent to Linus).
FWIW, it is ok to combine such changes into a single patch. You don't have to separate our pata_hpt37x and pata_hpt366 coding style cleanups. The main goal with separate patches is separating classes of changes, so that human reviewers and 'git bisect' may notice breakage at a useful, fine-grained level. Separating patches at the driver boundary is less useful from that perspective (though permissible, if that is your preferred method of working).
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
