Re: [PATCHv5 1/3] ARM: tegra: Add Tegra AHB driver

From: Arnd Bergmann
Date: Mon May 07 2012 - 04:59:46 EST


On Monday 07 May 2012, Hiroshi Doyu wrote:
> > On Mon, May 07, 2012 at 09:43:45AM +0300, Hiroshi DOYU wrote:
> > > diff --git a/drivers/Makefile b/drivers/Makefile
> > > index 5870322..d97e2e2 100644
> > > --- a/drivers/Makefile
> > > +++ b/drivers/Makefile
> > > @@ -18,7 +18,7 @@ obj-$(CONFIG_SFI) += sfi/
> > > # PnP must come after ACPI since it will eventually need to check if acpi
> > > # was used and do nothing if so
> > > obj-$(CONFIG_PNP) += pnp/
> > > -obj-$(CONFIG_ARM_AMBA) += amba/
> > > +obj-$(CONFIG_ARM) += amba/
> >
> > NAK. This makes amba only buildable for ARM which is not the intention.
> > Other architectures do have Primecells.
>
> What about the following?
>
> +obj-y += amba/

Looks ok to me, but which problem are you trying to solve anyway? Tegra
always enables ARM_AMBA, so AFAICT there is no difference between this
and the original code.

Arnd
--
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/