On Wed, Mar 11, 2015 at 11:06:08AM -0700, Ray Jui wrote:
> This adds the support for Broadcom iProc PCIe controller
> 
> pcie-iproc.c servers as the common core driver, and front-end bus
> interface needs to be added to support different bus interfaces
> 
> pcie-iproc-pltfm.c contains the support for the platform bus interface
> 
> Signed-off-by: Ray Jui <r...@broadcom.com>
> Reviewed-by: Scott Branden <sbran...@broadcom.com>
> ...

> +static int iproc_pcie_check_link(struct iproc_pcie *pcie, struct pci_bus 
> *bus)
> +{
> +     u8 hdr_type;
> +     u32 link_ctrl;
> +     u16 pos, link_status;
> +     int link_is_active = 0;
> +
> +     /* make sure we are not in EP mode */
> +     pci_bus_read_config_byte(bus, 0, PCI_HEADER_TYPE, &hdr_type);
> +     if ((hdr_type & 0x7f) != PCI_HEADER_TYPE_BRIDGE) {
> +             dev_err(pcie->dev, "in EP mode, hdr=0x08%x\n", hdr_type);

"0x08%x" doesn't look right.  I changed it to "%#02x"; is that what you
intended?
--
To unsubscribe from this list: send the line "unsubscribe devicetree" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Reply via email to