Hi Sergei, On Thu, Jan 12, 2012 at 17:30:15, Karicheri, Muralidharan wrote: > Sekhar & Sergei, > > This will affect the new SoCs that have capabilities to power up different > domains. Why do we need this enforcement? We are working on a DaVinci variant > SoC that can do power up multiple domains (That is why I had a patch earlier > to support > Multiple PD which is already part of the tree). So I don't want this patch to > go through unless I see a reason for pushing this.
I agree with Murali here. The cpu_is_ checking in the patch is going to quickly turn ugly as new SoCs with power domain control are added. On SoCs where domain power up procedure is not documented, even bootloader cannot be responsible for powering up the domain. The domain will be powered up in hardware itself; so the power domain status cannot show the domain as powered off. A better change in this area would be to add a timeout to the infinite loop reading epcpr. This will enable proper error reporting. Thanks, Sekhar _______________________________________________ Davinci-linux-open-source mailing list Davinci-linux-open-source@linux.davincidsp.com http://linux.davincidsp.com/mailman/listinfo/davinci-linux-open-source