[PATCH 22/33] net: Drop DM_PCI check from designware driver

Ramon Fried rfried.dev at gmail.com
Tue Jul 27 06:25:28 CEST 2021


On Mon, Jul 26, 2021 at 4:38 PM Simon Glass <sjg at chromium.org> wrote:
>
> We don't need this check anymore since when PCI is enabled, driver model
> is always used.
>
> Drop it.
>
> Signed-off-by: Simon Glass <sjg at chromium.org>
> ---
>
>  drivers/net/designware.c | 22 ++++++++++------------
>  1 file changed, 10 insertions(+), 12 deletions(-)
>
> diff --git a/drivers/net/designware.c b/drivers/net/designware.c
> index 5d92257e74d..5aaac603a0e 100644
> --- a/drivers/net/designware.c
> +++ b/drivers/net/designware.c
> @@ -756,16 +756,16 @@ int designware_eth_write_hwaddr(struct udevice *dev)
>
>  static int designware_eth_bind(struct udevice *dev)
>  {
> -#ifdef CONFIG_DM_PCI
> -       static int num_cards;
> -       char name[20];
> -
> -       /* Create a unique device name for PCI type devices */
> -       if (device_is_on_pci_bus(dev)) {
> -               sprintf(name, "eth_designware#%u", num_cards++);
> -               device_set_name(dev, name);
> +       if (IS_ENABLED(CONFIG_PCI)) {
> +               static int num_cards;
> +               char name[20];
> +
> +               /* Create a unique device name for PCI type devices */
> +               if (device_is_on_pci_bus(dev)) {
> +                       sprintf(name, "eth_designware#%u", num_cards++);
> +                       device_set_name(dev, name);
> +               }
>         }
> -#endif
>
>         return 0;
>  }
> @@ -831,12 +831,11 @@ int designware_eth_probe(struct udevice *dev)
>         else
>                 reset_deassert_bulk(&reset_bulk);
>
> -#ifdef CONFIG_DM_PCI
>         /*
>          * If we are on PCI bus, either directly attached to a PCI root port,
>          * or via a PCI bridge, fill in plat before we probe the hardware.
>          */
> -       if (device_is_on_pci_bus(dev)) {
> +       if (IS_ENABLED(CONFIG_PCI) && device_is_on_pci_bus(dev)) {
>                 dm_pci_read_config32(dev, PCI_BASE_ADDRESS_0, &iobase);
>                 iobase &= PCI_BASE_ADDRESS_MEM_MASK;
>                 iobase = dm_pci_mem_to_phys(dev, iobase);
> @@ -844,7 +843,6 @@ int designware_eth_probe(struct udevice *dev)
>                 pdata->iobase = iobase;
>                 pdata->phy_interface = PHY_INTERFACE_MODE_RMII;
>         }
> -#endif
>
>         debug("%s, iobase=%x, priv=%p\n", __func__, iobase, priv);
>         ioaddr = iobase;
> --
> 2.32.0.432.gabb21c7263-goog
>
Reviewed-by: Ramon Fried <rfried.dev at gmail.com>


More information about the U-Boot mailing list