[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <CAOh2x==Fud7k6VcxiiS2VBKsbfOJLZBsX=rtdV+QfdyYQEoY=Q@mail.gmail.com>
Date: Fri, 21 Sep 2012 11:34:27 +0530
From: viresh kumar <viresh.kumar@...aro.org>
To: Andy Shevchenko <andriy.shevchenko@...ux.intel.com>,
Shiraz HASHIM <shiraz.hashim@...com>
Cc: Vinod Koul <vinod.koul@...el.com>, spear-devel@...t.st.com,
linux-kernel@...r.kernel.org, Hein Tibosch <hein_tibosch@...oo.es>
Subject: Re: [PATCH 3/7] dw_dmac: get number of channels from hardware if possible
On Thu, Sep 20, 2012 at 3:10 PM, viresh kumar <viresh.kumar@...aro.org> wrote:
> On Thu, Sep 20, 2012 at 3:05 PM, Andy Shevchenko
> <andriy.shevchenko@...ux.intel.com> wrote:
>> On Tue, 2012-09-18 at 12:20 +0530, viresh kumar wrote:
>>> > @@ -1392,23 +1396,32 @@ static int __devinit dw_probe(struct platform_device *pdev)
>>> > + dw_params = dma_raw_readl(regs, DW_PARAMS);
>>>
>>> Is this valid for every SoC implementation. What if this configuration
>>> is not valid
>>> for a particular SoC and it is invalid to access this address? Or this
>>> gives a invalid
>>> value instead of returning 0?
>
>> Actually I didn't get it clearly from the documentation. We have only
>> one test report from Hein until now.
>
> @Shiraz: Can you please verify reading this register from u-boot? This
> is important before we apply this patch to linux-next.
Ok. I have verified from the documentation. This are valid registers for SPEAr.
@Shiraz: No need to check now. :)
--
viresh
--
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