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] [thread-next>] [day] [month] [year] [list]
Message-ID: <7023673.LvFx2qVVIh@steina-w>
Date: Thu, 08 May 2025 10:07:27 +0200
From: Alexander Stein <alexander.stein@...tq-group.com>
To: Esben Haabendal <esben@...nix.com>
Cc: Catalin Marinas <catalin.marinas@....com>, Will Deacon <will@...nel.org>,
 Stefan Wahren <wahrenst@....net>, linux-arm-kernel@...ts.infradead.org,
 linux-kernel@...r.kernel.org, Shawn Guo <shawnguo@...nel.org>
Subject: Re: [PATCH 1/1] arm64: Kconfig: Enable PINCTRL on i.MX platforms

Hi Esben,

Am Donnerstag, 8. Mai 2025, 08:44:22 CEST schrieb Esben Haabendal:
> "Alexander Stein" <alexander.stein@...tq-group.com> writes:
> 
> > Hi Stefan,
> >
> > Am Mittwoch, 7. Mai 2025, 16:30:33 CEST schrieb Stefan Wahren:
> >> Hi Alexander,
> >>
> >> [add Shawn and Esben]
> >>
> >> Am 07.05.25 um 14:44 schrieb Alexander Stein:
> >> > Select PINCTRL for NXP i.MX SoCs.
> >> could you please explain the motivation behind your change?
> >>
> >> Is it related to this commit 17d21001891402 ("ARM: imx: Allow user to
> >> disable pinctrl")?
> >
> > Ah, thanks for the pointer. It might be the case.
> 
> The goal of the patch mentioned above was to be able to build a kernel
> for LS1021A without pinctrl framework enabled, as LS1021A does not have
> a pinctrl driver.
> 
> With your patch, that would not be possible anymore.

Why? LS1021A is arm, not arm64 which this patch is touching only.

BTW: Commit b77bd3ba762f3 ("ARM: imx: Re-introduce the PINCTRL selection")
is actually doing the same for arm as there is some fallout from
17d21001891402.

> > I noticed that, when using arch/arm64/defconfig and disabling all
> > platforms despite ARCH_MXC before running make olddefconfig,
> > CONFIG_PINCTRL gets disabled as well. No platform is enabling it. I
> > noticed this when building in yocto and non-IMX platforms are disabled
> > for build time reasons.
> 
> But is that something that needs to be fixed?
> 
> It sounds like quite a special use-case, and why not simply enable
> CONFIG_PINCTRL in that case then?

PINCTRL is crucial for any SoC to even boot, so this is an option which has
to be set if that platform is enabled.

Best regards
Alexander
-- 
TQ-Systems GmbH | Mühlstraße 2, Gut Delling | 82229 Seefeld, Germany
Amtsgericht München, HRB 105018
Geschäftsführer: Detlef Schneider, Rüdiger Stahl, Stefan Schneider
http://www.tq-group.com/



Powered by blists - more mailing lists

Powered by Openwall GNU/*/Linux Powered by OpenVZ