linux-kernel - Re: [PATCH 8/9] arm64: dts: mediatek: mt8183-kukui: Fix pull-down/up-adv values

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 for Android: free password hash cracker in your pocket

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

Message-ID: <CACRpkdbWctNH0XJfcHfVJM9Etp0WCXpdyhhyaQemH-Xc0LDr0A@mail.gmail.com>
Date: Mon, 18 Aug 2025 17:22:05 +0200
From: Linus Walleij <linus.walleij@...aro.org>
To: Chen-Yu Tsai <wenst@...omium.org>
Cc: Julien Massot <julien.massot@...labora.com>, kernel@...labora.com, 
	Michael Turquette <mturquette@...libre.com>, Stephen Boyd <sboyd@...nel.org>, 
	Rob Herring <robh@...nel.org>, Krzysztof Kozlowski <krzk+dt@...nel.org>, Conor Dooley <conor+dt@...nel.org>, 
	Matthias Brugger <matthias.bgg@...il.com>, 
	AngeloGioacchino Del Regno <angelogioacchino.delregno@...labora.com>, Ikjoon Jang <ikjn@...omium.org>, 
	Enric Balletbo i Serra <eballetbo@...nel.org>, Weiyi Lu <weiyi.lu@...iatek.com>, 
	Eugen Hristev <eugen.hristev@...aro.org>, Liam Girdwood <lgirdwood@...il.com>, 
	Mark Brown <broonie@...nel.org>, Julien Massot <jmassot@...labora.com>, 
	Sean Wang <sean.wang@...nel.org>, linux-clk@...r.kernel.org, 
	devicetree@...r.kernel.org, linux-kernel@...r.kernel.org, 
	linux-arm-kernel@...ts.infradead.org, linux-mediatek@...ts.infradead.org, 
	linux-sound@...r.kernel.org, linux-gpio@...r.kernel.org
Subject: Re: [PATCH 8/9] arm64: dts: mediatek: mt8183-kukui: Fix
 pull-down/up-adv values

On Wed, Aug 6, 2025 at 8:38 AM Chen-Yu Tsai <wenst@...omium.org> wrote:
> On Fri, Aug 1, 2025 at 7:18 PM Julien Massot wrote

> >                 pins-clk {
> >                         pinmux = <PINMUX_GPIO124__FUNC_MSDC0_CLK>;
> >                         drive-strength = <MTK_DRIVE_14mA>;
> > -                       mediatek,pull-down-adv = <10>;
> > +                       mediatek,pull-down-adv = <2>;
>
>         bias-pull-down = <MTK_PUPD_SET_R1R0_10>;
>
> and so on.
>
> ChenYu

I agree with ChenYu, the more standardized properties are the better it is.

All the custom properties makes sense for an engineer working with just
that one SoC (like the SoC vendor...) but for field engineers who have
to use different SoCs every day this is just a big mess for the mind.

The standard properties are clear, concise and tell you exactly what
they are about.

The argument should be in Ohms though, according to the standard
bindings, but maybe the value of MTK_PUPD_SET_R1R0_10 is
something like that?

Yours,
Linus Walleij