Hello.
On 09-06-2011 06:36 PM, Sergei Shtylyov wrote:
Signed-off-by: Murali Karicheri<[email protected]>
[...]
+ pdctl = __raw_readl(psc_base + PDCTL + 4 * domain);
+ pdctl |= 0x1;
+ __raw_writel(pdctl, psc_base + PDCTL + 4 * domain);
ptcmd = 1<< domain;
__raw_writel(ptcmd, psc_base + PTCMD);
@@ -92,9 +92,10 @@ void davinci_psc_config(unsigned int domain, unsigned
int ctlr,
epcpr = __raw_readl(psc_base + EPCPR);
} while ((((epcpr>> domain)& 1) == 0));
Does this work for ARM domain, i.e. does it require an external power
supply too?
Not sure. Will someone actively using the DaVinci platforms answer this
question?
I may try to verify on my DM6446 if I'll find the time...
Hmm, DA8x0 documentation doesn't even mention the EPCPR register, so I'm not
sure how this loop is supposed to work on those chips... unless this path is
never actually executed -- which I'm trying to check now...
Have verified that the patch was never being executed indeed. If I make it
executed, the board happily locks up. So looks like this code has even more
issues...
Seeing the same on DM6446 BUT the test is wrong -- of course, I just can't
power down domain 0 and then power it up to see if the domain 0's bit gets set
in the EPCPR register. :-)
Well, given that domain 0 is always enabled, my above question doesn't make
much sense anyway, so your patch is fine with me. :-)
WBR, Sergei
_______________________________________________
Davinci-linux-open-source mailing list
[email protected]
http://linux.davincidsp.com/mailman/listinfo/davinci-linux-open-source
