RE: [PATCH v3 1/3] dt-bindings: reset: Document th1520 reset control

From: Kwanghoon Son
Date: Tue Sep 19 2023 - 01:08:47 EST


> > > + - const: thead,th1520-reset
> > > + - const: syscon
> >
> > iDumb question perhaps, but why is this a syscon?
>
> This is good point.
> In fact, I took it from the vendor kernel, and I tried to keep it as same as I could.
> Vendor also followed drivers/reset/reset-imx7.c
> As Rob said, if don't need it, I'll change it.
>
> Regards,
> Kwang.
>
I have investigated vendor kernels, but it is still not sure reset information.
I thought this is about sys_reg, but without datasheet, can't tell.
Therefore, should be stopped patch for a while until knows better comes out.

Kwang.

> > Cheers,
> > Conor.
> >
> > > +
> > > + reg:
> > > + maxItems: 1
> > > +
> > > + '#reset-cells':
> > > + const: 1