* Hiroshi Doyu wrote:
> diff --git a/drivers/of/of_dma.c b/drivers/of/of_dma.c
> new file mode 100644
> index 0000000..45c9e88
> --- /dev/null
> +++ b/drivers/of/of_dma.c
> @@ -0,0 +1,35 @@
> +/*
> + * Stolen from:
> + * "arch/microblaze/kernel/prom_parse.c"
> + * "arch/powerpc/kernel/prom_parse.c"
> + */
> +
> +#include <linux/of_address.h>
> +
> +void of_parse_dma_window(struct device_node *dn, const void *dma_window_prop,
> + unsigned long *busno, unsigned long *phys, unsigned long *size)
> +{
> + const __be32 *dma_window;
> + u32 cells;
> + const unsigned char *prop;There's no need for this to be const unsigned char *, const void * will do just as well. > + > + dma_window = dma_window_prop; > + > + /* busno is always one cell */ > + if (busno) > + *busno = be32_to_cpup(dma_window++); > + > + prop = of_get_property(dn, "#dma-address-cells", NULL); > + if (!prop) > + prop = of_get_property(dn, "#address-cells", NULL); > + > + cells = prop ? be32_to_cpup(prop) : of_n_addr_cells(dn); > + *phys = of_read_number(dma_window, cells); This should probably fail gracefully if phys == NULL, similar to what you do for busno. > + > + dma_window += cells; > + > + prop = of_get_property(dn, "#dma-size-cells", NULL); > + cells = prop ? be32_to_cpup(prop) : of_n_size_cells(dn); > + *size = of_read_number(dma_window, cells); Same here. > +} > + And you might want to add a EXPORT_SYMBOL(of_parse_dma_window) here so the function can be used from modules. Sorry for having you go through another round. I should have looked more closely before. Thierry
pgpEbjPP8OWKO.pgp
Description: PGP signature
_______________________________________________ devicetree-discuss mailing list devicetree-discuss@lists.ozlabs.org https://lists.ozlabs.org/listinfo/devicetree-discuss
