[PATCH] powerpc: Fixed duplicate copying in the early boot.

Segher Boessenkool segher at kernel.crashing.org
Tue Jun 18 02:13:36 AEST 2024


Hi!

On Mon, Jun 17, 2024 at 10:35:09AM +0800, Jinglin Wen wrote:
> +	cmplwi	cr0,r4,0	/* runtime base addr is zero */

Just write
   cmpwi r4,0

cr0 is the default, also implicit in many other instructions, please
don't clutter the source code.  All the extra stuff makes you miss the
things that do matter!

The "l" is unnecessary, you only care about equality here after all.

Should it not be cmpdi here, though?


Segher


More information about the Linuxppc-dev mailing list