On 12/20/2010 07:01 PM, Hauke Mehrtens wrote: > Use function pci_pcie_cap() instead of accessing struct member directly. > > CC: Larry Finger <Larry.Finger@xxxxxxxxxxxx> > Signed-off-by: Hauke Mehrtens <hauke@xxxxxxxxxx> > --- Is this change really necessary? The inline routine expands exactly to the code that you replaced. The change just adds one extra layer of obscurity. If this really is a desirable change, then add Tested-by: Larry Finger <Larry.Finger@xxxxxxxxxxxx> Larry -- To unsubscribe from this list: send the line "unsubscribe linux-wireless" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html