mbox series

[0/6] MIPS: OCTEON: cleanups, part II

Message ID 20181204201220.12667-1-aaro.koskinen@iki.fi (mailing list archive)
Headers show
Series MIPS: OCTEON: cleanups, part II | expand

Message

Aaro Koskinen Dec. 4, 2018, 8:12 p.m. UTC
Hi,

Continuing OCTEON cleanups:

- Enable all drivers in defconfig to get the full build coverage.

- Some small adjustements to platform code to allow mechanical deletion of
  a huge amount of unneeded union fields.

Boot tested on OCTEON+ (EdgeRouter Lite) and OCTEON 2 (EdgeRouter Pro).

Build tested with cavium_octeon_defconfig.

A.

Aaro Koskinen (6):
  MIPS: OCTEON: enable all OCTEON drivers in defconfig
  MIPS: OCTEON: octeon-usb: use common gpio_bit definition
  MIPS: OCTEON: cvmx_pko_mem_debug8: use oldest forward compatible
    definition
  MIPS: OCTEON: cvmx_mio_fus_dat3: use oldest forward compatible
    definition
  MIPS: OCTEON: cvmx_gmxx_inf_mode: use oldest forward compatible
    definition
  MIPS: OCTEON: delete redundant register definitions

 .../cavium-octeon/executive/cvmx-cmd-queue.c  |    2 +-
 .../cavium-octeon/executive/cvmx-helper.c     |    4 +-
 .../executive/cvmx-interrupt-rsl.c            |    2 +-
 .../cavium-octeon/executive/octeon-model.c    |   12 +-
 arch/mips/cavium-octeon/octeon-usb.c          |    6 +-
 arch/mips/configs/cavium_octeon_defconfig     |    2 +-
 arch/mips/include/asm/octeon/cvmx-agl-defs.h  |  699 ----------
 arch/mips/include/asm/octeon/cvmx-asxx-defs.h |  105 --
 arch/mips/include/asm/octeon/cvmx-dbg-defs.h  |    4 -
 arch/mips/include/asm/octeon/cvmx-dpi-defs.h  |  178 ---
 arch/mips/include/asm/octeon/cvmx-fpa-defs.h  |  247 ----
 arch/mips/include/asm/octeon/cvmx-gmxx-defs.h |  118 --
 arch/mips/include/asm/octeon/cvmx-gpio-defs.h |  116 --
 arch/mips/include/asm/octeon/cvmx-iob-defs.h  |  375 ------
 arch/mips/include/asm/octeon/cvmx-ipd-defs.h  |  538 --------
 arch/mips/include/asm/octeon/cvmx-l2t-defs.h  |    6 -
 arch/mips/include/asm/octeon/cvmx-led-defs.h  |   78 --
 arch/mips/include/asm/octeon/cvmx-lmcx-defs.h |  514 -------
 arch/mips/include/asm/octeon/cvmx-mio-defs.h  | 1197 -----------------
 arch/mips/include/asm/octeon/cvmx-mixx-defs.h |  136 --
 arch/mips/include/asm/octeon/cvmx-npei-defs.h |  295 ----
 arch/mips/include/asm/octeon/cvmx-npi-defs.h  |  235 ----
 arch/mips/include/asm/octeon/cvmx-pci-defs.h  |  392 ------
 arch/mips/include/asm/octeon/cvmx-pcsx-defs.h |  185 ---
 .../mips/include/asm/octeon/cvmx-pcsxx-defs.h |  146 --
 arch/mips/include/asm/octeon/cvmx-pemx-defs.h |  144 --
 .../mips/include/asm/octeon/cvmx-pescx-defs.h |   59 -
 arch/mips/include/asm/octeon/cvmx-pip-defs.h  |  688 ----------
 arch/mips/include/asm/octeon/cvmx-pko-defs.h  |  619 ---------
 arch/mips/include/asm/octeon/cvmx-pko.h       |    2 +-
 arch/mips/include/asm/octeon/cvmx-pow-defs.h  |  317 -----
 arch/mips/include/asm/octeon/cvmx-rnm-defs.h  |   53 -
 arch/mips/include/asm/octeon/cvmx-rst-defs.h  |   28 -
 arch/mips/include/asm/octeon/cvmx-smix-defs.h |   88 --
 arch/mips/include/asm/octeon/cvmx-spxx-defs.h |   62 -
 .../mips/include/asm/octeon/cvmx-sriox-defs.h |  123 --
 arch/mips/include/asm/octeon/cvmx-srxx-defs.h |   22 -
 arch/mips/include/asm/octeon/cvmx-stxx-defs.h |   64 -
 .../mips/include/asm/octeon/cvmx-uctlx-defs.h |   89 --
 39 files changed, 15 insertions(+), 7935 deletions(-)

Comments

Aaro Koskinen Dec. 5, 2018, 10:09 p.m. UTC | #1
Hi,

On Tue, Dec 04, 2018 at 10:12:14PM +0200, Aaro Koskinen wrote:
>   MIPS: OCTEON: delete redundant register definitions
[...]
>  39 files changed, 15 insertions(+), 7935 deletions(-)

Looks like the last patch didn't get through (around 500 KB, maybe hit
some limit). Please disregard this series for now, I'll rework the series
to do changes with smaller patches.

A.
Paul Burton Dec. 5, 2018, 10:20 p.m. UTC | #2
Hi Aaro,

On Thu, Dec 06, 2018 at 12:09:18AM +0200, Aaro Koskinen wrote:
> Hi,
> 
> On Tue, Dec 04, 2018 at 10:12:14PM +0200, Aaro Koskinen wrote:
> >   MIPS: OCTEON: delete redundant register definitions
> [...]
> >  39 files changed, 15 insertions(+), 7935 deletions(-)
> 
> Looks like the last patch didn't get through (around 500 KB, maybe hit
> some limit). Please disregard this series for now, I'll rework the series
> to do changes with smaller patches.

I actually applied this already - patch 6 didn't make it through the
mailing list but I did receive it personally.

Let me know ASAP if you'd prefer that I drop it.

Thanks,
    Paul
Aaro Koskinen Dec. 5, 2018, 10:25 p.m. UTC | #3
Hi,

On Wed, Dec 05, 2018 at 10:20:12PM +0000, Paul Burton wrote:
> On Thu, Dec 06, 2018 at 12:09:18AM +0200, Aaro Koskinen wrote:
> > Hi,
> > 
> > On Tue, Dec 04, 2018 at 10:12:14PM +0200, Aaro Koskinen wrote:
> > >   MIPS: OCTEON: delete redundant register definitions
> > [...]
> > >  39 files changed, 15 insertions(+), 7935 deletions(-)
> > 
> > Looks like the last patch didn't get through (around 500 KB, maybe hit
> > some limit). Please disregard this series for now, I'll rework the series
> > to do changes with smaller patches.
> 
> I actually applied this already - patch 6 didn't make it through the
> mailing list but I did receive it personally.

OK, that's fine then, thanks!

> Let me know ASAP if you'd prefer that I drop it.

No need to drop.

A.
Paul Burton Dec. 5, 2018, 10:39 p.m. UTC | #4
Hello,

Aaro Koskinen wrote:
> Hi,
> 
> Continuing OCTEON cleanups:
> 
> - Enable all drivers in defconfig to get the full build coverage.
> 
> - Some small adjustements to platform code to allow mechanical deletion of
> a huge amount of unneeded union fields.
> 
> Boot tested on OCTEON+ (EdgeRouter Lite) and OCTEON 2 (EdgeRouter Pro).
> 
> Build tested with cavium_octeon_defconfig.
> 
> A.
> 
> Aaro Koskinen (6):
> MIPS: OCTEON: enable all OCTEON drivers in defconfig
> MIPS: OCTEON: octeon-usb: use common gpio_bit definition
> MIPS: OCTEON: cvmx_pko_mem_debug8: use oldest forward compatible
> definition
> MIPS: OCTEON: cvmx_mio_fus_dat3: use oldest forward compatible
> definition
> MIPS: OCTEON: cvmx_gmxx_inf_mode: use oldest forward compatible
> definition
> MIPS: OCTEON: delete redundant register definitions
> 
> .../cavium-octeon/executive/cvmx-cmd-queue.c  |    2 +-
> .../cavium-octeon/executive/cvmx-helper.c     |    4 +-
> .../executive/cvmx-interrupt-rsl.c            |    2 +-
> .../cavium-octeon/executive/octeon-model.c    |   12 +-
> arch/mips/cavium-octeon/octeon-usb.c          |    6 +-
> arch/mips/configs/cavium_octeon_defconfig     |    2 +-
> arch/mips/include/asm/octeon/cvmx-agl-defs.h  |  699 ----------
> arch/mips/include/asm/octeon/cvmx-asxx-defs.h |  105 --
> arch/mips/include/asm/octeon/cvmx-dbg-defs.h  |    4 -
> arch/mips/include/asm/octeon/cvmx-dpi-defs.h  |  178 ---
> arch/mips/include/asm/octeon/cvmx-fpa-defs.h  |  247 ----
> arch/mips/include/asm/octeon/cvmx-gmxx-defs.h |  118 --
> arch/mips/include/asm/octeon/cvmx-gpio-defs.h |  116 --
> arch/mips/include/asm/octeon/cvmx-iob-defs.h  |  375 ------
> arch/mips/include/asm/octeon/cvmx-ipd-defs.h  |  538 --------
> arch/mips/include/asm/octeon/cvmx-l2t-defs.h  |    6 -
> arch/mips/include/asm/octeon/cvmx-led-defs.h  |   78 --
> arch/mips/include/asm/octeon/cvmx-lmcx-defs.h |  514 -------
> arch/mips/include/asm/octeon/cvmx-mio-defs.h  | 1197 -----------------
> arch/mips/include/asm/octeon/cvmx-mixx-defs.h |  136 --
> arch/mips/include/asm/octeon/cvmx-npei-defs.h |  295 ----
> arch/mips/include/asm/octeon/cvmx-npi-defs.h  |  235 ----
> arch/mips/include/asm/octeon/cvmx-pci-defs.h  |  392 ------
> arch/mips/include/asm/octeon/cvmx-pcsx-defs.h |  185 ---
> .../mips/include/asm/octeon/cvmx-pcsxx-defs.h |  146 --
> arch/mips/include/asm/octeon/cvmx-pemx-defs.h |  144 --
> .../mips/include/asm/octeon/cvmx-pescx-defs.h |   59 -
> arch/mips/include/asm/octeon/cvmx-pip-defs.h  |  688 ----------
> arch/mips/include/asm/octeon/cvmx-pko-defs.h  |  619 ---------
> arch/mips/include/asm/octeon/cvmx-pko.h       |    2 +-
> arch/mips/include/asm/octeon/cvmx-pow-defs.h  |  317 -----
> arch/mips/include/asm/octeon/cvmx-rnm-defs.h  |   53 -
> arch/mips/include/asm/octeon/cvmx-rst-defs.h  |   28 -
> arch/mips/include/asm/octeon/cvmx-smix-defs.h |   88 --
> arch/mips/include/asm/octeon/cvmx-spxx-defs.h |   62 -
> .../mips/include/asm/octeon/cvmx-sriox-defs.h |  123 --
> arch/mips/include/asm/octeon/cvmx-srxx-defs.h |   22 -
> arch/mips/include/asm/octeon/cvmx-stxx-defs.h |   64 -
> .../mips/include/asm/octeon/cvmx-uctlx-defs.h |   89 --
> 39 files changed, 15 insertions(+), 7935 deletions(-)

Series applied to mips-next.

Thanks,
    Paul

[ This message was auto-generated; if you believe anything is incorrect
  then please email paul.burton@mips.com to report it. ]