* [PATCH] hw/openrisc: use right OMPIC size variable
@ 2022-05-02 23:28 ` Jason A. Donenfeld
0 siblings, 0 replies; 16+ messages in thread
From: Jason A. Donenfeld @ 2022-05-02 23:28 UTC (permalink / raw)
To: qemu-devel, openrisc, shorne; +Cc: Jason A. Donenfeld
This appears to be a copy and paste error. The UART size was used
instead of the much smaller OMPIC size.
Signed-off-by: Jason A. Donenfeld <Jason@zx2c4.com>
---
hw/openrisc/openrisc_sim.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/hw/openrisc/openrisc_sim.c b/hw/openrisc/openrisc_sim.c
index 99b14940f4..bf434e9737 100644
--- a/hw/openrisc/openrisc_sim.c
+++ b/hw/openrisc/openrisc_sim.c
@@ -410,7 +410,7 @@ static void openrisc_sim_init(MachineState *machine)
if (smp_cpus > 1) {
openrisc_sim_ompic_init(state, or1ksim_memmap[OR1KSIM_OMPIC].base,
- or1ksim_memmap[OR1KSIM_UART].size,
+ or1ksim_memmap[OR1KSIM_OMPIC].size,
smp_cpus, cpus, OR1KSIM_OMPIC_IRQ);
}
--
2.35.1
^ permalink raw reply related [flat|nested] 16+ messages in thread
* [OpenRISC] [PATCH] hw/openrisc: use right OMPIC size variable
@ 2022-05-02 23:28 ` Jason A. Donenfeld
0 siblings, 0 replies; 16+ messages in thread
From: Jason A. Donenfeld @ 2022-05-02 23:28 UTC (permalink / raw)
To: openrisc
This appears to be a copy and paste error. The UART size was used
instead of the much smaller OMPIC size.
Signed-off-by: Jason A. Donenfeld <Jason@zx2c4.com>
---
hw/openrisc/openrisc_sim.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/hw/openrisc/openrisc_sim.c b/hw/openrisc/openrisc_sim.c
index 99b14940f4..bf434e9737 100644
--- a/hw/openrisc/openrisc_sim.c
+++ b/hw/openrisc/openrisc_sim.c
@@ -410,7 +410,7 @@ static void openrisc_sim_init(MachineState *machine)
if (smp_cpus > 1) {
openrisc_sim_ompic_init(state, or1ksim_memmap[OR1KSIM_OMPIC].base,
- or1ksim_memmap[OR1KSIM_UART].size,
+ or1ksim_memmap[OR1KSIM_OMPIC].size,
smp_cpus, cpus, OR1KSIM_OMPIC_IRQ);
}
--
2.35.1
^ permalink raw reply related [flat|nested] 16+ messages in thread
* Re: [PATCH] hw/openrisc: use right OMPIC size variable
2022-05-02 23:28 ` [OpenRISC] " Jason A. Donenfeld
@ 2022-05-02 23:59 ` Richard Henderson
-1 siblings, 0 replies; 16+ messages in thread
From: Richard Henderson @ 2022-05-02 23:59 UTC (permalink / raw)
To: Jason A. Donenfeld, qemu-devel, openrisc, shorne
On 5/2/22 16:28, Jason A. Donenfeld wrote:
> This appears to be a copy and paste error. The UART size was used
> instead of the much smaller OMPIC size.
>
> Signed-off-by: Jason A. Donenfeld<Jason@zx2c4.com>
> ---
> hw/openrisc/openrisc_sim.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
Reviewed-by: Richard Henderson <richard.henderson@linaro.org>
r~
^ permalink raw reply [flat|nested] 16+ messages in thread
* [OpenRISC] [PATCH] hw/openrisc: use right OMPIC size variable
@ 2022-05-02 23:59 ` Richard Henderson
0 siblings, 0 replies; 16+ messages in thread
From: Richard Henderson @ 2022-05-02 23:59 UTC (permalink / raw)
To: openrisc
On 5/2/22 16:28, Jason A. Donenfeld wrote:
> This appears to be a copy and paste error. The UART size was used
> instead of the much smaller OMPIC size.
>
> Signed-off-by: Jason A. Donenfeld<Jason@zx2c4.com>
> ---
> hw/openrisc/openrisc_sim.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
Reviewed-by: Richard Henderson <richard.henderson@linaro.org>
r~
^ permalink raw reply [flat|nested] 16+ messages in thread
* Re: [PATCH] hw/openrisc: use right OMPIC size variable
2022-05-02 23:59 ` [OpenRISC] " Richard Henderson
@ 2022-05-03 9:19 ` Stafford Horne
-1 siblings, 0 replies; 16+ messages in thread
From: Stafford Horne @ 2022-05-03 9:19 UTC (permalink / raw)
To: Richard Henderson; +Cc: Jason A. Donenfeld, qemu-devel, openrisc
On Mon, May 02, 2022 at 04:59:47PM -0700, Richard Henderson wrote:
> On 5/2/22 16:28, Jason A. Donenfeld wrote:
> > This appears to be a copy and paste error. The UART size was used
> > instead of the much smaller OMPIC size.
> >
> > Signed-off-by: Jason A. Donenfeld<Jason@zx2c4.com>
> > ---
> > hw/openrisc/openrisc_sim.c | 2 +-
> > 1 file changed, 1 insertion(+), 1 deletion(-)
>
> Reviewed-by: Richard Henderson <richard.henderson@linaro.org>
Thanks,
I will queue this one.
^ permalink raw reply [flat|nested] 16+ messages in thread
* [OpenRISC] [PATCH] hw/openrisc: use right OMPIC size variable
@ 2022-05-03 9:19 ` Stafford Horne
0 siblings, 0 replies; 16+ messages in thread
From: Stafford Horne @ 2022-05-03 9:19 UTC (permalink / raw)
To: openrisc
On Mon, May 02, 2022 at 04:59:47PM -0700, Richard Henderson wrote:
> On 5/2/22 16:28, Jason A. Donenfeld wrote:
> > This appears to be a copy and paste error. The UART size was used
> > instead of the much smaller OMPIC size.
> >
> > Signed-off-by: Jason A. Donenfeld<Jason@zx2c4.com>
> > ---
> > hw/openrisc/openrisc_sim.c | 2 +-
> > 1 file changed, 1 insertion(+), 1 deletion(-)
>
> Reviewed-by: Richard Henderson <richard.henderson@linaro.org>
Thanks,
I will queue this one.
^ permalink raw reply [flat|nested] 16+ messages in thread
* [PATCH v2] hw/openrisc: use right OMPIC size variable
2022-05-02 23:28 ` [OpenRISC] " Jason A. Donenfeld
@ 2022-05-03 9:45 ` Jason A. Donenfeld
-1 siblings, 0 replies; 16+ messages in thread
From: Jason A. Donenfeld @ 2022-05-03 9:45 UTC (permalink / raw)
To: qemu-devel, openrisc, shorne, richard.henderson; +Cc: Jason A. Donenfeld
This appears to be a copy and paste error. The UART size was used
instead of the much smaller OMPIC size. But actually that smaller OMPIC
size is wrong too and doesn't allow the IPI to work in Linux. So set it
to the old value.
Signed-off-by: Jason A. Donenfeld <Jason@zx2c4.com>
---
hw/openrisc/openrisc_sim.c | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/hw/openrisc/openrisc_sim.c b/hw/openrisc/openrisc_sim.c
index 99b14940f4..3218db6656 100644
--- a/hw/openrisc/openrisc_sim.c
+++ b/hw/openrisc/openrisc_sim.c
@@ -78,7 +78,7 @@ static const struct MemmapEntry {
[OR1KSIM_DRAM] = { 0x00000000, 0 },
[OR1KSIM_UART] = { 0x90000000, 0x100 },
[OR1KSIM_ETHOC] = { 0x92000000, 0x800 },
- [OR1KSIM_OMPIC] = { 0x98000000, 16 },
+ [OR1KSIM_OMPIC] = { 0x98000000, 0x100 },
};
static struct openrisc_boot_info {
@@ -410,7 +410,7 @@ static void openrisc_sim_init(MachineState *machine)
if (smp_cpus > 1) {
openrisc_sim_ompic_init(state, or1ksim_memmap[OR1KSIM_OMPIC].base,
- or1ksim_memmap[OR1KSIM_UART].size,
+ or1ksim_memmap[OR1KSIM_OMPIC].size,
smp_cpus, cpus, OR1KSIM_OMPIC_IRQ);
}
--
2.35.1
^ permalink raw reply related [flat|nested] 16+ messages in thread
* [OpenRISC] [PATCH v2] hw/openrisc: use right OMPIC size variable
@ 2022-05-03 9:45 ` Jason A. Donenfeld
0 siblings, 0 replies; 16+ messages in thread
From: Jason A. Donenfeld @ 2022-05-03 9:45 UTC (permalink / raw)
To: openrisc
This appears to be a copy and paste error. The UART size was used
instead of the much smaller OMPIC size. But actually that smaller OMPIC
size is wrong too and doesn't allow the IPI to work in Linux. So set it
to the old value.
Signed-off-by: Jason A. Donenfeld <Jason@zx2c4.com>
---
hw/openrisc/openrisc_sim.c | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/hw/openrisc/openrisc_sim.c b/hw/openrisc/openrisc_sim.c
index 99b14940f4..3218db6656 100644
--- a/hw/openrisc/openrisc_sim.c
+++ b/hw/openrisc/openrisc_sim.c
@@ -78,7 +78,7 @@ static const struct MemmapEntry {
[OR1KSIM_DRAM] = { 0x00000000, 0 },
[OR1KSIM_UART] = { 0x90000000, 0x100 },
[OR1KSIM_ETHOC] = { 0x92000000, 0x800 },
- [OR1KSIM_OMPIC] = { 0x98000000, 16 },
+ [OR1KSIM_OMPIC] = { 0x98000000, 0x100 },
};
static struct openrisc_boot_info {
@@ -410,7 +410,7 @@ static void openrisc_sim_init(MachineState *machine)
if (smp_cpus > 1) {
openrisc_sim_ompic_init(state, or1ksim_memmap[OR1KSIM_OMPIC].base,
- or1ksim_memmap[OR1KSIM_UART].size,
+ or1ksim_memmap[OR1KSIM_OMPIC].size,
smp_cpus, cpus, OR1KSIM_OMPIC_IRQ);
}
--
2.35.1
^ permalink raw reply related [flat|nested] 16+ messages in thread
* Re: [PATCH] hw/openrisc: use right OMPIC size variable
2022-05-03 9:19 ` [OpenRISC] " Stafford Horne
@ 2022-05-03 9:59 ` Jason A. Donenfeld
-1 siblings, 0 replies; 16+ messages in thread
From: Jason A. Donenfeld @ 2022-05-03 9:59 UTC (permalink / raw)
To: Stafford Horne; +Cc: Richard Henderson, qemu-devel, openrisc
On Tue, May 03, 2022 at 06:19:29PM +0900, Stafford Horne wrote:
> On Mon, May 02, 2022 at 04:59:47PM -0700, Richard Henderson wrote:
> > On 5/2/22 16:28, Jason A. Donenfeld wrote:
> > > This appears to be a copy and paste error. The UART size was used
> > > instead of the much smaller OMPIC size.
> > >
> > > Signed-off-by: Jason A. Donenfeld<Jason@zx2c4.com>
> > > ---
> > > hw/openrisc/openrisc_sim.c | 2 +-
> > > 1 file changed, 1 insertion(+), 1 deletion(-)
> >
> > Reviewed-by: Richard Henderson <richard.henderson@linaro.org>
>
> Thanks,
>
> I will queue this one.
>
Please take a look at the v2 I posted instead.
^ permalink raw reply [flat|nested] 16+ messages in thread
* [OpenRISC] [PATCH] hw/openrisc: use right OMPIC size variable
@ 2022-05-03 9:59 ` Jason A. Donenfeld
0 siblings, 0 replies; 16+ messages in thread
From: Jason A. Donenfeld @ 2022-05-03 9:59 UTC (permalink / raw)
To: openrisc
On Tue, May 03, 2022 at 06:19:29PM +0900, Stafford Horne wrote:
> On Mon, May 02, 2022 at 04:59:47PM -0700, Richard Henderson wrote:
> > On 5/2/22 16:28, Jason A. Donenfeld wrote:
> > > This appears to be a copy and paste error. The UART size was used
> > > instead of the much smaller OMPIC size.
> > >
> > > Signed-off-by: Jason A. Donenfeld<Jason@zx2c4.com>
> > > ---
> > > hw/openrisc/openrisc_sim.c | 2 +-
> > > 1 file changed, 1 insertion(+), 1 deletion(-)
> >
> > Reviewed-by: Richard Henderson <richard.henderson@linaro.org>
>
> Thanks,
>
> I will queue this one.
>
Please take a look at the v2 I posted instead.
^ permalink raw reply [flat|nested] 16+ messages in thread
* Re: [PATCH v2] hw/openrisc: use right OMPIC size variable
2022-05-03 9:45 ` [OpenRISC] " Jason A. Donenfeld
@ 2022-05-03 20:22 ` Stafford Horne
-1 siblings, 0 replies; 16+ messages in thread
From: Stafford Horne @ 2022-05-03 20:22 UTC (permalink / raw)
To: Jason A. Donenfeld; +Cc: qemu-devel, openrisc, richard.henderson
On Tue, May 03, 2022 at 11:45:33AM +0200, Jason A. Donenfeld wrote:
> This appears to be a copy and paste error. The UART size was used
> instead of the much smaller OMPIC size. But actually that smaller OMPIC
> size is wrong too and doesn't allow the IPI to work in Linux. So set it
> to the old value.
>
> Signed-off-by: Jason A. Donenfeld <Jason@zx2c4.com>
> ---
> hw/openrisc/openrisc_sim.c | 4 ++--
> 1 file changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/hw/openrisc/openrisc_sim.c b/hw/openrisc/openrisc_sim.c
> index 99b14940f4..3218db6656 100644
> --- a/hw/openrisc/openrisc_sim.c
> +++ b/hw/openrisc/openrisc_sim.c
> @@ -78,7 +78,7 @@ static const struct MemmapEntry {
> [OR1KSIM_DRAM] = { 0x00000000, 0 },
> [OR1KSIM_UART] = { 0x90000000, 0x100 },
> [OR1KSIM_ETHOC] = { 0x92000000, 0x800 },
> - [OR1KSIM_OMPIC] = { 0x98000000, 16 },
> + [OR1KSIM_OMPIC] = { 0x98000000, 0x100 },
Right, I missed this as part of my series. OMPIC will allocate 2 32-bit
registers per CPU. I documented this here:
- https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git/tree/drivers/irqchip/irq-ompic.c
I think what we will want here is something like:
[OR1KSIM_OMPIC] = { 0x98000000, 8 * OR1KSIM_CPUS_MAX },
> };
>
> static struct openrisc_boot_info {
> @@ -410,7 +410,7 @@ static void openrisc_sim_init(MachineState *machine)
>
> if (smp_cpus > 1) {
> openrisc_sim_ompic_init(state, or1ksim_memmap[OR1KSIM_OMPIC].base,
> - or1ksim_memmap[OR1KSIM_UART].size,
> + or1ksim_memmap[OR1KSIM_OMPIC].size,
> smp_cpus, cpus, OR1KSIM_OMPIC_IRQ);
> }
>
> --
> 2.35.1
>
^ permalink raw reply [flat|nested] 16+ messages in thread
* [PATCH v2] hw/openrisc: use right OMPIC size variable
@ 2022-05-03 20:22 ` Stafford Horne
0 siblings, 0 replies; 16+ messages in thread
From: Stafford Horne @ 2022-05-03 20:22 UTC (permalink / raw)
To: openrisc
On Tue, May 03, 2022 at 11:45:33AM +0200, Jason A. Donenfeld wrote:
> This appears to be a copy and paste error. The UART size was used
> instead of the much smaller OMPIC size. But actually that smaller OMPIC
> size is wrong too and doesn't allow the IPI to work in Linux. So set it
> to the old value.
>
> Signed-off-by: Jason A. Donenfeld <Jason@zx2c4.com>
> ---
> hw/openrisc/openrisc_sim.c | 4 ++--
> 1 file changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/hw/openrisc/openrisc_sim.c b/hw/openrisc/openrisc_sim.c
> index 99b14940f4..3218db6656 100644
> --- a/hw/openrisc/openrisc_sim.c
> +++ b/hw/openrisc/openrisc_sim.c
> @@ -78,7 +78,7 @@ static const struct MemmapEntry {
> [OR1KSIM_DRAM] = { 0x00000000, 0 },
> [OR1KSIM_UART] = { 0x90000000, 0x100 },
> [OR1KSIM_ETHOC] = { 0x92000000, 0x800 },
> - [OR1KSIM_OMPIC] = { 0x98000000, 16 },
> + [OR1KSIM_OMPIC] = { 0x98000000, 0x100 },
Right, I missed this as part of my series. OMPIC will allocate 2 32-bit
registers per CPU. I documented this here:
- https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git/tree/drivers/irqchip/irq-ompic.c
I think what we will want here is something like:
[OR1KSIM_OMPIC] = { 0x98000000, 8 * OR1KSIM_CPUS_MAX },
> };
>
> static struct openrisc_boot_info {
> @@ -410,7 +410,7 @@ static void openrisc_sim_init(MachineState *machine)
>
> if (smp_cpus > 1) {
> openrisc_sim_ompic_init(state, or1ksim_memmap[OR1KSIM_OMPIC].base,
> - or1ksim_memmap[OR1KSIM_UART].size,
> + or1ksim_memmap[OR1KSIM_OMPIC].size,
> smp_cpus, cpus, OR1KSIM_OMPIC_IRQ);
> }
>
> --
> 2.35.1
>
^ permalink raw reply [flat|nested] 16+ messages in thread
* Re: [PATCH v2] hw/openrisc: use right OMPIC size variable
2022-05-03 20:22 ` Stafford Horne
@ 2022-05-04 11:10 ` Jason A. Donenfeld
-1 siblings, 0 replies; 16+ messages in thread
From: Jason A. Donenfeld @ 2022-05-04 11:10 UTC (permalink / raw)
To: Stafford Horne; +Cc: QEMU Developers, openrisc, richard.henderson
On Tue, May 3, 2022 at 10:22 PM Stafford Horne <shorne@gmail.com> wrote:
>
> On Tue, May 03, 2022 at 11:45:33AM +0200, Jason A. Donenfeld wrote:
> > This appears to be a copy and paste error. The UART size was used
> > instead of the much smaller OMPIC size. But actually that smaller OMPIC
> > size is wrong too and doesn't allow the IPI to work in Linux. So set it
> > to the old value.
> >
> > Signed-off-by: Jason A. Donenfeld <Jason@zx2c4.com>
> > ---
> > hw/openrisc/openrisc_sim.c | 4 ++--
> > 1 file changed, 2 insertions(+), 2 deletions(-)
> >
> > diff --git a/hw/openrisc/openrisc_sim.c b/hw/openrisc/openrisc_sim.c
> > index 99b14940f4..3218db6656 100644
> > --- a/hw/openrisc/openrisc_sim.c
> > +++ b/hw/openrisc/openrisc_sim.c
> > @@ -78,7 +78,7 @@ static const struct MemmapEntry {
> > [OR1KSIM_DRAM] = { 0x00000000, 0 },
> > [OR1KSIM_UART] = { 0x90000000, 0x100 },
> > [OR1KSIM_ETHOC] = { 0x92000000, 0x800 },
> > - [OR1KSIM_OMPIC] = { 0x98000000, 16 },
> > + [OR1KSIM_OMPIC] = { 0x98000000, 0x100 },
>
> Right, I missed this as part of my series. OMPIC will allocate 2 32-bit
> registers per CPU. I documented this here:
>
> - https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git/tree/drivers/irqchip/irq-ompic.c
>
> I think what we will want here is something like:
>
> [OR1KSIM_OMPIC] = { 0x98000000, 8 * OR1KSIM_CPUS_MAX },
Do you want a v3 or are you going to fix it up yourself?
Jason
^ permalink raw reply [flat|nested] 16+ messages in thread
* [PATCH v2] hw/openrisc: use right OMPIC size variable
@ 2022-05-04 11:10 ` Jason A. Donenfeld
0 siblings, 0 replies; 16+ messages in thread
From: Jason A. Donenfeld @ 2022-05-04 11:10 UTC (permalink / raw)
To: openrisc
On Tue, May 3, 2022 at 10:22 PM Stafford Horne <shorne@gmail.com> wrote:
>
> On Tue, May 03, 2022 at 11:45:33AM +0200, Jason A. Donenfeld wrote:
> > This appears to be a copy and paste error. The UART size was used
> > instead of the much smaller OMPIC size. But actually that smaller OMPIC
> > size is wrong too and doesn't allow the IPI to work in Linux. So set it
> > to the old value.
> >
> > Signed-off-by: Jason A. Donenfeld <Jason@zx2c4.com>
> > ---
> > hw/openrisc/openrisc_sim.c | 4 ++--
> > 1 file changed, 2 insertions(+), 2 deletions(-)
> >
> > diff --git a/hw/openrisc/openrisc_sim.c b/hw/openrisc/openrisc_sim.c
> > index 99b14940f4..3218db6656 100644
> > --- a/hw/openrisc/openrisc_sim.c
> > +++ b/hw/openrisc/openrisc_sim.c
> > @@ -78,7 +78,7 @@ static const struct MemmapEntry {
> > [OR1KSIM_DRAM] = { 0x00000000, 0 },
> > [OR1KSIM_UART] = { 0x90000000, 0x100 },
> > [OR1KSIM_ETHOC] = { 0x92000000, 0x800 },
> > - [OR1KSIM_OMPIC] = { 0x98000000, 16 },
> > + [OR1KSIM_OMPIC] = { 0x98000000, 0x100 },
>
> Right, I missed this as part of my series. OMPIC will allocate 2 32-bit
> registers per CPU. I documented this here:
>
> - https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git/tree/drivers/irqchip/irq-ompic.c
>
> I think what we will want here is something like:
>
> [OR1KSIM_OMPIC] = { 0x98000000, 8 * OR1KSIM_CPUS_MAX },
Do you want a v3 or are you going to fix it up yourself?
Jason
^ permalink raw reply [flat|nested] 16+ messages in thread
* Re: [PATCH v2] hw/openrisc: use right OMPIC size variable
2022-05-04 11:10 ` Jason A. Donenfeld
@ 2022-05-04 22:23 ` Stafford Horne
-1 siblings, 0 replies; 16+ messages in thread
From: Stafford Horne @ 2022-05-04 22:23 UTC (permalink / raw)
To: Jason A. Donenfeld; +Cc: QEMU Developers, openrisc, richard.henderson
On Wed, May 04, 2022 at 01:10:04PM +0200, Jason A. Donenfeld wrote:
> On Tue, May 3, 2022 at 10:22 PM Stafford Horne <shorne@gmail.com> wrote:
> >
> > On Tue, May 03, 2022 at 11:45:33AM +0200, Jason A. Donenfeld wrote:
> > > This appears to be a copy and paste error. The UART size was used
> > > instead of the much smaller OMPIC size. But actually that smaller OMPIC
> > > size is wrong too and doesn't allow the IPI to work in Linux. So set it
> > > to the old value.
> > >
> > > Signed-off-by: Jason A. Donenfeld <Jason@zx2c4.com>
> > > ---
> > > hw/openrisc/openrisc_sim.c | 4 ++--
> > > 1 file changed, 2 insertions(+), 2 deletions(-)
> > >
> > > diff --git a/hw/openrisc/openrisc_sim.c b/hw/openrisc/openrisc_sim.c
> > > index 99b14940f4..3218db6656 100644
> > > --- a/hw/openrisc/openrisc_sim.c
> > > +++ b/hw/openrisc/openrisc_sim.c
> > > @@ -78,7 +78,7 @@ static const struct MemmapEntry {
> > > [OR1KSIM_DRAM] = { 0x00000000, 0 },
> > > [OR1KSIM_UART] = { 0x90000000, 0x100 },
> > > [OR1KSIM_ETHOC] = { 0x92000000, 0x800 },
> > > - [OR1KSIM_OMPIC] = { 0x98000000, 16 },
> > > + [OR1KSIM_OMPIC] = { 0x98000000, 0x100 },
> >
> > Right, I missed this as part of my series. OMPIC will allocate 2 32-bit
> > registers per CPU. I documented this here:
> >
> > - https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git/tree/drivers/irqchip/irq-ompic.c
> >
> > I think what we will want here is something like:
> >
> > [OR1KSIM_OMPIC] = { 0x98000000, 8 * OR1KSIM_CPUS_MAX },
>
> Do you want a v3 or are you going to fix it up yourself?
I'll fix it up.
-Stafford
^ permalink raw reply [flat|nested] 16+ messages in thread
* [PATCH v2] hw/openrisc: use right OMPIC size variable
@ 2022-05-04 22:23 ` Stafford Horne
0 siblings, 0 replies; 16+ messages in thread
From: Stafford Horne @ 2022-05-04 22:23 UTC (permalink / raw)
To: openrisc
On Wed, May 04, 2022 at 01:10:04PM +0200, Jason A. Donenfeld wrote:
> On Tue, May 3, 2022 at 10:22 PM Stafford Horne <shorne@gmail.com> wrote:
> >
> > On Tue, May 03, 2022 at 11:45:33AM +0200, Jason A. Donenfeld wrote:
> > > This appears to be a copy and paste error. The UART size was used
> > > instead of the much smaller OMPIC size. But actually that smaller OMPIC
> > > size is wrong too and doesn't allow the IPI to work in Linux. So set it
> > > to the old value.
> > >
> > > Signed-off-by: Jason A. Donenfeld <Jason@zx2c4.com>
> > > ---
> > > hw/openrisc/openrisc_sim.c | 4 ++--
> > > 1 file changed, 2 insertions(+), 2 deletions(-)
> > >
> > > diff --git a/hw/openrisc/openrisc_sim.c b/hw/openrisc/openrisc_sim.c
> > > index 99b14940f4..3218db6656 100644
> > > --- a/hw/openrisc/openrisc_sim.c
> > > +++ b/hw/openrisc/openrisc_sim.c
> > > @@ -78,7 +78,7 @@ static const struct MemmapEntry {
> > > [OR1KSIM_DRAM] = { 0x00000000, 0 },
> > > [OR1KSIM_UART] = { 0x90000000, 0x100 },
> > > [OR1KSIM_ETHOC] = { 0x92000000, 0x800 },
> > > - [OR1KSIM_OMPIC] = { 0x98000000, 16 },
> > > + [OR1KSIM_OMPIC] = { 0x98000000, 0x100 },
> >
> > Right, I missed this as part of my series. OMPIC will allocate 2 32-bit
> > registers per CPU. I documented this here:
> >
> > - https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git/tree/drivers/irqchip/irq-ompic.c
> >
> > I think what we will want here is something like:
> >
> > [OR1KSIM_OMPIC] = { 0x98000000, 8 * OR1KSIM_CPUS_MAX },
>
> Do you want a v3 or are you going to fix it up yourself?
I'll fix it up.
-Stafford
^ permalink raw reply [flat|nested] 16+ messages in thread
end of thread, other threads:[~2022-05-04 22:25 UTC | newest]
Thread overview: 16+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2022-05-02 23:28 [PATCH] hw/openrisc: use right OMPIC size variable Jason A. Donenfeld
2022-05-02 23:28 ` [OpenRISC] " Jason A. Donenfeld
2022-05-02 23:59 ` Richard Henderson
2022-05-02 23:59 ` [OpenRISC] " Richard Henderson
2022-05-03 9:19 ` Stafford Horne
2022-05-03 9:19 ` [OpenRISC] " Stafford Horne
2022-05-03 9:59 ` Jason A. Donenfeld
2022-05-03 9:59 ` [OpenRISC] " Jason A. Donenfeld
2022-05-03 9:45 ` [PATCH v2] " Jason A. Donenfeld
2022-05-03 9:45 ` [OpenRISC] " Jason A. Donenfeld
2022-05-03 20:22 ` Stafford Horne
2022-05-03 20:22 ` Stafford Horne
2022-05-04 11:10 ` Jason A. Donenfeld
2022-05-04 11:10 ` Jason A. Donenfeld
2022-05-04 22:23 ` Stafford Horne
2022-05-04 22:23 ` Stafford Horne
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.