* [U-Boot] [PATCH] simple-bus: enable suport for of-platdata
@ 2017-01-18 12:25 Kever Yang
2017-01-26 14:23 ` Simon Glass
0 siblings, 1 reply; 5+ messages in thread
From: Kever Yang @ 2017-01-18 12:25 UTC (permalink / raw)
To: u-boot
Just do nothing in post_bind if of-platdata enabled,
for there is no dm_scan_fdt_dev().
Signed-off-by: Kever Yang <kever.yang@rock-chips.com>
---
drivers/core/simple-bus.c | 4 ++++
1 file changed, 4 insertions(+)
diff --git a/drivers/core/simple-bus.c b/drivers/core/simple-bus.c
index 5c955da..9b79825 100644
--- a/drivers/core/simple-bus.c
+++ b/drivers/core/simple-bus.c
@@ -27,6 +27,9 @@ fdt_addr_t simple_bus_translate(struct udevice *dev, fdt_addr_t addr)
static int simple_bus_post_bind(struct udevice *dev)
{
+#if CONFIG_IS_ENABLED(OF_PLATDATA)
+ return 0;
+#else
u32 cell[3];
int ret;
@@ -41,6 +44,7 @@ static int simple_bus_post_bind(struct udevice *dev)
}
return dm_scan_fdt_dev(dev);
+#endif
}
UCLASS_DRIVER(simple_bus) = {
--
1.9.1
^ permalink raw reply related [flat|nested] 5+ messages in thread
* [U-Boot] [PATCH] simple-bus: enable suport for of-platdata
2017-01-18 12:25 [U-Boot] [PATCH] simple-bus: enable suport for of-platdata Kever Yang
@ 2017-01-26 14:23 ` Simon Glass
2017-02-05 0:24 ` Kever Yang
0 siblings, 1 reply; 5+ messages in thread
From: Simon Glass @ 2017-01-26 14:23 UTC (permalink / raw)
To: u-boot
On 18 January 2017 at 05:25, Kever Yang <kever.yang@rock-chips.com> wrote:
> Just do nothing in post_bind if of-platdata enabled,
> for there is no dm_scan_fdt_dev().
>
> Signed-off-by: Kever Yang <kever.yang@rock-chips.com>
> ---
>
> drivers/core/simple-bus.c | 4 ++++
> 1 file changed, 4 insertions(+)
Reviewed-by: Simon Glass <sjg@chromium.org>
^ permalink raw reply [flat|nested] 5+ messages in thread
* [U-Boot] [PATCH] simple-bus: enable suport for of-platdata
2017-01-26 14:23 ` Simon Glass
@ 2017-02-05 0:24 ` Kever Yang
2017-02-08 5:09 ` Simon Glass
0 siblings, 1 reply; 5+ messages in thread
From: Kever Yang @ 2017-02-05 0:24 UTC (permalink / raw)
To: u-boot
Hi Simon,
Found 1 typo on the subject, 'suport' should be 'support',
do I need to send a new patch or you can correct it when you apply?
Thanks,
- Kever
On 01/26/2017 10:23 PM, Simon Glass wrote:
> On 18 January 2017 at 05:25, Kever Yang <kever.yang@rock-chips.com> wrote:
>> Just do nothing in post_bind if of-platdata enabled,
>> for there is no dm_scan_fdt_dev().
>>
>> Signed-off-by: Kever Yang <kever.yang@rock-chips.com>
>> ---
>>
>> drivers/core/simple-bus.c | 4 ++++
>> 1 file changed, 4 insertions(+)
> Reviewed-by: Simon Glass <sjg@chromium.org>
>
>
>
^ permalink raw reply [flat|nested] 5+ messages in thread
* [U-Boot] [PATCH] simple-bus: enable suport for of-platdata
2017-02-05 0:24 ` Kever Yang
@ 2017-02-08 5:09 ` Simon Glass
2017-02-08 17:42 ` Simon Glass
0 siblings, 1 reply; 5+ messages in thread
From: Simon Glass @ 2017-02-08 5:09 UTC (permalink / raw)
To: u-boot
Hi Kever,
On 4 February 2017 at 17:24, Kever Yang <kever.yang@rock-chips.com> wrote:
>
> Hi Simon,
>
> Found 1 typo on the subject, 'suport' should be 'support',
> do I need to send a new patch or you can correct it when you apply?
I can correct it when I apply, should be soon.
- Simon
>
> Thanks,
> - Kever
>
> On 01/26/2017 10:23 PM, Simon Glass wrote:
>>
>> On 18 January 2017 at 05:25, Kever Yang <kever.yang@rock-chips.com> wrote:
>>>
>>> Just do nothing in post_bind if of-platdata enabled,
>>> for there is no dm_scan_fdt_dev().
>>>
>>> Signed-off-by: Kever Yang <kever.yang@rock-chips.com>
>>> ---
>>>
>>> drivers/core/simple-bus.c | 4 ++++
>>> 1 file changed, 4 insertions(+)
>>
>> Reviewed-by: Simon Glass <sjg@chromium.org>
>>
>>
>>
>
>
^ permalink raw reply [flat|nested] 5+ messages in thread
* [U-Boot] [PATCH] simple-bus: enable suport for of-platdata
2017-02-08 5:09 ` Simon Glass
@ 2017-02-08 17:42 ` Simon Glass
0 siblings, 0 replies; 5+ messages in thread
From: Simon Glass @ 2017-02-08 17:42 UTC (permalink / raw)
To: u-boot
On 7 February 2017 at 22:09, Simon Glass <sjg@chromium.org> wrote:
> Hi Kever,
>
> On 4 February 2017 at 17:24, Kever Yang <kever.yang@rock-chips.com> wrote:
>>
>> Hi Simon,
>>
>> Found 1 typo on the subject, 'suport' should be 'support',
>> do I need to send a new patch or you can correct it when you apply?
>
> I can correct it when I apply, should be soon.
>
> - Simon
>
>>
>> Thanks,
>> - Kever
>>
>> On 01/26/2017 10:23 PM, Simon Glass wrote:
>>>
>>> On 18 January 2017 at 05:25, Kever Yang <kever.yang@rock-chips.com> wrote:
>>>>
>>>> Just do nothing in post_bind if of-platdata enabled,
>>>> for there is no dm_scan_fdt_dev().
>>>>
>>>> Signed-off-by: Kever Yang <kever.yang@rock-chips.com>
>>>> ---
>>>>
>>>> drivers/core/simple-bus.c | 4 ++++
>>>> 1 file changed, 4 insertions(+)
>>>
>>> Reviewed-by: Simon Glass <sjg@chromium.org>
>>>
>>>
>>>
>>
>>
Corrected and
Applied to u-boot-dm, thanks!
^ permalink raw reply [flat|nested] 5+ messages in thread
end of thread, other threads:[~2017-02-08 17:42 UTC | newest]
Thread overview: 5+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2017-01-18 12:25 [U-Boot] [PATCH] simple-bus: enable suport for of-platdata Kever Yang
2017-01-26 14:23 ` Simon Glass
2017-02-05 0:24 ` Kever Yang
2017-02-08 5:09 ` Simon Glass
2017-02-08 17:42 ` Simon Glass
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.