Re: [PATCH 1/4] dt-bindings: reset: Add binding for Sophgo CV1800B reset controller

From: Jisheng Zhang
Date: Mon Nov 13 2023 - 09:12:44 EST


On Mon, Nov 13, 2023 at 01:36:54PM +0000, Conor Dooley wrote:
> On Mon, Nov 13, 2023 at 08:55:00AM +0800, Jisheng Zhang wrote:
> > Add devicetree binding for Sophgo CV1800B SoC reset controller.
> >
> > Signed-off-by: Jisheng Zhang <jszhang@xxxxxxxxxx>
>
> With the unterminated ifndef that was pointed out by the robots fixed,
> Reviewed-by: Conor Dooley <conor.dooley@xxxxxxxxxxxxx>
>
> > +/* 0-1 */
> > +/* 10 */
> > +/* 13 */
> > +/* 15 */
> > +/* 17 */
> > +/* 36-39 */
> > +/* 53-57 */
> > +/* 59-60 */
> > +/* 63-73 */
> > +/* 90 */
> > +/* 94 */
> > +/* 102-292 */
>
> There are quite a lot of gaps here, do you know why that is?

The tail bits are for cpusys, so I guess the SoC designer want to
seperate them with guard? I'm not sure.

>
> Thanks,
> Conor.