* [PATCH 1/1] arm64: dts: imx8mp-skov: add pinctrl-name 'reset' to fix CHECK_DTB warning @ 2025-12-29 16:51 Frank Li 2026-01-05 11:14 ` Marco Felsch 2026-01-06 21:43 ` Rob Herring 0 siblings, 2 replies; 7+ messages in thread From: Frank Li @ 2025-12-29 16:51 UTC (permalink / raw) To: Rob Herring, Krzysztof Kozlowski, Conor Dooley, Shawn Guo, Sascha Hauer, Pengutronix Kernel Team, Fabio Estevam, open list:OPEN FIRMWARE AND FLATTENED DEVICE TREE BINDINGS, open list:ARM/FREESCALE IMX / MXC ARM ARCHITECTURE, moderated list:ARM/FREESCALE IMX / MXC ARM ARCHITECTURE, open list Cc: imx Add pinctrl-name 'reset' to fix below CHECK_DTB warnings: arch/arm64/boot/dts/freescale/imx8mp-skov-basic.dtb: switch@5f (microchip,ksz9893): pinctrl-names: ['default'] is too short from schema $id: http://devicetree.org/schemas/net/dsa/microchip,ksz.yaml# The commit (e469b87e0fb0d dt-bindings: net: dsa: microchip: Add strap description to set SPI mode" force use two pinctrl-name 'default' and 'reset'. switch@5f doesn't use SPI, so it is safe to use the same pinctrl setting for both. Signed-off-by: Frank Li <Frank.Li@nxp.com> --- arch/arm64/boot/dts/freescale/imx8mp-skov-reva.dtsi | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/arch/arm64/boot/dts/freescale/imx8mp-skov-reva.dtsi b/arch/arm64/boot/dts/freescale/imx8mp-skov-reva.dtsi index 020f20c8ce667..7d2fc5fdd3bd1 100644 --- a/arch/arm64/boot/dts/freescale/imx8mp-skov-reva.dtsi +++ b/arch/arm64/boot/dts/freescale/imx8mp-skov-reva.dtsi @@ -373,8 +373,8 @@ &i2c4 { switch: switch@5f { compatible = "microchip,ksz9893"; - pinctrl-names = "default"; - pinctrl-0 = <&pinctrl_switch>; + pinctrl-names = "default", "reset"; + pinctrl-0 = <&pinctrl_switch>, <&pinctrl_switch>; reset-gpios = <&gpio5 1 GPIO_ACTIVE_LOW>; reg = <0x5f>; -- 2.34.1 ^ permalink raw reply related [flat|nested] 7+ messages in thread
* Re: [PATCH 1/1] arm64: dts: imx8mp-skov: add pinctrl-name 'reset' to fix CHECK_DTB warning 2025-12-29 16:51 [PATCH 1/1] arm64: dts: imx8mp-skov: add pinctrl-name 'reset' to fix CHECK_DTB warning Frank Li @ 2026-01-05 11:14 ` Marco Felsch 2026-01-05 17:21 ` Frank Li 2026-01-06 10:18 ` Peng Fan 2026-01-06 21:43 ` Rob Herring 1 sibling, 2 replies; 7+ messages in thread From: Marco Felsch @ 2026-01-05 11:14 UTC (permalink / raw) To: Frank Li Cc: Rob Herring, Krzysztof Kozlowski, Conor Dooley, Shawn Guo, Sascha Hauer, Pengutronix Kernel Team, Fabio Estevam, open list:OPEN FIRMWARE AND FLATTENED DEVICE TREE BINDINGS, open list:ARM/FREESCALE IMX / MXC ARM ARCHITECTURE, moderated list:ARM/FREESCALE IMX / MXC ARM ARCHITECTURE, open list Hi Frank, On 25-12-29, Frank Li wrote: > Add pinctrl-name 'reset' to fix below CHECK_DTB warnings: > arch/arm64/boot/dts/freescale/imx8mp-skov-basic.dtb: switch@5f (microchip,ksz9893): pinctrl-names: ['default'] is too short > from schema $id: http://devicetree.org/schemas/net/dsa/microchip,ksz.yaml# > > The commit (e469b87e0fb0d dt-bindings: net: dsa: microchip: Add strap > description to set SPI mode" force use two pinctrl-name 'default' and > 'reset'. switch@5f doesn't use SPI, so it is safe to use the same pinctrl > setting for both. please see: - https://lore.kernel.org/all/20251112084717.ea7fchu7jcz6dzsi@pengutronix.de/ I stumbled over the same warning, but came to the conclusion, that the dt-bindings should be fixed instead of workaround broken bindings within the dtb. Regards, Marco > > Signed-off-by: Frank Li <Frank.Li@nxp.com> > --- > arch/arm64/boot/dts/freescale/imx8mp-skov-reva.dtsi | 4 ++-- > 1 file changed, 2 insertions(+), 2 deletions(-) > > diff --git a/arch/arm64/boot/dts/freescale/imx8mp-skov-reva.dtsi b/arch/arm64/boot/dts/freescale/imx8mp-skov-reva.dtsi > index 020f20c8ce667..7d2fc5fdd3bd1 100644 > --- a/arch/arm64/boot/dts/freescale/imx8mp-skov-reva.dtsi > +++ b/arch/arm64/boot/dts/freescale/imx8mp-skov-reva.dtsi > @@ -373,8 +373,8 @@ &i2c4 { > > switch: switch@5f { > compatible = "microchip,ksz9893"; > - pinctrl-names = "default"; > - pinctrl-0 = <&pinctrl_switch>; > + pinctrl-names = "default", "reset"; > + pinctrl-0 = <&pinctrl_switch>, <&pinctrl_switch>; > reset-gpios = <&gpio5 1 GPIO_ACTIVE_LOW>; > reg = <0x5f>; > > -- > 2.34.1 > > > -- #gernperDu #CallMeByMyFirstName Pengutronix e.K. | | Steuerwalder Str. 21 | https://www.pengutronix.de/ | 31137 Hildesheim, Germany | Phone: +49-5121-206917-0 | Amtsgericht Hildesheim, HRA 2686 | Fax: +49-5121-206917-9 | ^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [PATCH 1/1] arm64: dts: imx8mp-skov: add pinctrl-name 'reset' to fix CHECK_DTB warning 2026-01-05 11:14 ` Marco Felsch @ 2026-01-05 17:21 ` Frank Li 2026-01-05 17:55 ` Marco Felsch 2026-01-06 10:18 ` Peng Fan 1 sibling, 1 reply; 7+ messages in thread From: Frank Li @ 2026-01-05 17:21 UTC (permalink / raw) To: Marco Felsch Cc: Rob Herring, Krzysztof Kozlowski, Conor Dooley, Shawn Guo, Sascha Hauer, Pengutronix Kernel Team, Fabio Estevam, open list:OPEN FIRMWARE AND FLATTENED DEVICE TREE BINDINGS, open list:ARM/FREESCALE IMX / MXC ARM ARCHITECTURE, moderated list:ARM/FREESCALE IMX / MXC ARM ARCHITECTURE, open list On Mon, Jan 05, 2026 at 12:14:48PM +0100, Marco Felsch wrote: > Hi Frank, > > On 25-12-29, Frank Li wrote: > > Add pinctrl-name 'reset' to fix below CHECK_DTB warnings: > > arch/arm64/boot/dts/freescale/imx8mp-skov-basic.dtb: switch@5f (microchip,ksz9893): pinctrl-names: ['default'] is too short > > from schema $id: http://devicetree.org/schemas/net/dsa/microchip,ksz.yaml# > > > > The commit (e469b87e0fb0d dt-bindings: net: dsa: microchip: Add strap > > description to set SPI mode" force use two pinctrl-name 'default' and > > 'reset'. switch@5f doesn't use SPI, so it is safe to use the same pinctrl > > setting for both. > > please see: > - https://lore.kernel.org/all/20251112084717.ea7fchu7jcz6dzsi@pengutronix.de/ > > I stumbled over the same warning, but came to the conclusion, that the > dt-bindings should be fixed instead of workaround broken bindings within > the dtb. I have not seen "the conclusion" at above thread. I think patch 'reset' may be reasonable for SPI pin, which used for stub at reset. And there are not good method to distinguish under SPI or I2C. Frank > > Regards, > Marco > > > > > Signed-off-by: Frank Li <Frank.Li@nxp.com> > > --- > > arch/arm64/boot/dts/freescale/imx8mp-skov-reva.dtsi | 4 ++-- > > 1 file changed, 2 insertions(+), 2 deletions(-) > > > > diff --git a/arch/arm64/boot/dts/freescale/imx8mp-skov-reva.dtsi b/arch/arm64/boot/dts/freescale/imx8mp-skov-reva.dtsi > > index 020f20c8ce667..7d2fc5fdd3bd1 100644 > > --- a/arch/arm64/boot/dts/freescale/imx8mp-skov-reva.dtsi > > +++ b/arch/arm64/boot/dts/freescale/imx8mp-skov-reva.dtsi > > @@ -373,8 +373,8 @@ &i2c4 { > > > > switch: switch@5f { > > compatible = "microchip,ksz9893"; > > - pinctrl-names = "default"; > > - pinctrl-0 = <&pinctrl_switch>; > > + pinctrl-names = "default", "reset"; > > + pinctrl-0 = <&pinctrl_switch>, <&pinctrl_switch>; > > reset-gpios = <&gpio5 1 GPIO_ACTIVE_LOW>; > > reg = <0x5f>; > > > > -- > > 2.34.1 > > > > > > > > -- > #gernperDu > #CallMeByMyFirstName > > Pengutronix e.K. | | > Steuerwalder Str. 21 | https://www.pengutronix.de/ | > 31137 Hildesheim, Germany | Phone: +49-5121-206917-0 | > Amtsgericht Hildesheim, HRA 2686 | Fax: +49-5121-206917-9 | ^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [PATCH 1/1] arm64: dts: imx8mp-skov: add pinctrl-name 'reset' to fix CHECK_DTB warning 2026-01-05 17:21 ` Frank Li @ 2026-01-05 17:55 ` Marco Felsch 0 siblings, 0 replies; 7+ messages in thread From: Marco Felsch @ 2026-01-05 17:55 UTC (permalink / raw) To: Frank Li Cc: Rob Herring, Krzysztof Kozlowski, Conor Dooley, Shawn Guo, Sascha Hauer, Pengutronix Kernel Team, Fabio Estevam, open list:OPEN FIRMWARE AND FLATTENED DEVICE TREE BINDINGS, open list:ARM/FREESCALE IMX / MXC ARM ARCHITECTURE, moderated list:ARM/FREESCALE IMX / MXC ARM ARCHITECTURE, open list On 26-01-05, Frank Li wrote: > On Mon, Jan 05, 2026 at 12:14:48PM +0100, Marco Felsch wrote: > > Hi Frank, > > > > On 25-12-29, Frank Li wrote: > > > Add pinctrl-name 'reset' to fix below CHECK_DTB warnings: > > > arch/arm64/boot/dts/freescale/imx8mp-skov-basic.dtb: switch@5f (microchip,ksz9893): pinctrl-names: ['default'] is too short > > > from schema $id: http://devicetree.org/schemas/net/dsa/microchip,ksz.yaml# > > > > > > The commit (e469b87e0fb0d dt-bindings: net: dsa: microchip: Add strap > > > description to set SPI mode" force use two pinctrl-name 'default' and > > > 'reset'. switch@5f doesn't use SPI, so it is safe to use the same pinctrl > > > setting for both. > > > > please see: > > - https://lore.kernel.org/all/20251112084717.ea7fchu7jcz6dzsi@pengutronix.de/ > > > > I stumbled over the same warning, but came to the conclusion, that the > > dt-bindings should be fixed instead of workaround broken bindings within > > the dtb. > > I have not seen "the conclusion" at above thread. I think patch 'reset' may > be reasonable for SPI pin, which used for stub at reset. And there are not > good method to distinguish under SPI or I2C. The linked-linked dt-patch from Bastien adapts the pinctrl-names unconditionally for all KSZ switches, while he writes within the commit message, that only KSZ8463 should be affected. Therefore the straps-rxd-gpios was added only for the microchip,ksz8463. Furthermore the pinctrl-names should be either: - default or - default - reset since the 'reset' pinctrl is optional if no strapping pins are connected. So IMHO the correct fix should be a dt-bindings fix. Regards, Marco > > Frank > > > > > Regards, > > Marco > > > > > > > > Signed-off-by: Frank Li <Frank.Li@nxp.com> > > > --- > > > arch/arm64/boot/dts/freescale/imx8mp-skov-reva.dtsi | 4 ++-- > > > 1 file changed, 2 insertions(+), 2 deletions(-) > > > > > > diff --git a/arch/arm64/boot/dts/freescale/imx8mp-skov-reva.dtsi b/arch/arm64/boot/dts/freescale/imx8mp-skov-reva.dtsi > > > index 020f20c8ce667..7d2fc5fdd3bd1 100644 > > > --- a/arch/arm64/boot/dts/freescale/imx8mp-skov-reva.dtsi > > > +++ b/arch/arm64/boot/dts/freescale/imx8mp-skov-reva.dtsi > > > @@ -373,8 +373,8 @@ &i2c4 { > > > > > > switch: switch@5f { > > > compatible = "microchip,ksz9893"; > > > - pinctrl-names = "default"; > > > - pinctrl-0 = <&pinctrl_switch>; > > > + pinctrl-names = "default", "reset"; > > > + pinctrl-0 = <&pinctrl_switch>, <&pinctrl_switch>; > > > reset-gpios = <&gpio5 1 GPIO_ACTIVE_LOW>; > > > reg = <0x5f>; > > > > > > -- > > > 2.34.1 > > > > > > > > > > > > > -- > > #gernperDu > > #CallMeByMyFirstName > > > > Pengutronix e.K. | | > > Steuerwalder Str. 21 | https://www.pengutronix.de/ | > > 31137 Hildesheim, Germany | Phone: +49-5121-206917-0 | > > Amtsgericht Hildesheim, HRA 2686 | Fax: +49-5121-206917-9 | > -- #gernperDu #CallMeByMyFirstName Pengutronix e.K. | | Steuerwalder Str. 21 | https://www.pengutronix.de/ | 31137 Hildesheim, Germany | Phone: +49-5121-206917-0 | Amtsgericht Hildesheim, HRA 2686 | Fax: +49-5121-206917-9 | ^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [PATCH 1/1] arm64: dts: imx8mp-skov: add pinctrl-name 'reset' to fix CHECK_DTB warning 2026-01-05 11:14 ` Marco Felsch 2026-01-05 17:21 ` Frank Li @ 2026-01-06 10:18 ` Peng Fan 2026-01-06 14:13 ` Marco Felsch 1 sibling, 1 reply; 7+ messages in thread From: Peng Fan @ 2026-01-06 10:18 UTC (permalink / raw) To: Marco Felsch Cc: Frank Li, Rob Herring, Krzysztof Kozlowski, Conor Dooley, Shawn Guo, Sascha Hauer, Pengutronix Kernel Team, Fabio Estevam, open list:OPEN FIRMWARE AND FLATTENED DEVICE TREE BINDINGS, open list:ARM/FREESCALE IMX / MXC ARM ARCHITECTURE, moderated list:ARM/FREESCALE IMX / MXC ARM ARCHITECTURE, open list Hi Marco, On Mon, Jan 05, 2026 at 12:14:48PM +0100, Marco Felsch wrote: >Hi Frank, > >On 25-12-29, Frank Li wrote: >> Add pinctrl-name 'reset' to fix below CHECK_DTB warnings: >> arch/arm64/boot/dts/freescale/imx8mp-skov-basic.dtb: switch@5f (microchip,ksz9893): pinctrl-names: ['default'] is too short >> from schema $id: http://devicetree.org/schemas/net/dsa/microchip,ksz.yaml# >> >> The commit (e469b87e0fb0d dt-bindings: net: dsa: microchip: Add strap >> description to set SPI mode" force use two pinctrl-name 'default' and >> 'reset'. switch@5f doesn't use SPI, so it is safe to use the same pinctrl >> setting for both. > >please see: >- https://lore.kernel.org/all/20251112084717.ea7fchu7jcz6dzsi@pengutronix.de/ > >I stumbled over the same warning, but came to the conclusion, that the >dt-bindings should be fixed instead of workaround broken bindings within >the dtb. If reset is optional, the dt-bindings should include "minItems:1" per my understanding. But.. > >> - pinctrl-names = "default"; >> - pinctrl-0 = <&pinctrl_switch>; >> + pinctrl-names = "default", "reset"; >> + pinctrl-0 = <&pinctrl_switch>, <&pinctrl_switch>; Per checking https://elixir.bootlin.com/linux/v6.18.3/source/drivers/net/dsa/microchip/ksz_common.c#L5372 seems "reset" is required in driver. Regards Peng >> reset-gpios = <&gpio5 1 GPIO_ACTIVE_LOW>; >> reg = <0x5f>; >> >> -- >> 2.34.1 >> >> >> > >-- >#gernperDu >#CallMeByMyFirstName > >Pengutronix e.K. | | >Steuerwalder Str. 21 | https://www.pengutronix.de/ | >31137 Hildesheim, Germany | Phone: +49-5121-206917-0 | >Amtsgericht Hildesheim, HRA 2686 | Fax: +49-5121-206917-9 | ^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [PATCH 1/1] arm64: dts: imx8mp-skov: add pinctrl-name 'reset' to fix CHECK_DTB warning 2026-01-06 10:18 ` Peng Fan @ 2026-01-06 14:13 ` Marco Felsch 0 siblings, 0 replies; 7+ messages in thread From: Marco Felsch @ 2026-01-06 14:13 UTC (permalink / raw) To: Peng Fan Cc: Frank Li, Rob Herring, Krzysztof Kozlowski, Conor Dooley, Shawn Guo, Sascha Hauer, Pengutronix Kernel Team, Fabio Estevam, open list:OPEN FIRMWARE AND FLATTENED DEVICE TREE BINDINGS, open list:ARM/FREESCALE IMX / MXC ARM ARCHITECTURE, moderated list:ARM/FREESCALE IMX / MXC ARM ARCHITECTURE, open list Hi Peng, On 26-01-06, Peng Fan wrote: > Hi Marco, > > On Mon, Jan 05, 2026 at 12:14:48PM +0100, Marco Felsch wrote: > >Hi Frank, > > > >On 25-12-29, Frank Li wrote: > >> Add pinctrl-name 'reset' to fix below CHECK_DTB warnings: > >> arch/arm64/boot/dts/freescale/imx8mp-skov-basic.dtb: switch@5f (microchip,ksz9893): pinctrl-names: ['default'] is too short > >> from schema $id: http://devicetree.org/schemas/net/dsa/microchip,ksz.yaml# > >> > >> The commit (e469b87e0fb0d dt-bindings: net: dsa: microchip: Add strap > >> description to set SPI mode" force use two pinctrl-name 'default' and > >> 'reset'. switch@5f doesn't use SPI, so it is safe to use the same pinctrl > >> setting for both. > > > >please see: > >- https://lore.kernel.org/all/20251112084717.ea7fchu7jcz6dzsi@pengutronix.de/ > > > >I stumbled over the same warning, but came to the conclusion, that the > >dt-bindings should be fixed instead of workaround broken bindings within > >the dtb. > > If reset is optional, the dt-bindings should include "minItems:1" per my > understanding. Not sure, since the pinctrl-names is just an enum list. That beeing said, I don't see it as required since the strapping may not be required for systems which do already has correct ext. strapping options. Therefore the binding should be always optional and the driver should be adapted. > But.. > > > > >> - pinctrl-names = "default"; > >> - pinctrl-0 = <&pinctrl_switch>; > >> + pinctrl-names = "default", "reset"; > >> + pinctrl-0 = <&pinctrl_switch>, <&pinctrl_switch>; > > Per checking > https://elixir.bootlin.com/linux/v6.18.3/source/drivers/net/dsa/microchip/ksz_common.c#L5372 > > seems "reset" is required in driver. For the ksz8463 but not for the ksz9893 which is used by the imx8mp-skov-reva.dtsi (here). Regards, Marco > > Regards > Peng > > >> reset-gpios = <&gpio5 1 GPIO_ACTIVE_LOW>; > >> reg = <0x5f>; > >> > >> -- > >> 2.34.1 > >> > >> > >> > > > >-- > >#gernperDu > >#CallMeByMyFirstName > > > >Pengutronix e.K. | | > >Steuerwalder Str. 21 | https://www.pengutronix.de/ | > >31137 Hildesheim, Germany | Phone: +49-5121-206917-0 | > >Amtsgericht Hildesheim, HRA 2686 | Fax: +49-5121-206917-9 | > -- #gernperDu #CallMeByMyFirstName Pengutronix e.K. | | Steuerwalder Str. 21 | https://www.pengutronix.de/ | 31137 Hildesheim, Germany | Phone: +49-5121-206917-0 | Amtsgericht Hildesheim, HRA 2686 | Fax: +49-5121-206917-9 | ^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [PATCH 1/1] arm64: dts: imx8mp-skov: add pinctrl-name 'reset' to fix CHECK_DTB warning 2025-12-29 16:51 [PATCH 1/1] arm64: dts: imx8mp-skov: add pinctrl-name 'reset' to fix CHECK_DTB warning Frank Li 2026-01-05 11:14 ` Marco Felsch @ 2026-01-06 21:43 ` Rob Herring 1 sibling, 0 replies; 7+ messages in thread From: Rob Herring @ 2026-01-06 21:43 UTC (permalink / raw) To: Frank Li Cc: Krzysztof Kozlowski, Conor Dooley, Shawn Guo, Sascha Hauer, Pengutronix Kernel Team, Fabio Estevam, open list:OPEN FIRMWARE AND FLATTENED DEVICE TREE BINDINGS, open list:ARM/FREESCALE IMX / MXC ARM ARCHITECTURE, moderated list:ARM/FREESCALE IMX / MXC ARM ARCHITECTURE, open list On Mon, Dec 29, 2025 at 10:51 AM Frank Li <Frank.Li@nxp.com> wrote: > > Add pinctrl-name 'reset' to fix below CHECK_DTB warnings: > arch/arm64/boot/dts/freescale/imx8mp-skov-basic.dtb: switch@5f (microchip,ksz9893): pinctrl-names: ['default'] is too short > from schema $id: http://devicetree.org/schemas/net/dsa/microchip,ksz.yaml# > > The commit (e469b87e0fb0d dt-bindings: net: dsa: microchip: Add strap > description to set SPI mode" force use two pinctrl-name 'default' and > 'reset'. switch@5f doesn't use SPI, so it is safe to use the same pinctrl > setting for both. > > Signed-off-by: Frank Li <Frank.Li@nxp.com> > --- > arch/arm64/boot/dts/freescale/imx8mp-skov-reva.dtsi | 4 ++-- > 1 file changed, 2 insertions(+), 2 deletions(-) > > diff --git a/arch/arm64/boot/dts/freescale/imx8mp-skov-reva.dtsi b/arch/arm64/boot/dts/freescale/imx8mp-skov-reva.dtsi > index 020f20c8ce667..7d2fc5fdd3bd1 100644 > --- a/arch/arm64/boot/dts/freescale/imx8mp-skov-reva.dtsi > +++ b/arch/arm64/boot/dts/freescale/imx8mp-skov-reva.dtsi > @@ -373,8 +373,8 @@ &i2c4 { > > switch: switch@5f { > compatible = "microchip,ksz9893"; > - pinctrl-names = "default"; > - pinctrl-0 = <&pinctrl_switch>; > + pinctrl-names = "default", "reset"; > + pinctrl-0 = <&pinctrl_switch>, <&pinctrl_switch>; I guess this is getting dropped anyways, but this is completely wrong. "reset" entry here corresponds to pinctrl-1 property. pinctrl-N properties can have any number of phandles, but those are all one name. Rob ^ permalink raw reply [flat|nested] 7+ messages in thread
end of thread, other threads:[~2026-01-06 21:43 UTC | newest] Thread overview: 7+ messages (download: mbox.gz follow: Atom feed -- links below jump to the message on this page -- 2025-12-29 16:51 [PATCH 1/1] arm64: dts: imx8mp-skov: add pinctrl-name 'reset' to fix CHECK_DTB warning Frank Li 2026-01-05 11:14 ` Marco Felsch 2026-01-05 17:21 ` Frank Li 2026-01-05 17:55 ` Marco Felsch 2026-01-06 10:18 ` Peng Fan 2026-01-06 14:13 ` Marco Felsch 2026-01-06 21:43 ` Rob Herring
This is a public inbox, see mirroring instructions for how to clone and mirror all data and code used for this inbox