[PATCH 5/5] irqchip: armada-370-xp: document the overall driver logic

Jason Cooper
2015-10-20T14:07:34Z

On Tue, Oct 20, 2015 at 04:00:21PM +0200, Thomas Petazzoni wrote:
> Hello,
>
> On Tue, 20 Oct 2015 15:23:55 +0200, Thomas Petazzoni wrote:
>
> > +/*
> > + * Overall diagram of the Armada XP interrupt controller:
> > + *
> > + * To CPU 0 To CPU 1
> > + *
> > + * /\ /\
> > + * || ||
> > + * +---------------+ +---------------+
> > + * | | | |
>
> Things are a bit off here.
>
> > + * | per-CPU | | per-CPU |
> > + * | mask/unmask | | mask/unmask |
> > + * | CPU0 | | CPU1 |
> > + * | | | |
> > + * +---------------+ +---------------+
> > + * /\ /\
>
> and here. So I can fix that up in the next version.
>
> If we're doing ASCII art, let's do it properly.

Agreed. It's not pressing. You can resubmit these last two separately
and we'll cue them up for 4.4.

thx,

Jason.