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;
Message-ID: <13c836b31909f2eb64dcf2a5feb22c2a3173bba2.camel@physik.fu-berlin.de>
Subject: Re: [PATCH 3/4] sh: remove superhyway bus support
From:   John Paul Adrian Glaubitz <glaubitz@physik.fu-berlin.de>
To:     Arnd Bergmann <arnd@kernel.org>,
        "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>
Date:   Thu, 03 Aug 2023 11:36:22 +0200
In-Reply-To: <4a3fae63-cf85-4e18-b785-1a438ec761aa@app.fastmail.com>
References: <20230802184849.1019466-3-arnd@kernel.org>
         <D28BDDE3-7FE6-4ACC-98B9-B6AB6D9518A3@gmail.com>
         <fecb873aefad8dd4c1d89935cf8f7790e9ba231d.camel@physik.fu-berlin.de>
         <4a3fae63-cf85-4e18-b785-1a438ec761aa@app.fastmail.com>
Content-Type: text/plain; charset="UTF-8"
Content-Transfer-Encoding: 8BIT
User-Agent: Evolution 3.48.4 
MIME-Version: 1.0
Precedence: bulk

Hi Arnd!

On Thu, 2023-08-03 at 11:19 +0200, Arnd Bergmann wrote:
> 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.

OK, we'll discuss this on the #linux-sh IRC channel and if no one objects,
I'm going to apply the series.

> 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.

It depends on what Yoshinori plans for the device tree conversion. The
latest version of his series was posted here [1]. But he promised to
send a rebased and updated version soon.

> 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.

Yes, that would be greatly appreciated. Please go ahead.

FWIW, I am currently on vacation until August 15th, so I might be a bit
slow with open source stuff. I am using the free time to get things done
on my TODO list.

Adrian

> [1] https://lore.kernel.org/all/20180507155543.GJ1392@brightrain.aerifal.cx/t/

-- 
 .''`.  John Paul Adrian Glaubitz
: :' :  Debian Developer
`. `'   Physicist
  `-    GPG: 62FF 8A75 84E0 2956 9546  0006 7426 3B37 F5B5 F913