Commit ab3aab29 authored by Christophe Leroy's avatar Christophe Leroy Committed by Michael Ellerman
Browse files

powerpc: Move update_power8_hid0() into its only user 



update_power8_hid0() is used only by powernv platform subcore.c

Move it there.

Signed-off-by: default avatarChristophe Leroy <christophe.leroy@csgroup.eu>
Signed-off-by: default avatarMichael Ellerman <mpe@ellerman.id.au>
Link: https://lore.kernel.org/r/37f41d74faa0c66f90b373e243e8b1ee37a1f6fa.1623219019.git.christophe.leroy@csgroup.eu
parent 77b0bed7

arch/powerpc/include/asm/reg.h

+0 −10
Original line number Diff line number Diff line
@@ -1446,16 +1446,6 @@ extern void scom970_write(unsigned int address, unsigned long value); 
struct pt_regs;



extern void ppc_save_regs(struct pt_regs *regs);



static inline void update_power8_hid0(unsigned long hid0)

{

	/*

	 *  The HID0 update on Power8 should at the very least be

	 *  preceded by a SYNC instruction followed by an ISYNC

	 *  instruction

	 */

	asm volatile("sync; mtspr %0,%1; isync":: "i"(SPRN_HID0), "r"(hid0));

}

#endif /* __ASSEMBLY__ */

#endif /* __KERNEL__ */

#endif /* _ASM_POWERPC_REG_H */

arch/powerpc/platforms/powernv/subcore.c

+10 −0
Original line number Diff line number Diff line
@@ -169,6 +169,16 @@ static void update_hid_in_slw(u64 hid0) 
	}

}



static inline void update_power8_hid0(unsigned long hid0)

{

	/*

	 *  The HID0 update on Power8 should at the very least be

	 *  preceded by a SYNC instruction followed by an ISYNC

	 *  instruction

	 */

	asm volatile("sync; mtspr %0,%1; isync":: "i"(SPRN_HID0), "r"(hid0));

}



static void unsplit_core(void)

{

	u64 hid0, mask;