On 18/05/2020 18.07, Pierre Morel wrote: > Let's make it possible to add and remove a custom io interrupt handler, > that can be used instead of the normal one. > > Signed-off-by: Pierre Morel <pmorel@xxxxxxxxxxxxx> > Reviewed-by: Thomas Huth <thuth@xxxxxxxxxx> > Reviewed-by: David Hildenbrand <david@xxxxxxxxxx> > Reviewed-by: Janosch Frank <frankja@xxxxxxxxxxxxx> > --- > lib/s390x/interrupt.c | 23 ++++++++++++++++++++++- > lib/s390x/interrupt.h | 8 ++++++++ > 2 files changed, 30 insertions(+), 1 deletion(-) > create mode 100644 lib/s390x/interrupt.h ... > diff --git a/lib/s390x/interrupt.h b/lib/s390x/interrupt.h > new file mode 100644 > index 0000000..323258d > --- /dev/null > +++ b/lib/s390x/interrupt.h > @@ -0,0 +1,8 @@ > +#ifndef __INTERRUPT_H > +#define __INTERRUPT_H Looking at this patch again, I noticed another nit: No double underscores at the beginning of header guards, please! That's reserved namespace. Simply use INTERRUPT_H or S390X_INTERRUPT_H or something similar instead. Thanks, Thomas