linux-kernel - Re: [PATCH v5 0/6] mips: Add DT bindings for MIPS CDMM and MIPS GIC

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives
Hash Suite: Windows password security audit tool. GUI, reports in PDF.
[<prev] [next>] [<thread-prev] [day] [month] [year] [list]
Message-ID: <20200721083008.GA9399@alpha.franken.de>
Date:   Tue, 21 Jul 2020 10:30:08 +0200
From:   Thomas Bogendoerfer <tsbogend@...ha.franken.de>
To:     Serge Semin <Sergey.Semin@...kalelectronics.ru>
Cc:     Thomas Gleixner <tglx@...utronix.de>,
        "Rafael J. Wysocki" <rjw@...ysocki.net>,
        Daniel Lezcano <daniel.lezcano@...aro.org>,
        Greg Kroah-Hartman <gregkh@...uxfoundation.org>,
        Serge Semin <fancer.lancer@...il.com>,
        Alexey Malahov <Alexey.Malahov@...kalelectronics.ru>,
        Rob Herring <robh+dt@...nel.org>,
        Arnd Bergmann <arnd@...db.de>,
        Jason Cooper <jason@...edaemon.net>,
        Marc Zyngier <maz@...nel.org>, James Hogan <jhogan@...nel.org>,
        linux-mips@...r.kernel.org, devicetree@...r.kernel.org,
        linux-kernel@...r.kernel.org
Subject: Re: [PATCH v5 0/6] mips: Add DT bindings for MIPS CDMM and MIPS GIC

On Tue, Jul 14, 2020 at 03:57:46PM +0300, Serge Semin wrote:
> Daniel, Rafael, Thomas (Gleixner), could you specifically take a look at
> the last patch in this series? If you are ok with that, please explicitly
> ack. We need at least one of your blessing to merge the series in, since
> the code and DT-related patches here have been mostly reviewed. We've
> missed the last merge window. It would be pity to miss the next one...
> 
> Regarding this patchset origin. Recently I've submitted a series of
> patchset's which provided multiple fixes for the MIPS arch subsystem and
> the MIPS GIC and DW APB Timer drivers, which were required for the
> Baikal-T1 SoC correctly working with those drivers. Mostly those patchsets
> have been already merged into the corresponding subsystems, but several
> patches have been left floating since noone really responded for review
> except Rob provided his approval regarding DT bindings. Thus in this
> patchset I've collected all the leftovers so not to loose them in a pale
> of the maintainers email logs.
> 
> The patchset includes the following updates: MIPS CPC and GIC DT bindings
> legacy text-based file are converted to the DT schema (Rob has already
> reviewed them), add MIPS CDMM DT node support to place the CDMM block at
> the platform-specific MMIO range, make sure MIPS CDMM is available for
> MIPS_R5 CPUs.
> 
> Seeing the series concerns the MIPS-related drivers it's better to merge
> it in through the MIPS repository:
> https://git.kernel.org/pub/scm/linux/kernel/git/mips/linux.git/
> 
> This patchset is rebased and tested on the mainline Linux kernel 5.7-rc4:
> base-commit: 0e698dfa2822 ("Linux 5.7-rc4")
> tag: v5.7-rc4
> 
> Suggestion.
> Since Paul isn't looking after the MIPS arch code anymore, Ralf hasn't
> been seen maintaining MIPS for a long time, Thomas is only responsible
> for the next part of it:
> 	F:      Documentation/devicetree/bindings/mips/
> 	F:      Documentation/mips/
> 	F:      arch/mips/
> 	F:      drivers/platform/mips/
> the MIPS-specific drivers like:
> 	F:	drivers/bus/mips_cdmm.c
> 	F:	drivers/irqchip/irq-mips-cpu.c
> 	F:	drivers/irqchip/irq-mips-gic.c
> 	F:	drivers/clocksource/mips-gic-timer.c
> 	F:	drivers/cpuidle/cpuidle-cps.c
> seem to be left for the subsystems maintainers to support. So if you don't
> mind or unless there is a better alternative, I can help with looking
> after them to ease the maintainers review burden and since I'll be working
> on our MIPS-based SoC drivers integrating into the mainline kernel repo
> anyway. Thomas agreed to join in maintaining that drivers.
> 
> Previous patchsets:
> mips: Prepare MIPS-arch code for Baikal-T1 SoC support:
> Link: https://lore.kernel.org/linux-mips/20200306124807.3596F80307C2@mail.baikalelectronics.ru
> Link: https://lore.kernel.org/linux-mips/20200506174238.15385-1-Sergey.Semin@baikalelectronics.ru
> Link: https://lore.kernel.org/linux-mips/20200521140725.29571-1-Sergey.Semin@baikalelectronics.ru
> 
> clocksource: Fix MIPS GIC and DW APB Timer for Baikal-T1 SoC support:
> Link: https://lore.kernel.org/linux-rtc/20200324174325.14213-1-Sergey.Semin@baikalelectronics.ru
> Link: https://lore.kernel.org/linux-rtc/20200506214107.25956-1-Sergey.Semin@baikalelectronics.ru
> Link: https://lore.kernel.org/linux-rtc/20200521005321.12129-1-Sergey.Semin@baikalelectronics.ru
> 
> Changelog prev:
> - Add yaml-based bindings file for MIPS CDMM dt-node.
> - Convert mti,mips-cpc to DT schema.
> - Use a shorter summary describing the bindings modification patches.
> - Rearrange the SoBs with adding Alexey' co-development tag.
> - Lowercase the hex numbers in the dt-bindings.
> 
> Changelog v2:
> - Resend.
> 
> Link: https://lore.kernel.org/linux-mips/20200601122121.15809-1-Sergey.Semin@baikalelectronics.ru
> Changelog v3:
> - Keep F: MAINTAINERS section alphabetically ordered.
> - Add Thomas as the co-maintainer of the MIPS CPU and GIC IRQchip, MIPS
>   GIC timer and MIPS CPS CPUidle drivers.
> 
> Link: https://lore.kernel.org/linux-mips/20200602100921.1155-1-Sergey.Semin@baikalelectronics.ru
> Changelog v4:
> - Resend.
> 
> Link: https://lore.kernel.org/linux-mips/20200617223201.23259-1-Sergey.Semin@baikalelectronics.ru
> Changelog v5:
> - Consider address and size cells being <1> by default for the DT examples.
> 
> Signed-off-by: Serge Semin <Sergey.Semin@...kalelectronics.ru>
> Cc: Alexey Malahov <Alexey.Malahov@...kalelectronics.ru>
> Cc: Rob Herring <robh+dt@...nel.org>
> Cc: Arnd Bergmann <arnd@...db.de>
> Cc: Jason Cooper <jason@...edaemon.net>
> Cc: Marc Zyngier <maz@...nel.org>
> Cc: James Hogan <jhogan@...nel.org>
> Cc: linux-mips@...r.kernel.org
> Cc: devicetree@...r.kernel.org
> Cc: linux-kernel@...r.kernel.org
> 
> Serge Semin (6):
>   dt-bindings: power: Convert mti,mips-cpc to DT schema
>   dt-bindings: interrupt-controller: Convert mti,gic to DT schema
>   dt-bindings: bus: Add MIPS CDMM controller
>   mips: cdmm: Add mti,mips-cdmm dtb node support
>   bus: cdmm: Add MIPS R5 arch support
>   MAINTAINERS: Add maintainers for MIPS core drivers
> 
>  .../bindings/bus/mti,mips-cdmm.yaml           |  35 +++++
>  .../interrupt-controller/mips-gic.txt         |  67 --------
>  .../interrupt-controller/mti,gic.yaml         | 148 ++++++++++++++++++
>  .../bindings/power/mti,mips-cpc.txt           |   8 -
>  .../bindings/power/mti,mips-cpc.yaml          |  35 +++++
>  MAINTAINERS                                   |  11 ++
>  drivers/bus/Kconfig                           |   2 +-
>  drivers/bus/mips_cdmm.c                       |  15 ++
>  8 files changed, 245 insertions(+), 76 deletions(-)
>  create mode 100644 Documentation/devicetree/bindings/bus/mti,mips-cdmm.yaml
>  delete mode 100644 Documentation/devicetree/bindings/interrupt-controller/mips-gic.txt
>  create mode 100644 Documentation/devicetree/bindings/interrupt-controller/mti,gic.yaml
>  delete mode 100644 Documentation/devicetree/bindings/power/mti,mips-cpc.txt
>  create mode 100644 Documentation/devicetree/bindings/power/mti,mips-cpc.yaml

series applied to mips-next.

Thomas.

-- 
Crap can work. Given enough thrust pigs will fly, but it's not necessarily a
good idea.                                                [ RFC1925, 2.3 ]