From: Nicolas Frattaroli <nicolas.frattaroli@collabora.com>
To: "Martin K. Petersen" <martin.petersen@oracle.com>
Cc: Rob Herring <robh@kernel.org>,
Alim Akhtar <alim.akhtar@samsung.com>,
Avri Altman <avri.altman@wdc.com>,
Bart Van Assche <bvanassche@acm.org>,
Krzysztof Kozlowski <krzk+dt@kernel.org>,
Conor Dooley <conor+dt@kernel.org>,
Matthias Brugger <matthias.bgg@gmail.com>,
AngeloGioacchino Del Regno
<angelogioacchino.delregno@collabora.com>,
Chunfeng Yun <chunfeng.yun@mediatek.com>,
Vinod Koul <vkoul@kernel.org>,
Kishon Vijay Abraham I <kishon@kernel.org>,
Peter Wang <peter.wang@mediatek.com>,
Stanley Jhu <chu.stanley@gmail.com>,
"James E.J. Bottomley" <James.Bottomley@hansenpartnership.com>,
"Martin K. Petersen" <martin.petersen@oracle.com>,
Philipp Zabel <p.zabel@pengutronix.de>,
Liam Girdwood <lgirdwood@gmail.com>,
Mark Brown <broonie@kernel.org>,
Chaotian Jing <Chaotian.Jing@mediatek.com>,
Neil Armstrong <neil.armstrong@linaro.org>,
Louis-Alexis Eyraud <louisalexis.eyraud@collabora.com>,
kernel@collabora.com, linux-scsi@vger.kernel.org,
devicetree@vger.kernel.org, linux-kernel@vger.kernel.org,
linux-arm-kernel@lists.infradead.org,
linux-mediatek@lists.infradead.org,
linux-phy@lists.infradead.org,
Conor Dooley <conor.dooley@microchip.com>
Subject: Re: [PATCH v9 03/23] dt-bindings: ufs: mediatek,ufs: Add mt8196 variant
Date: Mon, 09 Mar 2026 11:04:14 +0100 [thread overview]
Message-ID: <5973984.DvuYhMxLoT@workhorse> (raw)
In-Reply-To: <yq14imrwp3z.fsf@ca-mkp.ca.oracle.com>
On Saturday, 7 March 2026 19:01:17 Central European Standard Time Martin K. Petersen wrote:
>
> Nicolas,
>
> >> "ufs" is redundant as all the clocks are for UFS. Same comment on prior
> >> patch.
> >
> > Is this naming a big enough concern to block this series with two
> > explicit acks on this patch that fixes a wholly broken and useless
> > binding?
>
> It is if it comes from one of the DT maintainers.
>
> > I am trying to put out this dumpster fire of a downstream turd that
> > made its way into mainline as the review process has been completely
> > subverted, and is only getting worse with each passing month
>
> This has to stop. Please read Documentation/process/code-of-conduct.rst.
>
>
I apologise for my tone, it's my frustration getting the better of me.
I'll be handing off this series to someone else, so you won't have to
deal with me anymore.
I do ask however that you don't apply patches from MediaTek blindly;
if there's code to read an OF property, and that OF property is not
in the binding, then the patch should be rejected, even if there's an
Ack from the MediaTek maintainer.
next prev parent reply other threads:[~2026-03-09 10:04 UTC|newest]
Thread overview: 35+ messages / expand[flat|nested] mbox.gz Atom feed top
2026-03-06 13:24 [PATCH v9 00/23] MediaTek UFS Cleanup and MT8196 Enablement Nicolas Frattaroli
2026-03-06 13:24 ` [PATCH v9 01/23] dt-bindings: phy: Add mediatek,mt8196-ufsphy variant Nicolas Frattaroli
2026-03-06 13:24 ` [PATCH v9 02/23] dt-bindings: ufs: mediatek,ufs: Complete the binding Nicolas Frattaroli
2026-03-06 13:24 ` [PATCH v9 03/23] dt-bindings: ufs: mediatek,ufs: Add mt8196 variant Nicolas Frattaroli
2026-03-06 16:33 ` Rob Herring
2026-03-06 18:37 ` Nicolas Frattaroli
2026-03-07 18:01 ` Martin K. Petersen
2026-03-09 10:04 ` Nicolas Frattaroli [this message]
2026-03-10 18:21 ` Rob Herring
2026-03-16 9:34 ` AngeloGioacchino Del Regno
2026-03-10 18:10 ` Rob Herring
2026-03-06 13:24 ` [PATCH v9 04/23] scsi: ufs: mediatek: Move MTK_SIP_UFS_CONTROL to mtk_sip_svc.h Nicolas Frattaroli
2026-03-06 13:24 ` [PATCH v9 05/23] phy: mediatek: ufs: Add support for resets Nicolas Frattaroli
2026-03-06 13:24 ` [PATCH v9 06/23] scsi: ufs: mediatek: Rework resets Nicolas Frattaroli
2026-03-06 13:24 ` [PATCH v9 07/23] scsi: ufs: mediatek: Rework 0.9V regulator Nicolas Frattaroli
2026-03-06 13:24 ` [PATCH v9 08/23] scsi: ufs: mediatek: Rework init function Nicolas Frattaroli
2026-03-06 13:24 ` [PATCH v9 09/23] scsi: ufs: mediatek: Rework the crypt-boost stuff Nicolas Frattaroli
2026-03-06 13:24 ` [PATCH v9 10/23] scsi: ufs: mediatek: Handle misc host voltage regulators Nicolas Frattaroli
2026-03-06 13:24 ` [PATCH v9 11/23] scsi: ufs: mediatek: Remove undocumented downstream reset cruft Nicolas Frattaroli
2026-03-06 13:24 ` [PATCH v9 12/23] scsi: ufs: mediatek: Remove vendor kernel quirks cruft Nicolas Frattaroli
2026-03-06 13:24 ` [PATCH v9 13/23] scsi: ufs: mediatek: Use the common PHY framework Nicolas Frattaroli
2026-03-06 13:24 ` [PATCH v9 14/23] scsi: ufs: mediatek: Remove mediatek,ufs-broken-rtc property Nicolas Frattaroli
2026-03-06 13:24 ` [PATCH v9 15/23] scsi: ufs: mediatek: Rework _ufs_mtk_clk_scale error paths Nicolas Frattaroli
2026-03-06 13:24 ` [PATCH v9 16/23] scsi: ufs: mediatek: Clean up logging prints Nicolas Frattaroli
2026-03-06 13:24 ` [PATCH v9 17/23] scsi: ufs: mediatek: Rework ufs_mtk_wait_idle_state Nicolas Frattaroli
2026-03-06 13:24 ` [PATCH v9 18/23] scsi: ufs: mediatek: Don't acquire dvfsrc-vcore twice Nicolas Frattaroli
2026-03-06 13:25 ` [PATCH v9 19/23] scsi: ufs: mediatek: Rework hardware version reading Nicolas Frattaroli
2026-03-08 10:31 ` kernel test robot
2026-03-06 13:25 ` [PATCH v9 20/23] scsi: ufs: mediatek: Back up idle timer in per-instance struct Nicolas Frattaroli
2026-03-18 1:41 ` Chaotian Jing (井朝天)
2026-03-06 13:25 ` [PATCH v9 21/23] scsi: ufs: mediatek: Remove ret local from link_startup_notify Nicolas Frattaroli
2026-03-06 13:25 ` [PATCH v9 22/23] scsi: ufs: mediatek: Remove undocumented "clk-scale-up-vcore-min" Nicolas Frattaroli
2026-03-18 2:23 ` Chaotian Jing (井朝天)
2026-03-06 13:25 ` [PATCH v9 23/23] scsi: ufs: mediatek: Add MT8196 compatible, update copyright Nicolas Frattaroli
2026-03-18 3:15 ` Chaotian Jing (井朝天)
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=5973984.DvuYhMxLoT@workhorse \
--to=nicolas.frattaroli@collabora.com \
--cc=Chaotian.Jing@mediatek.com \
--cc=James.Bottomley@hansenpartnership.com \
--cc=alim.akhtar@samsung.com \
--cc=angelogioacchino.delregno@collabora.com \
--cc=avri.altman@wdc.com \
--cc=broonie@kernel.org \
--cc=bvanassche@acm.org \
--cc=chu.stanley@gmail.com \
--cc=chunfeng.yun@mediatek.com \
--cc=conor+dt@kernel.org \
--cc=conor.dooley@microchip.com \
--cc=devicetree@vger.kernel.org \
--cc=kernel@collabora.com \
--cc=kishon@kernel.org \
--cc=krzk+dt@kernel.org \
--cc=lgirdwood@gmail.com \
--cc=linux-arm-kernel@lists.infradead.org \
--cc=linux-kernel@vger.kernel.org \
--cc=linux-mediatek@lists.infradead.org \
--cc=linux-phy@lists.infradead.org \
--cc=linux-scsi@vger.kernel.org \
--cc=louisalexis.eyraud@collabora.com \
--cc=martin.petersen@oracle.com \
--cc=matthias.bgg@gmail.com \
--cc=neil.armstrong@linaro.org \
--cc=p.zabel@pengutronix.de \
--cc=peter.wang@mediatek.com \
--cc=robh@kernel.org \
--cc=vkoul@kernel.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox