On Wed, Aug 09, 2017 at 09:12:29AM +0200, Cédric Le Goater wrote:
> On 08/09/2017 05:55 AM, David Gibson wrote:
> > On Tue, Aug 08, 2017 at 10:56:13AM +0200, Cédric Le Goater wrote:
> >> xive_poke_esb() is performing a load/read so it is better named as
> >> xive_esb_read().
> >
> > Uh, patch seem
On 08/09/2017 05:55 AM, David Gibson wrote:
> On Tue, Aug 08, 2017 at 10:56:13AM +0200, Cédric Le Goater wrote:
>> xive_poke_esb() is performing a load/read so it is better named as
>> xive_esb_read().
>
> Uh, patch seems to mismatch the comment here, calling it
> xive_peek_esb() instead.
euh yes
On Tue, Aug 08, 2017 at 10:56:13AM +0200, Cédric Le Goater wrote:
> xive_poke_esb() is performing a load/read so it is better named as
> xive_esb_read().
Uh, patch seems to mismatch the comment here, calling it
xive_peek_esb() instead.
Does reading the ESB had a side effect on the hardware? If s
xive_poke_esb() is performing a load/read so it is better named as
xive_esb_read(). Also introduce a XIVE_ESB_LOAD_EOI read when EOI'ing
LSI interrupts.
Signed-off-by: Cédric Le Goater
---
arch/powerpc/sysdev/xive/common.c | 20 ++--
1 file changed, 10 insertions(+), 10 deletions