On Wed, Jan 4, 2017 at 1:48 AM, Andy Shevchenko <[email protected]> wrote: > On Wed, Jan 4, 2017 at 1:40 AM, Guenter Roeck <[email protected]> wrote: >> On Wed, Jan 04, 2017 at 12:39:59AM +0200, Andy Shevchenko wrote: >>> On Tue, Jan 3, 2017 at 4:39 PM, Guenter Roeck <[email protected]> wrote: >>> > Use pdev for struct platform_device, pcidev for struct pci_dev, and dev >>> > for struct device variables to improve consistency. >>> > >>> > Remove 'struct platform_device *dev;' from struct iTCO_wdt_private since >>> > it was unused. >>> >>> Would pci_dev work? >>> >> Sure, or maybe just 'pci'. Any preference ? > > Just slightly prefer pci_dev over others (pcidev, pci), but do not > insist. Up to you.
Or even struct device *dev; ... struct pci_dev *pci_dev = to_pci_dev(dev); in places where it's guaranteed to be PCI. -- With Best Regards, Andy Shevchenko

