Hello.

On 09/06/2011 06:17 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...

WBR, Sergei
_______________________________________________
Davinci-linux-open-source mailing list
[email protected]
http://linux.davincidsp.com/mailman/listinfo/davinci-linux-open-source

Reply via email to