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

Sebastian Huber sebastian.huber at embedded-brains.de
Wed Jul 21 19:08:02 UTC 2021


On 21/07/2021 21:04, Gedare Bloom wrote:
> On Wed, Jul 21, 2021 at 12:31 PM Sebastian Huber
> <sebastian.huber at embedded-brains.de>  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 interrupts. In production, you want tested code.
>>
> ok, thanks. My comments are all pretty minor, except for the
> terminology issues of "cause" but that wording already exists. post
> the v2 series, but I probably won't review it and you can check it in
> if no one complains. It's up to you if you want to work a different
> wording than "cause" -- I prefer "raise"

Thanks a lot for the review.

Joel, what is your opinion with respect to "cause" vs. "raise"?

-- 
embedded brains GmbH
Herr Sebastian HUBER
Dornierstr. 4
82178 Puchheim
Germany
email: sebastian.huber at embedded-brains.de
phone: +49-89-18 94 741 - 16
fax:   +49-89-18 94 741 - 08

Registergericht: Amtsgericht München
Registernummer: HRB 157899
Vertretungsberechtigte Geschäftsführer: Peter Rasmussen, Thomas Dörfler
Unsere Datenschutzerklärung finden Sie hier:
https://embedded-brains.de/datenschutzerklaerung/


More information about the devel mailing list