> -----Original Message----- > From: Bin Meng [mailto:bmeng...@gmail.com] > Sent: Monday, February 11, 2019 8:51 AM > To: Anup Patel <anup.pa...@wdc.com> > Cc: Rick Chen <r...@andestech.com>; Joe Hershberger > <joe.hershber...@ni.com>; Michal Simek <mon...@monstr.eu>; Michal > Simek <michal.si...@xilinx.com>; Lukas Auer > <lukas.a...@aisec.fraunhofer.de>; Masahiro Yamada > <yamada.masah...@socionext.com>; Simon Glass <s...@chromium.org>; > Alexander Graf <ag...@suse.de>; Palmer Dabbelt <pal...@sifive.com>; > Paul Walmsley <paul.walms...@sifive.com>; Atish Patra > <atish.pa...@wdc.com>; U-Boot Mailing List <u-boot@lists.denx.de>; Tom > Rini <tr...@konsulko.com> > Subject: Re: [PATCH v6 05/16] riscv: Add place-holder asm/arch/clk.h for > driver compilation > > On Sat, Feb 9, 2019 at 2:31 PM Anup Patel <anup.pa...@wdc.com> wrote: > > > > Some of the drivers expect asm/arch/clk.h to be provided by arch > > support code so we add place-holder asm/arch/clk.h for RISC-V support. > > > > Signed-off-by: Anup Patel <anup.pa...@wdc.com> > > --- > > arch/riscv/include/asm/arch/clk.h | 14 ++++++++++++++ > > 1 file changed, 14 insertions(+) > > create mode 100644 arch/riscv/include/asm/arch/clk.h > > I looked at this and understand why patch [1/16] and [2/16] are needed. But > can we do the same like other architectures, ie: creating directory > include/asm/arch-xxx instead? Creating a hard-coded include/asm/arch > directory prevents us from expanding SoC/platform specific support in the > future.
Even Tom suggested similar thing. I will use CREATE_ARCH_SYMLINK instead in next version of patchset. Regards, Anup _______________________________________________ U-Boot mailing list U-Boot@lists.denx.de https://lists.denx.de/listinfo/u-boot
