| Message ID | 20220325154048.467245-2-quic_jaehyoo@quicinc.com (mailing list archive) |
|---|---|
| State | New, archived |
| Headers | show |
| Series | [v2,1/5] ARM: dts: aspeed-g6: remove FWQSPID group in pinctrl dtsi | expand |
On Sat, 26 Mar 2022, at 02:10, Jae Hyun Yoo wrote: > FWSPIDQ2 and FWSPIDQ3 are not part of FWSPI18 interface so remove > FWQSPID group in pinctrl dtsi. These pins must be used with the > FWSPI pins that are dedicated for boot SPI interface which provides > same 3.3v logic level. > > Signed-off-by: Jae Hyun Yoo <quic_jaehyoo@quicinc.com> > Fixes: 2f6edb6bcb2f ("ARM: dts: aspeed: Fix AST2600 quad spi group") > --- > Changes in v2: > * Rebased it on the latest. > * Updated 'Fixes' while rebasing it on the latest. > > Note: > * Removing 'pinctrl_fwqspid_default' doesn't break any existing DT since > it has not been used in any DTs. Can you please send a patch to the bindings to drop the broken function/group given they never worked and no-one could be using them? Other than that, Reviewed-by: Andrew Jeffery <andrew@aj.id.au>
On 3/27/2022 8:09 PM, Andrew Jeffery wrote: > > > On Sat, 26 Mar 2022, at 02:10, Jae Hyun Yoo wrote: >> FWSPIDQ2 and FWSPIDQ3 are not part of FWSPI18 interface so remove >> FWQSPID group in pinctrl dtsi. These pins must be used with the >> FWSPI pins that are dedicated for boot SPI interface which provides >> same 3.3v logic level. >> >> Signed-off-by: Jae Hyun Yoo <quic_jaehyoo@quicinc.com> >> Fixes: 2f6edb6bcb2f ("ARM: dts: aspeed: Fix AST2600 quad spi group") >> --- >> Changes in v2: >> * Rebased it on the latest. >> * Updated 'Fixes' while rebasing it on the latest. >> >> Note: >> * Removing 'pinctrl_fwqspid_default' doesn't break any existing DT since >> it has not been used in any DTs. > > Can you please send a patch to the bindings to drop the broken > function/group given they never worked and no-one could be using them? Sure, I'll send a bindings patch too in v3. Thanks for your review! -Jae > Other than that, > > Reviewed-by: Andrew Jeffery <andrew@aj.id.au>
diff --git a/arch/arm/boot/dts/aspeed-g6-pinctrl.dtsi b/arch/arm/boot/dts/aspeed-g6-pinctrl.dtsi index e4775bbceecc..06d60a8540e9 100644 --- a/arch/arm/boot/dts/aspeed-g6-pinctrl.dtsi +++ b/arch/arm/boot/dts/aspeed-g6-pinctrl.dtsi @@ -117,11 +117,6 @@ pinctrl_fwspid_default: fwspid_default { groups = "FWSPID"; }; - pinctrl_fwqspid_default: fwqspid_default { - function = "FWSPID"; - groups = "FWQSPID"; - }; - pinctrl_fwspiwp_default: fwspiwp_default { function = "FWSPIWP"; groups = "FWSPIWP";
FWSPIDQ2 and FWSPIDQ3 are not part of FWSPI18 interface so remove FWQSPID group in pinctrl dtsi. These pins must be used with the FWSPI pins that are dedicated for boot SPI interface which provides same 3.3v logic level. Signed-off-by: Jae Hyun Yoo <quic_jaehyoo@quicinc.com> Fixes: 2f6edb6bcb2f ("ARM: dts: aspeed: Fix AST2600 quad spi group") --- Changes in v2: * Rebased it on the latest. * Updated 'Fixes' while rebasing it on the latest. Note: * Removing 'pinctrl_fwqspid_default' doesn't break any existing DT since it has not been used in any DTs. arch/arm/boot/dts/aspeed-g6-pinctrl.dtsi | 5 ----- 1 file changed, 5 deletions(-)