Message ID | 20210406120921.2484986-1-j.neuschaefer@gmx.net |
---|---|
Headers | show |
Series | Initial support for Nuvoton WPCM450 BMC SoC | expand |
On 4/6/21 5:09 AM, Jonathan Neuschäfer wrote: > Add a compatible string for WPCM450, which has essentially the same > watchdog mechanism as NPCM750. > > Signed-off-by: Jonathan Neuschäfer <j.neuschaefer@gmx.net> Acked-by: Guenter Roeck <linux@roeck-us.net> > --- > > v2: > - Added patch description > --- > drivers/watchdog/npcm_wdt.c | 1 + > 1 file changed, 1 insertion(+) > > diff --git a/drivers/watchdog/npcm_wdt.c b/drivers/watchdog/npcm_wdt.c > index 765577f11c8db..28a24caa2627c 100644 > --- a/drivers/watchdog/npcm_wdt.c > +++ b/drivers/watchdog/npcm_wdt.c > @@ -229,6 +229,7 @@ static int npcm_wdt_probe(struct platform_device *pdev) > > #ifdef CONFIG_OF > static const struct of_device_id npcm_wdt_match[] = { > + {.compatible = "nuvoton,wpcm450-wdt"}, > {.compatible = "nuvoton,npcm750-wdt"}, > {}, > }; > -- > 2.30.2 >
On Tue, Apr 6, 2021 at 2:09 PM Jonathan Neuschäfer <j.neuschaefer@gmx.net> wrote: > > This series adds basic support for the Nuvoton WPCM450 BMC SoC. It's an older > SoC but still commonly found on eBay, mostly in Supermicro X9 server boards. > > Third-party documentation is available at: https://github.com/neuschaefer/wpcm450/wiki > > Patches 1-4 add devicetree bindings for the WPCM450 SoC and its various parts. > Patches 5-7 add arch and driver support. Patches 8 and 9 add a devicetree for > the SoC and a board based on it. Patch 10 finally updates the MAINTAINERS file. > > Patch 2 requires "dt-bindings: arm: Convert nuvoton,npcm750 binding to YAML" > (https://lore.kernel.org/lkml/20210320164023.614059-1-j.neuschaefer@gmx.net/) Hi Jonathan, It appears these patches are doing roughly the right thing, and we may still be able to get them into v5.13, but I'm not sure what your plan for maintaining them is. The two options are that you either send your patches to be picked up by Joel, or you send everything directly to soc@kernel.org once it's fully reviewed. I only noticed your series when patch 9/10 made it into the soc@kernel.org patchwork because of the Cc, but none of the other ones did. If you end up with the second option, we can go through what this involves off-list. Regarding the Cc:soc@kernel.org, please add that only for patches that are already reviewed and ready to be picked up, ideally with a cover letter that describes what the plan is for merging. If you need me to review the platform code, use my arnd@arndb.de or arnd@kernel.org addresses. Arnd
On Tue, Apr 06, 2021 at 05:15:01PM +0200, Arnd Bergmann wrote: > On Tue, Apr 6, 2021 at 2:09 PM Jonathan Neuschäfer > <j.neuschaefer@gmx.net> wrote: > > > > This series adds basic support for the Nuvoton WPCM450 BMC SoC. It's an older > > SoC but still commonly found on eBay, mostly in Supermicro X9 server boards. > > > > Third-party documentation is available at: https://github.com/neuschaefer/wpcm450/wiki > > > > Patches 1-4 add devicetree bindings for the WPCM450 SoC and its various parts. > > Patches 5-7 add arch and driver support. Patches 8 and 9 add a devicetree for > > the SoC and a board based on it. Patch 10 finally updates the MAINTAINERS file. > > > > Patch 2 requires "dt-bindings: arm: Convert nuvoton,npcm750 binding to YAML" > > (https://lore.kernel.org/lkml/20210320164023.614059-1-j.neuschaefer@gmx.net/) > > Hi Jonathan, > > It appears these patches are doing roughly the right thing, and we may still > be able to get them into v5.13, but I'm not sure what your plan for maintaining > them is. The two options are that you either send your patches to be picked up > by Joel, or you send everything directly to soc@kernel.org once it's fully > reviewed. The route via Joel sounds alright with me. I've Cc'd him on this version of the series. > I only noticed your series when patch 9/10 made it into the soc@kernel.org > patchwork because of the Cc, but none of the other ones did. > > If you end up with the second option, we can go through what this involves > off-list. > > Regarding the Cc:soc@kernel.org, please add that only for patches that > are already reviewed and ready to be picked up, ideally with a cover letter > that describes what the plan is for merging. If you need me to review the > platform code, use my arnd@arndb.de or arnd@kernel.org addresses. Ah sorry, soc@kernel.org slipped in automatically because of get_maintainers.pl. The platform code[1] is rather simple, essentially just hooking up the nuvoton,wpcm450 compatible string. Thanks, Jonathan [1]: https://lore.kernel.org/lkml/20210406120921.2484986-6-j.neuschaefer@gmx.net/
On Tue, 06 Apr 2021 13:09:17 +0100, Jonathan Neuschäfer <j.neuschaefer@gmx.net> wrote: > > The WPCM450 AIC ("Advanced Interrupt Controller") is the interrupt > controller found in the Nuvoton WPCM450 SoC and other Winbond/Nuvoton > SoCs. > > The list of registers if based on the AMI vendor kernel and the > Nuvoton W90N745 datasheet. > > Although the hardware supports other interrupt modes, the driver only > supports high-level interrupts at the moment, because other modes could > not be tested so far. > > Signed-off-by: Jonathan Neuschäfer <j.neuschaefer@gmx.net> > --- > > v2: > - Rename IRQS macro to AIC_NUM_IRQS > - Fix IRQ range check > - Use linux/printk.h header instead of linux/console.h > - Add AIC_SCR_PRIORITY_MASK constant > - Add missing register descriptions > - Remove superfluous printk about IRQ flow type mismatch > - Use BIT() macro > - Rename _ack function to _eoi for accuracy, and use handle_fasteoi_irq > --- > arch/arm/mach-npcm/Kconfig | 1 + > drivers/irqchip/Kconfig | 6 ++ > drivers/irqchip/Makefile | 1 + > drivers/irqchip/irq-wpcm450-aic.c | 161 ++++++++++++++++++++++++++++++ > 4 files changed, 169 insertions(+) > create mode 100644 drivers/irqchip/irq-wpcm450-aic.c > > diff --git a/arch/arm/mach-npcm/Kconfig b/arch/arm/mach-npcm/Kconfig > index 658c8efb4ca14..a71cf1d189ae5 100644 > --- a/arch/arm/mach-npcm/Kconfig > +++ b/arch/arm/mach-npcm/Kconfig > @@ -10,6 +10,7 @@ config ARCH_WPCM450 > bool "Support for WPCM450 BMC (Hermon)" > depends on ARCH_MULTI_V5 > select CPU_ARM926T > + select WPCM450_AIC > select NPCM7XX_TIMER > help > General support for WPCM450 BMC (Hermon). I can't take this patch with this particular hunk, as I don't have this file in my tree. I can either drop this line, or delay the merging of this patch to a later point in time. The driver otherwise looks ready. Thanks, M.
On Wed, Apr 07, 2021 at 12:11:21PM +0100, Marc Zyngier wrote: > On Tue, 06 Apr 2021 13:09:17 +0100, > Jonathan Neuschäfer <j.neuschaefer@gmx.net> wrote: ... > > diff --git a/arch/arm/mach-npcm/Kconfig b/arch/arm/mach-npcm/Kconfig > > index 658c8efb4ca14..a71cf1d189ae5 100644 > > --- a/arch/arm/mach-npcm/Kconfig > > +++ b/arch/arm/mach-npcm/Kconfig > > @@ -10,6 +10,7 @@ config ARCH_WPCM450 > > bool "Support for WPCM450 BMC (Hermon)" > > depends on ARCH_MULTI_V5 > > select CPU_ARM926T > > + select WPCM450_AIC > > select NPCM7XX_TIMER > > help > > General support for WPCM450 BMC (Hermon). > > I can't take this patch with this particular hunk, as I don't have > this file in my tree. I can either drop this line, or delay the > merging of this patch to a later point in time. Hmm, indeed. Please apply the patch without this hunk. I'll reintroduce it in another patch. > > The driver otherwise looks ready. Good. Thanks, Jonathan
On Tue, 6 Apr 2021 14:09:11 +0200, Jonathan Neuschäfer wrote: > This series adds basic support for the Nuvoton WPCM450 BMC SoC. It's an older > SoC but still commonly found on eBay, mostly in Supermicro X9 server boards. > > Third-party documentation is available at: https://github.com/neuschaefer/wpcm450/wiki > > Patches 1-4 add devicetree bindings for the WPCM450 SoC and its various parts. > Patches 5-7 add arch and driver support. Patches 8 and 9 add a devicetree for > the SoC and a board based on it. Patch 10 finally updates the MAINTAINERS file. > > [...] Applied to irq/irqchip-next, thanks! [03/10] dt-bindings: interrupt-controller: Add nuvoton, wpcm450-aic commit: 7c18715546203a09f859dac2fe3ea8aceec5f235 [06/10] irqchip: Add driver for WPCM450 interrupt controller commit: fead4dd496631707549f414b4059afb86ea8fb80 Cheers, M.
On Tue, 6 Apr 2021 at 21:59, Jonathan Neuschäfer <j.neuschaefer@gmx.net> wrote: > > On Tue, Apr 06, 2021 at 05:15:01PM +0200, Arnd Bergmann wrote: > > On Tue, Apr 6, 2021 at 2:09 PM Jonathan Neuschäfer > > <j.neuschaefer@gmx.net> wrote: > > > > > > This series adds basic support for the Nuvoton WPCM450 BMC SoC. It's an older > > > SoC but still commonly found on eBay, mostly in Supermicro X9 server boards. > > > > > > Third-party documentation is available at: https://github.com/neuschaefer/wpcm450/wiki > > > > > > Patches 1-4 add devicetree bindings for the WPCM450 SoC and its various parts. > > > Patches 5-7 add arch and driver support. Patches 8 and 9 add a devicetree for > > > the SoC and a board based on it. Patch 10 finally updates the MAINTAINERS file. > > > > > > Patch 2 requires "dt-bindings: arm: Convert nuvoton,npcm750 binding to YAML" > > > (https://lore.kernel.org/lkml/20210320164023.614059-1-j.neuschaefer@gmx.net/) > > > > Hi Jonathan, > > > > It appears these patches are doing roughly the right thing, and we may still > > be able to get them into v5.13, but I'm not sure what your plan for maintaining > > them is. The two options are that you either send your patches to be picked up > > by Joel, or you send everything directly to soc@kernel.org once it's fully > > reviewed. > > The route via Joel sounds alright with me. I've Cc'd him on this version > of the series. I've had a look at the series and it looks good to me: Reviewed-by: Joel Stanley <joel@jms.id.au> Nice work Jonathan. I'll put this in it's own branch along with the bindings change it depends on and send a pull request to Arnd for v5.13. Cheers, Joel
On Fri, Apr 09, 2021 at 04:37:34AM +0000, Joel Stanley wrote: > On Tue, 6 Apr 2021 at 21:59, Jonathan Neuschäfer <j.neuschaefer@gmx.net> wrote: > > > > On Tue, Apr 06, 2021 at 05:15:01PM +0200, Arnd Bergmann wrote: > > > On Tue, Apr 6, 2021 at 2:09 PM Jonathan Neuschäfer > > > <j.neuschaefer@gmx.net> wrote: > > > > > > > > This series adds basic support for the Nuvoton WPCM450 BMC SoC. It's an older > > > > SoC but still commonly found on eBay, mostly in Supermicro X9 server boards. > > > > > > > > Third-party documentation is available at: https://github.com/neuschaefer/wpcm450/wiki > > > > > > > > Patches 1-4 add devicetree bindings for the WPCM450 SoC and its various parts. > > > > Patches 5-7 add arch and driver support. Patches 8 and 9 add a devicetree for > > > > the SoC and a board based on it. Patch 10 finally updates the MAINTAINERS file. > > > > > > > > Patch 2 requires "dt-bindings: arm: Convert nuvoton,npcm750 binding to YAML" > > > > (https://lore.kernel.org/lkml/20210320164023.614059-1-j.neuschaefer@gmx.net/) > > > > > > Hi Jonathan, > > > > > > It appears these patches are doing roughly the right thing, and we may still > > > be able to get them into v5.13, but I'm not sure what your plan for maintaining > > > them is. The two options are that you either send your patches to be picked up > > > by Joel, or you send everything directly to soc@kernel.org once it's fully > > > reviewed. > > > > The route via Joel sounds alright with me. I've Cc'd him on this version > > of the series. > > I've had a look at the series and it looks good to me: > > Reviewed-by: Joel Stanley <joel@jms.id.au> > > Nice work Jonathan. > > I'll put this in it's own branch along with the bindings change it > depends on and send a pull request to Arnd for v5.13. Thanks a bunch! A few patches are going through other branches (IRQ bindings+driver; watchdog bindings+driver probably, I guess). That leaves the following patches to go into your branch, AFAIUI: [PATCH v2 01/10] dt-bindings: vendor-prefixes: Add Supermicro [PATCH v2 02/10] dt-bindings: arm: npcm: Add nuvoton,wpcm450 compatible string [PATCH v2 05/10] ARM: npcm: Introduce Nuvoton WPCM450 SoC [PATCH v2 08/10] ARM: dts: Add devicetree for Nuvoton WPCM450 BMC chip [PATCH v2 09/10] ARM: dts: Add devicetree for Supermicro X9SCi-LN4F based on WPCM450 [PATCH v2 10/10] MAINTAINERS: Add entry for Nuvoton WPCM450 Jonathan
On Fri, Apr 9, 2021 at 9:58 AM Jonathan Neuschäfer <j.neuschaefer@gmx.net> wrote: > On Fri, Apr 09, 2021 at 04:37:34AM +0000, Joel Stanley wrote: > > On Tue, 6 Apr 2021 at 21:59, Jonathan Neuschäfer <j.neuschaefer@gmx.net> wrote: > > I've had a look at the series and it looks good to me: > > > > Reviewed-by: Joel Stanley <joel@jms.id.au> > > > > Nice work Jonathan. > > > > I'll put this in it's own branch along with the bindings change it > > depends on and send a pull request to Arnd for v5.13. > > Thanks a bunch! > > A few patches are going through other branches (IRQ bindings+driver; > watchdog bindings+driver probably, I guess). That leaves the following > patches to go into your branch, AFAIUI: > > [PATCH v2 01/10] dt-bindings: vendor-prefixes: Add Supermicro > [PATCH v2 02/10] dt-bindings: arm: npcm: Add nuvoton,wpcm450 compatible string > [PATCH v2 05/10] ARM: npcm: Introduce Nuvoton WPCM450 SoC > [PATCH v2 08/10] ARM: dts: Add devicetree for Nuvoton WPCM450 BMC chip > [PATCH v2 09/10] ARM: dts: Add devicetree for Supermicro X9SCi-LN4F based on WPCM450 > [PATCH v2 10/10] MAINTAINERS: Add entry for Nuvoton WPCM450 Actually for an initial merge, we sometimes just put all the patches into one branch in the soc tree to avoid conflicts. Unfortunately we already have a (trivial) conflict now anyway since I merged the irqchip driver for the Apple M1 SoC through the soc tree but the wpcm irqchip through the irqchip tree. You did nothing wrong here, this would have just been a way to make the initial merge a bit easier, and have a tree that is more easily bisectible when one branch in the merge window contains all the code that is needed for booting. Arnd