public inbox for u-boot@lists.denx.de
 help / color / mirror / Atom feed
From: Ziyuan Xu <xzy.xu@rock-chips.com>
To: u-boot@lists.denx.de
Subject: [U-Boot] [PATCH] rockchip: disable CONFIG_EFI_LOADER[Involving remittance information, please pay attention to the safety of property]
Date: Mon, 11 Jul 2016 09:25:18 +0800	[thread overview]
Message-ID: <5782F57E.2040700@rock-chips.com> (raw)
In-Reply-To: <CAPnjgZ0SF+kdDmhi7AnqZ=tT6=AT41iiyQ5Oi3vHiKbL4T-t_A@mail.gmail.com>

hi Simon,

Sorry, it's my mistake about CONFIG_EFI_LOADER, pls drop this patch.
At first, I doubt that  CONFIG_EFI_LOADER influyences cache coherence 
while enable fastboot function on rk3288 board. With further 
investigation, I fix it with http://patchwork.ozlabs.org/patch/645189/. 
Could you take a view on?
Thanks!

On 2016?06?30? 23:23, Simon Glass wrote:
> Hi Ziyuan,
>
> On 30 June 2016 at 00:28, Ziyuan Xu <xzy.xu@rock-chips.com> wrote:
>> Disable EFI applications feature for rockchip platform.
>>
>> Signed-off-by: Ziyuan Xu <xzy.xu@rock-chips.com>
>> ---
>>   configs/chromebook_jerry_defconfig | 1 +
>>   configs/evb-rk3036_defconfig       | 1 +
>>   configs/kylin-rk3036_defconfig     | 1 +
>>   configs/rock2_defconfig            | 1 +
>>   4 files changed, 4 insertions(+)
>>
>> diff --git a/configs/chromebook_jerry_defconfig b/configs/chromebook_jerry_defconfig
>> index d5bc515..b020066 100644
>> --- a/configs/chromebook_jerry_defconfig
>> +++ b/configs/chromebook_jerry_defconfig
>> @@ -10,6 +10,7 @@ CONFIG_DEFAULT_DEVICE_TREE="rk3288-jerry"
>>   CONFIG_SPL_STACK_R=y
>>   CONFIG_SPL_STACK_R_MALLOC_SIMPLE_LEN=0x2000
>>   CONFIG_HUSH_PARSER=y
>> +CONFIG_EFI_LOADER=n
> You should use the format:
>
> # CONFIG_EFI_LOADER is not set
>
>>   CONFIG_CMD_BOOTZ=y
>>   # CONFIG_CMD_IMLS is not set
>>   CONFIG_CMD_MMC=y
>> diff --git a/configs/evb-rk3036_defconfig b/configs/evb-rk3036_defconfig
>> index 9894fff..5f816ec 100644
>> --- a/configs/evb-rk3036_defconfig
>> +++ b/configs/evb-rk3036_defconfig
>> @@ -6,6 +6,7 @@ CONFIG_SPL_STACK_R_ADDR=0x80000
>>   CONFIG_DEFAULT_DEVICE_TREE="rk3036-sdk"
>>   CONFIG_SPL_STACK_R=y
>>   CONFIG_HUSH_PARSER=y
>> +CONFIG_EFI_LOADER=n
>>   CONFIG_CMD_BOOTZ=y
>>   # CONFIG_CMD_IMLS is not set
>>   CONFIG_CMD_MMC=y
>> diff --git a/configs/kylin-rk3036_defconfig b/configs/kylin-rk3036_defconfig
>> index 0ff6c6b..4a043af 100644
>> --- a/configs/kylin-rk3036_defconfig
>> +++ b/configs/kylin-rk3036_defconfig
>> @@ -7,6 +7,7 @@ CONFIG_DEFAULT_DEVICE_TREE="rk3036-sdk"
>>   CONFIG_SPL_STACK_R=y
>>   CONFIG_HUSH_PARSER=y
>>   CONFIG_CMD_BOOTZ=y
>> +CONFIG_EFI_LOADER=n
>>   # CONFIG_CMD_IMLS is not set
>>   CONFIG_CMD_MMC=y
>>   CONFIG_CMD_SF=y
>> diff --git a/configs/rock2_defconfig b/configs/rock2_defconfig
>> index 3e16b80..e07100e 100644
>> --- a/configs/rock2_defconfig
>> +++ b/configs/rock2_defconfig
>> @@ -8,6 +8,7 @@ CONFIG_DEFAULT_DEVICE_TREE="rk3288-rock2-square"
>>   CONFIG_SPL_STACK_R=y
>>   CONFIG_SPL_STACK_R_MALLOC_SIMPLE_LEN=0x2000
>>   CONFIG_HUSH_PARSER=y
>> +CONFIG_EFI_LOADER=n
>>   CONFIG_CMD_BOOTZ=y
>>   # CONFIG_CMD_IMLS is not set
>>   CONFIG_CMD_MMC=y
>> --
>> 1.9.1
>>
>>
> Regards,
> Simon
>
>
>

      reply	other threads:[~2016-07-11  1:25 UTC|newest]

Thread overview: 3+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2016-06-30  7:28 [U-Boot] [PATCH] rockchip: disable CONFIG_EFI_LOADER Ziyuan Xu
2016-06-30 15:23 ` Simon Glass
2016-07-11  1:25   ` Ziyuan Xu [this message]

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=5782F57E.2040700@rock-chips.com \
    --to=xzy.xu@rock-chips.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