Re: [PATCH] [PATCH] ARM: sunxi: doc: Add sun7i (A20) interrupt table

From: Maxime Ripard
Date: Fri Sep 20 2013 - 12:07:36 EST


Hi Mark,

On Fri, Sep 20, 2013 at 01:39:56PM +0100, Mark Rutland wrote:
> On Fri, Sep 20, 2013 at 12:12:40PM +0100, Oliver Schinagl wrote:
> > From: Oliver Schinagl <oliver@xxxxxxxxxxx>
> >
> > This patch adds some documentation about the Allwinner sun7i (A20) using
> > the GIC.
> >
> > Signed-off-by: Oliver Schinagl <oliver@xxxxxxxxxxx>
>
> NAK.
>
> This information is not a binding, and it's not a property of the
> interrupt controller. It's the integration of a full SoC, which should
> be described full in a dtsi where it can actually be used.
>
> The only other SoCs that have this are other sunxi variants, and I see
> no reason to have them here either.

This was mostly here because we had a very sparse documentation for
these SoCs when we first submitted the patches, and these informations
were not easy to find out. So a misplaced documentation is still better
than no documentation.

We do have the same documentation for the clocks here, mostly for the
same reason.

Maxime

--
Maxime Ripard, Free Electrons
Embedded Linux, Kernel and Android engineering
http://free-electrons.com

Attachment: signature.asc
Description: Digital signature