* [V2] Octeon: Several misc fixes
@ 2010-03-03 8:43 Yang Shi
2010-03-03 8:43 ` [PATCH 1/3] MIPS: Octeon: Remove superfluous on_each_cpu parameter Yang Shi
0 siblings, 1 reply; 11+ messages in thread
From: Yang Shi @ 2010-03-03 8:43 UTC (permalink / raw)
To: ddaney, ralf, f.fainelli; +Cc: linux-mips
Hi David & Ralf,
These are some misc build fixes for Octeon.
Fixed two spelling errors against the first version.
Yang Shi (3):
MIPS: Octeon: Remove superfluous on_each_cpu parameter
MIPS: Octeon: Remove redundant declaration of octeon_reserve32_memory
MIPS: Octeon: Add add_wired_entry decralation in header file
Thanks,
Yang
^ permalink raw reply [flat|nested] 11+ messages in thread
* [PATCH 1/3] MIPS: Octeon: Remove superfluous on_each_cpu parameter
2010-03-03 8:43 [V2] Octeon: Several misc fixes Yang Shi
@ 2010-03-03 8:43 ` Yang Shi
2010-03-03 8:43 ` [PATCH 2/3] MIPS: Octeon: Remove redundant declaration of octeon_reserve32_memory Yang Shi
2010-03-03 17:53 ` [PATCH 1/3] MIPS: Octeon: Remove superfluous on_each_cpu parameter David Daney
0 siblings, 2 replies; 11+ messages in thread
From: Yang Shi @ 2010-03-03 8:43 UTC (permalink / raw)
To: ddaney, ralf, f.fainelli; +Cc: linux-mips
Now, on_each_cpu just need three parameters, but the on_each_cpu
still uses four parameters in Octeon's setup.c. So, remove the
superfluous parameter.
Signed-off-by: Yang Shi <yang.shi@windriver.com>
---
arch/mips/cavium-octeon/setup.c | 2 +-
1 files changed, 1 insertions(+), 1 deletions(-)
diff --git a/arch/mips/cavium-octeon/setup.c b/arch/mips/cavium-octeon/setup.c
index b321d3b..4eaa35f 100644
--- a/arch/mips/cavium-octeon/setup.c
+++ b/arch/mips/cavium-octeon/setup.c
@@ -230,7 +230,7 @@ static void octeon_hal_setup_per_cpu_reserved32(void *unused)
void octeon_hal_setup_reserved32(void)
{
#ifdef CONFIG_CAVIUM_RESERVE32_USE_WIRED_TLB
- on_each_cpu(octeon_hal_setup_per_cpu_reserved32, NULL, 0, 1);
+ on_each_cpu(octeon_hal_setup_per_cpu_reserved32, NULL, 1);
#endif
}
--
1.6.3.3
^ permalink raw reply related [flat|nested] 11+ messages in thread
* [PATCH 2/3] MIPS: Octeon: Remove redundant declaration of octeon_reserve32_memory
2010-03-03 8:43 ` [PATCH 1/3] MIPS: Octeon: Remove superfluous on_each_cpu parameter Yang Shi
@ 2010-03-03 8:43 ` Yang Shi
2010-03-03 8:43 ` [PATCH 3/3] MIPS: Octeon: Add add_wired_entry decralation in header file Yang Shi
2010-03-03 18:25 ` [PATCH 2/3] MIPS: Octeon: Remove redundant declaration of octeon_reserve32_memory David Daney
2010-03-03 17:53 ` [PATCH 1/3] MIPS: Octeon: Remove superfluous on_each_cpu parameter David Daney
1 sibling, 2 replies; 11+ messages in thread
From: Yang Shi @ 2010-03-03 8:43 UTC (permalink / raw)
To: ddaney, ralf, f.fainelli; +Cc: linux-mips
In Octeon's setup.c, octeon_reserve32_memory is defined, so remove the
redundant extern declaration of this variable.
Signed-off-by: Yang Shi <yang.shi@windriver.com>
---
arch/mips/cavium-octeon/setup.c | 3 ---
1 files changed, 0 insertions(+), 3 deletions(-)
diff --git a/arch/mips/cavium-octeon/setup.c b/arch/mips/cavium-octeon/setup.c
index 4eaa35f..8309d68 100644
--- a/arch/mips/cavium-octeon/setup.c
+++ b/arch/mips/cavium-octeon/setup.c
@@ -45,9 +45,6 @@ extern struct plat_smp_ops octeon_smp_ops;
extern void pci_console_init(const char *arg);
#endif
-#ifdef CONFIG_CAVIUM_RESERVE32
-extern uint64_t octeon_reserve32_memory;
-#endif
static unsigned long long MAX_MEMORY = 512ull << 20;
struct octeon_boot_descriptor *octeon_boot_desc_ptr;
--
1.6.3.3
^ permalink raw reply related [flat|nested] 11+ messages in thread
* [PATCH 3/3] MIPS: Octeon: Add add_wired_entry decralation in header file
2010-03-03 8:43 ` [PATCH 2/3] MIPS: Octeon: Remove redundant declaration of octeon_reserve32_memory Yang Shi
@ 2010-03-03 8:43 ` Yang Shi
2010-03-03 18:24 ` David Daney
2010-03-03 18:25 ` [PATCH 2/3] MIPS: Octeon: Remove redundant declaration of octeon_reserve32_memory David Daney
1 sibling, 1 reply; 11+ messages in thread
From: Yang Shi @ 2010-03-03 8:43 UTC (permalink / raw)
To: ddaney, ralf, f.fainelli; +Cc: linux-mips
Octeon's setup.c uses add_wired_entry, but it is not declared
anywhere. Copy add_wired_entry decralation from pgtable-32.h to
pgtable-64.h and include asm/pgtable.h into Octeon's setup.c.
Signed-off-by: Yang Shi <yang.shi@windriver.com>
---
arch/mips/cavium-octeon/setup.c | 1 +
arch/mips/include/asm/pgtable-64.h | 6 ++++++
2 files changed, 7 insertions(+), 0 deletions(-)
diff --git a/arch/mips/cavium-octeon/setup.c b/arch/mips/cavium-octeon/setup.c
index 8309d68..d63b8e6 100644
--- a/arch/mips/cavium-octeon/setup.c
+++ b/arch/mips/cavium-octeon/setup.c
@@ -30,6 +30,7 @@
#include <asm/bootinfo.h>
#include <asm/sections.h>
#include <asm/time.h>
+#include <asm/pgtable.h>
#include <asm/octeon/octeon.h>
diff --git a/arch/mips/include/asm/pgtable-64.h b/arch/mips/include/asm/pgtable-64.h
index 26dc69d..85ee34d 100644
--- a/arch/mips/include/asm/pgtable-64.h
+++ b/arch/mips/include/asm/pgtable-64.h
@@ -23,6 +23,12 @@
#endif
/*
+ * - add_wired_entry() add a fixed TLB entry, and move wired register
+ */
+extern void add_wired_entry(unsigned long entrylo0, unsigned long entrylo1,
+ unsigned long entryhi, unsigned long pagemask);
+
+/*
* Each address space has 2 4K pages as its page directory, giving 1024
* (== PTRS_PER_PGD) 8 byte pointers to pmd tables. Each pmd table is a
* single 4K page, giving 512 (== PTRS_PER_PMD) 8 byte pointers to page
--
1.6.3.3
^ permalink raw reply related [flat|nested] 11+ messages in thread
* Re: [PATCH 1/3] MIPS: Octeon: Remove superfluous on_each_cpu parameter
2010-03-03 8:43 ` [PATCH 1/3] MIPS: Octeon: Remove superfluous on_each_cpu parameter Yang Shi
2010-03-03 8:43 ` [PATCH 2/3] MIPS: Octeon: Remove redundant declaration of octeon_reserve32_memory Yang Shi
@ 2010-03-03 17:53 ` David Daney
2010-03-04 2:08 ` Yang Shi
1 sibling, 1 reply; 11+ messages in thread
From: David Daney @ 2010-03-03 17:53 UTC (permalink / raw)
To: Yang Shi; +Cc: ralf, f.fainelli, linux-mips
On 03/03/2010 12:43 AM, Yang Shi wrote:
> Now, on_each_cpu just need three parameters, but the on_each_cpu
> still uses four parameters in Octeon's setup.c. So, remove the
> superfluous parameter.
>
> Signed-off-by: Yang Shi<yang.shi@windriver.com>
NAK! We are removing CONFIG_CAVIUM_RESERVE32_USE_WIRED_TLB completely.
I will send the removal patch soon.
David Daney
> ---
> arch/mips/cavium-octeon/setup.c | 2 +-
> 1 files changed, 1 insertions(+), 1 deletions(-)
>
> diff --git a/arch/mips/cavium-octeon/setup.c b/arch/mips/cavium-octeon/setup.c
> index b321d3b..4eaa35f 100644
> --- a/arch/mips/cavium-octeon/setup.c
> +++ b/arch/mips/cavium-octeon/setup.c
> @@ -230,7 +230,7 @@ static void octeon_hal_setup_per_cpu_reserved32(void *unused)
> void octeon_hal_setup_reserved32(void)
> {
> #ifdef CONFIG_CAVIUM_RESERVE32_USE_WIRED_TLB
> - on_each_cpu(octeon_hal_setup_per_cpu_reserved32, NULL, 0, 1);
> + on_each_cpu(octeon_hal_setup_per_cpu_reserved32, NULL, 1);
> #endif
> }
>
^ permalink raw reply [flat|nested] 11+ messages in thread
* Re: [PATCH 3/3] MIPS: Octeon: Add add_wired_entry decralation in header file
2010-03-03 8:43 ` [PATCH 3/3] MIPS: Octeon: Add add_wired_entry decralation in header file Yang Shi
@ 2010-03-03 18:24 ` David Daney
2010-03-09 14:16 ` Ralf Baechle
0 siblings, 1 reply; 11+ messages in thread
From: David Daney @ 2010-03-03 18:24 UTC (permalink / raw)
To: Yang Shi; +Cc: ralf, f.fainelli, linux-mips
On 03/03/2010 12:43 AM, Yang Shi wrote:
> Octeon's setup.c uses add_wired_entry, but it is not declared
> anywhere. Copy add_wired_entry decralation from pgtable-32.h to
> pgtable-64.h and include asm/pgtable.h into Octeon's setup.c.
>
> Signed-off-by: Yang Shi<yang.shi@windriver.com>
NAK!
We are removing the use of add_wired_entry(), so adding a declaration
will not be necessary.
David Daney
> ---
> arch/mips/cavium-octeon/setup.c | 1 +
> arch/mips/include/asm/pgtable-64.h | 6 ++++++
> 2 files changed, 7 insertions(+), 0 deletions(-)
>
> diff --git a/arch/mips/cavium-octeon/setup.c b/arch/mips/cavium-octeon/setup.c
> index 8309d68..d63b8e6 100644
> --- a/arch/mips/cavium-octeon/setup.c
> +++ b/arch/mips/cavium-octeon/setup.c
> @@ -30,6 +30,7 @@
> #include<asm/bootinfo.h>
> #include<asm/sections.h>
> #include<asm/time.h>
> +#include<asm/pgtable.h>
>
> #include<asm/octeon/octeon.h>
>
> diff --git a/arch/mips/include/asm/pgtable-64.h b/arch/mips/include/asm/pgtable-64.h
> index 26dc69d..85ee34d 100644
> --- a/arch/mips/include/asm/pgtable-64.h
> +++ b/arch/mips/include/asm/pgtable-64.h
> @@ -23,6 +23,12 @@
> #endif
>
> /*
> + * - add_wired_entry() add a fixed TLB entry, and move wired register
> + */
> +extern void add_wired_entry(unsigned long entrylo0, unsigned long entrylo1,
> + unsigned long entryhi, unsigned long pagemask);
> +
> +/*
> * Each address space has 2 4K pages as its page directory, giving 1024
> * (== PTRS_PER_PGD) 8 byte pointers to pmd tables. Each pmd table is a
> * single 4K page, giving 512 (== PTRS_PER_PMD) 8 byte pointers to page
^ permalink raw reply [flat|nested] 11+ messages in thread
* Re: [PATCH 2/3] MIPS: Octeon: Remove redundant declaration of octeon_reserve32_memory
2010-03-03 8:43 ` [PATCH 2/3] MIPS: Octeon: Remove redundant declaration of octeon_reserve32_memory Yang Shi
2010-03-03 8:43 ` [PATCH 3/3] MIPS: Octeon: Add add_wired_entry decralation in header file Yang Shi
@ 2010-03-03 18:25 ` David Daney
2010-03-11 0:52 ` Ralf Baechle
1 sibling, 1 reply; 11+ messages in thread
From: David Daney @ 2010-03-03 18:25 UTC (permalink / raw)
To: Yang Shi, ralf; +Cc: f.fainelli, linux-mips
On 03/03/2010 12:43 AM, Yang Shi wrote:
> In Octeon's setup.c, octeon_reserve32_memory is defined, so remove the
> redundant extern declaration of this variable.
>
> Signed-off-by: Yang Shi<yang.shi@windriver.com>
Acked-by: David Daney <ddaney@caviumnetworks.com>
This looks good to me. Thanks,
David Daney
> ---
> arch/mips/cavium-octeon/setup.c | 3 ---
> 1 files changed, 0 insertions(+), 3 deletions(-)
>
> diff --git a/arch/mips/cavium-octeon/setup.c b/arch/mips/cavium-octeon/setup.c
> index 4eaa35f..8309d68 100644
> --- a/arch/mips/cavium-octeon/setup.c
> +++ b/arch/mips/cavium-octeon/setup.c
> @@ -45,9 +45,6 @@ extern struct plat_smp_ops octeon_smp_ops;
> extern void pci_console_init(const char *arg);
> #endif
>
> -#ifdef CONFIG_CAVIUM_RESERVE32
> -extern uint64_t octeon_reserve32_memory;
> -#endif
> static unsigned long long MAX_MEMORY = 512ull<< 20;
>
> struct octeon_boot_descriptor *octeon_boot_desc_ptr;
^ permalink raw reply [flat|nested] 11+ messages in thread
* Re: [PATCH 1/3] MIPS: Octeon: Remove superfluous on_each_cpu parameter
2010-03-03 17:53 ` [PATCH 1/3] MIPS: Octeon: Remove superfluous on_each_cpu parameter David Daney
@ 2010-03-04 2:08 ` Yang Shi
0 siblings, 0 replies; 11+ messages in thread
From: Yang Shi @ 2010-03-04 2:08 UTC (permalink / raw)
To: David Daney; +Cc: ralf, f.fainelli, linux-mips
David Daney 写道:
> On 03/03/2010 12:43 AM, Yang Shi wrote:
>
>> Now, on_each_cpu just need three parameters, but the on_each_cpu
>> still uses four parameters in Octeon's setup.c. So, remove the
>> superfluous parameter.
>>
>> Signed-off-by: Yang Shi<yang.shi@windriver.com>
>>
>
>
> NAK! We are removing CONFIG_CAVIUM_RESERVE32_USE_WIRED_TLB completely.
>
> I will send the removal patch soon.
>
Thanks David. Will apply your patch to remove WIRED_TLB.
Regards,
Yang
> David Daney
>
>
>> ---
>> arch/mips/cavium-octeon/setup.c | 2 +-
>> 1 files changed, 1 insertions(+), 1 deletions(-)
>>
>> diff --git a/arch/mips/cavium-octeon/setup.c b/arch/mips/cavium-octeon/setup.c
>> index b321d3b..4eaa35f 100644
>> --- a/arch/mips/cavium-octeon/setup.c
>> +++ b/arch/mips/cavium-octeon/setup.c
>> @@ -230,7 +230,7 @@ static void octeon_hal_setup_per_cpu_reserved32(void *unused)
>> void octeon_hal_setup_reserved32(void)
>> {
>> #ifdef CONFIG_CAVIUM_RESERVE32_USE_WIRED_TLB
>> - on_each_cpu(octeon_hal_setup_per_cpu_reserved32, NULL, 0, 1);
>> + on_each_cpu(octeon_hal_setup_per_cpu_reserved32, NULL, 1);
>> #endif
>> }
>>
>>
>
>
>
^ permalink raw reply [flat|nested] 11+ messages in thread
* Re: [PATCH 3/3] MIPS: Octeon: Add add_wired_entry decralation in header file
2010-03-03 18:24 ` David Daney
@ 2010-03-09 14:16 ` Ralf Baechle
0 siblings, 0 replies; 11+ messages in thread
From: Ralf Baechle @ 2010-03-09 14:16 UTC (permalink / raw)
To: David Daney; +Cc: Yang Shi, f.fainelli, linux-mips
On Wed, Mar 03, 2010 at 10:24:11AM -0800, David Daney wrote:
> On 03/03/2010 12:43 AM, Yang Shi wrote:
> >Octeon's setup.c uses add_wired_entry, but it is not declared
> >anywhere. Copy add_wired_entry decralation from pgtable-32.h to
> >pgtable-64.h and include asm/pgtable.h into Octeon's setup.c.
> >
> >Signed-off-by: Yang Shi<yang.shi@windriver.com>
>
> NAK!
>
> We are removing the use of add_wired_entry(), so adding a
> declaration will not be necessary.
For explanation - add_wired_entry is a horrible API; it requires the user
to have knowledge about the TLB structure, code differently for 32-bit and
64-bit kernels. It's just the API of terror.
Ralf
^ permalink raw reply [flat|nested] 11+ messages in thread
* Re: [PATCH 2/3] MIPS: Octeon: Remove redundant declaration of octeon_reserve32_memory
2010-03-03 18:25 ` [PATCH 2/3] MIPS: Octeon: Remove redundant declaration of octeon_reserve32_memory David Daney
@ 2010-03-11 0:52 ` Ralf Baechle
0 siblings, 0 replies; 11+ messages in thread
From: Ralf Baechle @ 2010-03-11 0:52 UTC (permalink / raw)
To: David Daney; +Cc: Yang Shi, f.fainelli, linux-mips
On Wed, Mar 03, 2010 at 10:25:49AM -0800, David Daney wrote:
> On 03/03/2010 12:43 AM, Yang Shi wrote:
> >In Octeon's setup.c, octeon_reserve32_memory is defined, so remove the
> >redundant extern declaration of this variable.
> >
> >Signed-off-by: Yang Shi<yang.shi@windriver.com>
>
> Acked-by: David Daney <ddaney@caviumnetworks.com>
>
> This looks good to me. Thanks,
> David Daney
Applied, thanks.
Ralf
^ permalink raw reply [flat|nested] 11+ messages in thread
* [PATCH 2/3] MIPS: Octeon: Remove redundant declaration of octeon_reserve32_memory
2010-03-03 7:26 ` [PATCH 1/3] MIPS: Octeon: Remove superfluous on_each_cpu parameter Yang Shi
@ 2010-03-03 7:26 ` Yang Shi
0 siblings, 0 replies; 11+ messages in thread
From: Yang Shi @ 2010-03-03 7:26 UTC (permalink / raw)
To: ddaney, ralf; +Cc: linux-mips
In Octeon's setup.c, octeon_reserve32_memory is defined, so remove the
redundant extern declaration of this variable.
Signed-off-by: Yang Shi <yang.shi@windriver.com>
---
arch/mips/cavium-octeon/setup.c | 3 ---
1 files changed, 0 insertions(+), 3 deletions(-)
diff --git a/arch/mips/cavium-octeon/setup.c b/arch/mips/cavium-octeon/setup.c
index 4eaa35f..8309d68 100644
--- a/arch/mips/cavium-octeon/setup.c
+++ b/arch/mips/cavium-octeon/setup.c
@@ -45,9 +45,6 @@ extern struct plat_smp_ops octeon_smp_ops;
extern void pci_console_init(const char *arg);
#endif
-#ifdef CONFIG_CAVIUM_RESERVE32
-extern uint64_t octeon_reserve32_memory;
-#endif
static unsigned long long MAX_MEMORY = 512ull << 20;
struct octeon_boot_descriptor *octeon_boot_desc_ptr;
--
1.6.3.3
^ permalink raw reply related [flat|nested] 11+ messages in thread
end of thread, other threads:[~2010-03-11 0:53 UTC | newest]
Thread overview: 11+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2010-03-03 8:43 [V2] Octeon: Several misc fixes Yang Shi
2010-03-03 8:43 ` [PATCH 1/3] MIPS: Octeon: Remove superfluous on_each_cpu parameter Yang Shi
2010-03-03 8:43 ` [PATCH 2/3] MIPS: Octeon: Remove redundant declaration of octeon_reserve32_memory Yang Shi
2010-03-03 8:43 ` [PATCH 3/3] MIPS: Octeon: Add add_wired_entry decralation in header file Yang Shi
2010-03-03 18:24 ` David Daney
2010-03-09 14:16 ` Ralf Baechle
2010-03-03 18:25 ` [PATCH 2/3] MIPS: Octeon: Remove redundant declaration of octeon_reserve32_memory David Daney
2010-03-11 0:52 ` Ralf Baechle
2010-03-03 17:53 ` [PATCH 1/3] MIPS: Octeon: Remove superfluous on_each_cpu parameter David Daney
2010-03-04 2:08 ` Yang Shi
-- strict thread matches above, loose matches on Subject: below --
2010-03-03 7:26 Octeon: Several misc fixes Yang Shi
2010-03-03 7:26 ` [PATCH 1/3] MIPS: Octeon: Remove superfluous on_each_cpu parameter Yang Shi
2010-03-03 7:26 ` [PATCH 2/3] MIPS: Octeon: Remove redundant declaration of octeon_reserve32_memory Yang Shi
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.