[PATCH v2 1/5] ARM: dts: aspeed-g6: remove FWQSPID group in pinctrl dtsi
Jae Hyun Yoo
quic_jaehyoo at quicinc.com
Tue Mar 29 01:26:01 AEDT 2022
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 at 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.
>> * 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!
> Other than that,
> Reviewed-by: Andrew Jeffery <andrew at aj.id.au>
More information about the Linux-aspeed