[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <CAJF2gTS_KkQjyQPubc2+naYiammLjN8jJyptGtCe8Z56MAmT2A@mail.gmail.com>
Date: Tue, 25 Jun 2024 11:55:32 +0800
From: Guo Ren <guoren@...nel.org>
To: Conor Dooley <conor@...nel.org>
Cc: linux-riscv@...ts.infradead.org, Yangyu Chen <cyy@...self.name>,
Conor Dooley <conor.dooley@...rochip.com>, Damien Le Moal <dlemoal@...nel.org>,
Rob Herring <robh+dt@...nel.org>,
Krzysztof Kozlowski <krzysztof.kozlowski+dt@...aro.org>,
Dan Carpenter <dan.carpenter@...aro.org>, Paul Walmsley <paul.walmsley@...ive.com>,
Palmer Dabbelt <palmer@...belt.com>, Albert Ou <aou@...s.berkeley.edu>, devicetree@...r.kernel.org,
linux-kernel@...r.kernel.org
Subject: Re: (subset) [PATCH RESEND v8 0/6] riscv: add initial support for
Canaan Kendryte K230
On Mon, Jun 24, 2024 at 1:32 AM Conor Dooley <conor@...nel.org> wrote:
>
> On Mon, Jun 24, 2024 at 01:07:55AM +0800, Guo Ren wrote:
> > On Wed, Jun 19, 2024 at 6:45 PM Conor Dooley <conor@...nel.org> wrote:
> > >
> > > On Wed, Apr 10, 2024 at 11:30:25AM +0100, Conor Dooley wrote:
> > > > From: Conor Dooley <conor.dooley@...rochip.com>
> > > >
> > > > On Mon, 08 Apr 2024 00:26:58 +0800, Yangyu Chen wrote:
> > > > > K230 is an ideal chip for RISC-V Vector 1.0 evaluation now. Add initial
> > > > > support for it to allow more people to participate in building drivers
> > > > > to mainline for it.
> > > > >
> > > > > This kernel has been tested upon factory SDK [1] with
> > > > > k230_evb_only_linux_defconfig and patched mainline opensbi [2] to skip
> > > > > locked pmp and successfully booted to busybox on initrd with this log [3].
> > > > >
> > > > > [...]
> > > >
> > > > Applied to riscv-dt-for-next, thanks!
> > > >
> > > > [1/6] dt-bindings: riscv: Add T-HEAD C908 compatible
> > > > https://git.kernel.org/conor/c/64cbc46bb854
> > > > [2/6] dt-bindings: add Canaan K230 boards compatible strings
> > > > https://git.kernel.org/conor/c/b065da13ea9c
> > > > [3/6] dt-bindings: timer: Add Canaan K230 CLINT
> > > > https://git.kernel.org/conor/c/b3ae796d0a4f
> > > > [4/6] dt-bindings: interrupt-controller: Add Canaan K230 PLIC
> > > > https://git.kernel.org/conor/c/db54fda11b13
> > > > [5/6] riscv: dts: add initial canmv-k230 and k230-evb dts
> > > > https://git.kernel.org/conor/c/5db2c4dc413e
> > >
> > > After some discussion on the k1 thread
> > > (https://lore.kernel.org/all/ZnEOU7D00J8Jzy-1@xhacker/, https://lore.kernel.org/all/ZnA6pZLkI2StP8Hh@xhacker/)
> > > I am going to drop this series. It's not very useful in the current
> > > state and there's not really been any interest from people in getting
> > > the platform to a more complete state. Jisheng made some good points in
> > > the k1 thread about the missing clock controller stuff, and I think I'm
> > > going to make having basic things like clocks and where applicable
> > > resets and pinctrl the minimum requirement for the platforms I'm looking
> > > after.
> > Here is the k230 clock driver based on Linux-6.6:
> > https://github.com/ruyisdk/linux-xuantie-kernel/commit/196242fd9b9b4a191dab0c7c3c5bf851ed857d8d
> >
> > pinctrl:
> > https://github.com/ruyisdk/linux-xuantie-kernel/commit/baf26b6622c9de2ff64a6ed58eeeb98c8b2c828b
> >
> > No reset driver.
> >
> > Most of the k230 drivers are under Linux-5.10, and we are porting them
> > into the newest version of Linux, which takes time.
> >
> > So, if the clock & punctual drivers mentioned above could satisfy the
> > minimum requirements for the platforms, we will update the version of
> > this series as a supplement.
> >
> > Is that okay?
>
> I don't understand how that changes anything, these are all out of tree
> drivers based on an old kernel. I know that there are drivers for a lot
> of the peripherals that are in-use in the vendor tree etc. What I am
> looking to happen, before I apply patches for the k230, is that the clock
> dt-bindings will have landed in linux-next - and ideally the pinctrl ones
> too.
Okay, clock & pinctrl, the reply is clear to me. Thx.
>
> Thanks,
> Conor.
--
Best Regards
Guo Ren
Powered by blists - more mailing lists