Re: [U-Boot] [PATCH 10/12] cmd: qfw: do not depend on x86

2016-05-16 Thread Bin Meng
Hi Miao,

On Mon, May 16, 2016 at 5:44 PM, Miao Yan  wrote:
> Hi Bin,
>
> 2016-05-13 22:00 GMT+08:00 Bin Meng :
>> Hi Miao,
>>
>> On Fri, May 13, 2016 at 2:29 PM, Miao Yan  wrote:
>>> The qfw command interface used to depend on X86, this patch removes
>>> this restriction so it can be built for sandbox for testing. For normal
>>> usage, it can only be used with CONFIG_QEMU.
>>>
>>> Signed-off-by: Miao Yan 
>>> ---
>>
>> Reviewed-by: Bin Meng 
>>
>> Would you prepare a patch to enable this on Sandbox?
>
> I'll give it a try. Do you prefer to do that in this patch-set or submit 
> later ?
>

Please do in this patch-set so that we can test the sandbox build with
this qfw command.

Regards,
Bin
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot


Re: [U-Boot] [PATCH 10/12] cmd: qfw: do not depend on x86

2016-05-16 Thread Miao Yan
Hi Bin,

2016-05-13 22:00 GMT+08:00 Bin Meng :
> Hi Miao,
>
> On Fri, May 13, 2016 at 2:29 PM, Miao Yan  wrote:
>> The qfw command interface used to depend on X86, this patch removes
>> this restriction so it can be built for sandbox for testing. For normal
>> usage, it can only be used with CONFIG_QEMU.
>>
>> Signed-off-by: Miao Yan 
>> ---
>
> Reviewed-by: Bin Meng 
>
> Would you prepare a patch to enable this on Sandbox?

I'll give it a try. Do you prefer to do that in this patch-set or submit later ?

Miao

>
>>  cmd/Kconfig | 1 -
>>  1 file changed, 1 deletion(-)
>>
>> diff --git a/cmd/Kconfig b/cmd/Kconfig
>> index 08b761f..01870cb 100644
>> --- a/cmd/Kconfig
>> +++ b/cmd/Kconfig
>> @@ -595,7 +595,6 @@ config CMD_SOUND
>>
>>  config CMD_QEMU_FW_CFG
>> bool "qfw"
>> -   depends on X86
>> select QFW
>> help
>>   This provides access to the QEMU firmware interface.  The main
>> --
>
> Regards,
> Bin
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot


Re: [U-Boot] [PATCH 10/12] cmd: qfw: do not depend on x86

2016-05-13 Thread Bin Meng
Hi Miao,

On Fri, May 13, 2016 at 2:29 PM, Miao Yan  wrote:
> The qfw command interface used to depend on X86, this patch removes
> this restriction so it can be built for sandbox for testing. For normal
> usage, it can only be used with CONFIG_QEMU.
>
> Signed-off-by: Miao Yan 
> ---

Reviewed-by: Bin Meng 

Would you prepare a patch to enable this on Sandbox?

>  cmd/Kconfig | 1 -
>  1 file changed, 1 deletion(-)
>
> diff --git a/cmd/Kconfig b/cmd/Kconfig
> index 08b761f..01870cb 100644
> --- a/cmd/Kconfig
> +++ b/cmd/Kconfig
> @@ -595,7 +595,6 @@ config CMD_SOUND
>
>  config CMD_QEMU_FW_CFG
> bool "qfw"
> -   depends on X86
> select QFW
> help
>   This provides access to the QEMU firmware interface.  The main
> --

Regards,
Bin
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot


[U-Boot] [PATCH 10/12] cmd: qfw: do not depend on x86

2016-05-12 Thread Miao Yan
The qfw command interface used to depend on X86, this patch removes
this restriction so it can be built for sandbox for testing. For normal
usage, it can only be used with CONFIG_QEMU.

Signed-off-by: Miao Yan 
---
 cmd/Kconfig | 1 -
 1 file changed, 1 deletion(-)

diff --git a/cmd/Kconfig b/cmd/Kconfig
index 08b761f..01870cb 100644
--- a/cmd/Kconfig
+++ b/cmd/Kconfig
@@ -595,7 +595,6 @@ config CMD_SOUND
 
 config CMD_QEMU_FW_CFG
bool "qfw"
-   depends on X86
select QFW
help
  This provides access to the QEMU firmware interface.  The main
-- 
1.9.1

___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot