Re: [PATCH v5 4/5] pinctrl: qcom: ipq5018: allow it to be bulid on arm32

[exxxxkc <exxxxkc@getgoogleoff.me>]

On 10/4/24 7:44 PM, Linus Walleij wrote:
> On Thu, Oct 3, 2024 at 1:29 AM Karl Chan <exxxxkc@getgoogleoff.me> wrote:
>
>> There are some ipq5018 based device's firmware only can able to boot
>> arm32 but the pinctrl driver dont allow it to be compiled on
>> arm32.Therefore this patch needed for those devices.
>>
>> Signed-off-by: Karl Chan <exxxxkc@getgoogleoff.me>
> (...)
>> -       depends on ARM64 || COMPILE_TEST
>> +       depends on ARM || ARM64 || COMPILE_TEST
> Can we get some more context for this?
>
> Is this another one of those cases where the SoC is actually a
> 64bit core but vendors just choose to run it in 32bit mode because
> that is what they want or what is the reason here? AFAIK
> IPQ5018 is always Cortex-A53?
>
> I just want to know if this is something we should encourage
> or leave out-of-tree.
>
> Yours,
> Linus Walleij


Yeah it is like those MSM8916 based device that only has 32bit 
firmware.(e.g. samsung e5).
AFIK , other ipq5018 based linksys router has 64bit firmware but this 
one dont.
Also this router seem to be identical to linksys ER5500 but with 32bit 
firmware and isp exclusive.
(It is PCCW isp exclusive router (See 
https://support.linksys.com/kb/article/3723-en/) )