From mboxrd@z Thu Jan  1 00:00:00 1970
From: Rick Chen <rickchen36@gmail.com>
Date: Mon, 3 May 2021 10:34:23 +0800
Subject: [PATCH v7 0/8] Add FU740 chip and HiFive Unmatched board support
In-Reply-To: <752D002CFF5D0F4FA35C0100F1D73F3FE5E9B483@ATCPCS12.andestech.com>
References: <20210422091202.396956-1-green.wan@sifive.com>
 <CAJivOr4WQTEDfp9zUY+wYSUmL3qLBjcv-Z1W7bDK2KA0wwbUAw@mail.gmail.com>
 <CAEUhbmXPvhZGJhQJkP=Z7BWZSfb_pdEcE7AJ7VpUwo5tVWznxw@mail.gmail.com>
 <752D002CFF5D0F4FA35C0100F1D73F3FE5E9B483@ATCPCS12.andestech.com>
Message-ID: <CAN5B=e+nyqSiu+xMt3MDsKrjc=2D0sPz4Z+Hn7Xyi+NsRw5rPg@mail.gmail.com>
List-Id: <u-boot.lists.denx.de>
MIME-Version: 1.0
Content-Type: text/plain; charset="us-ascii"
Content-Transfer-Encoding: 7bit
To: u-boot@lists.denx.de

Hi Green,


I did not sign the Reviewed-by for this patch "board: sifive: add
HiFive Unmatched board support" from v1 to v6.
But it just has been tagged in [v7,7/8] board: sifive: add HiFive
Unmatched board support by yourself.

[v6,6/7] board: sifive: add HiFive Unmatched board support
https://patchwork.ozlabs.org/project/uboot/patch/20210408134020.238658-7-green.wan at sifive.com/

[v7,7/8] board: sifive: add HiFive Unmatched board support
https://patchwork.ozlabs.org/project/uboot/patch/20210422091202.396956-8-green.wan at sifive.com/

Actually I don't like this patch that you mix every things (arch/,
drivers/, common/, doc/)together in this patch.
But it is OK for now.

BTW, in [PATCH v7 1/8] riscv: cpu: fu740: Add support for cpu fu740
I found that arch/riscv/cpu/fu740/cpu.c and arch/riscv/fu540/cpu.c are
100% the same.

And about spl.c, they are only different in the annotation of Copyright
diff fu540/spl.c fu740/spl.c
3c3
<  * Copyright (C) 2020 SiFive, Inc
---
>  * Copyright (C) 2020-201 SiFive, Inc

About the cache.c, they are just different in one character

diff fu540/cache.c fu740/cache.c
3c3
<  * Copyright (C) 2020 SiFive, Inc
---
>  * Copyright (C) 2020-2021 SiFive, Inc
10d9
< #include <asm/global_data.h>
12a12
> #include <asm/global_data.h>
34c34
<                                            "sifive,fu540-c000-ccache");
---
>                                            "sifive,fu740-c000-ccache");


Originally, I am considering to tell you to re-use the same code base
instead of just copy and create.
After a few days of consideration, I feel it's OK for now.

About
[PATCH v7 2/8] drivers: clk: add fu740 support and
[PATCH v7 4/8] drivers: pci: add pcie support for fu740,
there are still not get any Reviewed-by till now.
For me, it will be better if someone can tag a Reviewed-by here.

Principally, it will be suggested to split drivers from RISC-V
relevant, do not mix them together as Palmer said.

Thanks,
Rick

> From: Bin Meng <bmeng.cn@gmail.com>
> Sent: Thursday, April 29, 2021 8:27 PM
> To: Green Wan <green.wan@sifive.com>
> Cc: Rick Jian-Zhi Chen(???) <rick@andestech.com>; Paul Walmsley <paul.walmsley@sifive.com>; Palmer Dabbelt <palmer@dabbelt.com>; Anup Patel <anup.patel@wdc.com>; Atish Patra <atish.patra@wdc.com>; Lukasz Majewski <lukma@denx.de>; Joe Hershberger <joe.hershberger@ni.com>; Ramon Fried <rfried.dev@gmail.com>; U-Boot Mailing List <u-boot@lists.denx.de>
> Subject: Re: [PATCH v7 0/8] Add FU740 chip and HiFive Unmatched board support
>
> Hi Green,
>
> On Thu, Apr 29, 2021 at 7:11 PM Green Wan <green.wan@sifive.com> wrote:
> >
> > Hi Bin,
> >
> > How should this patch set be proceeded?
> >
> > To summary the major changes,
> > - I've rebased to mainstream and merged pcie refactoring code which
> > based on pcie_dw_common.c
> > - separate unmatched dts into separated patch.
> >
>
> I don't have specific comments. Rick should pick this up via the riscv tree. Thanks!
>
> Regards,
> Bin