[PATCH 7/19] powerpc: interfaces to the hypervisor of Celleb

Anton Blanchard anton at samba.org
Sat Jan 13 23:46:38 EST 2007


Hi,

Just a few suggestions.

> +#define	HCALL_MEDIUM	or 	2,2,2

Doesnt look like this is used.

> +#define	HVSC	.long	0x44000022

Could you pick this up from the define in include/asm-powerpc instead?

> +#define	LOAD_CONST64(r, const)					\
> +	lis	r, ((const)>>48)&0xFFFF;			\
> +	ori	r, r, ((const)>>32)&0xFFFF;			\
> +	rldicr	r, r, 32, 31;					\
> +	oris	r, r, ((const)>>16)&0xFFFF;			\
> +	ori	r, r, ((const)>>0)&0xFFFF

It doesnt look like this is used either.

Anton



More information about the Linuxppc-dev mailing list