[U-Boot] [PATCH] simple-bus: enable suport for of-platdata

[U-Boot] [PATCH] simple-bus: enable suport for of-platdata

[Kever Yang]

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

[U-Boot] [PATCH] simple-bus: enable suport for of-platdata

[Simon Glass]

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>

[U-Boot] [PATCH] simple-bus: enable suport for of-platdata

[Kever Yang]

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>
>
>
>

[U-Boot] [PATCH] simple-bus: enable suport for of-platdata

[Simon Glass]

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>
>>
>>
>>
>
>

[U-Boot] [PATCH] simple-bus: enable suport for of-platdata

[Simon Glass]

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!