[PATCH v1 1/2] Documentation: dt: reset: add mediatek,syscon-reset binding
Peter Wang (王信友)
peter.wang at mediatek.com
Thu Jul 2 01:52:53 PDT 2026
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
More information about the Linux-mediatek
mailing list