random-mirrors/smaeul-u-boot
1
0
Fork 0
mirror of https://github.com/smaeul/u-boot.git synced 2025-10-14 04:46:01 +01:00
Code Issues Packages Projects Releases Wiki Activity

treewide: use dev_read_addr_*_ptr() where appropriate

Browse Source 
A follow-up to commit 842fb5de424e
("drivers: use devfdt_get_addr_size_index_ptr when cast to pointer")
and commit 320a1938b6f7
("drivers: use devfdt_get_addr_index_ptr when cast to pointer").

In addition to using the *_ptr variants of these functions where the
address is cast to a pointer, this also changes devfdt_get_addr_*() to
dev_read_addr_*() in a few places. Some variable and field types are
changed from fdt_addr_t or phys_addr_t to void* where the cast was
happening later.

This patch fixes a number of compile warnings when building a 32bit
U-Boot with CONFIG_PHYS_64BIT=y. In some places, it also fixes error
handling where the return value of dev_read_addr() etc. was checked for
NULL instead of FDT_ADDR_T_NONE.

Signed-off-by: Matthias Schiffer <matthias.schiffer@ew.tq-group.com>
Reviewed-by: Simon Glass <sjg@chromium.org>
This commit is contained in:
Matthias Schiffer 2023-09-27 15:33:34 +02:00 committed by Simon Glass
parent 7f18fb8a27
commit 5fecea171d
9 changed files with 55 additions and 59 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

16
arch/arm/mach-k3/sysfw-loader.c
View File

@ -321,7 +321,7 @@ exit:
static void *k3_sysfw_get_spi_addr(void) static void *k3_sysfw_get_spi_addr(void)
{ {
struct udevice *dev; struct udevice *dev;
fdt_addr_t addr; void *addr;
int ret; int ret;
unsigned int sf_bus = spl_spi_boot_bus(); unsigned int sf_bus = spl_spi_boot_bus();
@ -329,11 +329,11 @@ static void *k3_sysfw_get_spi_addr(void)
if (ret) if (ret)
return NULL; return NULL;
addr = dev_read_addr_index(dev, 1); addr = dev_read_addr_index_ptr(dev, 1);
if (addr == FDT_ADDR_T_NONE) if (!addr)
return NULL; return NULL;
return (void *)(addr + CONFIG_K3_SYSFW_IMAGE_SPI_OFFS); return addr + CONFIG_K3_SYSFW_IMAGE_SPI_OFFS;
} }
static void k3_sysfw_spi_copy(u32 *dst, u32 *src, size_t len) static void k3_sysfw_spi_copy(u32 *dst, u32 *src, size_t len)
@ -349,18 +349,18 @@ static void k3_sysfw_spi_copy(u32 *dst, u32 *src, size_t len)
static void *get_sysfw_hf_addr(void) static void *get_sysfw_hf_addr(void)
{ {
struct udevice *dev; struct udevice *dev;
fdt_addr_t addr; void *addr;
int ret; int ret;
ret = uclass_find_first_device(UCLASS_MTD, &dev); ret = uclass_find_first_device(UCLASS_MTD, &dev);
if (ret) if (ret)
return NULL; return NULL;
addr = dev_read_addr_index(dev, 1); addr = dev_read_addr_index_ptr(dev, 1);
if (addr == FDT_ADDR_T_NONE) if (!addr)
return NULL; return NULL;
return (void *)(addr + CONFIG_K3_SYSFW_IMAGE_SPI_OFFS); return addr + CONFIG_K3_SYSFW_IMAGE_SPI_OFFS;
} }
#endif #endif

5
drivers/dma/ti/k3-udma.c
View File

@ -1287,7 +1287,7 @@ static int udma_get_mmrs(struct udevice *dev)
u32 cap2, cap3, cap4; u32 cap2, cap3, cap4;
int i; int i;
ud->mmrs[MMR_GCFG] = (uint32_t *)devfdt_get_addr_name(dev, mmr_names[MMR_GCFG]); ud->mmrs[MMR_GCFG] = dev_read_addr_name_ptr(dev, mmr_names[MMR_GCFG]);
if (!ud->mmrs[MMR_GCFG]) if (!ud->mmrs[MMR_GCFG])
return -EINVAL; return -EINVAL;
@ -1325,8 +1325,7 @@ static int udma_get_mmrs(struct udevice *dev)
if (i == MMR_RCHANRT && ud->rchan_cnt == 0) if (i == MMR_RCHANRT && ud->rchan_cnt == 0)
continue; continue;
ud->mmrs[i] = (uint32_t *)devfdt_get_addr_name(dev, ud->mmrs[i] = dev_read_addr_name_ptr(dev, mmr_names[i]);
mmr_names[i]);
if (!ud->mmrs[i]) if (!ud->mmrs[i])
return -EINVAL; return -EINVAL;
} }

4
drivers/gpio/tegra186_gpio.c
View File

@ -176,8 +176,8 @@ static int tegra186_gpio_bind(struct udevice *parent)
if (parent_plat) if (parent_plat)
return 0; return 0;
regs = (uint32_t *)devfdt_get_addr_name(parent, "gpio"); regs = dev_read_addr_name_ptr(parent, "gpio");
if (regs == (uint32_t *)FDT_ADDR_T_NONE) if (!regs)
return -EINVAL; return -EINVAL;
for (port = 0; port < ctlr_data->port_count; port++) { for (port = 0; port < ctlr_data->port_count; port++) {

18
drivers/mailbox/k3-sec-proxy.c
View File

@ -84,9 +84,9 @@ struct k3_sec_proxy_mbox {
struct mbox_chan chan; struct mbox_chan chan;
struct k3_sec_proxy_desc *desc; struct k3_sec_proxy_desc *desc;
struct k3_sec_proxy_thread *chans; struct k3_sec_proxy_thread *chans;
phys_addr_t target_data; void *target_data;
phys_addr_t scfg; void *scfg;
phys_addr_t rt; void *rt;
}; };
static inline u32 sp_readl(void __iomem *addr, unsigned int offset) static inline u32 sp_readl(void __iomem *addr, unsigned int offset)
@ -319,20 +319,20 @@ static int k3_sec_proxy_of_to_priv(struct udevice *dev,
return -ENODEV; return -ENODEV;
} }
spm->target_data = devfdt_get_addr_name(dev, "target_data"); spm->target_data = dev_read_addr_name_ptr(dev, "target_data");
if (spm->target_data == FDT_ADDR_T_NONE) { if (!spm->target_data) {
dev_err(dev, "No reg property for target data base\n"); dev_err(dev, "No reg property for target data base\n");
return -EINVAL; return -EINVAL;
} }
spm->scfg = devfdt_get_addr_name(dev, "scfg"); spm->scfg = dev_read_addr_name_ptr(dev, "scfg");
if (spm->scfg == FDT_ADDR_T_NONE) { if (!spm->scfg) {
dev_err(dev, "No reg property for Secure Cfg base\n"); dev_err(dev, "No reg property for Secure Cfg base\n");
return -EINVAL; return -EINVAL;
} }
spm->rt = devfdt_get_addr_name(dev, "rt"); spm->rt = dev_read_addr_name_ptr(dev, "rt");
if (spm->rt == FDT_ADDR_T_NONE) { if (!spm->rt) {
dev_err(dev, "No reg property for Real Time Cfg base\n"); dev_err(dev, "No reg property for Real Time Cfg base\n");
return -EINVAL; return -EINVAL;
} }

12
drivers/phy/allwinner/phy-sun4i-usb.c
View File

@ -472,9 +472,9 @@ static int sun4i_usb_phy_probe(struct udevice *dev)
if (!data->cfg) if (!data->cfg)
return -EINVAL; return -EINVAL;
data->base = (void __iomem *)devfdt_get_addr_name(dev, "phy_ctrl"); data->base = (void __iomem *)dev_read_addr_name_ptr(dev, "phy_ctrl");
if (IS_ERR(data->base)) if (!data->base)
return PTR_ERR(data->base); return -EINVAL;
device_get_supply_regulator(dev, "usb0_vbus_power-supply", device_get_supply_regulator(dev, "usb0_vbus_power-supply",
&data->vbus_power_supply); &data->vbus_power_supply);
@ -555,9 +555,9 @@ static int sun4i_usb_phy_probe(struct udevice *dev)
if (i || data->cfg->phy0_dual_route) { if (i || data->cfg->phy0_dual_route) {
snprintf(name, sizeof(name), "pmu%d", i); snprintf(name, sizeof(name), "pmu%d", i);
phy->pmu = (void __iomem *)devfdt_get_addr_name(dev, name); phy->pmu = (void __iomem *)dev_read_addr_name_ptr(dev, name);
if (IS_ERR(phy->pmu)) if (!phy->pmu)
return PTR_ERR(phy->pmu); return -EINVAL;
} }
phy->id = i; phy->id = i;

4
drivers/phy/phy-bcm-sr-pcie.c
View File

@ -143,8 +143,8 @@ static int sr_pcie_phy_probe(struct udevice *dev)
core->dev = dev; core->dev = dev;
core->base = (void __iomem *)devfdt_get_addr_name(dev, "reg_base"); core->base = (void __iomem *)dev_read_addr_name_ptr(dev, "reg_base");
core->cdru = (void __iomem *)devfdt_get_addr_name(dev, "cdru_base"); core->cdru = (void __iomem *)dev_read_addr_name_ptr(dev, "cdru_base");
debug("ip base %p\n", core->base); debug("ip base %p\n", core->base);
debug("cdru base %p\n", core->cdru); debug("cdru base %p\n", core->cdru);

20
drivers/ram/k3-am654-ddrss.c
View File

@ -903,7 +903,7 @@ static int am654_ddrss_power_on(struct am654_ddrss_desc *ddrss)
static int am654_ddrss_ofdata_to_priv(struct udevice *dev) static int am654_ddrss_ofdata_to_priv(struct udevice *dev)
{ {
struct am654_ddrss_desc *ddrss = dev_get_priv(dev); struct am654_ddrss_desc *ddrss = dev_get_priv(dev);
phys_addr_t reg; void *reg;
int ret; int ret;
debug("%s(dev=%p)\n", __func__, dev); debug("%s(dev=%p)\n", __func__, dev);
@ -926,26 +926,26 @@ static int am654_ddrss_ofdata_to_priv(struct udevice *dev)
return ret; return ret;
} }
reg = devfdt_get_addr_name(dev, "ss"); reg = dev_read_addr_name_ptr(dev, "ss");
if (reg == FDT_ADDR_T_NONE) { if (!reg) {
dev_err(dev, "No reg property for DDRSS wrapper logic\n"); dev_err(dev, "No reg property for DDRSS wrapper logic\n");
return -EINVAL; return -EINVAL;
} }
ddrss->ddrss_ss_cfg = (void *)reg; ddrss->ddrss_ss_cfg = reg;
reg = devfdt_get_addr_name(dev, "ctl"); reg = dev_read_addr_name_ptr(dev, "ctl");
if (reg == FDT_ADDR_T_NONE) { if (!reg) {
dev_err(dev, "No reg property for Controller region\n"); dev_err(dev, "No reg property for Controller region\n");
return -EINVAL; return -EINVAL;
} }
ddrss->ddrss_ctl_cfg = (void *)reg; ddrss->ddrss_ctl_cfg = reg;
reg = devfdt_get_addr_name(dev, "phy"); reg = dev_read_addr_name_ptr(dev, "phy");
if (reg == FDT_ADDR_T_NONE) { if (!reg) {
dev_err(dev, "No reg property for PHY region\n"); dev_err(dev, "No reg property for PHY region\n");
return -EINVAL; return -EINVAL;
} }
ddrss->ddrss_phy_cfg = (void *)reg; ddrss->ddrss_phy_cfg = reg;
ret = dev_read_u32_array(dev, "ti,ss-reg", ret = dev_read_u32_array(dev, "ti,ss-reg",
(u32 *)&ddrss->params.ss_reg, (u32 *)&ddrss->params.ss_reg,

23
drivers/ram/k3-ddrss/k3-ddrss.c
View File

@ -331,32 +331,29 @@ static int k3_ddrss_ofdata_to_priv(struct udevice *dev)
{ {
struct k3_ddrss_desc *ddrss = dev_get_priv(dev); struct k3_ddrss_desc *ddrss = dev_get_priv(dev);
struct k3_ddrss_data *ddrss_data = (struct k3_ddrss_data *)dev_get_driver_data(dev); struct k3_ddrss_data *ddrss_data = (struct k3_ddrss_data *)dev_get_driver_data(dev);
phys_addr_t reg; void *reg;
int ret; int ret;
debug("%s(dev=%p)\n", __func__, dev); debug("%s(dev=%p)\n", __func__, dev);
reg = dev_read_addr_name(dev, "cfg"); reg = dev_read_addr_name_ptr(dev, "cfg");
if (reg == FDT_ADDR_T_NONE) { if (!reg) {
dev_err(dev, "No reg property for DDRSS wrapper logic\n"); dev_err(dev, "No reg property for DDRSS wrapper logic\n");
return -EINVAL; return -EINVAL;
} }
ddrss->ddrss_ctl_cfg = (void *)reg; ddrss->ddrss_ctl_cfg = reg;
reg = dev_read_addr_name(dev, "ctrl_mmr_lp4"); reg = dev_read_addr_name_ptr(dev, "ctrl_mmr_lp4");
if (reg == FDT_ADDR_T_NONE) { if (!reg) {
dev_err(dev, "No reg property for CTRL MMR\n"); dev_err(dev, "No reg property for CTRL MMR\n");
return -EINVAL; return -EINVAL;
} }
ddrss->ddrss_ctrl_mmr = (void *)reg; ddrss->ddrss_ctrl_mmr = reg;
reg = dev_read_addr_name(dev, "ss_cfg"); reg = dev_read_addr_name_ptr(dev, "ss_cfg");
if (reg == FDT_ADDR_T_NONE) { if (!reg)
dev_dbg(dev, "No reg property for SS Config region, but this is optional so continuing.\n"); dev_dbg(dev, "No reg property for SS Config region, but this is optional so continuing.\n");
ddrss->ddrss_ss_cfg = NULL; ddrss->ddrss_ss_cfg = reg;
} else {
ddrss->ddrss_ss_cfg = (void *)reg;
}
ret = power_domain_get_by_index(dev, &ddrss->ddrcfg_pwrdmn, 0); ret = power_domain_get_by_index(dev, &ddrss->ddrcfg_pwrdmn, 0);
if (ret) { if (ret) {

12
drivers/soc/ti/k3-navss-ringacc.c
View File

@ -987,10 +987,10 @@ static int k3_nav_ringacc_init(struct udevice *dev, struct k3_nav_ringacc *ringa
if (!base_cfg) if (!base_cfg)
return -EINVAL; return -EINVAL;
base_rt = (uint32_t *)devfdt_get_addr_name(dev, "rt"); base_rt = dev_read_addr_name_ptr(dev, "rt");
pr_debug("rt %p\n", base_rt); pr_debug("rt %p\n", base_rt);
if (IS_ERR(base_rt)) if (!base_rt)
return PTR_ERR(base_rt); return -EINVAL;
ringacc->rings = devm_kzalloc(dev, ringacc->rings = devm_kzalloc(dev,
sizeof(*ringacc->rings) * sizeof(*ringacc->rings) *
@ -1045,9 +1045,9 @@ struct k3_nav_ringacc *k3_ringacc_dmarings_init(struct udevice *dev,
ringacc->tisci = data->tisci; ringacc->tisci = data->tisci;
ringacc->tisci_dev_id = data->tisci_dev_id; ringacc->tisci_dev_id = data->tisci_dev_id;
base_rt = (uint32_t *)devfdt_get_addr_name(dev, "ringrt"); base_rt = dev_read_addr_name_ptr(dev, "ringrt");
if (IS_ERR(base_rt)) if (!base_rt)
return base_rt; return ERR_PTR(-EINVAL);
ringacc->rings = devm_kzalloc(dev, ringacc->rings = devm_kzalloc(dev,
sizeof(*ringacc->rings) * sizeof(*ringacc->rings) *