On Fri, 7 Aug 2009 11:30:02 -0600 Matthew Wilcox wrote: > On Thu, Aug 06, 2009 at 11:36:52AM +0900, Hidetoshi Seto wrote: > > Cleanups (nearly based on checkpatch). > > > > Before: total: 11 errors, 2 warnings, 0 checks, 842 lines checked > > After: total: 0 errors, 0 warnings, 0 checks, 842 lines checked > > > > Signed-off-by: Hidetoshi Seto <seto.hidetoshi@xxxxxxxxxxxxxx> > > > @@ -601,7 +600,8 @@ static int pci_msi_check_device(struct pci_dev* dev, int nvec, int type) > > if (nvec < 1) > > return -ERANGE; > > > > - /* Any bridge which does NOT route MSI transactions from it's > > + /* > > + * Any bridge which does NOT route MSI transactions from it's > > * secondary bus to it's primary bus must set NO_MSI flag on > > * the secondary pci_bus. > > * We expect only arch-specific PCI host bus controller driver > > While you're touching this comment, could you fix the it's/its mistake too? > > For native english speakers, the following rule sometimes helps: "its" > is possessive, like "his" and "hers". You wouldn't write hi's or her's, > so don't write it's. OTOH, "it's" means "it is" -- not possessive at all. > Non-native english speakers don't tend to get this one wrong. Probably > because they actually learn english instead of just using it ;-) > > Otherwise, Reviewed-by: Matthew Wilcox <willy@xxxxxxxxxxxxxxx> --- ~Randy LPC 2009, Sept. 23-25, Portland, Oregon http://linuxplumbersconf.org/2009/ -- To unsubscribe from this list: send the line "unsubscribe linux-pci" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
