[<prev] [next>] [<thread-prev] [day] [month] [year] [list]
Message-ID: <AANLkTinoMmz05KweBJQJ4Tr2cRNTp_SY6NOuX1GRzEcK@mail.gmail.com>
Date: Fri, 28 May 2010 03:11:58 +0200
From: Linus Walleij <linus.ml.walleij@...il.com>
To: Dan Williams <dan.j.williams@...el.com>
Cc: Russell King <rmk@....linux.org.uk>,
Linus Walleij <linus.walleij@...ricsson.com>,
linux-kernel@...r.kernel.org, STEricsson_nomadik_linux@...t.st.com
Subject: Re: [PATCH] DMAENGINE: DMA40 U8500 platform configuration
2010/5/28 Dan Williams <dan.j.williams@...el.com>:
> On Thu, May 27, 2010 at 3:12 PM, Russell King <rmk@....linux.org.uk> wrote:
>> On Thu, May 27, 2010 at 02:59:51PM -0700, Dan Williams wrote:
>>> On Thu, May 27, 2010 at 12:37 PM, Russell King <rmk@....linux.org.uk> wrote:
>>> > On Wed, May 26, 2010 at 09:17:42AM -0700, Dan Williams wrote:
>>> >> Requesting acked-by for:
>>> >>
>>> >> http://marc.info/?l=linux-kernel&m=127434177015990&w=2
>>> >
>>> > Well...
>>> >
>>> > +const static dma_addr_t dma40_tx_map[STEDMA40_NR_DEV] = {
>>> > +const static dma_addr_t dma40_rx_map[STEDMA40_NR_DEV] = {
>>>
>>> Yeah, no need to initialize static data to 0, I'll fix that up before
>>> submitting.
>>
>> static const
>>
>> not
>>
>> const static
>>
>
> Ah thanks, I was going to check into that and got distracted by the
> zero initializers...
>
> Also fixed up.
Thanks for fixing this Dan, appreciated!
Yours,
Linus Walleij
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
Powered by blists - more mailing lists