From: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>
To: Alexander Stein <alexander.stein@ew.tq-group.com>,
Rob Herring <robh+dt@kernel.org>,
Krzysztof Kozlowski <krzysztof.kozlowski+dt@linaro.org>,
Arnd Bergmann <arnd@arndb.de>, Olof Johansson <olof@lixom.net>,
Shawn Guo <shawnguo@kernel.org>, Li Yang <leoyang.li@nxp.com>,
Russell King <linux@armlinux.org.uk>, Marek Vasut <marex@denx.de>,
Marcel Ziswiler <marcel.ziswiler@toradex.com>
Cc: soc@kernel.org, devicetree@vger.kernel.org,
linux-arm-kernel@lists.infradead.org
Subject: Re: [PATCH 8/9] ARM: multi_v7_defconfig: Add options to support TQMLS102xA series
Date: Fri, 20 Jan 2023 12:35:41 +0100 [thread overview]
Message-ID: <e08e6325-4b2b-c1ce-b33a-877de2c0babe@linaro.org> (raw)
In-Reply-To: <2168770.1BCLMh4Saa@steina-w>
On 20/01/2023 12:12, Alexander Stein wrote:
>>> CONFIG_GPIO_MXC=y
>>> CONFIG_GPIO_RCAR=y
>>> CONFIG_GPIO_SYSCON=y
>>>
>>> @@ -493,6 +497,7 @@ CONFIG_GPIO_PCA953X=y
>>>
>>> CONFIG_GPIO_PCA953X_IRQ=y
>>> CONFIG_GPIO_PCF857X=y
>>> CONFIG_GPIO_PALMAS=y
>>>
>>> +CONFIG_GPIO_STMPE=y
>>
>> module
>
> That one as well, it's just a bool.
OK
>
>>> CONFIG_GPIO_TPS6586X=y
>>> CONFIG_GPIO_TPS65910=y
>>> CONFIG_GPIO_TWL4030=y
>>>
>>> @@ -533,6 +538,7 @@ CONFIG_SENSORS_INA2XX=m
>>>
>>> CONFIG_CPU_THERMAL=y
>>> CONFIG_DEVFREQ_THERMAL=y
>>> CONFIG_IMX_THERMAL=y
>>>
>>> +CONFIG_QORIQ_THERMAL=m
>>>
>>> CONFIG_ROCKCHIP_THERMAL=y
>>> CONFIG_RCAR_THERMAL=y
>>> CONFIG_ARMADA_THERMAL=y
>>>
>>> @@ -821,6 +827,8 @@ CONFIG_SND_SOC_MSM8916_WCD_ANALOG=m
>>>
>>> CONFIG_SND_SOC_MSM8916_WCD_DIGITAL=m
>>> CONFIG_SND_SOC_SGTL5000=m
>>> CONFIG_SND_SOC_STI_SAS=m
>>>
>>> +CONFIG_SND_SOC_TLV320AIC32X4=m
>>> +CONFIG_SND_SOC_TLV320AIC32X4_I2C=m
>>>
>>> CONFIG_SND_SOC_WM8978=m
>>> CONFIG_SND_AUDIO_GRAPH_CARD=m
>>> CONFIG_USB=y
>>>
>>> @@ -830,6 +838,7 @@ CONFIG_USB_XHCI_MVEBU=y
>>>
>>> CONFIG_USB_XHCI_TEGRA=m
>>> CONFIG_USB_BRCMSTB=m
>>> CONFIG_USB_EHCI_HCD=y
>>>
>>> +CONFIG_USB_EHCI_FSL=y
>>
>> module
>
> Well, apparently module autoload is broken due to wrong modalias value of ehci
> node created by fsl-mph-dr-of. This setting this as a module is currently not
> working.
But this is not valid argument for defconfig change. This must be
module. If autoloading does not work, fix that one.
>
>>> CONFIG_USB_EHCI_HCD_STI=y
>>> CONFIG_USB_EHCI_EXYNOS=m
>>> CONFIG_USB_EHCI_MV=m
>>>
>>> @@ -933,6 +942,8 @@ CONFIG_NEW_LEDS=y
>>>
>>> CONFIG_LEDS_CLASS=y
>>> CONFIG_LEDS_CLASS_FLASH=m
>>> CONFIG_LEDS_CPCAP=m
>>>
>>> +CONFIG_LEDS_PCA9532=m
>>> +CONFIG_LEDS_PCA9532_GPIO=y
>>>
>>> CONFIG_LEDS_GPIO=y
>>> CONFIG_LEDS_PWM=y
>>> CONFIG_LEDS_MAX8997=m
>>>
>>> @@ -949,6 +960,7 @@ CONFIG_LEDS_TRIGGER_DEFAULT_ON=y
>>>
>>> CONFIG_LEDS_TRIGGER_TRANSIENT=y
>>> CONFIG_LEDS_TRIGGER_CAMERA=y
>>> CONFIG_EDAC=y
>>>
>>> +CONFIG_EDAC_LAYERSCAPE=y
>>>
>>> CONFIG_EDAC_HIGHBANK_MC=y
>>> CONFIG_EDAC_HIGHBANK_L2=y
>>> CONFIG_RTC_CLASS=y
>>>
>>> @@ -962,6 +974,7 @@ CONFIG_RTC_DRV_MAX8997=m
>>>
>>> CONFIG_RTC_DRV_MAX77686=y
>>> CONFIG_RTC_DRV_RK808=m
>>> CONFIG_RTC_DRV_RS5C372=m
>>>
>>> +CONFIG_RTC_DRV_PCF85063=y
>>
>> module
>
> Okay, module it is. So I assume it is acceptable the system time is set much
> later after mounting rootfs then?
Modules can be loaded from initramfs, not rootfs.
Defconfig=y is only for critical stuff needed to bring initramfs. Time
is not needed for initramfs.
Best regards,
Krzysztof
next prev parent reply other threads:[~2023-01-20 11:35 UTC|newest]
Thread overview: 30+ messages / expand[flat|nested] mbox.gz Atom feed top
2023-01-19 14:42 [PATCH 0/9] TQMLS1021A support Alexander Stein
2023-01-19 14:42 ` [PATCH 1/9] dt-bindings: arm: fsl: add TQ-Systems LS1021A board Alexander Stein
2023-01-19 16:59 ` Krzysztof Kozlowski
2023-01-20 8:58 ` Alexander Stein
2023-01-20 9:07 ` Krzysztof Kozlowski
2023-01-19 14:42 ` [PATCH 2/9] ARM: dts: ls1021a: add TQ-Systems MBLS102xA device tree Alexander Stein
2023-01-19 17:03 ` Krzysztof Kozlowski
2023-01-20 10:08 ` Alexander Stein
2023-01-20 10:21 ` Krzysztof Kozlowski
2023-01-19 14:42 ` [PATCH 3/9] ARM: dts: ls1021a: add TQMLS1021A flash partition layout Alexander Stein
2023-01-19 14:42 ` [PATCH 4/9] ARM: dts: ls1021a: add TQMLS1021A/MBLS102xA LVDS TM070JVHG33 overlay Alexander Stein
2023-01-19 14:42 ` [PATCH 5/9] ARM: dts: ls1021a: add TQMLS1021A/MBLS102xA HDMI overlay Alexander Stein
2023-01-19 14:42 ` [PATCH 6/9] ARM: dts: ls1021a: add TQMLS1021A/MBLS102xA LVDS CDTECH DC44 overlay Alexander Stein
2023-01-19 14:42 ` [PATCH 7/9] ARM: dts: ls1021a: add TQMLS1021A/MBLS102xA LVDS CDTECH FC21 overlay Alexander Stein
2023-01-19 14:42 ` [PATCH 8/9] ARM: multi_v7_defconfig: Add options to support TQMLS102xA series Alexander Stein
2023-01-19 17:04 ` Krzysztof Kozlowski
2023-01-20 11:12 ` Alexander Stein
2023-01-20 11:35 ` Krzysztof Kozlowski [this message]
2023-01-20 14:57 ` Russell King (Oracle)
2023-01-21 20:27 ` Krzysztof Kozlowski
2023-01-21 22:33 ` Russell King (Oracle)
2023-01-19 14:42 ` [PATCH 9/9] [DNI] ARM: multi_v7_defconfig: Enable CONFIG_ARM_LPAE for multi_v7_config Alexander Stein
2023-01-19 15:09 ` Arnd Bergmann
2023-01-19 15:27 ` Alexander Stein
2023-01-19 16:07 ` Arnd Bergmann
2023-01-20 12:43 ` Alexander Stein
2023-01-20 14:00 ` Arnd Bergmann
2023-01-24 10:30 ` Alexander Stein
2023-01-24 11:37 ` Arnd Bergmann
2023-01-19 16:00 ` Russell King (Oracle)
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=e08e6325-4b2b-c1ce-b33a-877de2c0babe@linaro.org \
--to=krzysztof.kozlowski@linaro.org \
--cc=alexander.stein@ew.tq-group.com \
--cc=arnd@arndb.de \
--cc=devicetree@vger.kernel.org \
--cc=krzysztof.kozlowski+dt@linaro.org \
--cc=leoyang.li@nxp.com \
--cc=linux-arm-kernel@lists.infradead.org \
--cc=linux@armlinux.org.uk \
--cc=marcel.ziswiler@toradex.com \
--cc=marex@denx.de \
--cc=olof@lixom.net \
--cc=robh+dt@kernel.org \
--cc=shawnguo@kernel.org \
--cc=soc@kernel.org \
/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