Hello Cédric,
On 3/31/2022 8:56 AM, Cédric Le Goater wrote:
Hello Jae,
On 3/31/22 17:44, Jae Hyun Yoo wrote:
On 3/30/2022 10:50 PM, Joel Stanley wrote:
On Tue, 29 Mar 2022 at 17:40, Jae Hyun Yoo <quic_jaehyoo@xxxxxxxxxxx>
wrote:
I’m sending this patch series to fix current issues in AST2600 pinmux
settings while enabling quad mode SPI support.
FWSPI18 pins are basically 1.8v logic pins that are different from the
dedicated FWSPI pins that provide 3.3v logic level, so FWSPI18 pins
can’t
be grouped with FWSPIDQ2 and FWSPIDQ3, so this series fix the issue.
Also, fixes QSPI1 and QSPI2 function settings in AST2600 pinctrl
dtsi to
make it able to enable quad mode on SPI1 and SPI2 interfaces.
With this series, quad mode pinmux can be set like below.
FW SPI:
&fmc {
pinctrl-names = "default";
pinctrl-0 = <&pinctrl_fwqspi_default>;
}
SPI1:
&spi1 {
pinctrl-names = "default";
pinctrl-0 = <&pinctrl_qspi1_default>;
}
SPI2:
&spi2 {
pinctrl-names = "default";
pinctrl-0 = <&pinctrl_qspi2_default>;
}
Thanks. I hope to see a board from you that uses this soon :)
I'll send the patches as fixes once -rc1 is out.
Thanks Joel!
Yes, I would be able to send my BMC board dts soon.
Thanks in advance for your review on that too.
Out of curiosity, which driver are you using ? the one from SDK ?
I proposed a new one for upstream supporting all AST2400, AST2500, AST2600
controllers. I would be glad to have some feedback if you have time.
Yes, I saw your patch set of the new driver.
I'm currently using this fix with legacy aspeed-smc driver after
adding some fixes. I'll give it a try with your new driver as well and
will give you some feedback if I find any.
Thanks,
Jae
Thanks,
C.
