Re: [PATCH v3] ARM: configs: Add Freescale LS1021A defconfig

From: Horia GeantÄ
Date: Fri Aug 28 2015 - 06:23:57 EST


On 8/28/2015 11:04 AM, Wang Huan-B18965 wrote:
> Hi, Horia,
>
>> On 8/17/2015 6:42 AM, Wang Huan-B18965 wrote:
>>> Hi, Fabio,
>>
>> Please don't top post.
>>
>>>
>>> Based on the analysis and discussion, we prefer to use
>> ls1021a_defconfig and remove LS1021A from imx_v6_v7_defconfig.
>>
>> Alison, when you submit v4 for this patch (adding
>> CONFIG_ARCH_DMA_ADDR_T_64BIT=y as suggested), could you also send a
>> patch removing CONFIG_SOC_LS1021A from imx_v6_v7_config?
> [Alison Wang] I donât know CONFIG_ARCH_DMA_ADDR_T_64BIT=y is suggested.
> Could you help to explain the suggestion?

Please see
http://lists.infradead.org/pipermail/linux-arm-kernel/2011-November/072446.html

Instead of having CONFIG_ARCH_DMA_ADDR_T_64BIT=y in the defconfig, I
guess selecting it from Kconfig makes more sense:

diff --git a/arch/arm/mach-imx/Kconfig b/arch/arm/mach-imx/Kconfig
index 573536f1bb73..bb417e73af0f 100644
--- a/arch/arm/mach-imx/Kconfig
+++ b/arch/arm/mach-imx/Kconfig
@@ -563,6 +563,7 @@ config SOC_LS1021A
select HAVE_ARM_ARCH_TIMER
select PCI_DOMAINS if PCI
select ZONE_DMA if ARM_LPAE
+ select ARCH_DMA_ADDR_T_64BIT
help
This enables support for Freescale LS1021A processor.

Horia

--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/