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

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

[Alison Wang]

Add Freescale LS1021A initial defconfig file.
The LS1021A SoC is a dual-core Cortex-A7 based processor.

LS1021A has some special configure options against common V7 SOCs,
such as CONFIG_THUMB2_KERNEL, CONFIG_VMSPLIT_2G, CONFIG_VFP...

Enable I2C, LPUART, eDMA, WDT, GIANFAR, Sound, DSPI at default.

Signed-off-by: Haikun Wang <haikun.wang@freescale.com>
Signed-off-by: Alison Wang <alison.wang@freescale.com>
---
Changes since v2:
- Enable PSCI, as PSCI is enabled in u-boot.
Changes since v1:
- Enable GIANFAR, Sound and DSPI.

 arch/arm/configs/ls1021a_defconfig | 161 +++++++++++++++++++++++++++++++++++++
 1 file changed, 161 insertions(+)
 create mode 100644 arch/arm/configs/ls1021a_defconfig

diff --git a/arch/arm/configs/ls1021a_defconfig b/arch/arm/configs/ls1021a_defconfig
new file mode 100644
index 0000000..5576f67
--- /dev/null
+++ b/arch/arm/configs/ls1021a_defconfig
@@ -0,0 +1,161 @@
+CONFIG_SYSVIPC=y
+CONFIG_POSIX_MQUEUE=y
+CONFIG_NO_HZ_IDLE=y
+CONFIG_HIGH_RES_TIMERS=y
+CONFIG_LOG_BUF_SHIFT=16
+CONFIG_BLK_DEV_INITRD=y
+CONFIG_CC_OPTIMIZE_FOR_SIZE=y
+CONFIG_SYSCTL_SYSCALL=y
+CONFIG_KALLSYMS_ALL=y
+CONFIG_EMBEDDED=y
+CONFIG_PROFILING=y
+CONFIG_OPROFILE=y
+CONFIG_KPROBES=y
+CONFIG_JUMP_LABEL=y
+CONFIG_MODULES=y
+CONFIG_MODULE_FORCE_LOAD=y
+CONFIG_MODULE_UNLOAD=y
+CONFIG_BLK_CMDLINE_PARSER=y
+CONFIG_ARCH_MXC=y
+CONFIG_SOC_LS1021A=y
+CONFIG_ARM_LPAE=y
+CONFIG_PCI=y
+CONFIG_PCI_MSI=y
+CONFIG_SMP=y
+CONFIG_VMSPLIT_2G=y
+CONFIG_ARM_PSCI=y
+CONFIG_PREEMPT_VOLUNTARY=y
+CONFIG_THUMB2_KERNEL=y
+CONFIG_HIGHMEM=y
+CONFIG_CLEANCACHE=y
+CONFIG_FRONTSWAP=y
+CONFIG_CMDLINE="console=ttyS0,115200"
+CONFIG_VFP=y
+CONFIG_NEON=y
+CONFIG_KERNEL_MODE_NEON=y
+CONFIG_BINFMT_MISC=y
+CONFIG_NET=y
+CONFIG_PACKET=y
+CONFIG_UNIX=y
+CONFIG_UNIX_DIAG=y
+CONFIG_XFRM_USER=y
+CONFIG_NET_KEY=y
+CONFIG_INET=y
+CONFIG_IP_MULTICAST=y
+CONFIG_IP_ADVANCED_ROUTER=y
+CONFIG_IP_PNP=y
+CONFIG_IP_PNP_DHCP=y
+CONFIG_IP_MROUTE=y
+CONFIG_INET_AH=y
+CONFIG_INET_ESP=y
+CONFIG_INET_IPCOMP=y
+CONFIG_INET_UDP_DIAG=m
+CONFIG_NETFILTER=y
+CONFIG_DEVTMPFS=y
+CONFIG_DEVTMPFS_MOUNT=y
+CONFIG_MTD=y
+CONFIG_MTD_CMDLINE_PARTS=y
+CONFIG_MTD_BLOCK=y
+CONFIG_MTD_CFI=y
+CONFIG_MTD_CFI_ADV_OPTIONS=y
+CONFIG_MTD_CFI_BE_BYTE_SWAP=y
+CONFIG_MTD_CFI_GEOMETRY=y
+CONFIG_MTD_CFI_INTELEXT=y
+CONFIG_MTD_CFI_AMDSTD=y
+CONFIG_MTD_CFI_STAA=y
+CONFIG_MTD_PHYSMAP_OF=y
+CONFIG_MTD_NAND=y
+CONFIG_BLK_DEV_LOOP=y
+CONFIG_BLK_DEV_RAM=y
+CONFIG_BLK_DEV_RAM_SIZE=256000
+CONFIG_BLK_DEV_SD=y
+CONFIG_CHR_DEV_SG=y
+CONFIG_ATA=y
+CONFIG_NETDEVICES=y
+CONFIG_GIANFAR=y
+CONFIG_VITESSE_PHY=y
+CONFIG_BROADCOM_PHY=y
+CONFIG_REALTEK_PHY=y
+CONFIG_NATIONAL_PHY=y
+CONFIG_MICREL_PHY=y
+CONFIG_MDIO_BUS_MUX_MMIOREG=y
+CONFIG_INPUT_EVDEV=y
+CONFIG_SERIO_SERPORT=m
+CONFIG_SERIAL_8250=y
+CONFIG_SERIAL_8250_CONSOLE=y
+CONFIG_SERIAL_8250_EXTENDED=y
+CONFIG_SERIAL_8250_SHARE_IRQ=y
+CONFIG_SERIAL_OF_PLATFORM=y
+CONFIG_SERIAL_FSL_LPUART=y
+CONFIG_SERIAL_FSL_LPUART_CONSOLE=y
+CONFIG_HW_RANDOM=y
+CONFIG_I2C=y
+CONFIG_I2C_CHARDEV=y
+CONFIG_I2C_MUX=y
+CONFIG_I2C_MUX_PCA954x=y
+CONFIG_I2C_IMX=y
+CONFIG_SPI=y
+CONFIG_SPI_FSL_DSPI=y
+CONFIG_GPIO_SYSFS=y
+CONFIG_SENSORS_LM90=y
+CONFIG_SENSORS_INA2XX=y
+CONFIG_WATCHDOG=y
+CONFIG_IMX2_WDT=y
+CONFIG_REGULATOR=y
+CONFIG_REGULATOR_DEBUG=y
+CONFIG_REGULATOR_FIXED_VOLTAGE=y
+CONFIG_SOUND=y
+# CONFIG_SOUND_OSS_CORE_PRECLAIM is not set
+CONFIG_SND=y
+CONFIG_SND_MIXER_OSS=y
+CONFIG_SND_PCM_OSS=y
+# CONFIG_SND_ARM is not set
+CONFIG_SND_SOC=y
+CONFIG_SND_SOC_FSL_SAI=y
+CONFIG_SND_SOC_SGTL5000=y
+CONFIG_SND_SIMPLE_CARD=y
+CONFIG_MMC=y
+CONFIG_MMC_SDHCI=y
+CONFIG_MMC_SDHCI_PLTFM=y
+CONFIG_MMC_SDHCI_OF_ARASAN=y
+CONFIG_NEW_LEDS=y
+CONFIG_LEDS_CLASS=y
+CONFIG_LEDS_TRIGGERS=y
+CONFIG_RTC_CLASS=y
+CONFIG_RTC_DRV_DS3232=y
+CONFIG_DMADEVICES=y
+CONFIG_FSL_EDMA=y
+CONFIG_CLK_QORIQ=y
+CONFIG_MEMORY=y
+CONFIG_EXT2_FS=y
+CONFIG_EXT2_FS_XATTR=y
+CONFIG_EXT3_FS=y
+CONFIG_EXT4_FS=y
+CONFIG_FANOTIFY=y
+CONFIG_ISO9660_FS=m
+CONFIG_JOLIET=y
+CONFIG_ZISOFS=y
+CONFIG_UDF_FS=m
+CONFIG_MSDOS_FS=y
+CONFIG_VFAT_FS=y
+CONFIG_NTFS_FS=m
+CONFIG_TMPFS=y
+CONFIG_TMPFS_POSIX_ACL=y
+CONFIG_JFFS2_FS=y
+CONFIG_NFS_FS=y
+CONFIG_NFS_V4=y
+CONFIG_ROOT_NFS=y
+CONFIG_NLS_DEFAULT="cp437"
+CONFIG_NLS_CODEPAGE_437=y
+CONFIG_NLS_ASCII=y
+CONFIG_NLS_ISO8859_1=y
+CONFIG_NLS_ISO8859_2=y
+CONFIG_NLS_ISO8859_15=y
+CONFIG_NLS_UTF8=y
+CONFIG_DEBUG_SECTION_MISMATCH=y
+CONFIG_MAGIC_SYSRQ=y
+CONFIG_CRYPTO_LZO=y
+CONFIG_CRC_CCITT=m
+CONFIG_CRC_T10DIF=y
+CONFIG_CRC7=m
+CONFIG_LIBCRC32C=m
-- 
2.1.0.27.g96db324

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

[Horia Geantă]

On 8/7/2015 1:03 PM, Alison Wang wrote:
> Add Freescale LS1021A initial defconfig file.
> The LS1021A SoC is a dual-core Cortex-A7 based processor.
> 
> LS1021A has some special configure options against common V7 SOCs,
> such as CONFIG_THUMB2_KERNEL, CONFIG_VMSPLIT_2G, CONFIG_VFP...
> 
> Enable I2C, LPUART, eDMA, WDT, GIANFAR, Sound, DSPI at default.
> 
> Signed-off-by: Haikun Wang <haikun.wang@freescale.com>
> Signed-off-by: Alison Wang <alison.wang@freescale.com>
> ---
> Changes since v2:
> - Enable PSCI, as PSCI is enabled in u-boot.
> Changes since v1:
> - Enable GIANFAR, Sound and DSPI.

Looks like CONFIG_ARCH_DMA_ADDR_T_64BIT=y is missing.
For e.g. CAAM (crypto engine) is a DMA device able to use 40-bit bus
addresses.
Note that ARCH_DMA_ADDR_T_64BIT does not depend / is not selected by
ARM_LPAE.

Thanks,
Horia

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

[Horia Geantă]

On 8/12/2015 4:20 PM, Horia Geantă wrote:
> On 8/7/2015 1:03 PM, Alison Wang wrote:
>> Add Freescale LS1021A initial defconfig file.
>> The LS1021A SoC is a dual-core Cortex-A7 based processor.
>>
>> LS1021A has some special configure options against common V7 SOCs,
>> such as CONFIG_THUMB2_KERNEL, CONFIG_VMSPLIT_2G, CONFIG_VFP...
>>
>> Enable I2C, LPUART, eDMA, WDT, GIANFAR, Sound, DSPI at default.
>>
>> Signed-off-by: Haikun Wang <haikun.wang@freescale.com>
>> Signed-off-by: Alison Wang <alison.wang@freescale.com>
>> ---
>> Changes since v2:
>> - Enable PSCI, as PSCI is enabled in u-boot.
>> Changes since v1:
>> - Enable GIANFAR, Sound and DSPI.
> 
> Looks like CONFIG_ARCH_DMA_ADDR_T_64BIT=y is missing.
> For e.g. CAAM (crypto engine) is a DMA device able to use 40-bit bus
> addresses.
> Note that ARCH_DMA_ADDR_T_64BIT does not depend / is not selected by
> ARM_LPAE.

Another thing:
Commit 385c0fb0e2c4 ("ARM: imx_v6_v7_defconfig: Select LS1021A") added
CONFIG_SOC_LS1021A in imx_v6_v7_defconfig.

Besides this, booting a LS1021 AQDS board (Rev2) using
imx_v6_v7_defconfig did not work (stuck at "Starting kernel").

Horia

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

[Fabio Estevam]

On Fri, Aug 14, 2015 at 2:06 PM, Horia Geantă
<horia.geanta@freescale.com> wrote:

> Another thing:
> Commit 385c0fb0e2c4 ("ARM: imx_v6_v7_defconfig: Select LS1021A") added
> CONFIG_SOC_LS1021A in imx_v6_v7_defconfig.
>
> Besides this, booting a LS1021 AQDS board (Rev2) using
> imx_v6_v7_defconfig did not work (stuck at "Starting kernel").

What would be the recommended approach?

- Fix imx_v6_v7_defconfig so that it can make LS1021A to boot

or

- Use this proposed ls1021a_defconfig and remove LS1021A from
imx_v6_v7_defconfig?

Regards,

Fabio Estevam

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

[Huan Wang]

[-- Warning: decoded text below may be mangled, UTF-8 assumed --]
[-- Attachment #1: Type: text/plain; charset="utf-8", Size: 935 bytes --]

Hi, Fabio,

	Based on the analysis and discussion, we prefer to use ls1021a_defconfig and remove LS1021A from imx_v6_v7_defconfig.

	Thanks.

Best Regards,
Alison Wang

> On Fri, Aug 14, 2015 at 2:06 PM, Horia Geantă
> <horia.geanta@freescale.com> wrote:
> 
> > Another thing:
> > Commit 385c0fb0e2c4 ("ARM: imx_v6_v7_defconfig: Select LS1021A")
> added
> > CONFIG_SOC_LS1021A in imx_v6_v7_defconfig.
> >
> > Besides this, booting a LS1021 AQDS board (Rev2) using
> > imx_v6_v7_defconfig did not work (stuck at "Starting kernel").
> 
> What would be the recommended approach?
> 
> - Fix imx_v6_v7_defconfig so that it can make LS1021A to boot
> 
> or
> 
> - Use this proposed ls1021a_defconfig and remove LS1021A from
> imx_v6_v7_defconfig?
> 
> Regards,
> 
> Fabio Estevam
ÿôèº{.nÇ+‰·Ÿ®‰­†+%ŠËÿ±éݶ\x17¥Šwÿº{.nÇ+‰·¥Š{±þG«éÿŠ{ayº\x1dʇڙë,j\a­¢f£¢·hšïêÿ‘êçz_è®\x03(­éšŽŠÝ¢j"ú\x1a¶^[m§ÿÿ¾\a«þG«éÿ¢¸?™¨è­Ú&£ø§~á¶iO•æ¬z·švØ^\x14\x04\x1a¶^[m§ÿÿÃ\fÿ¶ìÿ¢¸?–I¥

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

[Horia Geantă]

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?

> 
> 	Thanks.
> 
> Best Regards,
> Alison Wang
> 
>> On Fri, Aug 14, 2015 at 2:06 PM, Horia Geantă
>> <horia.geanta@freescale.com> wrote:
>>
>>> Another thing:
>>> Commit 385c0fb0e2c4 ("ARM: imx_v6_v7_defconfig: Select LS1021A")
>> added
>>> CONFIG_SOC_LS1021A in imx_v6_v7_defconfig.
>>>
>>> Besides this, booting a LS1021 AQDS board (Rev2) using
>>> imx_v6_v7_defconfig did not work (stuck at "Starting kernel").
>>
>> What would be the recommended approach?
>>
>> - Fix imx_v6_v7_defconfig so that it can make LS1021A to boot
>>
>> or
>>
>> - Use this proposed ls1021a_defconfig and remove LS1021A from
>> imx_v6_v7_defconfig?
>>
>> Regards,
>>
>> Fabio Estevam

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

[Huan Wang]

[-- Warning: decoded text below may be mangled, UTF-8 assumed --]
[-- Attachment #1: Type: text/plain; charset="utf-8", Size: 1534 bytes --]

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?

I need Fabio's confirmation about removing CONFIG_SOC_LS1021A from imx_v6_v7_config.

Fabio, Do you agree it?
> 
> >
> > 	Thanks.
> >
> > Best Regards,
> > Alison Wang
> >
> >> On Fri, Aug 14, 2015 at 2:06 PM, Horia Geantă
> >> <horia.geanta@freescale.com> wrote:
> >>
> >>> Another thing:
> >>> Commit 385c0fb0e2c4 ("ARM: imx_v6_v7_defconfig: Select LS1021A")
> >> added
> >>> CONFIG_SOC_LS1021A in imx_v6_v7_defconfig.
> >>>
> >>> Besides this, booting a LS1021 AQDS board (Rev2) using
> >>> imx_v6_v7_defconfig did not work (stuck at "Starting kernel").
> >>
> >> What would be the recommended approach?
> >>
> >> - Fix imx_v6_v7_defconfig so that it can make LS1021A to boot
> >>
> >> or
> >>
> >> - Use this proposed ls1021a_defconfig and remove LS1021A from
> >> imx_v6_v7_defconfig?
> >>
> >> Regards,
> >>
> >> Fabio Estevam
> 

ÿôèº{.nÇ+‰·Ÿ®‰­†+%ŠËÿ±éݶ\x17¥Šwÿº{.nÇ+‰·¥Š{±þG«éÿŠ{ayº\x1dʇڙë,j\a­¢f£¢·hšïêÿ‘êçz_è®\x03(­éšŽŠÝ¢j"ú\x1a¶^[m§ÿÿ¾\a«þG«éÿ¢¸?™¨è­Ú&£ø§~á¶iO•æ¬z·švØ^\x14\x04\x1a¶^[m§ÿÿÃ\fÿ¶ìÿ¢¸?–I¥

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

[Horia Geantă]

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

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

[Fabio Estevam]

On Fri, Aug 28, 2015 at 5:04 AM, Huan Wang <alison.wang@freescale.com> wrote:

> I need Fabio's confirmation about removing CONFIG_SOC_LS1021A from imx_v6_v7_config.
>
> Fabio, Do you agree it?

Yes, if Shawn accepts ls1021a_defconfig then we should remove
CONFIG_SOC_LS1021A from imx_v6_v7_config.

Regards,

Fabio Estevam

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

[Huan Wang]

[-- Warning: decoded text below may be mangled, UTF-8 assumed --]
[-- Attachment #1: Type: text/plain; charset="utf-8", Size: 582 bytes --]

Hi, Shawn,

> On Fri, Aug 28, 2015 at 5:04 AM, Huan Wang <alison.wang@freescale.com>
> wrote:
> 
> > I need Fabio's confirmation about removing CONFIG_SOC_LS1021A from
> imx_v6_v7_config.
> >
> > Fabio, Do you agree it?
> 
> Yes, if Shawn accepts ls1021a_defconfig then we should remove
> CONFIG_SOC_LS1021A from imx_v6_v7_config.
> 
[Alison Wang] Could you help to review this patch?

Thanks.


Best Regards,
Alison Wang
ÿôèº{.nÇ+‰·Ÿ®‰­†+%ŠËÿ±éݶ\x17¥Šwÿº{.nÇ+‰·¥Š{±þG«éÿŠ{ayº\x1dʇڙë,j\a­¢f£¢·hšïêÿ‘êçz_è®\x03(­éšŽŠÝ¢j"ú\x1a¶^[m§ÿÿ¾\a«þG«éÿ¢¸?™¨è­Ú&£ø§~á¶iO•æ¬z·švØ^\x14\x04\x1a¶^[m§ÿÿÃ\fÿ¶ìÿ¢¸?–I¥

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

[Shawn Guo]

On Wed, Sep 16, 2015 at 08:22:56AM +0000, Huan Wang wrote:
> Hi, Shawn,
> 
> > On Fri, Aug 28, 2015 at 5:04 AM, Huan Wang <alison.wang@freescale.com>
> > wrote:
> > 
> > > I need Fabio's confirmation about removing CONFIG_SOC_LS1021A from
> > imx_v6_v7_config.
> > >
> > > Fabio, Do you agree it?
> > 
> > Yes, if Shawn accepts ls1021a_defconfig then we should remove
> > CONFIG_SOC_LS1021A from imx_v6_v7_config.
> > 
> [Alison Wang] Could you help to review this patch?

Can you please resend the patch with some explanation in the commit log
on why we need a new defconfig, i.e. why imx_v6_v7_config doesn't work
for LS1021A.

Shawn

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

[Huan Wang]

Hi, Shawn,

> On Wed, Sep 16, 2015 at 08:22:56AM +0000, Huan Wang wrote:
> > Hi, Shawn,
> >
> > > On Fri, Aug 28, 2015 at 5:04 AM, Huan Wang
> > > <alison.wang@freescale.com>
> > > wrote:
> > >
> > > > I need Fabio's confirmation about removing CONFIG_SOC_LS1021A
> from
> > > imx_v6_v7_config.
> > > >
> > > > Fabio, Do you agree it?
> > >
> > > Yes, if Shawn accepts ls1021a_defconfig then we should remove
> > > CONFIG_SOC_LS1021A from imx_v6_v7_config.
> > >
> > [Alison Wang] Could you help to review this patch?
> 
> Can you please resend the patch with some explanation in the commit log
> on why we need a new defconfig, i.e. why imx_v6_v7_config doesn't work
> for LS1021A.
[Alison Wang] I sent v4 patch. Please help to review.

Best Regards,
Alison Wang