Re: [PATCH] xtensa: Partially Revert "xtensa: Remove unnecessary of_platform_populate with default match table"

2016-07-26 Thread Kefeng Wang
On 2016/7/27 2:50, Max Filippov wrote: > On Tue, Jul 26, 2016 at 01:01:32PM -0500, Rob Herring wrote: >> This partially reverts commit 69d99e6c0d62 keeping only the main >> purpose of the original commit which is the removal of >> of_platform_populate() call. The moving of of_clk_init() caused ch

Re: [PATCH] xtensa: Partially Revert "xtensa: Remove unnecessary of_platform_populate with default match table"

2016-07-26 Thread Max Filippov
On Tue, Jul 26, 2016 at 02:13:52PM -0500, Rob Herring wrote: > >> I tried briefly running under QEMU, but didn't have success. If anyone has > >> up to date instructions that would be helpful as using these[1] didn't > >> seem to work. > > > > I think they're up to date. Another thing that need to

Re: [PATCH] xtensa: Partially Revert "xtensa: Remove unnecessary of_platform_populate with default match table"

2016-07-26 Thread Rob Herring
On Tue, Jul 26, 2016 at 1:50 PM, Max Filippov wrote: > On Tue, Jul 26, 2016 at 01:01:32PM -0500, Rob Herring wrote: >> This partially reverts commit 69d99e6c0d62 keeping only the main >> purpose of the original commit which is the removal of >> of_platform_populate() call. The moving of of_clk_ini

Re: [PATCH] xtensa: Partially Revert "xtensa: Remove unnecessary of_platform_populate with default match table"

2016-07-26 Thread Max Filippov
On Tue, Jul 26, 2016 at 01:01:32PM -0500, Rob Herring wrote: > This partially reverts commit 69d99e6c0d62 keeping only the main > purpose of the original commit which is the removal of > of_platform_populate() call. The moving of of_clk_init() caused changes > in the initialization order breaking b

[PATCH] xtensa: Partially Revert "xtensa: Remove unnecessary of_platform_populate with default match table"

2016-07-26 Thread Rob Herring
This partially reverts commit 69d99e6c0d62 keeping only the main purpose of the original commit which is the removal of of_platform_populate() call. The moving of of_clk_init() caused changes in the initialization order breaking booting. Fixes: 69d99e6c0d621f ("xtensa: Remove unnecessary of_platfo