[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <20151020140734.GF3953@io.lakedaemon.net>
Date: Tue, 20 Oct 2015 14:07:34 +0000
From: Jason Cooper <jason@...edaemon.net>
To: Thomas Petazzoni <thomas.petazzoni@...e-electrons.com>
Cc: Thomas Gleixner <tglx@...utronix.de>,
Marc Zyngier <marc.zyngier@....com>,
linux-kernel@...r.kernel.org, linux-arm-kernel@...ts.infradead.org,
Tawfik Bayouk <tawfik@...vell.com>,
Nadav Haklai <nadavh@...vell.com>,
Lior Amsalem <alior@...vell.com>, Andrew Lunn <andrew@...n.ch>,
Sebastian Hesselbarth <sebastian.hesselbarth@...il.com>,
Gregory Clement <gregory.clement@...e-electrons.com>
Subject: Re: [PATCH 5/5] irqchip: armada-370-xp: document the overall driver
logic
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.
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
Powered by blists - more mailing lists