[PATCH v19 00/17] RISC-V Kendryte K210 support improvements

[Damien Le Moal]

On 2021/02/19 15:29, Palmer Dabbelt wrote:
> On Tue, 09 Feb 2021 21:02:13 PST (-0800), Damien Le Moal wrote:
>> This series of patches improves support for boards based on the Canaan
>> Kendryte K210 RISC-V dual core SoC. Minimal support for this SoC is
>> already included in the kernel. These patches complete it, enabling
>> support for most peripherals present on the SoC as well as introducing
>> device trees for the various K210 boards available on the market today
>> from SiPeed and Kendryte.
>>
>> The patches here are the remaining patches not yet applied.
>>
>> Patch 1 adds the SoC clock driver.
> 
> I don't see a Review or Ack for this one, so I haven't taken it.  Not sure if I
> just missed it or if it's going in through the clock tree.

Stephen sent his reviewed-by. It is on lore:

https://lore.kernel.org/linux-riscv/161301087300.1254594.18438542581140302342@swboyd.mtv.corp.google.com/

There are a few additional nits he pointed out but I will send an incremental
patch for these later. I think this patch needs to go through the risc-v tree
since it touches the sysctl driver too.

> 
>> Patches 2 to 9 add and update device tree bindings documentation to
>> document the K210 device tree files. In particular, patch 2 and 3
>> define compatible strings used for the SoC CPU and for the K210 based
>> boards supported.
>>
>> Patches 10 to 15 update the existing K210 base device tree and add new
>> device tree files for several K210 based boards: MAIX Bit, MAIXDUINO,
>> MAIX Dock and MAIX Go boards from SiPeed and the KD233 development
>> board from Canaan.
> 
> Looks like I already have patch 10.

How come ? This version is the fixed one which removes undocumented dt nodes.
Did you queue an older version ?

> 
> The other DT files have an entry for a microphone with an undocumented
> compatible string, which triggers checkpatch.  I've removed it, along with the
> audio entry that references it.  I don't really have an opinion on how that
> stuff is in the DT, just that we need to document whatever we put in there.  If
> it breaks something I'm happy to swap it out for another version, I'm not
> planning on sending it up until next week (I'm sening a PR this week, just with
> some older stuff).

Works for me. Thanks.

> 
> Thanks!


-- 
Damien Le Moal
Western Digital Research