mirror of https://lore.kernel.org/lkml/
 help / color / mirror / Atom feed
* [PATCH] openrisc: add cache way information to cpuinfo
@ 2013-04-27 18:02 Stefan Kristiansson
  2013-08-01  9:46 ` [ORLinux] " Jonas Bonn
  0 siblings, 1 reply; 2+ messages in thread
From: Stefan Kristiansson @ 2013-04-27 18:02 UTC (permalink / raw)
  To: linux-kernel, linux; +Cc: Stefan Kristiansson

Motivation for this is to be able to print the way information
properly in print_cpuinfo(), instead of hardcoding it to one.

Signed-off-by: Stefan Kristiansson <stefan.kristiansson@saunalahti.fi>
---
 arch/openrisc/include/asm/cpuinfo.h |  2 ++
 arch/openrisc/kernel/setup.c        | 20 +++++++++++++-------
 2 files changed, 15 insertions(+), 7 deletions(-)

diff --git a/arch/openrisc/include/asm/cpuinfo.h b/arch/openrisc/include/asm/cpuinfo.h
index 917318b..ec10679 100644
--- a/arch/openrisc/include/asm/cpuinfo.h
+++ b/arch/openrisc/include/asm/cpuinfo.h
@@ -24,9 +24,11 @@ struct cpuinfo {
 
 	u32 icache_size;
 	u32 icache_block_size;
+	u32 icache_ways;
 
 	u32 dcache_size;
 	u32 dcache_block_size;
+	u32 dcache_ways;
 };
 
 extern struct cpuinfo cpuinfo;
diff --git a/arch/openrisc/kernel/setup.c b/arch/openrisc/kernel/setup.c
index f4d5bed..6b0e2c4 100644
--- a/arch/openrisc/kernel/setup.c
+++ b/arch/openrisc/kernel/setup.c
@@ -129,13 +129,15 @@ static void print_cpuinfo(void)
 	if (upr & SPR_UPR_DCP)
 		printk(KERN_INFO
 		       "-- dcache: %4d bytes total, %2d bytes/line, %d way(s)\n",
-		       cpuinfo.dcache_size, cpuinfo.dcache_block_size, 1);
+		       cpuinfo.dcache_size, cpuinfo.dcache_block_size,
+		       cpuinfo.dcache_ways);
 	else
 		printk(KERN_INFO "-- dcache disabled\n");
 	if (upr & SPR_UPR_ICP)
 		printk(KERN_INFO
 		       "-- icache: %4d bytes total, %2d bytes/line, %d way(s)\n",
-		       cpuinfo.icache_size, cpuinfo.icache_block_size, 1);
+		       cpuinfo.icache_size, cpuinfo.icache_block_size,
+		       cpuinfo.icache_ways);
 	else
 		printk(KERN_INFO "-- icache disabled\n");
 
@@ -167,25 +169,25 @@ void __init setup_cpuinfo(void)
 {
 	struct device_node *cpu;
 	unsigned long iccfgr, dccfgr;
-	unsigned long cache_set_size, cache_ways;
+	unsigned long cache_set_size;
 
 	cpu = of_find_compatible_node(NULL, NULL, "opencores,or1200-rtlsvn481");
 	if (!cpu)
 		panic("No compatible CPU found in device tree...\n");
 
 	iccfgr = mfspr(SPR_ICCFGR);
-	cache_ways = 1 << (iccfgr & SPR_ICCFGR_NCW);
+	cpuinfo.icache_ways = 1 << (iccfgr & SPR_ICCFGR_NCW);
 	cache_set_size = 1 << ((iccfgr & SPR_ICCFGR_NCS) >> 3);
 	cpuinfo.icache_block_size = 16 << ((iccfgr & SPR_ICCFGR_CBS) >> 7);
 	cpuinfo.icache_size =
-	    cache_set_size * cache_ways * cpuinfo.icache_block_size;
+	    cache_set_size * cpuinfo.icache_ways * cpuinfo.icache_block_size;
 
 	dccfgr = mfspr(SPR_DCCFGR);
-	cache_ways = 1 << (dccfgr & SPR_DCCFGR_NCW);
+	cpuinfo.dcache_ways = 1 << (dccfgr & SPR_DCCFGR_NCW);
 	cache_set_size = 1 << ((dccfgr & SPR_DCCFGR_NCS) >> 3);
 	cpuinfo.dcache_block_size = 16 << ((dccfgr & SPR_DCCFGR_CBS) >> 7);
 	cpuinfo.dcache_size =
-	    cache_set_size * cache_ways * cpuinfo.dcache_block_size;
+	    cache_set_size * cpuinfo.dcache_ways * cpuinfo.dcache_block_size;
 
 	if (of_property_read_u32(cpu, "clock-frequency",
 				 &cpuinfo.clock_frequency)) {
@@ -336,8 +338,10 @@ static int show_cpuinfo(struct seq_file *m, void *v)
 			  "frequency\t: %ld\n"
 			  "dcache size\t: %d bytes\n"
 			  "dcache block size\t: %d bytes\n"
+			  "dcache ways\t: %d\n"
 			  "icache size\t: %d bytes\n"
 			  "icache block size\t: %d bytes\n"
+			  "icache ways\t: %d\n"
 			  "immu\t\t: %d entries, %lu ways\n"
 			  "dmmu\t\t: %d entries, %lu ways\n"
 			  "bogomips\t: %lu.%02lu\n",
@@ -346,8 +350,10 @@ static int show_cpuinfo(struct seq_file *m, void *v)
 			  loops_per_jiffy * HZ,
 			  cpuinfo.dcache_size,
 			  cpuinfo.dcache_block_size,
+			  cpuinfo.dcache_ways,
 			  cpuinfo.icache_size,
 			  cpuinfo.icache_block_size,
+			  cpuinfo.icache_ways,
 			  1 << ((mfspr(SPR_DMMUCFGR) & SPR_DMMUCFGR_NTS) >> 2),
 			  1 + (mfspr(SPR_DMMUCFGR) & SPR_DMMUCFGR_NTW),
 			  1 << ((mfspr(SPR_IMMUCFGR) & SPR_IMMUCFGR_NTS) >> 2),
-- 
1.8.1.2


^ permalink raw reply	[flat|nested] 2+ messages in thread

* Re: [ORLinux] [PATCH] openrisc: add cache way information to cpuinfo
  2013-04-27 18:02 [PATCH] openrisc: add cache way information to cpuinfo Stefan Kristiansson
@ 2013-08-01  9:46 ` Jonas Bonn
  0 siblings, 0 replies; 2+ messages in thread
From: Jonas Bonn @ 2013-08-01  9:46 UTC (permalink / raw)
  To: Stefan Kristiansson; +Cc: linux-kernel, linux

On 04/27/2013 08:02 PM, Stefan Kristiansson wrote:
> Motivation for this is to be able to print the way information
> properly in print_cpuinfo(), instead of hardcoding it to one.
>
> Signed-off-by: Stefan Kristiansson <stefan.kristiansson@saunalahti.fi>

Applied.  Thanks.
/Jonas

> ---
>   arch/openrisc/include/asm/cpuinfo.h |  2 ++
>   arch/openrisc/kernel/setup.c        | 20 +++++++++++++-------
>   2 files changed, 15 insertions(+), 7 deletions(-)
>
> diff --git a/arch/openrisc/include/asm/cpuinfo.h b/arch/openrisc/include/asm/cpuinfo.h
> index 917318b..ec10679 100644
> --- a/arch/openrisc/include/asm/cpuinfo.h
> +++ b/arch/openrisc/include/asm/cpuinfo.h
> @@ -24,9 +24,11 @@ struct cpuinfo {
>
>   	u32 icache_size;
>   	u32 icache_block_size;
> +	u32 icache_ways;
>
>   	u32 dcache_size;
>   	u32 dcache_block_size;
> +	u32 dcache_ways;
>   };
>
>   extern struct cpuinfo cpuinfo;
> diff --git a/arch/openrisc/kernel/setup.c b/arch/openrisc/kernel/setup.c
> index f4d5bed..6b0e2c4 100644
> --- a/arch/openrisc/kernel/setup.c
> +++ b/arch/openrisc/kernel/setup.c
> @@ -129,13 +129,15 @@ static void print_cpuinfo(void)
>   	if (upr & SPR_UPR_DCP)
>   		printk(KERN_INFO
>   		       "-- dcache: %4d bytes total, %2d bytes/line, %d way(s)\n",
> -		       cpuinfo.dcache_size, cpuinfo.dcache_block_size, 1);
> +		       cpuinfo.dcache_size, cpuinfo.dcache_block_size,
> +		       cpuinfo.dcache_ways);
>   	else
>   		printk(KERN_INFO "-- dcache disabled\n");
>   	if (upr & SPR_UPR_ICP)
>   		printk(KERN_INFO
>   		       "-- icache: %4d bytes total, %2d bytes/line, %d way(s)\n",
> -		       cpuinfo.icache_size, cpuinfo.icache_block_size, 1);
> +		       cpuinfo.icache_size, cpuinfo.icache_block_size,
> +		       cpuinfo.icache_ways);
>   	else
>   		printk(KERN_INFO "-- icache disabled\n");
>
> @@ -167,25 +169,25 @@ void __init setup_cpuinfo(void)
>   {
>   	struct device_node *cpu;
>   	unsigned long iccfgr, dccfgr;
> -	unsigned long cache_set_size, cache_ways;
> +	unsigned long cache_set_size;
>
>   	cpu = of_find_compatible_node(NULL, NULL, "opencores,or1200-rtlsvn481");
>   	if (!cpu)
>   		panic("No compatible CPU found in device tree...\n");
>
>   	iccfgr = mfspr(SPR_ICCFGR);
> -	cache_ways = 1 << (iccfgr & SPR_ICCFGR_NCW);
> +	cpuinfo.icache_ways = 1 << (iccfgr & SPR_ICCFGR_NCW);
>   	cache_set_size = 1 << ((iccfgr & SPR_ICCFGR_NCS) >> 3);
>   	cpuinfo.icache_block_size = 16 << ((iccfgr & SPR_ICCFGR_CBS) >> 7);
>   	cpuinfo.icache_size =
> -	    cache_set_size * cache_ways * cpuinfo.icache_block_size;
> +	    cache_set_size * cpuinfo.icache_ways * cpuinfo.icache_block_size;
>
>   	dccfgr = mfspr(SPR_DCCFGR);
> -	cache_ways = 1 << (dccfgr & SPR_DCCFGR_NCW);
> +	cpuinfo.dcache_ways = 1 << (dccfgr & SPR_DCCFGR_NCW);
>   	cache_set_size = 1 << ((dccfgr & SPR_DCCFGR_NCS) >> 3);
>   	cpuinfo.dcache_block_size = 16 << ((dccfgr & SPR_DCCFGR_CBS) >> 7);
>   	cpuinfo.dcache_size =
> -	    cache_set_size * cache_ways * cpuinfo.dcache_block_size;
> +	    cache_set_size * cpuinfo.dcache_ways * cpuinfo.dcache_block_size;
>
>   	if (of_property_read_u32(cpu, "clock-frequency",
>   				 &cpuinfo.clock_frequency)) {
> @@ -336,8 +338,10 @@ static int show_cpuinfo(struct seq_file *m, void *v)
>   			  "frequency\t: %ld\n"
>   			  "dcache size\t: %d bytes\n"
>   			  "dcache block size\t: %d bytes\n"
> +			  "dcache ways\t: %d\n"
>   			  "icache size\t: %d bytes\n"
>   			  "icache block size\t: %d bytes\n"
> +			  "icache ways\t: %d\n"
>   			  "immu\t\t: %d entries, %lu ways\n"
>   			  "dmmu\t\t: %d entries, %lu ways\n"
>   			  "bogomips\t: %lu.%02lu\n",
> @@ -346,8 +350,10 @@ static int show_cpuinfo(struct seq_file *m, void *v)
>   			  loops_per_jiffy * HZ,
>   			  cpuinfo.dcache_size,
>   			  cpuinfo.dcache_block_size,
> +			  cpuinfo.dcache_ways,
>   			  cpuinfo.icache_size,
>   			  cpuinfo.icache_block_size,
> +			  cpuinfo.icache_ways,
>   			  1 << ((mfspr(SPR_DMMUCFGR) & SPR_DMMUCFGR_NTS) >> 2),
>   			  1 + (mfspr(SPR_DMMUCFGR) & SPR_DMMUCFGR_NTW),
>   			  1 << ((mfspr(SPR_IMMUCFGR) & SPR_IMMUCFGR_NTS) >> 2),
>


^ permalink raw reply	[flat|nested] 2+ messages in thread

end of thread, other threads:[~2013-08-01 10:06 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2013-04-27 18:02 [PATCH] openrisc: add cache way information to cpuinfo Stefan Kristiansson
2013-08-01  9:46 ` [ORLinux] " Jonas Bonn

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox

all inboxes | Powered by JetHome®