linux-kernel - Re: [PATCH 4/6] drm: ci: Enable configs to fix mt8173 boot hang issue

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [day] [month] [year] [list]

Message-ID: <2dc6f9fa-06fb-44d8-97cb-6b5a983b7539@collabora.com>
Date:   Fri, 8 Sep 2023 17:56:36 +0530
From:   David Heidelberg <david.heidelberg@...labora.com>
To:     Vignesh Raman <vignesh.raman@...labora.com>,
        AngeloGioacchino Del Regno 
        <angelogioacchino.delregno@...labora.com>,
        dri-devel@...ts.freedesktop.org,
        Matthias Brugger <matthias.bgg@...il.com>
Cc:     helen.koike@...labora.com, guilherme.gallo@...labora.com,
        sergi.blanch.torne@...labora.com, daniels@...labora.com,
        emma@...olt.net, robclark@...edesktop.org,
        gustavo.padovan@...labora.com, robdclark@...gle.com,
        anholt@...gle.com, maarten.lankhorst@...ux.intel.com,
        mripard@...nel.org, tzimmermann@...e.de, airlied@...il.com,
        daniel@...ll.ch, corbet@....net, neil.armstrong@...aro.org,
        khilman@...libre.com, jbrunet@...libre.com,
        martin.blumenstingl@...glemail.com, heiko@...ech.de,
        linux-doc@...r.kernel.org, linux-kernel@...r.kernel.org,
        linux-arm-kernel@...ts.infradead.org,
        linux-mediatek@...ts.infradead.org,
        linux-amlogic@...ts.infradead.org,
        linux-rockchip@...ts.infradead.org
Subject: Re: [PATCH 4/6] drm: ci: Enable configs to fix mt8173 boot hang issue

On 08/09/2023 13:23, Vignesh Raman wrote:

> Hi Angelo,
>
> On 07/09/23 16:16, AngeloGioacchino Del Regno wrote:
>> Il 25/08/23 14:24, Vignesh Raman ha scritto:
>>> Enable regulator
>>> Enable MT6397 RTC driver
>>>
>>> Signed-off-by: Vignesh Raman <vignesh.raman@...labora.com>
>>> ---
>>>   drivers/gpu/drm/ci/arm64.config | 2 ++
>>>   1 file changed, 2 insertions(+)
>>>
>>> diff --git a/drivers/gpu/drm/ci/arm64.config 
>>> b/drivers/gpu/drm/ci/arm64.config
>>> index 817e18ddfd4f..ea7a6cceff40 100644
>>> --- a/drivers/gpu/drm/ci/arm64.config
>>> +++ b/drivers/gpu/drm/ci/arm64.config
>>> @@ -184,6 +184,8 @@ CONFIG_HW_RANDOM_MTK=y
>>>   CONFIG_MTK_DEVAPC=y
>>>   CONFIG_PWM_MTK_DISP=y
>>>   CONFIG_MTK_CMDQ=y
>>> +CONFIG_REGULATOR_DA9211=y
>>> +CONFIG_RTC_DRV_MT6397=y
>>
>> I wonder if it'd be a better idea to simply add those to the 
>> defconfig instead as
>>
>> CONFIG_REGULATOR_DA9211=m
>> CONFIG_RTC_DRV_MT6397=m
>>
>> Any opinion on this? Matthias? Anyone else?
>
> CONFIG_RTC_DRV_MT6397=m is already present in defconfig.
> We can also add CONFIG_REGULATOR_DA9211=m to defconfig.

We need add config snippet override for CONFIG_REGULATOR_DA9211=y 
because our CI don't have any initrd with modules, we just immediately 
continue boot from NFS, where we load rest of the boot non-essential 
modules.

For generic kernel, I assume CONFIG_REGULATOR_DA9211=m should be added.

David

>
> Regards,
> Vignesh

-- 
David Heidelberg
Consultant Software Engineer

Collabora Ltd.
Platinum Building, St John's Innovation Park, Cambridge CB4 0DS, UK
Registered in England & Wales, no. 5513718


Download attachment "OpenPGP_0x69F567861C1EC014.asc" of type "application/pgp-keys" (682 bytes)

Download attachment "OpenPGP_signature.asc" of type "application/pgp-signature" (229 bytes)