[prev in list] [next in list] [prev in thread] [next in thread]
List: linux-mips
Subject: Re: [PATCH 1/12] mips: PMC MSP71xx core platform
From: Marc St-Jean <Marc_St-Jean () pmc-sierra ! com>
Date: 2007-06-29 17:18:09
Message-ID: 46853ED1.6050600 () pmc-sierra ! com
[Download RAW message or body]
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
[prev in list] [next in list] [prev in thread] [next in thread]
Configure |
About |
News |
Add a list |
Sponsored by KoreLogic