RE: [PATCH 2/3] aspeed-video: clear spurious interrupt bits unconditionally

2020-12-22 Thread Ryan Chen
t; ; Linux Kernel Mailing List > ; Jae Hyun Yoo > Subject: Re: [PATCH 2/3] aspeed-video: clear spurious interrupt bits > unconditionally > > On Tue, Dec 22, 2020 at 08:53:33PM CST, Ryan Chen wrote: > >> -Original Message- > >> From: Joel Stanley > >&

Re: [PATCH 2/3] aspeed-video: clear spurious interrupt bits unconditionally

2020-12-22 Thread Zev Weiss
; linux-aspeed ; Linux Kernel Mailing List ; Jae Hyun Yoo Subject: Re: [PATCH 2/3] aspeed-video: clear spurious interrupt bits unconditionally On Tue, 22 Dec 2020 at 19:14, Zev Weiss wrote: > > On Mon, Dec 21, 2020 at 10:47:37PM CST, Joel Stanley wrote: > >On Tue, 15 Dec 2020 at 02:4

RE: [PATCH 2/3] aspeed-video: clear spurious interrupt bits unconditionally

2020-12-22 Thread Ryan Chen
t; ; Linux Kernel Mailing List > ; Jae Hyun Yoo > Subject: Re: [PATCH 2/3] aspeed-video: clear spurious interrupt bits > unconditionally > > On Tue, 22 Dec 2020 at 19:14, Zev Weiss wrote: > > > > On Mon, Dec 21, 2020 at 10:47:37PM CST, Joel Stanley w

Re: [PATCH 2/3] aspeed-video: clear spurious interrupt bits unconditionally

2020-12-22 Thread Joel Stanley
On Tue, 22 Dec 2020 at 19:14, Zev Weiss wrote: > > On Mon, Dec 21, 2020 at 10:47:37PM CST, Joel Stanley wrote: > >On Tue, 15 Dec 2020 at 02:46, Zev Weiss wrote: > >> > >> Instead of testing and conditionally clearing them one by one, we can > >> instead just unconditionally clear them all at

Re: [PATCH 2/3] aspeed-video: clear spurious interrupt bits unconditionally

2020-12-22 Thread Zev Weiss
On Mon, Dec 21, 2020 at 10:47:37PM CST, Joel Stanley wrote: On Tue, 15 Dec 2020 at 02:46, Zev Weiss wrote: Instead of testing and conditionally clearing them one by one, we can instead just unconditionally clear them all at once. Signed-off-by: Zev Weiss I had a poke at the assembly and

Re: [PATCH 2/3] aspeed-video: clear spurious interrupt bits unconditionally

2020-12-21 Thread Joel Stanley
On Tue, 15 Dec 2020 at 02:46, Zev Weiss wrote: > > Instead of testing and conditionally clearing them one by one, we can > instead just unconditionally clear them all at once. > > Signed-off-by: Zev Weiss I had a poke at the assembly and it looks like GCC is clearing the bits unconditionally