Re: [PATCH 20/41] sparc/irq: Implement new interrupt directives

2021-07-22 Thread Sebastian Huber
On 22/07/2021 08:43, Chris Johns wrote: On 22/7/21 5:08 am, Sebastian Huber wrote: On 21/07/2021 21:04, Gedare Bloom wrote: On Wed, Jul 21, 2021 at 12:31 PM Sebastian Huber   wrote: On 21/07/2021 20:28, Gedare Bloom wrote: Why not throw an error here instead? In production, you wouldn't

Re: [PATCH 20/41] sparc/irq: Implement new interrupt directives

2021-07-22 Thread Chris Johns
On 22/7/21 5:08 am, Sebastian Huber wrote: > On 21/07/2021 21:04, Gedare Bloom wrote: >> On Wed, Jul 21, 2021 at 12:31 PM Sebastian Huber >>   wrote: >>> On 21/07/2021 20:28, Gedare Bloom wrote: Why not throw an error here instead? In production, you wouldn't want this code... >>> The

Re: [PATCH 20/41] sparc/irq: Implement new interrupt directives

2021-07-21 Thread Sebastian Huber
On 21/07/2021 21:04, Gedare Bloom wrote: On Wed, Jul 21, 2021 at 12:31 PM Sebastian Huber wrote: On 21/07/2021 20:28, Gedare Bloom wrote: Why not throw an error here instead? In production, you wouldn't want this code... The main issue is the bad chip design. If we don't have this code, we

Re: [PATCH 20/41] sparc/irq: Implement new interrupt directives

2021-07-21 Thread Gedare Bloom
On Wed, Jul 21, 2021 at 12:31 PM Sebastian Huber wrote: > > On 21/07/2021 20:28, Gedare Bloom wrote: > > Why not throw an error here instead? In production, you wouldn't want > > this code... > > The main issue is the bad chip design. If we don't have this code, we > can't test the extended

Re: [PATCH 20/41] sparc/irq: Implement new interrupt directives

2021-07-21 Thread Sebastian Huber
On 21/07/2021 20:28, Gedare Bloom wrote: Why not throw an error here instead? In production, you wouldn't want this code... The main issue is the bad chip design. If we don't have this code, we can't test the extended interrupts. In production, you want tested code. +

Re: [PATCH 20/41] sparc/irq: Implement new interrupt directives

2021-07-21 Thread Gedare Bloom
On Mon, Jul 12, 2021 at 6:50 AM Sebastian Huber wrote: > > Update #3269. > --- > bsps/sparc/leon3/start/eirq.c | 86 +++--- > bsps/sparc/shared/irq/irq-shared.c | 17 -- > 2 files changed, 91 insertions(+), 12 deletions(-) > > diff --git

[PATCH 20/41] sparc/irq: Implement new interrupt directives

2021-07-12 Thread Sebastian Huber
Update #3269. --- bsps/sparc/leon3/start/eirq.c | 86 +++--- bsps/sparc/shared/irq/irq-shared.c | 17 -- 2 files changed, 91 insertions(+), 12 deletions(-) diff --git a/bsps/sparc/leon3/start/eirq.c b/bsps/sparc/leon3/start/eirq.c index d6373bf00a..4f5f910106