Subject: Re: CVS commit: src/sys/arch/powerpc
To: None <shige@netbsd.org>
From: Izumi Tsutsui <tsutsui@ceres.dti.ne.jp>
List: source-changes
Date: 01/22/2005 20:36:37
In article <20050118171125.D9D882DA1D@cvs.netbsd.org>
shige@netbsd.org wrote:

> Module Name:	src
> Committed By:	shige
> Date:		Tue Jan 18 17:11:25 UTC 2005
> 
> Modified Files:
> 	src/sys/arch/powerpc/ibm4xx: ibm40x_machdep.c
> 	src/sys/arch/powerpc/include/ibm4xx: cpu.h
> Added Files:
> 	src/sys/arch/powerpc/ibm4xx: ibm4xx_machdep.c
> 
> Log Message:
> Arrange some machine-dependent code.
>   - ibm40x_machdep.c: ibm40x specific
> 	. ibm40x_memsize_init
> 	. mem_regions
> 	. other functions are moved to machdep.c or ibm4xx_machdep.c.
>   - ibm4xx_machdep.c: ibm4xx specific
> 	. ibm4xx_init (moved from ibm40x_machdep.c)
> 	. ibm4xx_install_extint (moved from ibm40x_machdep.c)
> 	. ibm4xx_cpu_startup (moved from ibm40x_machdep.c:ibm4xx_startup)
> 	. ibm4xx_dumpsys

- Why do we have both powerpc/ibm4xx/ibm4xx_machdep.c and
  powerpc/ibm4xx/machdep.c?
- IMO consinit() should be in port-specific machdep.c rather than
  common ibm4xx/machdep.c, shouldn't it?
---
Izumi Tsutsui
tsutsui@ceres.dti.ne.jp