On Wed, Aug 21, 2024 at 08:29:07PM +0530, Siddharth Vadapalli wrote:

> Add support for the Cadence PCIe Controller present on TI's K3 SoCs.
> This driver is an adaptation of the Linux driver.
> 
> Signed-off-by: Siddharth Vadapalli <s-vadapa...@ti.com>
[snip]
> diff --git a/drivers/pci/pcie_cdns_ti.c b/drivers/pci/pcie_cdns_ti.c
> new file mode 100644
> index 0000000000..2b9bba19b4
> --- /dev/null
> +++ b/drivers/pci/pcie_cdns_ti.c
> @@ -0,0 +1,645 @@
> +// SPDX-License-Identifier: GPL-2.0-only OR MIT
> +/*
> + * Copyright (C) 2024 Texas Instruments Incorporated - https://www.ti.com
> + *
> + * PCIe controller driver for TI's K3 SoCs with Cadence PCIe controller
> + *
> + * Ported from the Linux driver - drivers/pci/controller/cadence/pci-j721e.c
> + *
> + * Author: Siddharth Vadapalli <s-vadapa...@ti.com>
> + *
> + */
> +
> +#include <asm/global_data.h>
> +#include <asm/gpio.h>
> +#include <clk-uclass.h>
> +#include <dm.h>
> +#include <dm/device_compat.h>
> +#include <generic-phy.h>
> +#include <linux/bitops.h>
> +#include <linux/delay.h>
> +#include <linux/err.h>
> +#include <linux/io.h>
> +#include <linux/log2.h>
> +#include <log.h>
> +#include <pci.h>
> +#include <power-domain.h>
> +#include <regmap.h>
> +#include <syscon.h>

Please audit this list to make sure you need everything because..

> +
> +DECLARE_GLOBAL_DATA_PTR;

You don't reference gd in this code at all, so don't need this.

So please also make sure the giant list of defines is needed.
> +#define usleep_range(a, b) udelay((b))

Per checkpatch.pl.conf:
# Not Linux, so we don't recommend usleep_range() over udelay()

-- 
Tom

Attachment: signature.asc
Description: PGP signature

Reply via email to