linux-kernel - Re: [PATCH v4 12/25] scsi: ufs: mediatek: Remove vendor kernel quirks cruft

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: <13960383.uLZWGnKmhe@workhorse>
Date: Thu, 08 Jan 2026 10:28:00 +0100
From: Nicolas Frattaroli <nicolas.frattaroli@...labora.com>
To: "chu.stanley@...il.com" <chu.stanley@...il.com>,
 "robh@...nel.org" <robh@...nel.org>,
 Chunfeng Yun (云春峰) <Chunfeng.Yun@...iatek.com>,
 "kishon@...nel.org" <kishon@...nel.org>,
 "James.Bottomley@...senPartnership.com"
 <James.Bottomley@...senpartnership.com>,
 "bvanassche@....org" <bvanassche@....org>,
 AngeloGioacchino Del Regno <angelogioacchino.delregno@...labora.com>,
 "neil.armstrong@...aro.org" <neil.armstrong@...aro.org>,
 "conor+dt@...nel.org" <conor+dt@...nel.org>,
 Chaotian Jing (井朝天) <Chaotian.Jing@...iatek.com>,
 "lgirdwood@...il.com" <lgirdwood@...il.com>,
 "vkoul@...nel.org" <vkoul@...nel.org>,
 "krzk+dt@...nel.org" <krzk+dt@...nel.org>,
 "p.zabel@...gutronix.de" <p.zabel@...gutronix.de>,
 "alim.akhtar@...sung.com" <alim.akhtar@...sung.com>,
 "matthias.bgg@...il.com" <matthias.bgg@...il.com>,
 "avri.altman@....com" <avri.altman@....com>,
 "martin.petersen@...cle.com" <martin.petersen@...cle.com>,
 "broonie@...nel.org" <broonie@...nel.org>,
 Peter Wang (王信友) <peter.wang@...iatek.com>
Cc: "linux-scsi@...r.kernel.org" <linux-scsi@...r.kernel.org>,
 "devicetree@...r.kernel.org" <devicetree@...r.kernel.org>,
 "linux-kernel@...r.kernel.org" <linux-kernel@...r.kernel.org>,
 "linux-arm-kernel@...ts.infradead.org"
 <linux-arm-kernel@...ts.infradead.org>,
 "linux-phy@...ts.infradead.org" <linux-phy@...ts.infradead.org>,
 "linux-mediatek@...ts.infradead.org" <linux-mediatek@...ts.infradead.org>,
 Louis-Alexis Eyraud <louisalexis.eyraud@...labora.com>,
 "kernel@...labora.com" <kernel@...labora.com>
Subject:
 Re: [PATCH v4 12/25] scsi: ufs: mediatek: Remove vendor kernel quirks cruft

On Tuesday, 6 January 2026 14:25:22 Central European Standard Time Peter Wang (王信友) wrote:
> On Thu, 2025-12-18 at 13:55 +0100, Nicolas Frattaroli wrote:
> > 
> > Both ufs_mtk_vreg_fix_vcc and ufs_mtk_vreg_fix_vccqx look like they
> > are
> > vendor kernel hacks to work around existing downstream device trees.
> > Mainline does not need or want them, so remove them.
> > 
> 
> Hi Nicolas,
> 
> This is a flexible approach to implement one software supporting
> multiple
> hardware configurations. Because you cannot guarantee that your SOC
> will 
> always use UFS 2.0 or UFS 3.0, or that the PMIC you use will only have
> one set.

By "one software supporting multiple hardware configurations", do you
mean one device tree? Because if so, I don't think that's a good idea.
Device tree is meant to describe non-enumerable hardware.

Even if you want to make it easier for your customers to ship one image
for several SKUs, there's better ways to do this than having drivers
fix up individual DT nodes. The platform firmware like u-boot can choose
a DT based on differences it can probe. E.g. on Radxa ROCK 5B/5B+ boards,
we have u-boot choose between the 5B and 5B+ DT based on whether LPDDR5
is present, as 5B does not have LPDDR5, so as long as u-boot is told it's
either a ROCK 5B or ROCK 5B+, it can figure out which one specifically based
on that. Similarly, for whichever boards this is for, there may be
differences that can be probed to disambiguate between several SKUs of the
board as long as it's known it must be at least one of those SKUs.

> 
> Thanks
> Peter
> 
> 
>