| Message ID | 20200502100628.24809-21-pragnesh.patel@sifive.com |
|---|---|
| State | Superseded |
| Headers | show |
| Series | RISC-V SiFive FU540 support SPL | expand |
On Sat, May 2, 2020 at 6:09 PM Pragnesh Patel <pragnesh.patel at sifive.com> wrote: > > Enable all cache ways from U-Boot proper. > > Signed-off-by: Pragnesh Patel <pragnesh.patel at sifive.com> > --- > arch/riscv/cpu/fu540/Makefile | 1 + > arch/riscv/cpu/fu540/cache.c | 53 +++++++++++++++++++++++ > arch/riscv/include/asm/arch-fu540/cache.h | 14 ++++++ > board/sifive/fu540/fu540.c | 10 ++++- > 4 files changed, 76 insertions(+), 2 deletions(-) > create mode 100644 arch/riscv/cpu/fu540/cache.c > create mode 100644 arch/riscv/include/asm/arch-fu540/cache.h > > diff --git a/arch/riscv/cpu/fu540/Makefile b/arch/riscv/cpu/fu540/Makefile > index 043fb961a5..088205ef57 100644 > --- a/arch/riscv/cpu/fu540/Makefile > +++ b/arch/riscv/cpu/fu540/Makefile > @@ -8,4 +8,5 @@ obj-y += spl.o > else > obj-y += dram.o > obj-y += cpu.o > +obj-y += cache.o > endif > diff --git a/arch/riscv/cpu/fu540/cache.c b/arch/riscv/cpu/fu540/cache.c > new file mode 100644 > index 0000000000..d2dae7f17a > --- /dev/null > +++ b/arch/riscv/cpu/fu540/cache.c > @@ -0,0 +1,53 @@ > +// SPDX-License-Identifier: GPL-2.0+ > +/* > + * Copyright (C) 2020 SiFive, Inc > + * > + * Authors: > + * Pragnesh Patel <pragnesh.patel at sifive.com> > + */ > + > +#include <common.h> > +#include <asm/io.h> > +#include <linux/bitops.h> > + > +/* Register offsets */ > +#define CACHE_CONFIG 0x000 > +#define CACHE_ENABLE 0x008 > + > +#define MASK_NUM_WAYS GENMASK(15, 8) > +#define NUM_WAYS_SHIFT 8 > + > +DECLARE_GLOBAL_DATA_PTR; > + > +int cache_enable_ways(void) > +{ > + const void *blob = gd->fdt_blob; > + int node = (-FDT_ERR_NOTFOUND); > + fdt_addr_t base; > + u32 config; > + u32 ways; > + > + volatile u32 *enable; > + > + node = fdt_node_offset_by_compatible(blob, -1, > + "sifive,fu540-c000-ccache"); > + > + if (node < 0) > + return node; > + > + base = fdtdec_get_addr(blob, node, "reg"); > + if (base == FDT_ADDR_T_NONE) > + return FDT_ADDR_T_NONE; > + > + config = readl((volatile u32 *)base + CACHE_CONFIG); > + ways = (config & MASK_NUM_WAYS) >> NUM_WAYS_SHIFT; > + > + enable = (volatile u32 *)(base + CACHE_ENABLE); > + > + /* memory barrier */ > + mb(); > + (*enable) = ways - 1; > + /* memory barrier */ > + mb(); > + return 0; > +} > diff --git a/arch/riscv/include/asm/arch-fu540/cache.h b/arch/riscv/include/asm/arch-fu540/cache.h > new file mode 100644 > index 0000000000..135a17c679 > --- /dev/null > +++ b/arch/riscv/include/asm/arch-fu540/cache.h > @@ -0,0 +1,14 @@ > +/* SPDX-License-Identifier: GPL-2.0+ */ > +/* > + * Copyright (C) 2020 SiFive, Inc. > + * > + * Authors: > + * Pragnesh Patel <pragnesh.patel at sifve.com> > + */ > + > +#ifndef _CACHE_SIFIVE_H > +#define _CACHE_SIFIVE_H > + > +int cache_enable_ways(void); > + > +#endif /* _CACHE_SIFIVE_H */ > diff --git a/board/sifive/fu540/fu540.c b/board/sifive/fu540/fu540.c > index d05529a86b..131fee8898 100644 > --- a/board/sifive/fu540/fu540.c > +++ b/board/sifive/fu540/fu540.c > @@ -12,6 +12,7 @@ > #include <linux/io.h> > #include <misc.h> > #include <spl.h> > +#include <asm/arch/cache.h> > > /* > * This define is a value used for error/unknown serial. > @@ -111,8 +112,13 @@ int misc_init_r(void) > > int board_init(void) > { > - /* For now nothing to do here. */ > - > + int ret; nits: need a blank line > + /* enable all cache ways */ > + ret = cache_enable_ways(); > + if (ret) { > + debug("%s: could not enable cache ways\n", __func__); > + return ret; > + } nits: need a blank line > return 0; > } > Reviewed-by: Bin Meng <bmeng.cn at gmail.com> Regards, Bin
Hi Bin, >-----Original Message----- >From: Bin Meng <bmeng.cn at gmail.com> >Sent: 02 May 2020 18:26 >To: Pragnesh Patel <pragnesh.patel at sifive.com> >Cc: U-Boot Mailing List <u-boot at lists.denx.de>; Atish Patra ><atish.patra at wdc.com>; Palmer Dabbelt <palmerdabbelt at google.com>; Paul >Walmsley <paul.walmsley at sifive.com>; Jagan Teki ><jagan at amarulasolutions.com>; Troy Benjegerdes ><troy.benjegerdes at sifive.com>; Anup Patel <anup.patel at wdc.com>; Sagar >Kadam <sagar.kadam at sifive.com>; Rick Chen <rick at andestech.com>; Palmer >Dabbelt <palmer at dabbelt.com> >Subject: Re: [PATCH v7 20/22] riscv: sifive: fu540: enable all cache ways from >U-Boot proper > >[External Email] Do not click links or attachments unless you recognize the >sender and know the content is safe > >On Sat, May 2, 2020 at 6:09 PM Pragnesh Patel <pragnesh.patel at sifive.com> >wrote: >> >> Enable all cache ways from U-Boot proper. >> >> Signed-off-by: Pragnesh Patel <pragnesh.patel at sifive.com> >> --- >> arch/riscv/cpu/fu540/Makefile | 1 + >> arch/riscv/cpu/fu540/cache.c | 53 +++++++++++++++++++++++ >> arch/riscv/include/asm/arch-fu540/cache.h | 14 ++++++ >> board/sifive/fu540/fu540.c | 10 ++++- >> 4 files changed, 76 insertions(+), 2 deletions(-) create mode 100644 >> arch/riscv/cpu/fu540/cache.c create mode 100644 >> arch/riscv/include/asm/arch-fu540/cache.h >> >> diff --git a/arch/riscv/cpu/fu540/Makefile >> b/arch/riscv/cpu/fu540/Makefile index 043fb961a5..088205ef57 100644 >> --- a/arch/riscv/cpu/fu540/Makefile >> +++ b/arch/riscv/cpu/fu540/Makefile >> @@ -8,4 +8,5 @@ obj-y += spl.o >> else >> obj-y += dram.o >> obj-y += cpu.o >> +obj-y += cache.o >> endif >> diff --git a/arch/riscv/cpu/fu540/cache.c >> b/arch/riscv/cpu/fu540/cache.c new file mode 100644 index >> 0000000000..d2dae7f17a >> --- /dev/null >> +++ b/arch/riscv/cpu/fu540/cache.c >> @@ -0,0 +1,53 @@ >> +// SPDX-License-Identifier: GPL-2.0+ >> +/* >> + * Copyright (C) 2020 SiFive, Inc >> + * >> + * Authors: >> + * Pragnesh Patel <pragnesh.patel at sifive.com> >> + */ >> + >> +#include <common.h> >> +#include <asm/io.h> >> +#include <linux/bitops.h> >> + >> +/* Register offsets */ >> +#define CACHE_CONFIG 0x000 >> +#define CACHE_ENABLE 0x008 >> + >> +#define MASK_NUM_WAYS GENMASK(15, 8) #define NUM_WAYS_SHIFT 8 >> + >> +DECLARE_GLOBAL_DATA_PTR; >> + >> +int cache_enable_ways(void) >> +{ >> + const void *blob = gd->fdt_blob; >> + int node = (-FDT_ERR_NOTFOUND); >> + fdt_addr_t base; >> + u32 config; >> + u32 ways; >> + >> + volatile u32 *enable; >> + >> + node = fdt_node_offset_by_compatible(blob, -1, >> + >> + "sifive,fu540-c000-ccache"); >> + >> + if (node < 0) >> + return node; >> + >> + base = fdtdec_get_addr(blob, node, "reg"); >> + if (base == FDT_ADDR_T_NONE) >> + return FDT_ADDR_T_NONE; >> + >> + config = readl((volatile u32 *)base + CACHE_CONFIG); >> + ways = (config & MASK_NUM_WAYS) >> NUM_WAYS_SHIFT; >> + >> + enable = (volatile u32 *)(base + CACHE_ENABLE); >> + >> + /* memory barrier */ >> + mb(); >> + (*enable) = ways - 1; >> + /* memory barrier */ >> + mb(); >> + return 0; >> +} >> diff --git a/arch/riscv/include/asm/arch-fu540/cache.h >> b/arch/riscv/include/asm/arch-fu540/cache.h >> new file mode 100644 >> index 0000000000..135a17c679 >> --- /dev/null >> +++ b/arch/riscv/include/asm/arch-fu540/cache.h >> @@ -0,0 +1,14 @@ >> +/* SPDX-License-Identifier: GPL-2.0+ */ >> +/* >> + * Copyright (C) 2020 SiFive, Inc. >> + * >> + * Authors: >> + * Pragnesh Patel <pragnesh.patel at sifve.com> >> + */ >> + >> +#ifndef _CACHE_SIFIVE_H >> +#define _CACHE_SIFIVE_H >> + >> +int cache_enable_ways(void); >> + >> +#endif /* _CACHE_SIFIVE_H */ >> diff --git a/board/sifive/fu540/fu540.c b/board/sifive/fu540/fu540.c >> index d05529a86b..131fee8898 100644 >> --- a/board/sifive/fu540/fu540.c >> +++ b/board/sifive/fu540/fu540.c >> @@ -12,6 +12,7 @@ >> #include <linux/io.h> >> #include <misc.h> >> #include <spl.h> >> +#include <asm/arch/cache.h> >> >> /* >> * This define is a value used for error/unknown serial. >> @@ -111,8 +112,13 @@ int misc_init_r(void) >> >> int board_init(void) >> { >> - /* For now nothing to do here. */ >> - >> + int ret; > >nits: need a blank line Will update in v8. > >> + /* enable all cache ways */ >> + ret = cache_enable_ways(); >> + if (ret) { >> + debug("%s: could not enable cache ways\n", __func__); >> + return ret; >> + } > >nits: need a blank line Will update in v8. > >> return 0; >> } >> > >Reviewed-by: Bin Meng <bmeng.cn at gmail.com> > >Regards, >Bin
diff --git a/arch/riscv/cpu/fu540/Makefile b/arch/riscv/cpu/fu540/Makefile index 043fb961a5..088205ef57 100644 --- a/arch/riscv/cpu/fu540/Makefile +++ b/arch/riscv/cpu/fu540/Makefile @@ -8,4 +8,5 @@ obj-y += spl.o else obj-y += dram.o obj-y += cpu.o +obj-y += cache.o endif diff --git a/arch/riscv/cpu/fu540/cache.c b/arch/riscv/cpu/fu540/cache.c new file mode 100644 index 0000000000..d2dae7f17a --- /dev/null +++ b/arch/riscv/cpu/fu540/cache.c @@ -0,0 +1,53 @@ +// SPDX-License-Identifier: GPL-2.0+ +/* + * Copyright (C) 2020 SiFive, Inc + * + * Authors: + * Pragnesh Patel <pragnesh.patel at sifive.com> + */ + +#include <common.h> +#include <asm/io.h> +#include <linux/bitops.h> + +/* Register offsets */ +#define CACHE_CONFIG 0x000 +#define CACHE_ENABLE 0x008 + +#define MASK_NUM_WAYS GENMASK(15, 8) +#define NUM_WAYS_SHIFT 8 + +DECLARE_GLOBAL_DATA_PTR; + +int cache_enable_ways(void) +{ + const void *blob = gd->fdt_blob; + int node = (-FDT_ERR_NOTFOUND); + fdt_addr_t base; + u32 config; + u32 ways; + + volatile u32 *enable; + + node = fdt_node_offset_by_compatible(blob, -1, + "sifive,fu540-c000-ccache"); + + if (node < 0) + return node; + + base = fdtdec_get_addr(blob, node, "reg"); + if (base == FDT_ADDR_T_NONE) + return FDT_ADDR_T_NONE; + + config = readl((volatile u32 *)base + CACHE_CONFIG); + ways = (config & MASK_NUM_WAYS) >> NUM_WAYS_SHIFT; + + enable = (volatile u32 *)(base + CACHE_ENABLE); + + /* memory barrier */ + mb(); + (*enable) = ways - 1; + /* memory barrier */ + mb(); + return 0; +} diff --git a/arch/riscv/include/asm/arch-fu540/cache.h b/arch/riscv/include/asm/arch-fu540/cache.h new file mode 100644 index 0000000000..135a17c679 --- /dev/null +++ b/arch/riscv/include/asm/arch-fu540/cache.h @@ -0,0 +1,14 @@ +/* SPDX-License-Identifier: GPL-2.0+ */ +/* + * Copyright (C) 2020 SiFive, Inc. + * + * Authors: + * Pragnesh Patel <pragnesh.patel at sifve.com> + */ + +#ifndef _CACHE_SIFIVE_H +#define _CACHE_SIFIVE_H + +int cache_enable_ways(void); + +#endif /* _CACHE_SIFIVE_H */ diff --git a/board/sifive/fu540/fu540.c b/board/sifive/fu540/fu540.c index d05529a86b..131fee8898 100644 --- a/board/sifive/fu540/fu540.c +++ b/board/sifive/fu540/fu540.c @@ -12,6 +12,7 @@ #include <linux/io.h> #include <misc.h> #include <spl.h> +#include <asm/arch/cache.h> /* * This define is a value used for error/unknown serial. @@ -111,8 +112,13 @@ int misc_init_r(void) int board_init(void) { - /* For now nothing to do here. */ - + int ret; + /* enable all cache ways */ + ret = cache_enable_ways(); + if (ret) { + debug("%s: could not enable cache ways\n", __func__); + return ret; + } return 0; }
Enable all cache ways from U-Boot proper. Signed-off-by: Pragnesh Patel <pragnesh.patel at sifive.com> --- arch/riscv/cpu/fu540/Makefile | 1 + arch/riscv/cpu/fu540/cache.c | 53 +++++++++++++++++++++++ arch/riscv/include/asm/arch-fu540/cache.h | 14 ++++++ board/sifive/fu540/fu540.c | 10 ++++- 4 files changed, 76 insertions(+), 2 deletions(-) create mode 100644 arch/riscv/cpu/fu540/cache.c create mode 100644 arch/riscv/include/asm/arch-fu540/cache.h