Dear Prabhakar Kushwaha,
In message 4F56DEB0.6060500@freescale.com you wrote:
- _mas0 = MAS0_TLBSEL(1) |
MAS0_ESEL(CONFIG_DEBUGGER_TEMP_TLB);You are using an undocumented CONFIG_ option here.
This CONFIG_ is defined as part of documentation patch sent in this series
CONFIG_ options must be documented in the README.
+#if defined(CONFIG_E500_V1_V2)&& !defined(CONFIG_SYS_RAMBOOT)
What if no such debug support is needed, and code size hurts?
sure it will. I agree with you. But this piece of code overcome debug restriction of e500 and e500v2 cores and I think fixing restriction is a good way to go.
What do we do to disable this code if no debug support is needed, and the code size is considered critical?
+#define CONFIG_DEBUGGER_TEMP_TLB 3
Undocumented!
This CONFIG_ is defined as part of documentation patch sent in this series
This is not sufficient, see above.
Best regards,
Wolfgang Denk