[U-Boot] [PATCH v6 05/16] riscv: Add place-holder asm/arch/clk.h for driver compilation

[Bin Meng]

On Sat, Feb 9, 2019 at 2:31 PM Anup Patel <Anup.Patel at 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.patel at 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.

>
> diff --git a/arch/riscv/include/asm/arch/clk.h b/arch/riscv/include/asm/arch/clk.h
> new file mode 100644
> index 0000000000..1631f5f0bd
> --- /dev/null
> +++ b/arch/riscv/include/asm/arch/clk.h
> @@ -0,0 +1,14 @@
> +/* SPDX-License-Identifier: GPL-2.0+ */
> +/*
> + * Copyright (c) 2019 Western Digital Corporation or its affiliates.
> + *
> + * Authors:
> + *   Anup Patel <anup.patel at wdc.com>
> + */
> +
> +#ifndef __ASM_RISCV_ARCH_CLK_H
> +#define __ASM_RISCV_ARCH_CLK_H
> +
> +/* Note: This is a placeholder header for driver compilation. */
> +
> +#endif
> --

Regards,
Bin