[Skiboot] [PATCH] Remove POWER7 and POWER7+ support

Oliver oohall at gmail.com
Mon Jun 3 17:18:22 AEST 2019


On Mon, Jun 3, 2019 at 3:39 PM Stewart Smith <stewart at linux.ibm.com> wrote:
>
> It's been a good long while since either OPAL POWER7 user touched a
> machine, and even longer since they'd have been okay using an old
> version rather than tracking master.
>
> There's also been no testing of OPAL on POWER7 systems for an awfully
> long time, so it's pretty safe to assume that it's very much bitrotted.
>
> It also saves a whole 14kb of xz compressed payload space.
>
> Signed-off-by: Stewart Smith <stewart at linux.ibm.com>

Cool, I was planning on doing the same.

> ---
>  asm/head.S                                    |   58 -
>  core/affinity.c                               |    4 +-
>  core/chip.c                                   |    6 +-
>  core/cpu.c                                    |   18 +-
>  core/init.c                                   |    7 +-
>  core/interrupts.c                             |   10 -
>  core/test/run-cpufeatures.c                   |    2 +-
>  core/test/run-flash-firmware-versions.c       |    2 +-
>  .../opal-pci-get-hub-diag-data-50.rst         |   55 +-
>  external/xscom-utils/adu_scoms.py             |    6 +-
>  external/xscom-utils/getscom.c                |    7 -
>  hdata/cpu-common.c                            |   16 -
>  hdata/iohub.c                                 |   94 -
>  hdata/pcia.c                                  |    6 +-
>  hdata/spira.c                                 |  114 +-
>  hdata/spira.h                                 |    6 -
>  hdata/test/hdata_to_dt.c                      |   13 +-
>  hw/Makefile.inc                               |    3 +-

>  hw/cec.c                                      |    2 +-
As far as I can tell everything in here is only used by P7 code.

>  hw/chiptod.c                                  |    3 -
>  hw/dts.c                                      |   64 -
>  hw/fsp/fsp.c                                  |    1 -
>  hw/gx.c                                       |  158 -
>  hw/nx-842.c                                   |    6 +-
>  hw/nx-crypto.c                                |   16 +-
>  hw/nx-rng.c                                   |    6 +-
>  hw/p7ioc-inits.c                              | 1096 ------
>  hw/p7ioc-phb.c                                | 3242 -----------------
>  hw/p7ioc.c                                    |  688 ----
>  hw/psi.c                                      |  105 +-
>  hw/slw.c                                      |    6 +-
>  hw/xscom.c                                    |   10 +-
>  include/chip.h                                |   24 +-
>  include/gx.h                                  |   53 -
>  include/interrupts.h                          |  122 -
>  include/mem-map.h                             |    5 +-
>  include/nx.h                                  |   44 +-
>  include/opal-api.h                            |  110 -
>  include/p7ioc-regs.h                          |  444 ---
>  include/p7ioc.h                               |  367 --
>  include/processor.h                           |   12 +-
>  include/psi.h                                 |   14 +-
>  include/skiboot.h                             |    2 -

>  platforms/ibm-fsp/apollo-pci.c                |    1 -

All of apollo-pci.c and apollo.c can go. They're the platform files
for the P7 FSP systems.

I'm not going to pick through 8000 lines of deleted code, so:

Enthusiasticly-Acked-by: Oliver O'Halloran <oohall at gmail.com>


More information about the Skiboot mailing list