All of lore.kernel.org
 help / color / mirror / Atom feed
* [PATCH] ARM: mvebu: Enable NEON support in defconfig
@ 2014-03-19 22:38 Gregory CLEMENT
  2014-03-19 23:44 ` Arnaud Ebalard
  2014-03-20 13:19 ` Jason Cooper
  0 siblings, 2 replies; 4+ messages in thread
From: Gregory CLEMENT @ 2014-03-19 22:38 UTC (permalink / raw)
  To: linux-arm-kernel

CONFIG_NEON is no more turning on unconditionally when Armada 375 or
Armada 38x are selected. However we still want it selected for the
reference configuration.

Signed-off-by: Gregory CLEMENT <gregory.clement@free-electrons.com>
---
 arch/arm/configs/mvebu_v7_defconfig | 1 +
 1 file changed, 1 insertion(+)

diff --git a/arch/arm/configs/mvebu_v7_defconfig b/arch/arm/configs/mvebu_v7_defconfig
index 62b6a37626c5..36797dbfa610 100644
--- a/arch/arm/configs/mvebu_v7_defconfig
+++ b/arch/arm/configs/mvebu_v7_defconfig
@@ -26,6 +26,7 @@ CONFIG_ZBOOT_ROM_BSS=0x0
 CONFIG_ARM_APPENDED_DTB=y
 CONFIG_ARM_ATAG_DTB_COMPAT=y
 CONFIG_VFP=y
+CONFIG_NEON=y
 CONFIG_NET=y
 CONFIG_INET=y
 CONFIG_BT=y
-- 
1.8.1.2

^ permalink raw reply related	[flat|nested] 4+ messages in thread

* [PATCH] ARM: mvebu: Enable NEON support in defconfig
  2014-03-19 22:38 [PATCH] ARM: mvebu: Enable NEON support in defconfig Gregory CLEMENT
@ 2014-03-19 23:44 ` Arnaud Ebalard
  2014-03-20  0:09   ` Gregory CLEMENT
  2014-03-20 13:19 ` Jason Cooper
  1 sibling, 1 reply; 4+ messages in thread
From: Arnaud Ebalard @ 2014-03-19 23:44 UTC (permalink / raw)
  To: linux-arm-kernel

Hi Gregory,

Gregory CLEMENT <gregory.clement@free-electrons.com> writes:

> CONFIG_NEON is no more turning on unconditionally when Armada 375 or
> Armada 38x are selected. However we still want it selected for the
> reference configuration.
>
> Signed-off-by: Gregory CLEMENT <gregory.clement@free-electrons.com>
> ---
>  arch/arm/configs/mvebu_v7_defconfig | 1 +
>  1 file changed, 1 insertion(+)
>
> diff --git a/arch/arm/configs/mvebu_v7_defconfig b/arch/arm/configs/mvebu_v7_defconfig
> index 62b6a37626c5..36797dbfa610 100644
> --- a/arch/arm/configs/mvebu_v7_defconfig
> +++ b/arch/arm/configs/mvebu_v7_defconfig
> @@ -26,6 +26,7 @@ CONFIG_ZBOOT_ROM_BSS=0x0
>  CONFIG_ARM_APPENDED_DTB=y
>  CONFIG_ARM_ATAG_DTB_COMPAT=y
>  CONFIG_VFP=y
> +CONFIG_NEON=y
>  CONFIG_NET=y
>  CONFIG_INET=y
>  CONFIG_BT=y

Will the lack of support by the hardware be detected automatically
(Armada 370 does not have neon support for instance)? Or does the
logic relies on the fact that CONFIG_KERNEL_MODE_NEON is disabled?

Cheers,

a+

^ permalink raw reply	[flat|nested] 4+ messages in thread

* [PATCH] ARM: mvebu: Enable NEON support in defconfig
  2014-03-19 23:44 ` Arnaud Ebalard
@ 2014-03-20  0:09   ` Gregory CLEMENT
  0 siblings, 0 replies; 4+ messages in thread
From: Gregory CLEMENT @ 2014-03-20  0:09 UTC (permalink / raw)
  To: linux-arm-kernel

Hi Arnaud,

On 20/03/2014 00:44, Arnaud Ebalard wrote:
> Hi Gregory,
> 
> Gregory CLEMENT <gregory.clement@free-electrons.com> writes:
> 
>> CONFIG_NEON is no more turning on unconditionally when Armada 375 or
>> Armada 38x are selected. However we still want it selected for the
>> reference configuration.
>>
>> Signed-off-by: Gregory CLEMENT <gregory.clement@free-electrons.com>
>> ---
>>  arch/arm/configs/mvebu_v7_defconfig | 1 +
>>  1 file changed, 1 insertion(+)
>>
>> diff --git a/arch/arm/configs/mvebu_v7_defconfig b/arch/arm/configs/mvebu_v7_defconfig
>> index 62b6a37626c5..36797dbfa610 100644
>> --- a/arch/arm/configs/mvebu_v7_defconfig
>> +++ b/arch/arm/configs/mvebu_v7_defconfig
>> @@ -26,6 +26,7 @@ CONFIG_ZBOOT_ROM_BSS=0x0
>>  CONFIG_ARM_APPENDED_DTB=y
>>  CONFIG_ARM_ATAG_DTB_COMPAT=y
>>  CONFIG_VFP=y
>> +CONFIG_NEON=y
>>  CONFIG_NET=y
>>  CONFIG_INET=y
>>  CONFIG_BT=y
> 
> Will the lack of support by the hardware be detected automatically
> (Armada 370 does not have neon support for instance)? Or does the

yes the lack of support by the hardware should be detected automatically,
whereas if CONFIG_NEON is not selected NEON will never be used even if
the hardware is present.

Gregory

> logic relies on the fact that CONFIG_KERNEL_MODE_NEON is disabled?
> 
> Cheers,
> 
> a+
> 


-- 
Gregory Clement, Free Electrons
Kernel, drivers, real-time and embedded Linux
development, consulting, training and support.
http://free-electrons.com

^ permalink raw reply	[flat|nested] 4+ messages in thread

* [PATCH] ARM: mvebu: Enable NEON support in defconfig
  2014-03-19 22:38 [PATCH] ARM: mvebu: Enable NEON support in defconfig Gregory CLEMENT
  2014-03-19 23:44 ` Arnaud Ebalard
@ 2014-03-20 13:19 ` Jason Cooper
  1 sibling, 0 replies; 4+ messages in thread
From: Jason Cooper @ 2014-03-20 13:19 UTC (permalink / raw)
  To: linux-arm-kernel



> On Mar 19, 2014, at 18:38, Gregory CLEMENT <gregory.clement@free-electrons.com> wrote:
> 
> CONFIG_NEON is no more turning on unconditionally when Armada 375 or
> Armada 38x are selected. However we still want it selected for the
> reference configuration.
> 
> Signed-off-by: Gregory CLEMENT <gregory.clement@free-electrons.com>
> ---
> arch/arm/configs/mvebu_v7_defconfig | 1 +
> 1 file changed, 1 insertion(+)

Acked-by: Jason Cooper <jason@lakedaemon.net>

Arnd,

Mind including this with your series of fixes?

thx,

Jason.

> diff --git a/arch/arm/configs/mvebu_v7_defconfig b/arch/arm/configs/mvebu_v7_defconfig
> index 62b6a37626c5..36797dbfa610 100644
> --- a/arch/arm/configs/mvebu_v7_defconfig
> +++ b/arch/arm/configs/mvebu_v7_defconfig
> @@ -26,6 +26,7 @@ CONFIG_ZBOOT_ROM_BSS=0x0
> CONFIG_ARM_APPENDED_DTB=y
> CONFIG_ARM_ATAG_DTB_COMPAT=y
> CONFIG_VFP=y
> +CONFIG_NEON=y
> CONFIG_NET=y
> CONFIG_INET=y
> CONFIG_BT=y
> -- 
> 1.8.1.2
> 

^ permalink raw reply	[flat|nested] 4+ messages in thread

end of thread, other threads:[~2014-03-20 13:19 UTC | newest]

Thread overview: 4+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2014-03-19 22:38 [PATCH] ARM: mvebu: Enable NEON support in defconfig Gregory CLEMENT
2014-03-19 23:44 ` Arnaud Ebalard
2014-03-20  0:09   ` Gregory CLEMENT
2014-03-20 13:19 ` Jason Cooper

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.