From: Michal Simek <michal.simek@xilinx.com>
To: u-boot@lists.denx.de
Subject: [U-Boot] [PATCH 2/2] zynq: defconfig: Move CONFIG_OF_* to Kconfig
Date: Thu, 02 Jul 2015 11:53:13 +0200 [thread overview]
Message-ID: <55950A09.5040109@xilinx.com> (raw)
In-Reply-To: <CAD6G_RS-Z8kqF75kW+MngjcaTKMbeMXnSoxKSK5mfhjYk15aqg@mail.gmail.com>
On 07/02/2015 11:19 AM, Jagan Teki wrote:
> On 2 July 2015 at 14:11, Michal Simek <michal.simek@xilinx.com> wrote:
>> On 07/02/2015 08:11 AM, Jagan Teki wrote:
>>> On 2 July 2015 at 06:50, Masahiro Yamada <yamada.masahiro@socionext.com> wrote:
>>>> 2015-07-02 2:08 GMT+09:00 Jagan Teki <jteki@openedev.com>:
>>>>> On 29 June 2015 at 14:33, Jagan Teki <jteki@openedev.com> wrote:
>>>>>> This commit moves:
>>>>>> - CONFIG_OF_CONTROL
>>>>>> - SPL_DISABLE_OF_CONTROL
>>>>>>
>>>>>> from zynq_*_defconfig files into arch/arm/Kconfig "config ARCH_ZYNQ"
>>>>>>
>>>>>> Signed-off-by: Jagan Teki <jteki@openedev.com>
>>>>>> Cc: Michal Simek <michal.simek@xilinx.com>
>>>>>> Cc: Siva Durga Prasad Paladugu <sivadur@xilinx.com>
>>>>>> Cc: Masahiro Yamada <yamada.masahiro@socionext.com>
>>>>>> ---
>>>>>> arch/arm/Kconfig | 2 ++
>>>>>> configs/zynq_microzed_defconfig | 2 --
>>>>>> configs/zynq_picozed_defconfig | 1 -
>>>>>> configs/zynq_zc702_defconfig | 1 -
>>>>>> configs/zynq_zc706_defconfig | 1 -
>>>>>> configs/zynq_zc70x_defconfig | 2 --
>>>>>> configs/zynq_zc770_xm010_defconfig | 2 --
>>>>>> configs/zynq_zc770_xm012_defconfig | 2 --
>>>>>> configs/zynq_zc770_xm013_defconfig | 2 --
>>>>>> configs/zynq_zed_defconfig | 2 --
>>>>>> configs/zynq_zybo_defconfig | 2 --
>>>>>> 11 files changed, 2 insertions(+), 17 deletions(-)
>>>>>>
>>>>>> diff --git a/arch/arm/Kconfig b/arch/arm/Kconfig
>>>>>> index 5f026f0..192d9cf 100644
>>>>>> --- a/arch/arm/Kconfig
>>>>>> +++ b/arch/arm/Kconfig
>>>>>> @@ -670,6 +670,8 @@ config ARCH_ZYNQ
>>>>>> bool "Xilinx Zynq Platform"
>>>>>> select CPU_V7
>>>>>> select SUPPORT_SPL
>>>>>> + select OF_CONTROL
>>>>>> + select SPL_DISABLE_OF_CONTROL
>>>>>> select DM
>>>>>> select DM_SPI
>>>>>> select DM_SPI_FLASH
>>>>>> diff --git a/configs/zynq_microzed_defconfig b/configs/zynq_microzed_defconfig
>>>>>> index 533cdb5..c878924 100644
>>>>>> --- a/configs/zynq_microzed_defconfig
>>>>>> +++ b/configs/zynq_microzed_defconfig
>>>>>> @@ -10,5 +10,3 @@ CONFIG_FIT_SIGNATURE=y
>>>>>> # CONFIG_CMD_IMLS is not set
>>>>>> # CONFIG_CMD_FLASH is not set
>>>>>> # CONFIG_CMD_SETEXPR is not set
>>>>>> -CONFIG_OF_CONTROL=y
>>>>>> -CONFIG_SPL_DISABLE_OF_CONTROL=y
>>>>>> diff --git a/configs/zynq_picozed_defconfig b/configs/zynq_picozed_defconfig
>>>>>> index 76a76e0..af77a9d 100644
>>>>>> --- a/configs/zynq_picozed_defconfig
>>>>>> +++ b/configs/zynq_picozed_defconfig
>>>>>> @@ -6,4 +6,3 @@ CONFIG_SPL=y
>>>>>> # CONFIG_CMD_IMLS is not set
>>>>>> # CONFIG_CMD_FLASH is not set
>>>>>> # CONFIG_CMD_SETEXPR is not set
>>>>>> -CONFIG_OF_CONTROL=y
>>>>>> diff --git a/configs/zynq_zc702_defconfig b/configs/zynq_zc702_defconfig
>>>>>> index 2e40508..5dde452 100644
>>>>>> --- a/configs/zynq_zc702_defconfig
>>>>>> +++ b/configs/zynq_zc702_defconfig
>>>>>> @@ -10,4 +10,3 @@ CONFIG_FIT_SIGNATURE=y
>>>>>> # CONFIG_CMD_IMLS is not set
>>>>>> # CONFIG_CMD_FLASH is not set
>>>>>> # CONFIG_CMD_SETEXPR is not set
>>>>>> -CONFIG_OF_CONTROL=y
>>>>>> diff --git a/configs/zynq_zc706_defconfig b/configs/zynq_zc706_defconfig
>>>>>> index 7c3713e..0f96d16 100644
>>>>>> --- a/configs/zynq_zc706_defconfig
>>>>>> +++ b/configs/zynq_zc706_defconfig
>>>>>> @@ -10,4 +10,3 @@ CONFIG_FIT_SIGNATURE=y
>>>>>> # CONFIG_CMD_IMLS is not set
>>>>>> # CONFIG_CMD_FLASH is not set
>>>>>> # CONFIG_CMD_SETEXPR is not set
>>>>>> -CONFIG_OF_CONTROL=y
>>>>>> diff --git a/configs/zynq_zc70x_defconfig b/configs/zynq_zc70x_defconfig
>>>>>> index 41c04bb..525c538 100644
>>>>>> --- a/configs/zynq_zc70x_defconfig
>>>>>> +++ b/configs/zynq_zc70x_defconfig
>>>>>> @@ -10,5 +10,3 @@ CONFIG_FIT_SIGNATURE=y
>>>>>> # CONFIG_CMD_IMLS is not set
>>>>>> # CONFIG_CMD_FLASH is not set
>>>>>> # CONFIG_CMD_SETEXPR is not set
>>>>>> -CONFIG_OF_CONTROL=y
>>>>>> -CONFIG_SPL_DISABLE_OF_CONTROL=y
>>>>>> diff --git a/configs/zynq_zc770_xm010_defconfig b/configs/zynq_zc770_xm010_defconfig
>>>>>> index 73b5854..f1fc283 100644
>>>>>> --- a/configs/zynq_zc770_xm010_defconfig
>>>>>> +++ b/configs/zynq_zc770_xm010_defconfig
>>>>>> @@ -11,6 +11,4 @@ CONFIG_SYS_EXTRA_OPTIONS="ZC770_XM010"
>>>>>> # CONFIG_CMD_IMLS is not set
>>>>>> # CONFIG_CMD_FLASH is not set
>>>>>> # CONFIG_CMD_SETEXPR is not set
>>>>>> -CONFIG_OF_CONTROL=y
>>>>>> -CONFIG_SPL_DISABLE_OF_CONTROL=y
>>>>>> CONFIG_SPI_FLASH=y
>>>>>> diff --git a/configs/zynq_zc770_xm012_defconfig b/configs/zynq_zc770_xm012_defconfig
>>>>>> index 0d5c77e..73ed13d 100644
>>>>>> --- a/configs/zynq_zc770_xm012_defconfig
>>>>>> +++ b/configs/zynq_zc770_xm012_defconfig
>>>>>> @@ -9,5 +9,3 @@ CONFIG_FIT_VERBOSE=y
>>>>>> CONFIG_FIT_SIGNATURE=y
>>>>>> CONFIG_SYS_EXTRA_OPTIONS="ZC770_XM012"
>>>>>> # CONFIG_CMD_SETEXPR is not set
>>>>>> -CONFIG_OF_CONTROL=y
>>>>>> -CONFIG_SPL_DISABLE_OF_CONTROL=y
>>>>>> diff --git a/configs/zynq_zc770_xm013_defconfig b/configs/zynq_zc770_xm013_defconfig
>>>>>> index 8d320a1..211a41d 100644
>>>>>> --- a/configs/zynq_zc770_xm013_defconfig
>>>>>> +++ b/configs/zynq_zc770_xm013_defconfig
>>>>>> @@ -11,5 +11,3 @@ CONFIG_SYS_EXTRA_OPTIONS="ZC770_XM013"
>>>>>> # CONFIG_CMD_IMLS is not set
>>>>>> # CONFIG_CMD_FLASH is not set
>>>>>> # CONFIG_CMD_SETEXPR is not set
>>>>>> -CONFIG_OF_CONTROL=y
>>>>>> -CONFIG_SPL_DISABLE_OF_CONTROL=y
>>>>>> diff --git a/configs/zynq_zed_defconfig b/configs/zynq_zed_defconfig
>>>>>> index 9f37849..f3c63f9 100644
>>>>>> --- a/configs/zynq_zed_defconfig
>>>>>> +++ b/configs/zynq_zed_defconfig
>>>>>> @@ -10,5 +10,3 @@ CONFIG_FIT_SIGNATURE=y
>>>>>> # CONFIG_CMD_IMLS is not set
>>>>>> # CONFIG_CMD_FLASH is not set
>>>>>> # CONFIG_CMD_SETEXPR is not set
>>>>>> -CONFIG_OF_CONTROL=y
>>>>>> -CONFIG_SPL_DISABLE_OF_CONTROL=y
>>>>>> diff --git a/configs/zynq_zybo_defconfig b/configs/zynq_zybo_defconfig
>>>>>> index e25573b..0c6117d 100644
>>>>>> --- a/configs/zynq_zybo_defconfig
>>>>>> +++ b/configs/zynq_zybo_defconfig
>>>>>> @@ -10,5 +10,3 @@ CONFIG_FIT_SIGNATURE=y
>>>>>> # CONFIG_CMD_IMLS is not set
>>>>>> # CONFIG_CMD_FLASH is not set
>>>>>> # CONFIG_CMD_SETEXPR is not set
>>>>>> -CONFIG_OF_CONTROL=y
>>>>>> -CONFIG_SPL_DISABLE_OF_CONTROL=y
>>>>>> --
>>>>>> 1.9.1
>>>>>>
>>>>>
>>>>> Applied to u-boot-spi/master
>>>>>
>>>>
>>>>
>>>> Why u-boot-spi/master, not u-boot-microblaze/zynq ?
>>>
>>> As I sent these patches, as part of zynq dm support and dm-spi.
>>>
>>>>
>>>>
>>>> Note this patch will change the behavior.
>>>>
>>>> It is dropping the non-DeviceTree boot support,
>>>> so I'd like Michal's Acked-by before moving on.
>>>
>>> Verified the device-tree and along with dm-spi on zynq.
>>>
>>> Michal, Could you comment on this.
>>
>> I don't have picozed board here to be able to test it.
>> And doesn't matter because it it is applied to mainline already.
>>
>> I am just now sure why these big changes are coming after rc3.
>
> The only reason I have applied these as DM and DM_SPI added
> in previous patches and picozed requires OF_CONTROL to built properly.
>
> And we have verified this on microzed and xm010 as well.
No problem with this for now. We can fix it later if something is
broken. We want to switch to DT anyway.
Thanks,
Michal
next prev parent reply other threads:[~2015-07-02 9:53 UTC|newest]
Thread overview: 9+ messages / expand[flat|nested] mbox.gz Atom feed top
2015-06-29 9:03 [U-Boot] [PATCH 1/2] defconfig: Add CONFIG_OF_CONTROL in zynq_picozed Jagan Teki
2015-06-29 9:03 ` [U-Boot] [PATCH 2/2] zynq: defconfig: Move CONFIG_OF_* to Kconfig Jagan Teki
2015-07-01 17:08 ` Jagan Teki
2015-07-02 1:20 ` Masahiro Yamada
2015-07-02 6:11 ` Jagan Teki
2015-07-02 8:41 ` Michal Simek
2015-07-02 9:19 ` Jagan Teki
2015-07-02 9:53 ` Michal Simek [this message]
2015-07-01 17:07 ` [U-Boot] [PATCH 1/2] defconfig: Add CONFIG_OF_CONTROL in zynq_picozed Jagan Teki
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=55950A09.5040109@xilinx.com \
--to=michal.simek@xilinx.com \
--cc=u-boot@lists.denx.de \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox