Re: [PATCH 4/4] imx: add PCI fixup for PEX860X on Gateworks board

[Shawn Guo <shawn.guo-QSEj5FYQhm4dnm+yROfE0A@public.gmane.org>]

On Tue, Oct 22, 2013 at 09:51:28PM -0700, Tim Harvey wrote:

<snip>

> @@ -78,6 +80,37 @@ static int ksz9031rn_phy_fixup(struct phy_device *dev)
>  	return 0;
>  }
>  
> +/*
> + * fixup for PLX PEX8909 bridge to configure GPIO1-7 as output High
> + * as they are used for slots1-7 PERST#
> + */
> +static void mx6_ventana_pciesw_early_fixup(struct pci_dev *dev)

Please use 'imx' instead of 'mx' where you need an imx prefix.  In this
case, it's even fine without the prefix.  So I just drop the 'mx6_' from
function name, and applied the patch.  Thanks.

Shawn

> +{
> +	u32 dw;
> +
> +	if (!of_machine_is_compatible("gw,ventana"))
> +		return;
> +
> +	if (dev->devfn != 0)
> +		return;
> +
> +	pci_read_config_dword(dev, 0x62c, &dw);
> +	dw |= 0xaaa8; // GPIO1-7 outputs
> +	pci_write_config_dword(dev, 0x62c, dw);
> +
> +	pci_read_config_dword(dev, 0x644, &dw);
> +	dw |= 0xfe;   // GPIO1-7 output high
> +	pci_write_config_dword(dev, 0x644, dw);
> +
> +	msleep(100);
> +}
> +DECLARE_PCI_FIXUP_EARLY(PCI_VENDOR_ID_PLX, 0x8609,
> +	mx6_ventana_pciesw_early_fixup);
> +DECLARE_PCI_FIXUP_EARLY(PCI_VENDOR_ID_PLX, 0x8606,
> +	mx6_ventana_pciesw_early_fixup);
> +DECLARE_PCI_FIXUP_EARLY(PCI_VENDOR_ID_PLX, 0x8604,
> +	mx6_ventana_pciesw_early_fixup);
> +
>  static int ar8031_phy_fixup(struct phy_device *dev)
>  {
>  	u16 val;
> -- 
> 1.7.9.5
> 

--
To unsubscribe from this list: send the line "unsubscribe devicetree" in
the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html