[Skiboot] [PATCH v9 0/2] Add SBE timer support

Stewart Smith stewart at linux.ibm.com
Tue May 1 13:25:28 AEST 2018


Vasant Hegde <hegdevasant at linux.vnet.ibm.com> writes:
> This patchset adds SBE timer support.
>
> Patch 1 moves P8 timer code to separate file.
> Patch 2 adds P9 timer support.
>
> This patchset depends on SBE base driver patchset [1].
>
> [1] https://lists.ozlabs.org/pipermail/skiboot/2018-April/011142.html
>
> Changes in v9:
>   - Reworked interrupt handler code to handle one bit at a time.
>
> Changes in v8:
>   - Fixed poller code
>   - Fixed timer schedule re-arming issue
>
> Changes in v7:
>   - Addressed Ben's review comments.
>   - Major changes:
>     converted timer message from sync to async
>
> Changes in v6:
>   - Split base driver and timer support to separate patchset
>   - Reworked timer code based on Ben's review comment
>   - Changed timer granularity from ms to us.
>
> Changes in v5:
>   - Rebased on top of upstream master
>   - Synced to latest SBE-Host spec
>   - Fixed sbe_sync_msg()
>   - Removed continous timer call
>
> Changes in v4:
>   - Used PNOR info idata pointer to find primary chip
>   - Fixed few return value issues
>
> Testing:
>   - Tested this patchset on P9 machine and it working fine.
>
>
> Vasant Hegde (2):
>   Move P8 timer code to separate file
>   SBE: Add timer support
>
>  core/interrupts.c     |   6 +-
>  core/test/run-timer.c |  10 ++-
>  core/timer.c          |  22 ++++--
>  hw/Makefile.inc       |   2 +-
>  hw/sbe-p8.c           | 203 ++++++++++++++++++++++++++++++++++++++++++++++++++
>  hw/sbe-p9.c           | 186 ++++++++++++++++++++++++++++++++++++++++++++-
>  hw/slw.c              | 185 +--------------------------------------------
>  include/sbe-p8.h      |  28 +++++++
>  include/sbe-p9.h      |   6 ++
>  include/skiboot.h     |   6 --
>  10 files changed, 454 insertions(+), 200 deletions(-)
>  create mode 100644 hw/sbe-p8.c
>  create mode 100644 include/sbe-p8.h

Thanks! series merged to master as of
d7e7bdcd4accff9fb75579b43be72bc3c201d161.

-- 
Stewart Smith
OPAL Architect, IBM.



More information about the Skiboot mailing list