From: "Peter Wang (王信友)" <peter.wang@mediatek.com>
To: "conor@kernel.org" <conor@kernel.org>
Cc: "matthias.bgg@gmail.com" <matthias.bgg@gmail.com>,
"Alice Chao (趙珮均)" <Alice.Chao@mediatek.com>,
"krzysztof.kozlowski@linaro.org" <krzysztof.kozlowski@linaro.org>,
"Ed Tsai (蔡宗軒)" <Ed.Tsai@mediatek.com>,
"AngeloGioacchino Del Regno"
<angelogioacchino.delregno@collabora.com>,
"robh@kernel.org" <robh@kernel.org>,
"linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>,
"Chun-Hung Wu (巫駿宏)" <Chun-hung.Wu@mediatek.com>,
wsd_upstream <wsd_upstream@mediatek.com>,
"p.zabel@pengutronix.de" <p.zabel@pengutronix.de>,
"linux-mediatek@lists.infradead.org"
<linux-mediatek@lists.infradead.org>,
"linux-devicetree@vger.kernel.org"
<linux-devicetree@vger.kernel.org>,
"conor+dt@kernel.org" <conor+dt@kernel.org>,
"Naomi Chu (朱詠田)" <Naomi.Chu@mediatek.com>
Subject: Re: [PATCH v1 1/2] Documentation: dt: reset: add mediatek,syscon-reset binding
Date: Thu, 2 Jul 2026 08:52:53 +0000 [thread overview]
Message-ID: <4fedff0acaa249f4ce1e874f7f791d8cbe52556b.camel@mediatek.com> (raw)
In-Reply-To: <20260701-city-during-7d76a326f2f4@spud>
On Wed, 2026-07-01 at 17:57 +0100, Conor Dooley wrote:
> From what I recall, mediatek ufs is a mess with lots of vendor kernel
> type things slipping into mainline without proper review on the DT
> front.
> Because of that, I at least am going to require that everything is
> done
> completely (and perhaps excessively) by the book here, including
> introducing complete bindings for syscon regions rather than partial
> bits for components like this one.
>
Hi Conor,
I am sorry for giving you that impression. Yes, it was our
oversight due to a lack of experience, and we missed some
binding-related documentation. We are working on completing
them step by step, as we are doing now. We hope you can
kindly allow us to address the missing parts gradually,
instead of requiring everything to be completed all at once.
> > This approach is also consistent with the existing 'ti,syscon-
> > reset'
> > binding, which uses a similar per-entry table property 'ti,reset-
> > bits'
> > to describe reset lines within a syscon block.
>
> This was done about 10 years ago, I would not consider it a guide for
> what's acceptable today.
>
> Thanks,
> Conor.
This should simply be a difference in hardware design and usage,
with software deciding whether to opt for flexibility or a
fixed approach. And it's not that the previous method was 'correct'
and the current one is 'incorrect,' is it?
Thanks
Peter
next prev parent reply other threads:[~2026-07-02 8:53 UTC|newest]
Thread overview: 13+ messages / expand[flat|nested] mbox.gz Atom feed top
2026-06-26 7:46 [PATCH v1 0/2] reset: mediatek: add syscon-based reset controller peter.wang
2026-06-26 7:46 ` [PATCH v1 1/2] Documentation: dt: reset: add mediatek,syscon-reset binding peter.wang
2026-06-26 8:33 ` Philipp Zabel
2026-06-26 15:53 ` Conor Dooley
2026-07-01 6:36 ` Peter Wang (王信友)
2026-07-01 6:35 ` Peter Wang (王信友)
2026-07-01 16:57 ` Conor Dooley
2026-07-02 8:52 ` Peter Wang (王信友) [this message]
2026-07-02 18:44 ` Conor Dooley
2026-07-02 8:55 ` AngeloGioacchino Del Regno
2026-07-02 9:39 ` Peter Wang (王信友)
2026-07-03 9:21 ` Philipp Zabel
2026-06-26 7:46 ` [PATCH v1 2/2] reset: mediatek: add syscon-based reset controller driver peter.wang
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=4fedff0acaa249f4ce1e874f7f791d8cbe52556b.camel@mediatek.com \
--to=peter.wang@mediatek.com \
--cc=Alice.Chao@mediatek.com \
--cc=Chun-hung.Wu@mediatek.com \
--cc=Ed.Tsai@mediatek.com \
--cc=Naomi.Chu@mediatek.com \
--cc=angelogioacchino.delregno@collabora.com \
--cc=conor+dt@kernel.org \
--cc=conor@kernel.org \
--cc=krzysztof.kozlowski@linaro.org \
--cc=linux-devicetree@vger.kernel.org \
--cc=linux-kernel@vger.kernel.org \
--cc=linux-mediatek@lists.infradead.org \
--cc=matthias.bgg@gmail.com \
--cc=p.zabel@pengutronix.de \
--cc=robh@kernel.org \
--cc=wsd_upstream@mediatek.com \
/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