Re: CVS commit: src/sys/arch/sh3/sh3

["Valeriy E. Ushakov" <uwe%stderr.spb.ru@localhost>]

On Wed, Dec 12, 2012 at 13:34:49 +0000, Izumi Tsutsui wrote:

> Module Name:  src
> Committed By: tsutsui
> Date:         Wed Dec 12 13:34:49 UTC 2012
> 
> Modified Files:
>       src/sys/arch/sh3/sh3: cache_sh4.c
> 
> Log Message:
> Fix fallouts in rev 1.19:
>  
> http://www.nerv.org/~ryo/netbsd/netbsd/?q=id:20080316T191753Z.1654448ada03ce3c4668f3fe472796d0b771e147
>  - revert RUN_P1 -> PAD_P1_SWITCH changes where RUN_P1 is
>    actually required  (all icache CCIA ops still need RUN_P2)

Why is it required?  The return address is in P1, so the normal "ret"
will jump back to P1.  Or is there some inlining I missed?


>  - sh4_dcache_wbinv_all() and sh4_dcache_wbinv_range_index()
>    (which manipulate CCDA arrays) are no longer have RUN_P2 so
>    we can't call them directly from sh4_icache_sync_all() and
>    sh4_icache_sync_range_index() funcitons;  use function pointers
>    (which have appropriate addresses) instead for 7750 and 7750S
> 
> 
> To generate a diff of this commit:
> cvs rdiff -u -r1.20 -r1.21 src/sys/arch/sh3/sh3/cache_sh4.c

-uwe