On Thu, Sep 20, 2012 at 3:10 PM, viresh kumar <[email protected]> wrote: > On Thu, Sep 20, 2012 at 3:05 PM, Andy Shevchenko > <[email protected]> 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 [email protected] More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
