Re: [PATCH v2 1/6] ARM: tegra: export apb dma readl/writel
From: Peter De Schrijver
Date: Tue Jan 07 2014 - 07:54:32 EST
On Mon, Jan 06, 2014 at 09:07:26PM +0100, Stephen Warren wrote:
> On 12/24/2013 06:32 AM, Peter De Schrijver wrote:
> > Export APB DMA readl and writel. These are needed because we can't access
> > the fuses directly on Tegra20 without potentially causing a system hang.
>
> This does a lot more than just export the functions; it also changes the
> prototypes. I'm not sure why the prototype change was required/useful.
> This needs to be mentioned in the commit description/subject, and the
> reasoning explained too.
>
> > diff --git a/include/linux/tegra-soc.h b/include/linux/tegra-soc.h
>
> > +#if defined(CONFIG_TEGRA20_APB_DMA)
> > +void tegra_apb_io_init(void);
>
> This seems to be an internal function; why is this one publicly
> prototyped and/or stubbed?
>
Ah. this might be a leftover from a previous iteration... Need to check.
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/