[PATCH v1 1/2] Documentation: dt: reset: add mediatek,syscon-reset binding

Conor Dooley conor at kernel.org
Thu Jul 2 11:44:11 PDT 2026


On Thu, Jul 02, 2026 at 08:52:53AM +0000, Peter Wang (王信友) wrote:
> 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.

No, sorry. I expect to see a complete binding for this system added in
one go.

> > > 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.
> 
> 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?

I'm not sure what you mean here by "previous" and "current", since both
approaches are the same just with different compatibles. Times change
and so does the standard for what is acceptable.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 228 bytes
Desc: not available
URL: <http://lists.infradead.org/pipermail/linux-mediatek/attachments/20260702/92928915/attachment-0001.sig>


More information about the Linux-mediatek mailing list