Re: [PATCH] bsps/riscv: Clear interrupt complete before disable

2023-03-14 Thread Sebastian Huber
On 14.03.23 14:37, padmarao.beg...@microchip.com wrote: On Tue, 2023-03-14 at 13:37 +0100, Sebastian Huber wrote: On 14.03.23 13:31,padmarao.beg...@microchip.com wrote: Hi Sebastian, On Mon, 2023-03-06 at 14:11 +0100, Sebastian Huber wrote: On 06.03.23 13:00,padmarao.beg...@microchip.com

Re: [PATCH] bsps/riscv: Clear interrupt complete before disable

2023-03-14 Thread Padmarao.Begari
> On Tue, 2023-03-14 at 13:37 +0100, Sebastian Huber wrote: > > On 14.03.23 13:31, padmarao.beg...@microchip.com wrote: > > Hi Sebastian, > > > > > On Mon, 2023-03-06 at 14:11 +0100, Sebastian Huber wrote: > > > > > > > > > On 06.03.23 13:00,padmarao.beg...@microchip.com wrote: > > > > > On

Re: [PATCH] bsps/riscv: Clear interrupt complete before disable

2023-03-14 Thread Sebastian Huber
On 14.03.23 13:31, padmarao.beg...@microchip.com wrote: Hi Sebastian, On Mon, 2023-03-06 at 14:11 +0100, Sebastian Huber wrote: On 06.03.23 13:00,padmarao.beg...@microchip.com wrote: On Mon, 2023-03-06 at 11:19 +0100, Sebastian Huber wrote: On 06.03.23

Re: [PATCH] bsps/riscv: Clear interrupt complete before disable

2023-03-14 Thread Padmarao.Begari
Hi Sebastian, >On Mon, 2023-03-06 at 14:11 +0100, Sebastian Huber wrote: > > > On 06.03.23 13:00, padmarao.beg...@microchip.com wrote: > > > On Mon, 2023-03-06 at 11:19 +0100, Sebastian Huber wrote: > > > > > > On 06.03.23 10:24,padmarao.beg...@microchip.com wrote: > > > > Hi Sebastian, > > >

Re: [PATCH] bsps/riscv: Clear interrupt complete before disable

2023-03-06 Thread Padmarao.Begari
> On Mon, 2023-03-06 at 14:11 +0100, Sebastian Huber wrote: > > On 06.03.23 13:00, padmarao.beg...@microchip.com wrote: > > > On Mon, 2023-03-06 at 11:19 +0100, Sebastian Huber wrote: > > > > > > On 06.03.23 10:24,padmarao.beg...@microchip.com wrote: > > > > Hi Sebastian, > > > > > > > > > On

Re: [PATCH] bsps/riscv: Clear interrupt complete before disable

2023-03-06 Thread Sebastian Huber
On 06.03.23 13:00, padmarao.beg...@microchip.com wrote: On Mon, 2023-03-06 at 11:19 +0100, Sebastian Huber wrote: On 06.03.23 10:24,padmarao.beg...@microchip.com wrote: Hi Sebastian, On Mon, 2023-03-06 at 09:41 +0100, Sebastian Huber wrote: On 06.03.23 09:37,padmarao.beg...@microchip.com

Re: [PATCH] bsps/riscv: Clear interrupt complete before disable

2023-03-06 Thread Padmarao.Begari
> On Mon, 2023-03-06 at 11:19 +0100, Sebastian Huber wrote: > > On 06.03.23 10:24, padmarao.beg...@microchip.com wrote: > > Hi Sebastian, > > > > > On Mon, 2023-03-06 at 09:41 +0100, Sebastian Huber wrote: > > > > > > On 06.03.23 09:37, padmarao.beg...@microchip.com wrote: > > > > > Is > > > >

Re: [PATCH] bsps/riscv: Clear interrupt complete before disable

2023-03-06 Thread Sebastian Huber
On 06.03.23 10:24, padmarao.beg...@microchip.com wrote: Hi Sebastian, On Mon, 2023-03-06 at 09:41 +0100, Sebastian Huber wrote: On 06.03.23 09:37, padmarao.beg...@microchip.com wrote: Is the claim complete message ignored if the interrupt is disabled? Yes. Is this an intended and

Re: [PATCH] bsps/riscv: Clear interrupt complete before disable

2023-03-06 Thread Padmarao.Begari
Hi Sebastian, > On Mon, 2023-03-06 at 09:41 +0100, Sebastian Huber wrote: > > On 06.03.23 09:37, padmarao.beg...@microchip.com wrote: > > > Is > > > the claim complete message ignored if the interrupt is disabled? > > > > > Yes. > > Is this an intended and documented behaviour of the PLIC?

Re: [PATCH] bsps/riscv: Clear interrupt complete before disable

2023-03-06 Thread Sebastian Huber
On 06.03.23 09:37, padmarao.beg...@microchip.com wrote: Is the claim complete message ignored if the interrupt is disabled? Yes. Is this an intended and documented behaviour of the PLIC? This is a hardware bug from my point of view. If we really need a software workround just for the PLIC

Re: [PATCH] bsps/riscv: Clear interrupt complete before disable

2023-03-06 Thread Padmarao.Begari
Hi Sebastian, > On Mon, 2023-03-06 at 08:01 +0100, Sebastian Huber wrote: > > Hello Padmarao, > > On 03.03.23 15:55, padmarao.beg...@microchip.com wrote: > > > On Thu, 2023-03-02 at 15:18 +0100, Sebastian Huber wrote: > > > > > > > > > On 27.02.23 16:51, Padmarao Begari wrote: > > > > The

Re: [PATCH] bsps/riscv: Clear interrupt complete before disable

2023-03-05 Thread Sebastian Huber
Hello Padmarao, On 03.03.23 15:55, padmarao.beg...@microchip.com wrote: On Thu, 2023-03-02 at 15:18 +0100, Sebastian Huber wrote: On 27.02.23 16:51, Padmarao Begari wrote: The interrupt complete should clear with the interrupt number before disabling the interrupt in the PLIC to get the next

Re: [PATCH] bsps/riscv: Clear interrupt complete before disable

2023-03-03 Thread Padmarao.Begari
Hi Sebastian, > On Thu, 2023-03-02 at 15:18 +0100, Sebastian Huber wrote: > > > On 27.02.23 16:51, Padmarao Begari wrote: > > The interrupt complete should clear with the interrupt > > number before disabling the interrupt in the PLIC to > > get the next interrupt. > > Which problem does this

Re: [PATCH] bsps/riscv: Clear interrupt complete before disable

2023-03-02 Thread Sebastian Huber
On 27.02.23 16:51, Padmarao Begari wrote: The interrupt complete should clear with the interrupt number before disabling the interrupt in the PLIC to get the next interrupt. Which problem does this patch address? -- embedded brains GmbH Herr Sebastian HUBER Dornierstr. 4 82178 Puchheim

[PATCH] bsps/riscv: Clear interrupt complete before disable

2023-02-27 Thread Padmarao Begari
The interrupt complete should clear with the interrupt number before disabling the interrupt in the PLIC to get the next interrupt. --- bsps/riscv/riscv/irq/irq.c | 4 1 file changed, 4 insertions(+) diff --git a/bsps/riscv/riscv/irq/irq.c b/bsps/riscv/riscv/irq/irq.c index