>> + if (host->ops && host->ops->phb_prepare) {
>> + error = host->ops->phb_prepare(host);
>> + if(error) {
>
> Whitespace error.
Will fix.
>
>> --- a/include/linux/pci.h
>> +++ b/include/linux/pci.h
>> @@ -400,6 +400,13 @@ static inline int pci_channel_offline(struct pci_dev
>> *pdev)
>> return (pdev->error_state != pci_channel_io_normal);
>> }
>>
>> +struct pci_host_bridge;
>> +struct pci_host_bridge_ops {
>> + void (*phb_set_root_bus_speed)(struct pci_host_bridge *host);
>> + int (*phb_prepare)(struct pci_host_bridge *host);
>> + void (*phb_of_scan_bus)(struct pci_host_bridge *);
>
> These function pointers don't need a "phb_" prefix.
Will update them.
>
> .
>
--
Thanks!
Yijing
--
To unsubscribe from this list: send the line "unsubscribe linux-m68k" in
the body of a message to [email protected]
More majordomo info at http://vger.kernel.org/majordomo-info.html
