* [U-Boot] [PATCH] dm: core: Add dev_get_addr_size_index() to retrieve addr and size
@ 2016-11-28 9:46 Stefan Roese
2016-11-30 2:35 ` Simon Glass
0 siblings, 1 reply; 3+ messages in thread
From: Stefan Roese @ 2016-11-28 9:46 UTC (permalink / raw)
To: u-boot
The currently available functions accessing the 'reg' property of a
device only retrieve the address. Sometimes its also necessary to
retrieve the size described by the 'reg' property. This patch adds
the new function dev_get_addr_size_index() which retrieves both,
the address and the size described by the 'reg' property.
Signed-off-by: Stefan Roese <sr@denx.de>
Cc: Simon Glass <sjg@chromium.org>
---
drivers/core/device.c | 22 ++++++++++++++++++++++
include/dm/device.h | 16 ++++++++++++++++
2 files changed, 38 insertions(+)
diff --git a/drivers/core/device.c b/drivers/core/device.c
index dcf5d9d..ec43654 100644
--- a/drivers/core/device.c
+++ b/drivers/core/device.c
@@ -693,6 +693,28 @@ fdt_addr_t dev_get_addr_index(struct udevice *dev, int index)
#endif
}
+fdt_addr_t dev_get_addr_size_index(struct udevice *dev, int index,
+ fdt_size_t *size)
+{
+#if CONFIG_IS_ENABLED(OF_CONTROL)
+ /*
+ * Only get the size in this first call. We'll get the addr in the
+ * next call to the exisiting dev_get_xxx function which handles
+ * all config options.
+ */
+ fdtdec_get_addr_size_auto_noparent(gd->fdt_blob, dev->of_offset,
+ "reg", 1, size, false);
+
+ /*
+ * Get the base address via the existing function which handles
+ * all Kconfig cases
+ */
+ return dev_get_addr_index(dev, index);
+#else
+ return FDT_ADDR_T_NONE;
+#endif
+}
+
fdt_addr_t dev_get_addr_name(struct udevice *dev, const char *name)
{
#if CONFIG_IS_ENABLED(OF_CONTROL)
diff --git a/include/dm/device.h b/include/dm/device.h
index babf8ac..9948bd4 100644
--- a/include/dm/device.h
+++ b/include/dm/device.h
@@ -497,6 +497,22 @@ void *dev_map_physmem(struct udevice *dev, unsigned long size);
fdt_addr_t dev_get_addr_index(struct udevice *dev, int index);
/**
+ * dev_get_addr_size_index() - Get the indexed reg property of a device
+ *
+ * Returns the address and size specified in the 'reg' property of a device.
+ *
+ * @dev: Pointer to a device
+ * @index: the 'reg' property can hold a list of <addr, size> pairs
+ * and @index is used to select which one is required
+ * @size: Pointer to size varible - this function returns the size
+ * specified in the 'reg' property here
+ *
+ * @return addr
+ */
+fdt_addr_t dev_get_addr_size_index(struct udevice *dev, int index,
+ fdt_size_t *size);
+
+/**
* dev_get_addr_name() - Get the reg property of a device, indexed by name
*
* @dev: Pointer to a device
--
2.10.2
^ permalink raw reply related [flat|nested] 3+ messages in thread
* [U-Boot] [PATCH] dm: core: Add dev_get_addr_size_index() to retrieve addr and size
2016-11-28 9:46 [U-Boot] [PATCH] dm: core: Add dev_get_addr_size_index() to retrieve addr and size Stefan Roese
@ 2016-11-30 2:35 ` Simon Glass
2016-11-30 6:15 ` Stefan Roese
0 siblings, 1 reply; 3+ messages in thread
From: Simon Glass @ 2016-11-30 2:35 UTC (permalink / raw)
To: u-boot
Hi Stefan,
On 28 November 2016 at 02:46, Stefan Roese <sr@denx.de> wrote:
> The currently available functions accessing the 'reg' property of a
> device only retrieve the address. Sometimes its also necessary to
> retrieve the size described by the 'reg' property. This patch adds
> the new function dev_get_addr_size_index() which retrieves both,
> the address and the size described by the 'reg' property.
>
> Signed-off-by: Stefan Roese <sr@denx.de>
> Cc: Simon Glass <sjg@chromium.org>
> ---
> drivers/core/device.c | 22 ++++++++++++++++++++++
> include/dm/device.h | 16 ++++++++++++++++
> 2 files changed, 38 insertions(+)
>
> diff --git a/drivers/core/device.c b/drivers/core/device.c
> index dcf5d9d..ec43654 100644
> --- a/drivers/core/device.c
> +++ b/drivers/core/device.c
> @@ -693,6 +693,28 @@ fdt_addr_t dev_get_addr_index(struct udevice *dev, int index)
> #endif
> }
>
> +fdt_addr_t dev_get_addr_size_index(struct udevice *dev, int index,
> + fdt_size_t *size)
> +{
> +#if CONFIG_IS_ENABLED(OF_CONTROL)
> + /*
> + * Only get the size in this first call. We'll get the addr in the
> + * next call to the exisiting dev_get_xxx function which handles
> + * all config options.
> + */
> + fdtdec_get_addr_size_auto_noparent(gd->fdt_blob, dev->of_offset,
> + "reg", 1, size, false);
Does this take account of 'index'?
> +
> + /*
> + * Get the base address via the existing function which handles
> + * all Kconfig cases
> + */
> + return dev_get_addr_index(dev, index);
> +#else
> + return FDT_ADDR_T_NONE;
> +#endif
> +}
> +
> fdt_addr_t dev_get_addr_name(struct udevice *dev, const char *name)
> {
> #if CONFIG_IS_ENABLED(OF_CONTROL)
> diff --git a/include/dm/device.h b/include/dm/device.h
> index babf8ac..9948bd4 100644
> --- a/include/dm/device.h
> +++ b/include/dm/device.h
> @@ -497,6 +497,22 @@ void *dev_map_physmem(struct udevice *dev, unsigned long size);
> fdt_addr_t dev_get_addr_index(struct udevice *dev, int index);
>
> /**
> + * dev_get_addr_size_index() - Get the indexed reg property of a device
> + *
> + * Returns the address and size specified in the 'reg' property of a device.
> + *
> + * @dev: Pointer to a device
> + * @index: the 'reg' property can hold a list of <addr, size> pairs
> + * and @index is used to select which one is required
> + * @size: Pointer to size varible - this function returns the size
> + * specified in the 'reg' property here
> + *
> + * @return addr
> + */
> +fdt_addr_t dev_get_addr_size_index(struct udevice *dev, int index,
> + fdt_size_t *size);
> +
> +/**
> * dev_get_addr_name() - Get the reg property of a device, indexed by name
> *
> * @dev: Pointer to a device
> --
> 2.10.2
>
Regards,
Simon
^ permalink raw reply [flat|nested] 3+ messages in thread
* [U-Boot] [PATCH] dm: core: Add dev_get_addr_size_index() to retrieve addr and size
2016-11-30 2:35 ` Simon Glass
@ 2016-11-30 6:15 ` Stefan Roese
0 siblings, 0 replies; 3+ messages in thread
From: Stefan Roese @ 2016-11-30 6:15 UTC (permalink / raw)
To: u-boot
Hi Simon,
On 30.11.2016 03:35, Simon Glass wrote:
> On 28 November 2016 at 02:46, Stefan Roese <sr@denx.de> wrote:
>> The currently available functions accessing the 'reg' property of a
>> device only retrieve the address. Sometimes its also necessary to
>> retrieve the size described by the 'reg' property. This patch adds
>> the new function dev_get_addr_size_index() which retrieves both,
>> the address and the size described by the 'reg' property.
>>
>> Signed-off-by: Stefan Roese <sr@denx.de>
>> Cc: Simon Glass <sjg@chromium.org>
>> ---
>> drivers/core/device.c | 22 ++++++++++++++++++++++
>> include/dm/device.h | 16 ++++++++++++++++
>> 2 files changed, 38 insertions(+)
>>
>> diff --git a/drivers/core/device.c b/drivers/core/device.c
>> index dcf5d9d..ec43654 100644
>> --- a/drivers/core/device.c
>> +++ b/drivers/core/device.c
>> @@ -693,6 +693,28 @@ fdt_addr_t dev_get_addr_index(struct udevice *dev, int index)
>> #endif
>> }
>>
>> +fdt_addr_t dev_get_addr_size_index(struct udevice *dev, int index,
>> + fdt_size_t *size)
>> +{
>> +#if CONFIG_IS_ENABLED(OF_CONTROL)
>> + /*
>> + * Only get the size in this first call. We'll get the addr in the
>> + * next call to the exisiting dev_get_xxx function which handles
>> + * all config options.
>> + */
>> + fdtdec_get_addr_size_auto_noparent(gd->fdt_blob, dev->of_offset,
>> + "reg", 1, size, false);
>
> Does this take account of 'index'?
No, copy and paste error. Thanks for spotting. v2 will follow soon.
Thanks,
Stefan
^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2016-11-30 6:15 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2016-11-28 9:46 [U-Boot] [PATCH] dm: core: Add dev_get_addr_size_index() to retrieve addr and size Stefan Roese
2016-11-30 2:35 ` Simon Glass
2016-11-30 6:15 ` Stefan Roese
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox