On Tue, 12 Jun 2007, Domen Puncer wrote:

> Index: work-powerpc.git/include/asm-powerpc/mpc5200_rtc.h
> ===================================================================
> --- /dev/null
> +++ work-powerpc.git/include/asm-powerpc/mpc5200_rtc.h
> @@ -0,0 +1,49 @@
> +#ifndef _MPC5200_RTC_H_
> +#define _MPC5200_RTC_H_
> +
> +/* RTC */
> +struct mpc52xx_rtc {

Is it really worth putting a header that's only ever going to be used by 1 
file under drivers/rtc under include/asm-*/? If this struct is really only 
for rtc-mpc52xx, just define it .c?

Thanks
Guennadi
---
Guennadi Liakhovetski
_______________________________________________
Linuxppc-embedded mailing list
[email protected]
https://ozlabs.org/mailman/listinfo/linuxppc-embedded

Reply via email to