[U-Boot] [PATCH v3 4/4] arm: ls102xa: Enable Driver Model SPI for ls1021atwr

[Wang Haikun]

On 6/25/2015 9:24 PM, Jagan Teki wrote:
> On 19 June 2015 at 13:44, Jagan Teki <jteki at openedev.com> wrote:
>> On 19 June 2015 at 13:20, Wang Haikun <Haikun.Wang at freescale.com> wrote:
>>> On 6/19/2015 3:34 PM, Jagan Teki wrote:
>>>> On 18 June 2015 at 12:24, Jagan Teki <jteki at openedev.com> wrote:
>>>>> On 18 June 2015 at 07:50, Wang Haikun <Haikun.Wang at freescale.com> wrote:
>>>>>> On 6/17/2015 8:30 PM, Simon Glass wrote:
>>>>>>> Hi,
>>>>>>>
>>>>>>> On 17 June 2015 at 03:36, Bin Meng <bmeng.cn at gmail.com> wrote:
>>>>>>>> Hi Haikun,
>>>>>>>>
>>>>>>>> On Mon, May 18, 2015 at 9:25 PM, Haikun Wang <haikun.wang at freescale.com> wrote:
>>>>>>>>> From: Haikun Wang <Haikun.Wang at freescale.com>
>>>>>>>>>
>>>>>>>>> Enable Driver Model SPI for ls1021atwr board.
>>>>>>>>> DSPI and QSPI only be enabled when boot from QSPI.
>>>>>>>>> DSPI and QSPI are compatible under Driver Model SPI.
>>>>>>>>>
>>>>>>>>> Signed-off-by: Haikun Wang <Haikun.Wang at freescale.com>
>>>>>>>>> Change-Id: I6342807da7725ae8b678952117c8758c75a61d3d
>>>>
>>>> Where is this commit id? I couldn't see it on "git log"
>>> Hi Jagan,
>>>
>>> It is not a git commit ID, it is a code review task ID of gerrit in fact.
>>> I'm sorry again for forgetting remove it when submit patch.
>>>
>>> Best regards,
>>> Wang Haikun
>>>>
>>>>>>>>> Reviewed-on: http://git.am.freescale.net:8181/33447>
>>>>>> Best regards,
>>>>>> Wang Haikun
>>>>>>>
>>>>>>>>
>>>>>>>> Is this URL Freescale internal? I cannot access it.
>>>>>>>>
>>>>>>>
>>>>>>> Looks like it. BTW patman will remove these Gerrit tags automatically.
>>>>>> Yes, it is our internal URL.
>>>>>> I forget to remove it.
>>>>>> It couldn't be better if it will be removed automatically.
>>>>>
>>>>> I will remove if something not remove automatically.
>>>>>
>>>>> Anyone have any comments on these patch-set, I'm planning to take these.
>>>>>
>>>>> https://patchwork.ozlabs.org/patch/473391/
>>>>> https://patchwork.ozlabs.org/patch/473392/
>
>>>>> https://patchwork.ozlabs.org/patch/473393/
>>>>> https://patchwork.ozlabs.org/patch/473394/
> Few CONFIG_DM_* were missing in these two patches, did these patches
> have any dependencies with other series, please look at and let me know.

I found that macro IS_ENABLED(CONFIG_XXX) is only work with configure 
option in Kconfig, and DM core code use IS_ENABLED(), so configure 
option in head file can't work, so remove CONFIG_OF_CONTROL 
CONFIG_OF_SEPARATE CONFIG_DM CONFIG_DM_SPI in the head file.
>
> thanks!
>