From mboxrd@z Thu Jan  1 00:00:00 1970
From: Jagan Teki <jteki@openedev.com>
Date: Fri, 26 Jun 2015 12:13:06 +0530
Subject: [U-Boot] [PATCH v3 4/4] arm: ls102xa: Enable Driver Model SPI
	for ls1021atwr
In-Reply-To: <BN1PR0301MB0612EF5108036006A26DA7A3F8AD0@BN1PR0301MB0612.namprd03.prod.outlook.com>
References: <1431955503-10979-1-git-send-email-haikun.wang@freescale.com>
 <CAEUhbmUvkS3Y0Z-XGs2jg3kTzfo+a5vbMermXWdmA9Niqi+b4Q@mail.gmail.com>
 <CAPnjgZ1g7smRE=QFqW23ro1TtKK3NEYJwEDfO7b6mm_SPyeaRg@mail.gmail.com>
 <DM2PR0301MB062328200CE51A31B74F1EF2F8A50@DM2PR0301MB0623.namprd03.prod.outlook.com>
 <CAD6G_RQLmmfeujufE6vYdK70ZM0dLcbxGvDOx_JpY1qdgTN-6Q@mail.gmail.com>
 <CAD6G_RTcpDb22jKFYWjOD5CUcLEy4ts=ns_xxj+H1K0rErHg9Q@mail.gmail.com>
 <BN1PR0301MB061251FCF24B815ADA684D5DF8A40@BN1PR0301MB0612.namprd03.prod.outlook.com>
 <CAD6G_RQjvozacpCYUHS=wonaeSncgrynt2KTdv3iDUOQdfo-wQ@mail.gmail.com>
 <CAD6G_RQN3CKuAP3h2cZSBoAVc1sRF28HVXKiW_kMYDUycwW0vg@mail.gmail.com>
 <BN1PR0301MB0612EF5108036006A26DA7A3F8AD0@BN1PR0301MB0612.namprd03.prod.outlook.com>
Message-ID: <CAD6G_RR+2EKOeqKbG-A2hF_fNBvhiHUOyr8XS5Jc402_8rqfSg@mail.gmail.com>
List-Id: <u-boot.lists.denx.de>
MIME-Version: 1.0
Content-Type: text/plain; charset="us-ascii"
Content-Transfer-Encoding: 7bit
To: u-boot@lists.denx.de

On 26 June 2015 at 08:43, Wang Haikun <Haikun.Wang@freescale.com> wrote:
> On 6/25/2015 9:24 PM, Jagan Teki wrote:
>> On 19 June 2015 at 13:44, Jagan Teki <jteki@openedev.com> wrote:
>>> On 19 June 2015 at 13:20, Wang Haikun <Haikun.Wang@freescale.com> wrote:
>>>> On 6/19/2015 3:34 PM, Jagan Teki wrote:
>>>>> On 18 June 2015 at 12:24, Jagan Teki <jteki@openedev.com> wrote:
>>>>>> On 18 June 2015 at 07:50, Wang Haikun <Haikun.Wang@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@gmail.com> wrote:
>>>>>>>>> Hi Haikun,
>>>>>>>>>
>>>>>>>>> On Mon, May 18, 2015 at 9:25 PM, Haikun Wang <haikun.wang@freescale.com> wrote:
>>>>>>>>>> From: Haikun Wang <Haikun.Wang@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@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.

Please send the patch for this fix, found build errors by picking these patches.

thanks!
-- 
Jagan | openedev.