Re: [PATCH 1/12] mips: PMC MSP71xx core platform

[Marc St-Jean <Marc_St-Jean@pmc-sierra.com>]

Yoichi Yuasa wrote:
> Hi,
> 
> On Thu, 28 Jun 2007 19:05:30 -0600
> Marc St-Jean <stjeanma@pmc-sierra.com> wrote:
> 
>  > diff --git a/arch/mips/pmc-sierra/msp71xx/msp_hwbutton.c 
> b/arch/mips/pmc-sierra/msp71xx/msp_hwbutton.c
>  > new file mode 100644
>  > index 0000000..6fa8572
>  > --- /dev/null
>  > +++ b/arch/mips/pmc-sierra/msp71xx/msp_hwbutton.c
>  > @@ -0,0 +1,179 @@
>  > +/*
>  > + * Sets up interrupt handlers for various hardware switches which are
>  > + * connected to interrupt lines.
>  > + *
>  > + * Copyright 2005-2207 PMC-Sierra, Inc.

Hi Yoichi,
I'm just making sure future updates are accounted for! ;)

>  > +
>  > +#ifdef CONFIG_PMC_MSP7120_GW
> 
> You have already set "obj-$(CONFIG_PMC_MSP7120_GW) += msp_hwbutton.o" in 
> Makefile.
> It is not necessary.
> 
>  > +
>  > +static int __init msp_hwbutton_setup(void)
>  > +{
>  > +#ifdef CONFIG_PMC_MSP7120_GW
> 
> same.

The file was actually coded so boards other than the MSP7120_GW can make use of 
it. At some point the "obj-$(CONFIG_PMC_MSP7120_GW) += msp_hwbutton.o" will be 
removed from the Makefile. I'd prefer to leave the board dependent code in the 
source at this time, but of course if this will gate acceptance I will remove it.

Thanks,
Marc