MIME-Version: 1.0
References: <20230123214924.27476-1-afd@ti.com> <f8922fbc-13cc-4f08-a8e8-08d39ab7d63c@app.fastmail.com>
 <3ab320ef-dfbb-eee4-f895-6f1caa6cc66f@ti.com>
In-Reply-To: <3ab320ef-dfbb-eee4-f895-6f1caa6cc66f@ti.com>
From:   Daniel Tang <dt.tangr@gmail.com>
Date:   Sat, 28 Jan 2023 13:00:49 +1100
Message-ID: <CAPnH9dnGZxbMGy7c_7PFqw6R1PSVba00XWK8rUN4MEsWU7=9=g@mail.gmail.com>
Subject: Re: [PATCH v5 0/9] TI-Nspire cleanups
To:     Andrew Davis <afd@ti.com>
Cc:     Arnd Bergmann <arnd@arndb.de>, Lee Jones <lee@kernel.org>,
        Rob Herring <robh+dt@kernel.org>,
        Krzysztof Kozlowski <krzysztof.kozlowski+dt@linaro.org>,
        Linus Walleij <linus.walleij@linaro.org>,
        Geert Uytterhoeven <geert+renesas@glider.be>,
        Fabian Vogt <fabian@ritter-vogt.de>,
        devicetree@vger.kernel.org, linux-arm-kernel@lists.infradead.org,
        linux-kernel@vger.kernel.org
Content-Type: text/plain; charset="UTF-8"
Precedence: bulk

On Tue, 24 Jan 2023 at 10:56, Andrew Davis <afd@ti.com> wrote:
>
> On 1/23/23 3:57 PM, Arnd Bergmann wrote:
> > On Mon, Jan 23, 2023, at 22:49, Andrew Davis wrote:
> >> Hello all,
> >>
> >> This series is an extended version of the series started here[0]
> >> and here[1].
> >>
> >> We break out what was the first patch into one for DTS change and
> >> one for code changes as suggested by Krzysztof. Those are now patches
> >> 2 and 8 of this series (I kept the ACKs, hope that is okay).
> >>
> >> As also pointed out by Krzysztof syscon nodes need a specific
> >> compatible, add that as patch 1.
> >>
> >> While I was adding that, I noticed some other dtbs_check issues,
> >> so while here fixed some of those up too (patches 3-6).
> >
> > Looks all good to me, if there are no final comments within the
> > next few days, can you send this all to:soc@kernel.org, either as
> > separate patches or as two pull requests (dts and code) based
> > on 6.2-rc-1?
> >
>
> Sure thing.
>
> > I think I previously commented on the lack of an entry in
> > the MAINTAINERS. It would be nice to have one more patch to
> > add this, whichever way you want to split this with Daniel,
> > adding one or both of you as maintainer and/or reviewer.
> > I'd keep that separate from this series though, to not hold
> > it up any longer.
> >
>
> Daniel,
>
> Do you want to do this? If not I can add myself as a reviewer, the
> point of this series is to boil down the support to just the DTS file,
> at which point hopefully we won't need too much maintenance.
>
> Andrew
>

I would prefer that someone who is more active in this project be the
maintainer (I've not been working on this platform for many years now,
and don't currently have the means to test changes).

Fabian, would you be OK being listed as a maintainer for now? If not,
I'll get in touch with some other project team members to find a
maintainer.

Cheers,
Daniel

> >       Arnd