linux-kernel - Re: [PATCH] stmmac: pci: Add support for LS7A bridge chip

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <51aeec81-400f-c63b-3a22-e5384ff848f8@flygoat.com>
Date:   Sat, 11 Jul 2020 09:35:05 +0800
From:   Jiaxun Yang <jiaxun.yang@...goat.com>
To:     Zhi Li <lizhi01@...ngson.cn>, peppe.cavallaro@...com,
        alexandre.torgue@...com, joabreu@...opsys.com, davem@...emloft.net,
        kuba@...nel.org, mcoquelin.stm32@...il.com
Cc:     netdev@...r.kernel.org, linux-kernel@...r.kernel.org,
        lixuefeng@...ngson.cn, chenhc@...ote.com, yangtiezhu@...ngson.cn,
        Hongbin Li <lihongbin@...ngson.cn>
Subject: Re: [PATCH] stmmac: pci: Add support for LS7A bridge chip



在 2020/7/10 16:51, Zhi Li 写道:
> Add gmac platform data to support LS7A bridge chip.
>
> Co-developed-by: Hongbin Li <lihongbin@...ngson.cn>
> Signed-off-by: Hongbin Li <lihongbin@...ngson.cn>
> Signed-off-by: Zhi Li <lizhi01@...ngson.cn>
> ---
>   drivers/net/ethernet/stmicro/stmmac/stmmac_pci.c | 22 ++++++++++++++++++++++
>   1 file changed, 22 insertions(+)
>
> diff --git a/drivers/net/ethernet/stmicro/stmmac/stmmac_pci.c b/drivers/net/ethernet/stmicro/stmmac/stmmac_pci.c
> index 272cb47..dab2a40 100644
> --- a/drivers/net/ethernet/stmicro/stmmac/stmmac_pci.c
> +++ b/drivers/net/ethernet/stmicro/stmmac/stmmac_pci.c
> @@ -138,6 +138,24 @@ static const struct stmmac_pci_info snps_gmac5_pci_info = {
>   	.setup = snps_gmac5_default_data,
>   };
>   
> +static int loongson_default_data(struct pci_dev *pdev, struct plat_stmmacenent_data *plat)
> +{
> +	common_default_data(plat);
> +
> +	plat->bus_id = pci_dev_id(pdev);
> +	plat->phy_addr = 0;
> +	plat->interface = PHY_INTERFACE_MODE_GMII;
> +
> +	plat->dma_cfg->pbl = 32;
> +	plat->dma_cfg->pblx8 = true;
> +
> +	return 0;
> +}
> +
> +static struct stmmac_pci_info loongson_pci_info = {
> +	.setup = loongson_default_data;
> +};
> +
>   /**
>    * stmmac_pci_probe
>    *
> @@ -204,6 +222,8 @@ static int stmmac_pci_probe(struct pci_dev *pdev,
>   	res.addr = pcim_iomap_table(pdev)[i];
>   	res.wol_irq = pdev->irq;
>   	res.irq = pdev->irq;
> +	if (pdev->vendor == PCI_VENDOR_ID_LOONGSON)
> +		res.lpi_irq = pdev->irq + 1;

This can never work.
We're allocating IRQs by irq_domain, not ID.
Please describe IRQ in DeviceTree, and *DO NOT* sne dout untested patch.

Thanks.

>   
>   	return stmmac_dvr_probe(&pdev->dev, plat, &res);
>   }
> @@ -273,11 +293,13 @@ static SIMPLE_DEV_PM_OPS(stmmac_pm_ops, stmmac_pci_suspend, stmmac_pci_resume);
>   
>   #define PCI_DEVICE_ID_STMMAC_STMMAC		0x1108
>   #define PCI_DEVICE_ID_SYNOPSYS_GMAC5_ID		0x7102
> +#define PCI_DEVICE_ID_LOONGSON_GMAC		0x7a03
>   
>   static const struct pci_device_id stmmac_id_table[] = {
>   	{ PCI_DEVICE_DATA(STMMAC, STMMAC, &stmmac_pci_info) },
>   	{ PCI_DEVICE_DATA(STMICRO, MAC, &stmmac_pci_info) },
>   	{ PCI_DEVICE_DATA(SYNOPSYS, GMAC5_ID, &snps_gmac5_pci_info) },
> +	{ PCI_DEVICE_DATA(LOONGSON, GMAC, &loongson_pci_info) },
>   	{}
>   };
>   
- Jiaxun