[Skiboot] [PATCH v3 2/4] Move P8 timer code to separate file
hegdevasant at linux.vnet.ibm.com
Tue Mar 14 17:16:20 AEDT 2017
On 03/14/2017 11:33 AM, Benjamin Herrenschmidt wrote:
> On Tue, 2017-03-14 at 10:38 +0530, Vasant Hegde wrote:
>> 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)
>> Let me know which one is better.
> What would you win by having a single file other than more clutter ?
Nothing much except number of files ;-)
> I prefer keeping them separate as they have strictly nothing in common.
Agree. There is no common code. I will keep it as separate and changes p9
function to something like
More information about the Skiboot