Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) client-ip=2620:137:e000::1:20;
Feedback-ID: i36794607:Fastmail
User-Agent: Cyrus-JMAP/3.9.0-alpha0-624-g7714e4406d-fm-20230801.001-g7714e440
Mime-Version: 1.0
Message-Id: <4a3fae63-cf85-4e18-b785-1a438ec761aa@app.fastmail.com>
In-Reply-To: <fecb873aefad8dd4c1d89935cf8f7790e9ba231d.camel@physik.fu-berlin.de>
References: <20230802184849.1019466-3-arnd@kernel.org>
 <D28BDDE3-7FE6-4ACC-98B9-B6AB6D9518A3@gmail.com>
 <fecb873aefad8dd4c1d89935cf8f7790e9ba231d.camel@physik.fu-berlin.de>
Date:   Thu, 03 Aug 2023 11:19:03 +0200
From:   "Arnd Bergmann" <arnd@kernel.org>
To:     "John Paul Adrian Glaubitz" <glaubitz@physik.fu-berlin.de>,
        "D. Jeff Dionne" <djeffdionne@gmail.com>
Cc:     linux-sh@vger.kernel.org, "Rich Felker" <dalias@libc.org>,
        "Yoshinori Sato" <ysato@users.sourceforge.jp>,
        "Geert Uytterhoeven" <geert+renesas@glider.be>,
        linux-kernel@vger.kernel.org, "Arnd Bergmann" <arnd@arndb.de>
Subject: Re: [PATCH 3/4] sh: remove superhyway bus support
Content-Type: text/plain;charset=utf-8
Content-Transfer-Encoding: quoted-printable
Precedence: bulk

On Thu, Aug 3, 2023, at 10:44, John Paul Adrian Glaubitz wrote:
> On Thu, 2023-08-03 at 16:58 +0900, D. Jeff Dionne wrote:
>> On Aug 3, 2023, at 04:15, Arnd Bergmann <arnd@kernel.org> wrote:
>> >=20
>> > From: Arnd Bergmann <arnd@arndb.de>
>> >=20
>> > superhyway was only referenced on sh4-202, which is now gone, so re=
move it all as well.
>> >=20
>> > I could find no trace of anything ever calling
>> > superhyway_register_driver(), not in the git history but also not o=
n the web, so I assume this has never served any purpose on mainline ker=
nels.
>>=20
>> I don=E2=80=99t know, but I think it is fairly safe to assume that th=
ere were no superhyway implementations other than internal to SuperH Co =
(or Hitachi).  Probably not at ST either.
>>=20
>> I think this board, and infrastructure, can go without affecting any =
actual (even historical) user.  If anyone wants further conformation tha=
t there are/were no users of this in the wild, raise a flag and I will f=
ind out.
>
> OK, I'll think about applying this series. The thing is, we're going t=
o=20
> convert SuperH
> to device trees anyways. We're waiting now Yoshinori to post a rebased=20
> version of the
> patch series.

Applying this first should definitely help with the DT conversion,
especially not having to create a bus specific binding for superhyway
would help, as converting that to DT would be a complete rewrite
but also be untestable without drivers attaching to the bus.

I would also recommend trying to eliminate most of the SoC
support for chips that only support a reference board but no
products or known user of the reference board itself. While
a lot of the conversion could be done fairly mechanical, at
least the clk driver for each chip is a huge effort.

I looked at the clk conversion in the past, as this is not just
needed for the DT work, but also to remove CONFIG_HAVE_LEGACY_CLK.
The patch series I did a while ago renames the sh clk interfaces
to no longer conflict with COMMON_CLK, which should allow it
to coexist with a DT-enabled platform in the same kernel build.
Let me know if you'd like me to dig out and rebase that series,
it probably still applies and may help you here.

     Arnd