[PATCH 2/2] gpio/aspeed-sgpio: don't enable all interrupts by default
Jeremy Kerr
jk at codeconstruct.com.au
Fri Sep 11 11:12:11 AEST 2020
Hi Joel,
> And also removing the enabling of IRQs. This part makes sense, as
> it's what the commit message says.
>
> If you think a sensible default should be single edge (and I would
> agree with that change), perhaps update the comment to say "set
> single edge trigger mode" and mention it in your commit message.
OK, shall do. That was my intention with the "reasonable defaults"
reference, but being explicit about that better here.
I'll send a v2 with an updated commit message.
Cheers,
Jeremy
More information about the Linux-aspeed
mailing list