Message ID | 20190814081955.7122-1-j-keerthy@ti.com |
---|---|
State | Superseded |
Delegated to: | Simon Glass |
Headers | show |
Series | [U-Boot,RFT] core: of_addr: Correct the size type of of_get_address to fdt_size_t | expand |
Hi Keerthy, On Wed, Aug 14, 2019 at 10:21 AM Keerthy <j-keerthy@ti.com> wrote: > > Currently the size parameter is deined as u64 type. s/deined/defined/ > Correct the size type of of_get_address to fdt_size_t > so that both 64 bit and 32 bit architectures are taken > care of. Please, reference the initial bug report https://patchwork.ozlabs.org/patch/1090094/#2212555 and, ideally, add the "Fixes: <commit-adding-the-issue>" line. Please, add: Reported-by: Eugeniu Rosca <rosca.eugeniu@gmail.com> > Signed-off-by: Keerthy <j-keerthy@ti.com> > --- > drivers/core/of_addr.c | 4 ++-- > drivers/core/ofnode.c | 2 +- > include/dm/of_addr.h | 2 +- > 3 files changed, 4 insertions(+), 4 deletions(-) Given my test results in https://patchwork.ozlabs.org/patch/1090094/#2238022: Tested-by: Eugeniu Rosca <rosca.eugeniu@gmail.com> Thank you very much.
Superseded by https://patchwork.ozlabs.org/patch/1146933/ ("core: of_addr: Correct the size type of of_get_address to fdt_size_t")
diff --git a/drivers/core/of_addr.c b/drivers/core/of_addr.c index 4e256d9926..812a400b19 100644 --- a/drivers/core/of_addr.c +++ b/drivers/core/of_addr.c @@ -122,7 +122,7 @@ static void dev_count_cells(const struct device_node *np, int *nap, int *nsp) } const __be32 *of_get_address(const struct device_node *dev, int index, - u64 *size, unsigned int *flags) + fdt_size_t *size, unsigned int *flags) { const __be32 *prop; int psize; @@ -347,7 +347,7 @@ int of_address_to_resource(const struct device_node *dev, int index, struct resource *r) { const __be32 *addrp; - u64 size; + fdt_size_t size; unsigned int flags; const char *name = NULL; diff --git a/drivers/core/ofnode.c b/drivers/core/ofnode.c index 2ac73af934..b21b5183a3 100644 --- a/drivers/core/ofnode.c +++ b/drivers/core/ofnode.c @@ -260,7 +260,7 @@ fdt_addr_t ofnode_get_addr_size_index(ofnode node, int index, fdt_size_t *size) uint flags; prop_val = of_get_address(ofnode_to_np(node), index, - (u64 *)size, &flags); + size, &flags); if (!prop_val) return FDT_ADDR_T_NONE; diff --git a/include/dm/of_addr.h b/include/dm/of_addr.h index 3fa1ffce81..52443ef296 100644 --- a/include/dm/of_addr.h +++ b/include/dm/of_addr.h @@ -58,7 +58,7 @@ u64 of_translate_dma_address(const struct device_node *no, const __be32 *in_addr * @return pointer to address which can be read */ const __be32 *of_get_address(const struct device_node *no, int index, - u64 *size, unsigned int *flags); + fdt_size_t *size, unsigned int *flags); struct resource;
Currently the size parameter is deined as u64 type. Correct the size type of of_get_address to fdt_size_t so that both 64 bit and 32 bit architectures are taken care of. Signed-off-by: Keerthy <j-keerthy@ti.com> --- drivers/core/of_addr.c | 4 ++-- drivers/core/ofnode.c | 2 +- include/dm/of_addr.h | 2 +- 3 files changed, 4 insertions(+), 4 deletions(-)