[Skiboot] [PATCH v3 2/4] Move P8 timer code to separate file
Benjamin Herrenschmidt
benh at kernel.crashing.org
Tue Mar 14 17:03:31 AEDT 2017
On Tue, 2017-03-14 at 10:38 +0530, Vasant Hegde wrote:
> @Ben,
> As you suggested, I moved P8 timer code to separate file. But now I'm
> thinking it may be good to have single file (sbe.c) and add all SBE related code
> their (both P8 timer code and new P9 code).
>
> In P8, we have used direct SCOMs for SBE timer. But from P9 onwards we are
> getting MBOX messages for SBE operations. So we can have something like below
>
>
> void sbe_timer_init(void)
> {
> if (proc_gen == proc_gen_p8)
> p8_sbe_timer();
> else
> sbe_timer();
> }
>
> Let me know which one is better.
What would you win by having a single file other than more clutter ?
I prefer keeping them separate as they have strictly nothing in common.
Cheers,
Ben.
More information about the Skiboot
mailing list