* [PATCH v3 3/3] arm64: dts: rockchip: k3566-quartz64-a: adds sata variant @ 2022-09-16 16:00 ` Alessandro Carminati 0 siblings, 0 replies; 18+ messages in thread From: Alessandro Carminati @ 2022-09-16 16:00 UTC (permalink / raw) To: Rob Herring, Krzysztof Kozlowski, Heiko Stuebner, devicetree, linux-arm-kernel, linux-rockchip, linux-kernel The Quartz64 board is built upon Rockchip RK3566. Rockchip RK3566 has two combo phys. The first connects USB3 and SATA ctrl1, and the second PCIe lane and SATA ctrl2. The second combo phy is hardwired to the PCIe slot, where for the first, the hardware on the board provides both the USB3 connector and the SATA connector. This DT allows the users to switch the combo phy to the SATA connector. Signed-off-by: Alessandro Carminati <alessandro.carminati@gmail.com> --- arch/arm64/boot/dts/rockchip/Makefile | 1 + arch/arm64/boot/dts/rockchip/rk3566-quartz64-a-sata.dts | 9 +++++++++ 2 files changed, 10 insertions(+) create mode 100644 arch/arm64/boot/dts/rockchip/rk3566-quartz64-a-sata.dts diff --git a/arch/arm64/boot/dts/rockchip/Makefile b/arch/arm64/boot/dts/rockchip/Makefile index 8c843f6fc3cc..1d5dd91d1a34 100644 --- a/arch/arm64/boot/dts/rockchip/Makefile +++ b/arch/arm64/boot/dts/rockchip/Makefile @@ -60,6 +60,7 @@ dtb-$(CONFIG_ARCH_ROCKCHIP) += rk3399pro-rock-pi-n10.dtb dtb-$(CONFIG_ARCH_ROCKCHIP) += rk3566-pinenote-v1.1.dtb dtb-$(CONFIG_ARCH_ROCKCHIP) += rk3566-pinenote-v1.2.dtb dtb-$(CONFIG_ARCH_ROCKCHIP) += rk3566-quartz64-a-usb3.dts +dtb-$(CONFIG_ARCH_ROCKCHIP) += rk3566-quartz64-a-sata.dts dtb-$(CONFIG_ARCH_ROCKCHIP) += rk3566-quartz64-b.dtb dtb-$(CONFIG_ARCH_ROCKCHIP) += rk3566-roc-pc.dtb dtb-$(CONFIG_ARCH_ROCKCHIP) += rk3566-soquartz-cm4.dtb diff --git a/arch/arm64/boot/dts/rockchip/rk3566-quartz64-a-sata.dts b/arch/arm64/boot/dts/rockchip/rk3566-quartz64-a-sata.dts new file mode 100644 index 000000000000..8620df7ec01e --- /dev/null +++ b/arch/arm64/boot/dts/rockchip/rk3566-quartz64-a-sata.dts @@ -0,0 +1,9 @@ +// SPDX-License-Identifier: (GPL-2.0+ OR MIT) + +/dts-v1/; + +#include "rk3566-quartz64-a.dtsi" + +&sata1 { + status = "okay"; +}; -- 2.34.1 ^ permalink raw reply related [flat|nested] 18+ messages in thread
* [PATCH v3 3/3] arm64: dts: rockchip: k3566-quartz64-a: adds sata variant @ 2022-09-16 16:00 ` Alessandro Carminati 0 siblings, 0 replies; 18+ messages in thread From: Alessandro Carminati @ 2022-09-16 16:00 UTC (permalink / raw) To: Rob Herring, Krzysztof Kozlowski, Heiko Stuebner, devicetree, linux-arm-kernel, linux-rockchip, linux-kernel The Quartz64 board is built upon Rockchip RK3566. Rockchip RK3566 has two combo phys. The first connects USB3 and SATA ctrl1, and the second PCIe lane and SATA ctrl2. The second combo phy is hardwired to the PCIe slot, where for the first, the hardware on the board provides both the USB3 connector and the SATA connector. This DT allows the users to switch the combo phy to the SATA connector. Signed-off-by: Alessandro Carminati <alessandro.carminati@gmail.com> --- arch/arm64/boot/dts/rockchip/Makefile | 1 + arch/arm64/boot/dts/rockchip/rk3566-quartz64-a-sata.dts | 9 +++++++++ 2 files changed, 10 insertions(+) create mode 100644 arch/arm64/boot/dts/rockchip/rk3566-quartz64-a-sata.dts diff --git a/arch/arm64/boot/dts/rockchip/Makefile b/arch/arm64/boot/dts/rockchip/Makefile index 8c843f6fc3cc..1d5dd91d1a34 100644 --- a/arch/arm64/boot/dts/rockchip/Makefile +++ b/arch/arm64/boot/dts/rockchip/Makefile @@ -60,6 +60,7 @@ dtb-$(CONFIG_ARCH_ROCKCHIP) += rk3399pro-rock-pi-n10.dtb dtb-$(CONFIG_ARCH_ROCKCHIP) += rk3566-pinenote-v1.1.dtb dtb-$(CONFIG_ARCH_ROCKCHIP) += rk3566-pinenote-v1.2.dtb dtb-$(CONFIG_ARCH_ROCKCHIP) += rk3566-quartz64-a-usb3.dts +dtb-$(CONFIG_ARCH_ROCKCHIP) += rk3566-quartz64-a-sata.dts dtb-$(CONFIG_ARCH_ROCKCHIP) += rk3566-quartz64-b.dtb dtb-$(CONFIG_ARCH_ROCKCHIP) += rk3566-roc-pc.dtb dtb-$(CONFIG_ARCH_ROCKCHIP) += rk3566-soquartz-cm4.dtb diff --git a/arch/arm64/boot/dts/rockchip/rk3566-quartz64-a-sata.dts b/arch/arm64/boot/dts/rockchip/rk3566-quartz64-a-sata.dts new file mode 100644 index 000000000000..8620df7ec01e --- /dev/null +++ b/arch/arm64/boot/dts/rockchip/rk3566-quartz64-a-sata.dts @@ -0,0 +1,9 @@ +// SPDX-License-Identifier: (GPL-2.0+ OR MIT) + +/dts-v1/; + +#include "rk3566-quartz64-a.dtsi" + +&sata1 { + status = "okay"; +}; -- 2.34.1 _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel ^ permalink raw reply related [flat|nested] 18+ messages in thread
* [PATCH v3 3/3] arm64: dts: rockchip: k3566-quartz64-a: adds sata variant @ 2022-09-16 16:00 ` Alessandro Carminati 0 siblings, 0 replies; 18+ messages in thread From: Alessandro Carminati @ 2022-09-16 16:00 UTC (permalink / raw) To: Rob Herring, Krzysztof Kozlowski, Heiko Stuebner, devicetree, linux-arm-kernel, linux-rockchip, linux-kernel The Quartz64 board is built upon Rockchip RK3566. Rockchip RK3566 has two combo phys. The first connects USB3 and SATA ctrl1, and the second PCIe lane and SATA ctrl2. The second combo phy is hardwired to the PCIe slot, where for the first, the hardware on the board provides both the USB3 connector and the SATA connector. This DT allows the users to switch the combo phy to the SATA connector. Signed-off-by: Alessandro Carminati <alessandro.carminati@gmail.com> --- arch/arm64/boot/dts/rockchip/Makefile | 1 + arch/arm64/boot/dts/rockchip/rk3566-quartz64-a-sata.dts | 9 +++++++++ 2 files changed, 10 insertions(+) create mode 100644 arch/arm64/boot/dts/rockchip/rk3566-quartz64-a-sata.dts diff --git a/arch/arm64/boot/dts/rockchip/Makefile b/arch/arm64/boot/dts/rockchip/Makefile index 8c843f6fc3cc..1d5dd91d1a34 100644 --- a/arch/arm64/boot/dts/rockchip/Makefile +++ b/arch/arm64/boot/dts/rockchip/Makefile @@ -60,6 +60,7 @@ dtb-$(CONFIG_ARCH_ROCKCHIP) += rk3399pro-rock-pi-n10.dtb dtb-$(CONFIG_ARCH_ROCKCHIP) += rk3566-pinenote-v1.1.dtb dtb-$(CONFIG_ARCH_ROCKCHIP) += rk3566-pinenote-v1.2.dtb dtb-$(CONFIG_ARCH_ROCKCHIP) += rk3566-quartz64-a-usb3.dts +dtb-$(CONFIG_ARCH_ROCKCHIP) += rk3566-quartz64-a-sata.dts dtb-$(CONFIG_ARCH_ROCKCHIP) += rk3566-quartz64-b.dtb dtb-$(CONFIG_ARCH_ROCKCHIP) += rk3566-roc-pc.dtb dtb-$(CONFIG_ARCH_ROCKCHIP) += rk3566-soquartz-cm4.dtb diff --git a/arch/arm64/boot/dts/rockchip/rk3566-quartz64-a-sata.dts b/arch/arm64/boot/dts/rockchip/rk3566-quartz64-a-sata.dts new file mode 100644 index 000000000000..8620df7ec01e --- /dev/null +++ b/arch/arm64/boot/dts/rockchip/rk3566-quartz64-a-sata.dts @@ -0,0 +1,9 @@ +// SPDX-License-Identifier: (GPL-2.0+ OR MIT) + +/dts-v1/; + +#include "rk3566-quartz64-a.dtsi" + +&sata1 { + status = "okay"; +}; -- 2.34.1 _______________________________________________ Linux-rockchip mailing list Linux-rockchip@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-rockchip ^ permalink raw reply related [flat|nested] 18+ messages in thread
* Re: [PATCH v3 3/3] arm64: dts: rockchip: k3566-quartz64-a: adds sata variant 2022-09-16 16:00 ` Alessandro Carminati (?) @ 2022-09-17 1:40 ` Peter Geis -1 siblings, 0 replies; 18+ messages in thread From: Peter Geis @ 2022-09-17 1:40 UTC (permalink / raw) To: Alessandro Carminati Cc: Rob Herring, Krzysztof Kozlowski, Heiko Stuebner, devicetree, linux-arm-kernel, linux-rockchip, linux-kernel On Fri, Sep 16, 2022 at 12:06 PM Alessandro Carminati <alessandro.carminati@gmail.com> wrote: > > The Quartz64 board is built upon Rockchip RK3566. > Rockchip RK3566 has two combo phys. > The first connects USB3 and SATA ctrl1, and the second PCIe lane and SATA > ctrl2. > The second combo phy is hardwired to the PCIe slot, where for the first, > the hardware on the board provides both the USB3 connector and the SATA > connector. > This DT allows the users to switch the combo phy to the SATA connector. Good Evening, NACK to this whole series. Neither works correctly in the hardware as is, and USB3 was decided to be left enabled as the SATA port will be removed completely in the next revision. Very Respectfully, Peter Geis > > Signed-off-by: Alessandro Carminati <alessandro.carminati@gmail.com> > --- > arch/arm64/boot/dts/rockchip/Makefile | 1 + > arch/arm64/boot/dts/rockchip/rk3566-quartz64-a-sata.dts | 9 +++++++++ > 2 files changed, 10 insertions(+) > create mode 100644 arch/arm64/boot/dts/rockchip/rk3566-quartz64-a-sata.dts > > diff --git a/arch/arm64/boot/dts/rockchip/Makefile b/arch/arm64/boot/dts/rockchip/Makefile > index 8c843f6fc3cc..1d5dd91d1a34 100644 > --- a/arch/arm64/boot/dts/rockchip/Makefile > +++ b/arch/arm64/boot/dts/rockchip/Makefile > @@ -60,6 +60,7 @@ dtb-$(CONFIG_ARCH_ROCKCHIP) += rk3399pro-rock-pi-n10.dtb > dtb-$(CONFIG_ARCH_ROCKCHIP) += rk3566-pinenote-v1.1.dtb > dtb-$(CONFIG_ARCH_ROCKCHIP) += rk3566-pinenote-v1.2.dtb > dtb-$(CONFIG_ARCH_ROCKCHIP) += rk3566-quartz64-a-usb3.dts > +dtb-$(CONFIG_ARCH_ROCKCHIP) += rk3566-quartz64-a-sata.dts > dtb-$(CONFIG_ARCH_ROCKCHIP) += rk3566-quartz64-b.dtb > dtb-$(CONFIG_ARCH_ROCKCHIP) += rk3566-roc-pc.dtb > dtb-$(CONFIG_ARCH_ROCKCHIP) += rk3566-soquartz-cm4.dtb > diff --git a/arch/arm64/boot/dts/rockchip/rk3566-quartz64-a-sata.dts b/arch/arm64/boot/dts/rockchip/rk3566-quartz64-a-sata.dts > new file mode 100644 > index 000000000000..8620df7ec01e > --- /dev/null > +++ b/arch/arm64/boot/dts/rockchip/rk3566-quartz64-a-sata.dts > @@ -0,0 +1,9 @@ > +// SPDX-License-Identifier: (GPL-2.0+ OR MIT) > + > +/dts-v1/; > + > +#include "rk3566-quartz64-a.dtsi" > + > +&sata1 { > + status = "okay"; > +}; > -- > 2.34.1 > > > _______________________________________________ > Linux-rockchip mailing list > Linux-rockchip@lists.infradead.org > http://lists.infradead.org/mailman/listinfo/linux-rockchip ^ permalink raw reply [flat|nested] 18+ messages in thread
* Re: [PATCH v3 3/3] arm64: dts: rockchip: k3566-quartz64-a: adds sata variant @ 2022-09-17 1:40 ` Peter Geis 0 siblings, 0 replies; 18+ messages in thread From: Peter Geis @ 2022-09-17 1:40 UTC (permalink / raw) To: Alessandro Carminati Cc: Rob Herring, Krzysztof Kozlowski, Heiko Stuebner, devicetree, linux-arm-kernel, linux-rockchip, linux-kernel On Fri, Sep 16, 2022 at 12:06 PM Alessandro Carminati <alessandro.carminati@gmail.com> wrote: > > The Quartz64 board is built upon Rockchip RK3566. > Rockchip RK3566 has two combo phys. > The first connects USB3 and SATA ctrl1, and the second PCIe lane and SATA > ctrl2. > The second combo phy is hardwired to the PCIe slot, where for the first, > the hardware on the board provides both the USB3 connector and the SATA > connector. > This DT allows the users to switch the combo phy to the SATA connector. Good Evening, NACK to this whole series. Neither works correctly in the hardware as is, and USB3 was decided to be left enabled as the SATA port will be removed completely in the next revision. Very Respectfully, Peter Geis > > Signed-off-by: Alessandro Carminati <alessandro.carminati@gmail.com> > --- > arch/arm64/boot/dts/rockchip/Makefile | 1 + > arch/arm64/boot/dts/rockchip/rk3566-quartz64-a-sata.dts | 9 +++++++++ > 2 files changed, 10 insertions(+) > create mode 100644 arch/arm64/boot/dts/rockchip/rk3566-quartz64-a-sata.dts > > diff --git a/arch/arm64/boot/dts/rockchip/Makefile b/arch/arm64/boot/dts/rockchip/Makefile > index 8c843f6fc3cc..1d5dd91d1a34 100644 > --- a/arch/arm64/boot/dts/rockchip/Makefile > +++ b/arch/arm64/boot/dts/rockchip/Makefile > @@ -60,6 +60,7 @@ dtb-$(CONFIG_ARCH_ROCKCHIP) += rk3399pro-rock-pi-n10.dtb > dtb-$(CONFIG_ARCH_ROCKCHIP) += rk3566-pinenote-v1.1.dtb > dtb-$(CONFIG_ARCH_ROCKCHIP) += rk3566-pinenote-v1.2.dtb > dtb-$(CONFIG_ARCH_ROCKCHIP) += rk3566-quartz64-a-usb3.dts > +dtb-$(CONFIG_ARCH_ROCKCHIP) += rk3566-quartz64-a-sata.dts > dtb-$(CONFIG_ARCH_ROCKCHIP) += rk3566-quartz64-b.dtb > dtb-$(CONFIG_ARCH_ROCKCHIP) += rk3566-roc-pc.dtb > dtb-$(CONFIG_ARCH_ROCKCHIP) += rk3566-soquartz-cm4.dtb > diff --git a/arch/arm64/boot/dts/rockchip/rk3566-quartz64-a-sata.dts b/arch/arm64/boot/dts/rockchip/rk3566-quartz64-a-sata.dts > new file mode 100644 > index 000000000000..8620df7ec01e > --- /dev/null > +++ b/arch/arm64/boot/dts/rockchip/rk3566-quartz64-a-sata.dts > @@ -0,0 +1,9 @@ > +// SPDX-License-Identifier: (GPL-2.0+ OR MIT) > + > +/dts-v1/; > + > +#include "rk3566-quartz64-a.dtsi" > + > +&sata1 { > + status = "okay"; > +}; > -- > 2.34.1 > > > _______________________________________________ > Linux-rockchip mailing list > Linux-rockchip@lists.infradead.org > http://lists.infradead.org/mailman/listinfo/linux-rockchip _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel ^ permalink raw reply [flat|nested] 18+ messages in thread
* Re: [PATCH v3 3/3] arm64: dts: rockchip: k3566-quartz64-a: adds sata variant @ 2022-09-17 1:40 ` Peter Geis 0 siblings, 0 replies; 18+ messages in thread From: Peter Geis @ 2022-09-17 1:40 UTC (permalink / raw) To: Alessandro Carminati Cc: Rob Herring, Krzysztof Kozlowski, Heiko Stuebner, devicetree, linux-arm-kernel, linux-rockchip, linux-kernel On Fri, Sep 16, 2022 at 12:06 PM Alessandro Carminati <alessandro.carminati@gmail.com> wrote: > > The Quartz64 board is built upon Rockchip RK3566. > Rockchip RK3566 has two combo phys. > The first connects USB3 and SATA ctrl1, and the second PCIe lane and SATA > ctrl2. > The second combo phy is hardwired to the PCIe slot, where for the first, > the hardware on the board provides both the USB3 connector and the SATA > connector. > This DT allows the users to switch the combo phy to the SATA connector. Good Evening, NACK to this whole series. Neither works correctly in the hardware as is, and USB3 was decided to be left enabled as the SATA port will be removed completely in the next revision. Very Respectfully, Peter Geis > > Signed-off-by: Alessandro Carminati <alessandro.carminati@gmail.com> > --- > arch/arm64/boot/dts/rockchip/Makefile | 1 + > arch/arm64/boot/dts/rockchip/rk3566-quartz64-a-sata.dts | 9 +++++++++ > 2 files changed, 10 insertions(+) > create mode 100644 arch/arm64/boot/dts/rockchip/rk3566-quartz64-a-sata.dts > > diff --git a/arch/arm64/boot/dts/rockchip/Makefile b/arch/arm64/boot/dts/rockchip/Makefile > index 8c843f6fc3cc..1d5dd91d1a34 100644 > --- a/arch/arm64/boot/dts/rockchip/Makefile > +++ b/arch/arm64/boot/dts/rockchip/Makefile > @@ -60,6 +60,7 @@ dtb-$(CONFIG_ARCH_ROCKCHIP) += rk3399pro-rock-pi-n10.dtb > dtb-$(CONFIG_ARCH_ROCKCHIP) += rk3566-pinenote-v1.1.dtb > dtb-$(CONFIG_ARCH_ROCKCHIP) += rk3566-pinenote-v1.2.dtb > dtb-$(CONFIG_ARCH_ROCKCHIP) += rk3566-quartz64-a-usb3.dts > +dtb-$(CONFIG_ARCH_ROCKCHIP) += rk3566-quartz64-a-sata.dts > dtb-$(CONFIG_ARCH_ROCKCHIP) += rk3566-quartz64-b.dtb > dtb-$(CONFIG_ARCH_ROCKCHIP) += rk3566-roc-pc.dtb > dtb-$(CONFIG_ARCH_ROCKCHIP) += rk3566-soquartz-cm4.dtb > diff --git a/arch/arm64/boot/dts/rockchip/rk3566-quartz64-a-sata.dts b/arch/arm64/boot/dts/rockchip/rk3566-quartz64-a-sata.dts > new file mode 100644 > index 000000000000..8620df7ec01e > --- /dev/null > +++ b/arch/arm64/boot/dts/rockchip/rk3566-quartz64-a-sata.dts > @@ -0,0 +1,9 @@ > +// SPDX-License-Identifier: (GPL-2.0+ OR MIT) > + > +/dts-v1/; > + > +#include "rk3566-quartz64-a.dtsi" > + > +&sata1 { > + status = "okay"; > +}; > -- > 2.34.1 > > > _______________________________________________ > Linux-rockchip mailing list > Linux-rockchip@lists.infradead.org > http://lists.infradead.org/mailman/listinfo/linux-rockchip _______________________________________________ Linux-rockchip mailing list Linux-rockchip@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-rockchip ^ permalink raw reply [flat|nested] 18+ messages in thread
* Re: [PATCH v3 3/3] arm64: dts: rockchip: k3566-quartz64-a: adds sata variant 2022-09-17 1:40 ` Peter Geis (?) @ 2022-09-17 6:42 ` Heiko Stuebner -1 siblings, 0 replies; 18+ messages in thread From: Heiko Stuebner @ 2022-09-17 6:42 UTC (permalink / raw) To: Alessandro Carminati, Peter Geis Cc: Rob Herring, Krzysztof Kozlowski, devicetree, linux-arm-kernel, linux-rockchip, linux-kernel Hi Peter, Am Samstag, 17. September 2022, 03:40:07 CEST schrieb Peter Geis: > On Fri, Sep 16, 2022 at 12:06 PM Alessandro Carminati > <alessandro.carminati@gmail.com> wrote: > > > > The Quartz64 board is built upon Rockchip RK3566. > > Rockchip RK3566 has two combo phys. > > The first connects USB3 and SATA ctrl1, and the second PCIe lane and SATA > > ctrl2. > > The second combo phy is hardwired to the PCIe slot, where for the first, > > the hardware on the board provides both the USB3 connector and the SATA > > connector. > > This DT allows the users to switch the combo phy to the SATA connector. > > Good Evening, > > NACK to this whole series. Neither works correctly in the hardware as > is, Just for my understanding for the future, sata not working is that a bug in the soc or the board? > and USB3 was decided to be left enabled as the SATA port will be > removed completely in the next revision. That is good to know. Thanks for the heads up :-) Heiko > > Signed-off-by: Alessandro Carminati <alessandro.carminati@gmail.com> > > --- > > arch/arm64/boot/dts/rockchip/Makefile | 1 + > > arch/arm64/boot/dts/rockchip/rk3566-quartz64-a-sata.dts | 9 +++++++++ > > 2 files changed, 10 insertions(+) > > create mode 100644 arch/arm64/boot/dts/rockchip/rk3566-quartz64-a-sata.dts > > > > diff --git a/arch/arm64/boot/dts/rockchip/Makefile b/arch/arm64/boot/dts/rockchip/Makefile > > index 8c843f6fc3cc..1d5dd91d1a34 100644 > > --- a/arch/arm64/boot/dts/rockchip/Makefile > > +++ b/arch/arm64/boot/dts/rockchip/Makefile > > @@ -60,6 +60,7 @@ dtb-$(CONFIG_ARCH_ROCKCHIP) += rk3399pro-rock-pi-n10.dtb > > dtb-$(CONFIG_ARCH_ROCKCHIP) += rk3566-pinenote-v1.1.dtb > > dtb-$(CONFIG_ARCH_ROCKCHIP) += rk3566-pinenote-v1.2.dtb > > dtb-$(CONFIG_ARCH_ROCKCHIP) += rk3566-quartz64-a-usb3.dts > > +dtb-$(CONFIG_ARCH_ROCKCHIP) += rk3566-quartz64-a-sata.dts > > dtb-$(CONFIG_ARCH_ROCKCHIP) += rk3566-quartz64-b.dtb > > dtb-$(CONFIG_ARCH_ROCKCHIP) += rk3566-roc-pc.dtb > > dtb-$(CONFIG_ARCH_ROCKCHIP) += rk3566-soquartz-cm4.dtb > > diff --git a/arch/arm64/boot/dts/rockchip/rk3566-quartz64-a-sata.dts b/arch/arm64/boot/dts/rockchip/rk3566-quartz64-a-sata.dts > > new file mode 100644 > > index 000000000000..8620df7ec01e > > --- /dev/null > > +++ b/arch/arm64/boot/dts/rockchip/rk3566-quartz64-a-sata.dts > > @@ -0,0 +1,9 @@ > > +// SPDX-License-Identifier: (GPL-2.0+ OR MIT) > > + > > +/dts-v1/; > > + > > +#include "rk3566-quartz64-a.dtsi" > > + > > +&sata1 { > > + status = "okay"; > > +}; > > -- > > 2.34.1 > > > > > > _______________________________________________ > > Linux-rockchip mailing list > > Linux-rockchip@lists.infradead.org > > http://lists.infradead.org/mailman/listinfo/linux-rockchip > ^ permalink raw reply [flat|nested] 18+ messages in thread
* Re: [PATCH v3 3/3] arm64: dts: rockchip: k3566-quartz64-a: adds sata variant @ 2022-09-17 6:42 ` Heiko Stuebner 0 siblings, 0 replies; 18+ messages in thread From: Heiko Stuebner @ 2022-09-17 6:42 UTC (permalink / raw) To: Alessandro Carminati, Peter Geis Cc: Rob Herring, Krzysztof Kozlowski, devicetree, linux-arm-kernel, linux-rockchip, linux-kernel Hi Peter, Am Samstag, 17. September 2022, 03:40:07 CEST schrieb Peter Geis: > On Fri, Sep 16, 2022 at 12:06 PM Alessandro Carminati > <alessandro.carminati@gmail.com> wrote: > > > > The Quartz64 board is built upon Rockchip RK3566. > > Rockchip RK3566 has two combo phys. > > The first connects USB3 and SATA ctrl1, and the second PCIe lane and SATA > > ctrl2. > > The second combo phy is hardwired to the PCIe slot, where for the first, > > the hardware on the board provides both the USB3 connector and the SATA > > connector. > > This DT allows the users to switch the combo phy to the SATA connector. > > Good Evening, > > NACK to this whole series. Neither works correctly in the hardware as > is, Just for my understanding for the future, sata not working is that a bug in the soc or the board? > and USB3 was decided to be left enabled as the SATA port will be > removed completely in the next revision. That is good to know. Thanks for the heads up :-) Heiko > > Signed-off-by: Alessandro Carminati <alessandro.carminati@gmail.com> > > --- > > arch/arm64/boot/dts/rockchip/Makefile | 1 + > > arch/arm64/boot/dts/rockchip/rk3566-quartz64-a-sata.dts | 9 +++++++++ > > 2 files changed, 10 insertions(+) > > create mode 100644 arch/arm64/boot/dts/rockchip/rk3566-quartz64-a-sata.dts > > > > diff --git a/arch/arm64/boot/dts/rockchip/Makefile b/arch/arm64/boot/dts/rockchip/Makefile > > index 8c843f6fc3cc..1d5dd91d1a34 100644 > > --- a/arch/arm64/boot/dts/rockchip/Makefile > > +++ b/arch/arm64/boot/dts/rockchip/Makefile > > @@ -60,6 +60,7 @@ dtb-$(CONFIG_ARCH_ROCKCHIP) += rk3399pro-rock-pi-n10.dtb > > dtb-$(CONFIG_ARCH_ROCKCHIP) += rk3566-pinenote-v1.1.dtb > > dtb-$(CONFIG_ARCH_ROCKCHIP) += rk3566-pinenote-v1.2.dtb > > dtb-$(CONFIG_ARCH_ROCKCHIP) += rk3566-quartz64-a-usb3.dts > > +dtb-$(CONFIG_ARCH_ROCKCHIP) += rk3566-quartz64-a-sata.dts > > dtb-$(CONFIG_ARCH_ROCKCHIP) += rk3566-quartz64-b.dtb > > dtb-$(CONFIG_ARCH_ROCKCHIP) += rk3566-roc-pc.dtb > > dtb-$(CONFIG_ARCH_ROCKCHIP) += rk3566-soquartz-cm4.dtb > > diff --git a/arch/arm64/boot/dts/rockchip/rk3566-quartz64-a-sata.dts b/arch/arm64/boot/dts/rockchip/rk3566-quartz64-a-sata.dts > > new file mode 100644 > > index 000000000000..8620df7ec01e > > --- /dev/null > > +++ b/arch/arm64/boot/dts/rockchip/rk3566-quartz64-a-sata.dts > > @@ -0,0 +1,9 @@ > > +// SPDX-License-Identifier: (GPL-2.0+ OR MIT) > > + > > +/dts-v1/; > > + > > +#include "rk3566-quartz64-a.dtsi" > > + > > +&sata1 { > > + status = "okay"; > > +}; > > -- > > 2.34.1 > > > > > > _______________________________________________ > > Linux-rockchip mailing list > > Linux-rockchip@lists.infradead.org > > http://lists.infradead.org/mailman/listinfo/linux-rockchip > _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel ^ permalink raw reply [flat|nested] 18+ messages in thread
* Re: [PATCH v3 3/3] arm64: dts: rockchip: k3566-quartz64-a: adds sata variant @ 2022-09-17 6:42 ` Heiko Stuebner 0 siblings, 0 replies; 18+ messages in thread From: Heiko Stuebner @ 2022-09-17 6:42 UTC (permalink / raw) To: Alessandro Carminati, Peter Geis Cc: Rob Herring, Krzysztof Kozlowski, devicetree, linux-arm-kernel, linux-rockchip, linux-kernel Hi Peter, Am Samstag, 17. September 2022, 03:40:07 CEST schrieb Peter Geis: > On Fri, Sep 16, 2022 at 12:06 PM Alessandro Carminati > <alessandro.carminati@gmail.com> wrote: > > > > The Quartz64 board is built upon Rockchip RK3566. > > Rockchip RK3566 has two combo phys. > > The first connects USB3 and SATA ctrl1, and the second PCIe lane and SATA > > ctrl2. > > The second combo phy is hardwired to the PCIe slot, where for the first, > > the hardware on the board provides both the USB3 connector and the SATA > > connector. > > This DT allows the users to switch the combo phy to the SATA connector. > > Good Evening, > > NACK to this whole series. Neither works correctly in the hardware as > is, Just for my understanding for the future, sata not working is that a bug in the soc or the board? > and USB3 was decided to be left enabled as the SATA port will be > removed completely in the next revision. That is good to know. Thanks for the heads up :-) Heiko > > Signed-off-by: Alessandro Carminati <alessandro.carminati@gmail.com> > > --- > > arch/arm64/boot/dts/rockchip/Makefile | 1 + > > arch/arm64/boot/dts/rockchip/rk3566-quartz64-a-sata.dts | 9 +++++++++ > > 2 files changed, 10 insertions(+) > > create mode 100644 arch/arm64/boot/dts/rockchip/rk3566-quartz64-a-sata.dts > > > > diff --git a/arch/arm64/boot/dts/rockchip/Makefile b/arch/arm64/boot/dts/rockchip/Makefile > > index 8c843f6fc3cc..1d5dd91d1a34 100644 > > --- a/arch/arm64/boot/dts/rockchip/Makefile > > +++ b/arch/arm64/boot/dts/rockchip/Makefile > > @@ -60,6 +60,7 @@ dtb-$(CONFIG_ARCH_ROCKCHIP) += rk3399pro-rock-pi-n10.dtb > > dtb-$(CONFIG_ARCH_ROCKCHIP) += rk3566-pinenote-v1.1.dtb > > dtb-$(CONFIG_ARCH_ROCKCHIP) += rk3566-pinenote-v1.2.dtb > > dtb-$(CONFIG_ARCH_ROCKCHIP) += rk3566-quartz64-a-usb3.dts > > +dtb-$(CONFIG_ARCH_ROCKCHIP) += rk3566-quartz64-a-sata.dts > > dtb-$(CONFIG_ARCH_ROCKCHIP) += rk3566-quartz64-b.dtb > > dtb-$(CONFIG_ARCH_ROCKCHIP) += rk3566-roc-pc.dtb > > dtb-$(CONFIG_ARCH_ROCKCHIP) += rk3566-soquartz-cm4.dtb > > diff --git a/arch/arm64/boot/dts/rockchip/rk3566-quartz64-a-sata.dts b/arch/arm64/boot/dts/rockchip/rk3566-quartz64-a-sata.dts > > new file mode 100644 > > index 000000000000..8620df7ec01e > > --- /dev/null > > +++ b/arch/arm64/boot/dts/rockchip/rk3566-quartz64-a-sata.dts > > @@ -0,0 +1,9 @@ > > +// SPDX-License-Identifier: (GPL-2.0+ OR MIT) > > + > > +/dts-v1/; > > + > > +#include "rk3566-quartz64-a.dtsi" > > + > > +&sata1 { > > + status = "okay"; > > +}; > > -- > > 2.34.1 > > > > > > _______________________________________________ > > Linux-rockchip mailing list > > Linux-rockchip@lists.infradead.org > > http://lists.infradead.org/mailman/listinfo/linux-rockchip > _______________________________________________ Linux-rockchip mailing list Linux-rockchip@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-rockchip ^ permalink raw reply [flat|nested] 18+ messages in thread
* Re: [PATCH v3 3/3] arm64: dts: rockchip: k3566-quartz64-a: adds sata variant 2022-09-17 6:42 ` Heiko Stuebner (?) @ 2022-09-17 11:23 ` Peter Geis -1 siblings, 0 replies; 18+ messages in thread From: Peter Geis @ 2022-09-17 11:23 UTC (permalink / raw) To: Heiko Stuebner Cc: Alessandro Carminati, Rob Herring, Krzysztof Kozlowski, devicetree, linux-arm-kernel, linux-rockchip, linux-kernel On Sat, Sep 17, 2022 at 2:42 AM Heiko Stuebner <heiko@sntech.de> wrote: Good Morning Heiko, > > Hi Peter, > > Am Samstag, 17. September 2022, 03:40:07 CEST schrieb Peter Geis: > > On Fri, Sep 16, 2022 at 12:06 PM Alessandro Carminati > > <alessandro.carminati@gmail.com> wrote: > > > > > > The Quartz64 board is built upon Rockchip RK3566. > > > Rockchip RK3566 has two combo phys. > > > The first connects USB3 and SATA ctrl1, and the second PCIe lane and SATA > > > ctrl2. > > > The second combo phy is hardwired to the PCIe slot, where for the first, > > > the hardware on the board provides both the USB3 connector and the SATA > > > connector. > > > This DT allows the users to switch the combo phy to the SATA connector. > > > > Good Evening, > > > > NACK to this whole series. Neither works correctly in the hardware as > > is, > > Just for my understanding for the future, sata not working is that a bug > in the soc or the board? This is a board level problem. Attempting to build a device that had both ports electrically connected without a switch chip created a device where neither worked correctly. The SATA controllers themselves are amazing. I've used both nvme and sata m2 drives on the model b for example. > > > and USB3 was decided to be left enabled as the SATA port will be > > removed completely in the next revision. > > That is good to know. Thanks for the heads up :-) In regards to this sort of stuff in the future, we're working on fragment overlay support in U-Boot to work around the kernel's lack of support. If I remember correctly EDK2 will be implementing the switch in firmware as well. Devices that support both (at least ones I maintain) will have both in the dts, with the less likely use case left disabled. End users can simply switch which one is enabled if they want. Very Respectfully, Peter > > Heiko > > > > > Signed-off-by: Alessandro Carminati <alessandro.carminati@gmail.com> > > > --- > > > arch/arm64/boot/dts/rockchip/Makefile | 1 + > > > arch/arm64/boot/dts/rockchip/rk3566-quartz64-a-sata.dts | 9 +++++++++ > > > 2 files changed, 10 insertions(+) > > > create mode 100644 arch/arm64/boot/dts/rockchip/rk3566-quartz64-a-sata.dts > > > > > > diff --git a/arch/arm64/boot/dts/rockchip/Makefile b/arch/arm64/boot/dts/rockchip/Makefile > > > index 8c843f6fc3cc..1d5dd91d1a34 100644 > > > --- a/arch/arm64/boot/dts/rockchip/Makefile > > > +++ b/arch/arm64/boot/dts/rockchip/Makefile > > > @@ -60,6 +60,7 @@ dtb-$(CONFIG_ARCH_ROCKCHIP) += rk3399pro-rock-pi-n10.dtb > > > dtb-$(CONFIG_ARCH_ROCKCHIP) += rk3566-pinenote-v1.1.dtb > > > dtb-$(CONFIG_ARCH_ROCKCHIP) += rk3566-pinenote-v1.2.dtb > > > dtb-$(CONFIG_ARCH_ROCKCHIP) += rk3566-quartz64-a-usb3.dts > > > +dtb-$(CONFIG_ARCH_ROCKCHIP) += rk3566-quartz64-a-sata.dts > > > dtb-$(CONFIG_ARCH_ROCKCHIP) += rk3566-quartz64-b.dtb > > > dtb-$(CONFIG_ARCH_ROCKCHIP) += rk3566-roc-pc.dtb > > > dtb-$(CONFIG_ARCH_ROCKCHIP) += rk3566-soquartz-cm4.dtb > > > diff --git a/arch/arm64/boot/dts/rockchip/rk3566-quartz64-a-sata.dts b/arch/arm64/boot/dts/rockchip/rk3566-quartz64-a-sata.dts > > > new file mode 100644 > > > index 000000000000..8620df7ec01e > > > --- /dev/null > > > +++ b/arch/arm64/boot/dts/rockchip/rk3566-quartz64-a-sata.dts > > > @@ -0,0 +1,9 @@ > > > +// SPDX-License-Identifier: (GPL-2.0+ OR MIT) > > > + > > > +/dts-v1/; > > > + > > > +#include "rk3566-quartz64-a.dtsi" > > > + > > > +&sata1 { > > > + status = "okay"; > > > +}; > > > -- > > > 2.34.1 > > > > > > > > > _______________________________________________ > > > Linux-rockchip mailing list > > > Linux-rockchip@lists.infradead.org > > > http://lists.infradead.org/mailman/listinfo/linux-rockchip > > > > > > ^ permalink raw reply [flat|nested] 18+ messages in thread
* Re: [PATCH v3 3/3] arm64: dts: rockchip: k3566-quartz64-a: adds sata variant @ 2022-09-17 11:23 ` Peter Geis 0 siblings, 0 replies; 18+ messages in thread From: Peter Geis @ 2022-09-17 11:23 UTC (permalink / raw) To: Heiko Stuebner Cc: Alessandro Carminati, Rob Herring, Krzysztof Kozlowski, devicetree, linux-arm-kernel, linux-rockchip, linux-kernel On Sat, Sep 17, 2022 at 2:42 AM Heiko Stuebner <heiko@sntech.de> wrote: Good Morning Heiko, > > Hi Peter, > > Am Samstag, 17. September 2022, 03:40:07 CEST schrieb Peter Geis: > > On Fri, Sep 16, 2022 at 12:06 PM Alessandro Carminati > > <alessandro.carminati@gmail.com> wrote: > > > > > > The Quartz64 board is built upon Rockchip RK3566. > > > Rockchip RK3566 has two combo phys. > > > The first connects USB3 and SATA ctrl1, and the second PCIe lane and SATA > > > ctrl2. > > > The second combo phy is hardwired to the PCIe slot, where for the first, > > > the hardware on the board provides both the USB3 connector and the SATA > > > connector. > > > This DT allows the users to switch the combo phy to the SATA connector. > > > > Good Evening, > > > > NACK to this whole series. Neither works correctly in the hardware as > > is, > > Just for my understanding for the future, sata not working is that a bug > in the soc or the board? This is a board level problem. Attempting to build a device that had both ports electrically connected without a switch chip created a device where neither worked correctly. The SATA controllers themselves are amazing. I've used both nvme and sata m2 drives on the model b for example. > > > and USB3 was decided to be left enabled as the SATA port will be > > removed completely in the next revision. > > That is good to know. Thanks for the heads up :-) In regards to this sort of stuff in the future, we're working on fragment overlay support in U-Boot to work around the kernel's lack of support. If I remember correctly EDK2 will be implementing the switch in firmware as well. Devices that support both (at least ones I maintain) will have both in the dts, with the less likely use case left disabled. End users can simply switch which one is enabled if they want. Very Respectfully, Peter > > Heiko > > > > > Signed-off-by: Alessandro Carminati <alessandro.carminati@gmail.com> > > > --- > > > arch/arm64/boot/dts/rockchip/Makefile | 1 + > > > arch/arm64/boot/dts/rockchip/rk3566-quartz64-a-sata.dts | 9 +++++++++ > > > 2 files changed, 10 insertions(+) > > > create mode 100644 arch/arm64/boot/dts/rockchip/rk3566-quartz64-a-sata.dts > > > > > > diff --git a/arch/arm64/boot/dts/rockchip/Makefile b/arch/arm64/boot/dts/rockchip/Makefile > > > index 8c843f6fc3cc..1d5dd91d1a34 100644 > > > --- a/arch/arm64/boot/dts/rockchip/Makefile > > > +++ b/arch/arm64/boot/dts/rockchip/Makefile > > > @@ -60,6 +60,7 @@ dtb-$(CONFIG_ARCH_ROCKCHIP) += rk3399pro-rock-pi-n10.dtb > > > dtb-$(CONFIG_ARCH_ROCKCHIP) += rk3566-pinenote-v1.1.dtb > > > dtb-$(CONFIG_ARCH_ROCKCHIP) += rk3566-pinenote-v1.2.dtb > > > dtb-$(CONFIG_ARCH_ROCKCHIP) += rk3566-quartz64-a-usb3.dts > > > +dtb-$(CONFIG_ARCH_ROCKCHIP) += rk3566-quartz64-a-sata.dts > > > dtb-$(CONFIG_ARCH_ROCKCHIP) += rk3566-quartz64-b.dtb > > > dtb-$(CONFIG_ARCH_ROCKCHIP) += rk3566-roc-pc.dtb > > > dtb-$(CONFIG_ARCH_ROCKCHIP) += rk3566-soquartz-cm4.dtb > > > diff --git a/arch/arm64/boot/dts/rockchip/rk3566-quartz64-a-sata.dts b/arch/arm64/boot/dts/rockchip/rk3566-quartz64-a-sata.dts > > > new file mode 100644 > > > index 000000000000..8620df7ec01e > > > --- /dev/null > > > +++ b/arch/arm64/boot/dts/rockchip/rk3566-quartz64-a-sata.dts > > > @@ -0,0 +1,9 @@ > > > +// SPDX-License-Identifier: (GPL-2.0+ OR MIT) > > > + > > > +/dts-v1/; > > > + > > > +#include "rk3566-quartz64-a.dtsi" > > > + > > > +&sata1 { > > > + status = "okay"; > > > +}; > > > -- > > > 2.34.1 > > > > > > > > > _______________________________________________ > > > Linux-rockchip mailing list > > > Linux-rockchip@lists.infradead.org > > > http://lists.infradead.org/mailman/listinfo/linux-rockchip > > > > > > _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel ^ permalink raw reply [flat|nested] 18+ messages in thread
* Re: [PATCH v3 3/3] arm64: dts: rockchip: k3566-quartz64-a: adds sata variant @ 2022-09-17 11:23 ` Peter Geis 0 siblings, 0 replies; 18+ messages in thread From: Peter Geis @ 2022-09-17 11:23 UTC (permalink / raw) To: Heiko Stuebner Cc: Alessandro Carminati, Rob Herring, Krzysztof Kozlowski, devicetree, linux-arm-kernel, linux-rockchip, linux-kernel On Sat, Sep 17, 2022 at 2:42 AM Heiko Stuebner <heiko@sntech.de> wrote: Good Morning Heiko, > > Hi Peter, > > Am Samstag, 17. September 2022, 03:40:07 CEST schrieb Peter Geis: > > On Fri, Sep 16, 2022 at 12:06 PM Alessandro Carminati > > <alessandro.carminati@gmail.com> wrote: > > > > > > The Quartz64 board is built upon Rockchip RK3566. > > > Rockchip RK3566 has two combo phys. > > > The first connects USB3 and SATA ctrl1, and the second PCIe lane and SATA > > > ctrl2. > > > The second combo phy is hardwired to the PCIe slot, where for the first, > > > the hardware on the board provides both the USB3 connector and the SATA > > > connector. > > > This DT allows the users to switch the combo phy to the SATA connector. > > > > Good Evening, > > > > NACK to this whole series. Neither works correctly in the hardware as > > is, > > Just for my understanding for the future, sata not working is that a bug > in the soc or the board? This is a board level problem. Attempting to build a device that had both ports electrically connected without a switch chip created a device where neither worked correctly. The SATA controllers themselves are amazing. I've used both nvme and sata m2 drives on the model b for example. > > > and USB3 was decided to be left enabled as the SATA port will be > > removed completely in the next revision. > > That is good to know. Thanks for the heads up :-) In regards to this sort of stuff in the future, we're working on fragment overlay support in U-Boot to work around the kernel's lack of support. If I remember correctly EDK2 will be implementing the switch in firmware as well. Devices that support both (at least ones I maintain) will have both in the dts, with the less likely use case left disabled. End users can simply switch which one is enabled if they want. Very Respectfully, Peter > > Heiko > > > > > Signed-off-by: Alessandro Carminati <alessandro.carminati@gmail.com> > > > --- > > > arch/arm64/boot/dts/rockchip/Makefile | 1 + > > > arch/arm64/boot/dts/rockchip/rk3566-quartz64-a-sata.dts | 9 +++++++++ > > > 2 files changed, 10 insertions(+) > > > create mode 100644 arch/arm64/boot/dts/rockchip/rk3566-quartz64-a-sata.dts > > > > > > diff --git a/arch/arm64/boot/dts/rockchip/Makefile b/arch/arm64/boot/dts/rockchip/Makefile > > > index 8c843f6fc3cc..1d5dd91d1a34 100644 > > > --- a/arch/arm64/boot/dts/rockchip/Makefile > > > +++ b/arch/arm64/boot/dts/rockchip/Makefile > > > @@ -60,6 +60,7 @@ dtb-$(CONFIG_ARCH_ROCKCHIP) += rk3399pro-rock-pi-n10.dtb > > > dtb-$(CONFIG_ARCH_ROCKCHIP) += rk3566-pinenote-v1.1.dtb > > > dtb-$(CONFIG_ARCH_ROCKCHIP) += rk3566-pinenote-v1.2.dtb > > > dtb-$(CONFIG_ARCH_ROCKCHIP) += rk3566-quartz64-a-usb3.dts > > > +dtb-$(CONFIG_ARCH_ROCKCHIP) += rk3566-quartz64-a-sata.dts > > > dtb-$(CONFIG_ARCH_ROCKCHIP) += rk3566-quartz64-b.dtb > > > dtb-$(CONFIG_ARCH_ROCKCHIP) += rk3566-roc-pc.dtb > > > dtb-$(CONFIG_ARCH_ROCKCHIP) += rk3566-soquartz-cm4.dtb > > > diff --git a/arch/arm64/boot/dts/rockchip/rk3566-quartz64-a-sata.dts b/arch/arm64/boot/dts/rockchip/rk3566-quartz64-a-sata.dts > > > new file mode 100644 > > > index 000000000000..8620df7ec01e > > > --- /dev/null > > > +++ b/arch/arm64/boot/dts/rockchip/rk3566-quartz64-a-sata.dts > > > @@ -0,0 +1,9 @@ > > > +// SPDX-License-Identifier: (GPL-2.0+ OR MIT) > > > + > > > +/dts-v1/; > > > + > > > +#include "rk3566-quartz64-a.dtsi" > > > + > > > +&sata1 { > > > + status = "okay"; > > > +}; > > > -- > > > 2.34.1 > > > > > > > > > _______________________________________________ > > > Linux-rockchip mailing list > > > Linux-rockchip@lists.infradead.org > > > http://lists.infradead.org/mailman/listinfo/linux-rockchip > > > > > > _______________________________________________ Linux-rockchip mailing list Linux-rockchip@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-rockchip ^ permalink raw reply [flat|nested] 18+ messages in thread
* Re: [PATCH v3 3/3] arm64: dts: rockchip: k3566-quartz64-a: adds sata variant 2022-09-17 11:23 ` Peter Geis (?) @ 2022-09-17 13:50 ` Alessandro Carminati -1 siblings, 0 replies; 18+ messages in thread From: Alessandro Carminati @ 2022-09-17 13:50 UTC (permalink / raw) To: Peter Geis Cc: Heiko Stuebner, Rob Herring, Krzysztof Kozlowski, devicetree, linux-arm-kernel, linux-rockchip, linux-kernel Hello Peter, Thank you for the valuable details you added to this thread. If I understand correctly, the SATA controller has hardware-related issues if some electrical conditions are met by devices connected to the two SoC ports. Here an example of a faulty device layout would be helpful. But I guess this is not such common situation if you just connect a SATA device to the board. On Sat, Sep 17, 2022 at 07:23:39AM -0400, Peter Geis wrote: > On Sat, Sep 17, 2022 at 2:42 AM Heiko Stuebner <heiko@sntech.de> wrote: > > Good Morning Heiko, > > > > > > Hi Peter, > > > > Am Samstag, 17. September 2022, 03:40:07 CEST schrieb Peter Geis: > > > On Fri, Sep 16, 2022 at 12:06 PM Alessandro Carminati > > > <alessandro.carminati@gmail.com> wrote: > > > > > > > > The Quartz64 board is built upon Rockchip RK3566. > > > > Rockchip RK3566 has two combo phys. > > > > The first connects USB3 and SATA ctrl1, and the second PCIe lane and SATA > > > > ctrl2. > > > > The second combo phy is hardwired to the PCIe slot, where for the first, > > > > the hardware on the board provides both the USB3 connector and the SATA > > > > connector. > > > > This DT allows the users to switch the combo phy to the SATA connector. > > > > > > Good Evening, > > > > > > NACK to this whole series. Neither works correctly in the hardware as > > > is, > > > > Just for my understanding for the future, sata not working is that a bug > > in the soc or the board? > > This is a board level problem. Attempting to build a device that had > both ports electrically connected without a switch chip created a > device where neither worked correctly. The SATA controllers themselves > are amazing. I've used both nvme and sata m2 drives on the model b for > example. > > > > > > and USB3 was decided to be left enabled as the SATA port will be > > > removed completely in the next revision. > > > > That is good to know. Thanks for the heads up :-) > > In regards to this sort of stuff in the future, we're working on > fragment overlay support in U-Boot to work around the kernel's lack of > support. If I remember correctly EDK2 will be implementing the switch > in firmware as well. Devices that support both (at least ones I > maintain) will have both in the dts, with the less likely use case > left disabled. End users can simply switch which one is enabled if > they want. Reading through your message, I have the impression that you are trying to solve the problem on the firmware side. I want to express my admiration for this effort. I think that this is the right approach to solve this kind of problem, and that the more appropriate place to be for device trees is on the firmware and not in the kernel. Currently, the kernel includes a considerable amount of device trees, and the Quarttz64-a device tree is already upstream. As I understand, there's currently an effort to standardize the already existing device trees and give direction to the newcomers. In a recent interaction with Krzysztof Kozlowski, I learned the already existing device trees are likely not to respond to these regulations. Sooner or later, each upstream device tree will need to be adjusted, and the currently upstreamed quartz64-a DTS is one of these. I understand you are working on the u-boot side, possibly the EDK2. They alone are more than 80% of all the firmware running at this moment, but there's still a non-neglectable number of boots that use something else. All these words to say: * Krzysztof confirmed the upstreamed device tree for the quartz64 needs to be adjusted to meet the device trees node name regulation. * The work needed to add the SATA support is minimal. * Having this SATA DTS is not completely useless since numerous SATA configurations work smoothly. I am willing to work on this patch to make it suitable to be upstreamed. Regards Alessandro > > Very Respectfully, > Peter > > > > > Heiko > > > > > > > > Signed-off-by: Alessandro Carminati <alessandro.carminati@gmail.com> > > > > --- > > > > arch/arm64/boot/dts/rockchip/Makefile | 1 + > > > > arch/arm64/boot/dts/rockchip/rk3566-quartz64-a-sata.dts | 9 +++++++++ > > > > 2 files changed, 10 insertions(+) > > > > create mode 100644 arch/arm64/boot/dts/rockchip/rk3566-quartz64-a-sata.dts > > > > > > > > diff --git a/arch/arm64/boot/dts/rockchip/Makefile b/arch/arm64/boot/dts/rockchip/Makefile > > > > index 8c843f6fc3cc..1d5dd91d1a34 100644 > > > > --- a/arch/arm64/boot/dts/rockchip/Makefile > > > > +++ b/arch/arm64/boot/dts/rockchip/Makefile > > > > @@ -60,6 +60,7 @@ dtb-$(CONFIG_ARCH_ROCKCHIP) += rk3399pro-rock-pi-n10.dtb > > > > dtb-$(CONFIG_ARCH_ROCKCHIP) += rk3566-pinenote-v1.1.dtb > > > > dtb-$(CONFIG_ARCH_ROCKCHIP) += rk3566-pinenote-v1.2.dtb > > > > dtb-$(CONFIG_ARCH_ROCKCHIP) += rk3566-quartz64-a-usb3.dts > > > > +dtb-$(CONFIG_ARCH_ROCKCHIP) += rk3566-quartz64-a-sata.dts > > > > dtb-$(CONFIG_ARCH_ROCKCHIP) += rk3566-quartz64-b.dtb > > > > dtb-$(CONFIG_ARCH_ROCKCHIP) += rk3566-roc-pc.dtb > > > > dtb-$(CONFIG_ARCH_ROCKCHIP) += rk3566-soquartz-cm4.dtb > > > > diff --git a/arch/arm64/boot/dts/rockchip/rk3566-quartz64-a-sata.dts b/arch/arm64/boot/dts/rockchip/rk3566-quartz64-a-sata.dts > > > > new file mode 100644 > > > > index 000000000000..8620df7ec01e > > > > --- /dev/null > > > > +++ b/arch/arm64/boot/dts/rockchip/rk3566-quartz64-a-sata.dts > > > > @@ -0,0 +1,9 @@ > > > > +// SPDX-License-Identifier: (GPL-2.0+ OR MIT) > > > > + > > > > +/dts-v1/; > > > > + > > > > +#include "rk3566-quartz64-a.dtsi" > > > > + > > > > +&sata1 { > > > > + status = "okay"; > > > > +}; > > > > -- > > > > 2.34.1 > > > > > > > > > > > > _______________________________________________ > > > > Linux-rockchip mailing list > > > > Linux-rockchip@lists.infradead.org > > > > http://lists.infradead.org/mailman/listinfo/linux-rockchip > > > > > > > > > > > ^ permalink raw reply [flat|nested] 18+ messages in thread
* Re: [PATCH v3 3/3] arm64: dts: rockchip: k3566-quartz64-a: adds sata variant @ 2022-09-17 13:50 ` Alessandro Carminati 0 siblings, 0 replies; 18+ messages in thread From: Alessandro Carminati @ 2022-09-17 13:50 UTC (permalink / raw) To: Peter Geis Cc: Heiko Stuebner, Rob Herring, Krzysztof Kozlowski, devicetree, linux-arm-kernel, linux-rockchip, linux-kernel Hello Peter, Thank you for the valuable details you added to this thread. If I understand correctly, the SATA controller has hardware-related issues if some electrical conditions are met by devices connected to the two SoC ports. Here an example of a faulty device layout would be helpful. But I guess this is not such common situation if you just connect a SATA device to the board. On Sat, Sep 17, 2022 at 07:23:39AM -0400, Peter Geis wrote: > On Sat, Sep 17, 2022 at 2:42 AM Heiko Stuebner <heiko@sntech.de> wrote: > > Good Morning Heiko, > > > > > > Hi Peter, > > > > Am Samstag, 17. September 2022, 03:40:07 CEST schrieb Peter Geis: > > > On Fri, Sep 16, 2022 at 12:06 PM Alessandro Carminati > > > <alessandro.carminati@gmail.com> wrote: > > > > > > > > The Quartz64 board is built upon Rockchip RK3566. > > > > Rockchip RK3566 has two combo phys. > > > > The first connects USB3 and SATA ctrl1, and the second PCIe lane and SATA > > > > ctrl2. > > > > The second combo phy is hardwired to the PCIe slot, where for the first, > > > > the hardware on the board provides both the USB3 connector and the SATA > > > > connector. > > > > This DT allows the users to switch the combo phy to the SATA connector. > > > > > > Good Evening, > > > > > > NACK to this whole series. Neither works correctly in the hardware as > > > is, > > > > Just for my understanding for the future, sata not working is that a bug > > in the soc or the board? > > This is a board level problem. Attempting to build a device that had > both ports electrically connected without a switch chip created a > device where neither worked correctly. The SATA controllers themselves > are amazing. I've used both nvme and sata m2 drives on the model b for > example. > > > > > > and USB3 was decided to be left enabled as the SATA port will be > > > removed completely in the next revision. > > > > That is good to know. Thanks for the heads up :-) > > In regards to this sort of stuff in the future, we're working on > fragment overlay support in U-Boot to work around the kernel's lack of > support. If I remember correctly EDK2 will be implementing the switch > in firmware as well. Devices that support both (at least ones I > maintain) will have both in the dts, with the less likely use case > left disabled. End users can simply switch which one is enabled if > they want. Reading through your message, I have the impression that you are trying to solve the problem on the firmware side. I want to express my admiration for this effort. I think that this is the right approach to solve this kind of problem, and that the more appropriate place to be for device trees is on the firmware and not in the kernel. Currently, the kernel includes a considerable amount of device trees, and the Quarttz64-a device tree is already upstream. As I understand, there's currently an effort to standardize the already existing device trees and give direction to the newcomers. In a recent interaction with Krzysztof Kozlowski, I learned the already existing device trees are likely not to respond to these regulations. Sooner or later, each upstream device tree will need to be adjusted, and the currently upstreamed quartz64-a DTS is one of these. I understand you are working on the u-boot side, possibly the EDK2. They alone are more than 80% of all the firmware running at this moment, but there's still a non-neglectable number of boots that use something else. All these words to say: * Krzysztof confirmed the upstreamed device tree for the quartz64 needs to be adjusted to meet the device trees node name regulation. * The work needed to add the SATA support is minimal. * Having this SATA DTS is not completely useless since numerous SATA configurations work smoothly. I am willing to work on this patch to make it suitable to be upstreamed. Regards Alessandro > > Very Respectfully, > Peter > > > > > Heiko > > > > > > > > Signed-off-by: Alessandro Carminati <alessandro.carminati@gmail.com> > > > > --- > > > > arch/arm64/boot/dts/rockchip/Makefile | 1 + > > > > arch/arm64/boot/dts/rockchip/rk3566-quartz64-a-sata.dts | 9 +++++++++ > > > > 2 files changed, 10 insertions(+) > > > > create mode 100644 arch/arm64/boot/dts/rockchip/rk3566-quartz64-a-sata.dts > > > > > > > > diff --git a/arch/arm64/boot/dts/rockchip/Makefile b/arch/arm64/boot/dts/rockchip/Makefile > > > > index 8c843f6fc3cc..1d5dd91d1a34 100644 > > > > --- a/arch/arm64/boot/dts/rockchip/Makefile > > > > +++ b/arch/arm64/boot/dts/rockchip/Makefile > > > > @@ -60,6 +60,7 @@ dtb-$(CONFIG_ARCH_ROCKCHIP) += rk3399pro-rock-pi-n10.dtb > > > > dtb-$(CONFIG_ARCH_ROCKCHIP) += rk3566-pinenote-v1.1.dtb > > > > dtb-$(CONFIG_ARCH_ROCKCHIP) += rk3566-pinenote-v1.2.dtb > > > > dtb-$(CONFIG_ARCH_ROCKCHIP) += rk3566-quartz64-a-usb3.dts > > > > +dtb-$(CONFIG_ARCH_ROCKCHIP) += rk3566-quartz64-a-sata.dts > > > > dtb-$(CONFIG_ARCH_ROCKCHIP) += rk3566-quartz64-b.dtb > > > > dtb-$(CONFIG_ARCH_ROCKCHIP) += rk3566-roc-pc.dtb > > > > dtb-$(CONFIG_ARCH_ROCKCHIP) += rk3566-soquartz-cm4.dtb > > > > diff --git a/arch/arm64/boot/dts/rockchip/rk3566-quartz64-a-sata.dts b/arch/arm64/boot/dts/rockchip/rk3566-quartz64-a-sata.dts > > > > new file mode 100644 > > > > index 000000000000..8620df7ec01e > > > > --- /dev/null > > > > +++ b/arch/arm64/boot/dts/rockchip/rk3566-quartz64-a-sata.dts > > > > @@ -0,0 +1,9 @@ > > > > +// SPDX-License-Identifier: (GPL-2.0+ OR MIT) > > > > + > > > > +/dts-v1/; > > > > + > > > > +#include "rk3566-quartz64-a.dtsi" > > > > + > > > > +&sata1 { > > > > + status = "okay"; > > > > +}; > > > > -- > > > > 2.34.1 > > > > > > > > > > > > _______________________________________________ > > > > Linux-rockchip mailing list > > > > Linux-rockchip@lists.infradead.org > > > > http://lists.infradead.org/mailman/listinfo/linux-rockchip > > > > > > > > > > > _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel ^ permalink raw reply [flat|nested] 18+ messages in thread
* Re: [PATCH v3 3/3] arm64: dts: rockchip: k3566-quartz64-a: adds sata variant @ 2022-09-17 13:50 ` Alessandro Carminati 0 siblings, 0 replies; 18+ messages in thread From: Alessandro Carminati @ 2022-09-17 13:50 UTC (permalink / raw) To: Peter Geis Cc: Heiko Stuebner, Rob Herring, Krzysztof Kozlowski, devicetree, linux-arm-kernel, linux-rockchip, linux-kernel Hello Peter, Thank you for the valuable details you added to this thread. If I understand correctly, the SATA controller has hardware-related issues if some electrical conditions are met by devices connected to the two SoC ports. Here an example of a faulty device layout would be helpful. But I guess this is not such common situation if you just connect a SATA device to the board. On Sat, Sep 17, 2022 at 07:23:39AM -0400, Peter Geis wrote: > On Sat, Sep 17, 2022 at 2:42 AM Heiko Stuebner <heiko@sntech.de> wrote: > > Good Morning Heiko, > > > > > > Hi Peter, > > > > Am Samstag, 17. September 2022, 03:40:07 CEST schrieb Peter Geis: > > > On Fri, Sep 16, 2022 at 12:06 PM Alessandro Carminati > > > <alessandro.carminati@gmail.com> wrote: > > > > > > > > The Quartz64 board is built upon Rockchip RK3566. > > > > Rockchip RK3566 has two combo phys. > > > > The first connects USB3 and SATA ctrl1, and the second PCIe lane and SATA > > > > ctrl2. > > > > The second combo phy is hardwired to the PCIe slot, where for the first, > > > > the hardware on the board provides both the USB3 connector and the SATA > > > > connector. > > > > This DT allows the users to switch the combo phy to the SATA connector. > > > > > > Good Evening, > > > > > > NACK to this whole series. Neither works correctly in the hardware as > > > is, > > > > Just for my understanding for the future, sata not working is that a bug > > in the soc or the board? > > This is a board level problem. Attempting to build a device that had > both ports electrically connected without a switch chip created a > device where neither worked correctly. The SATA controllers themselves > are amazing. I've used both nvme and sata m2 drives on the model b for > example. > > > > > > and USB3 was decided to be left enabled as the SATA port will be > > > removed completely in the next revision. > > > > That is good to know. Thanks for the heads up :-) > > In regards to this sort of stuff in the future, we're working on > fragment overlay support in U-Boot to work around the kernel's lack of > support. If I remember correctly EDK2 will be implementing the switch > in firmware as well. Devices that support both (at least ones I > maintain) will have both in the dts, with the less likely use case > left disabled. End users can simply switch which one is enabled if > they want. Reading through your message, I have the impression that you are trying to solve the problem on the firmware side. I want to express my admiration for this effort. I think that this is the right approach to solve this kind of problem, and that the more appropriate place to be for device trees is on the firmware and not in the kernel. Currently, the kernel includes a considerable amount of device trees, and the Quarttz64-a device tree is already upstream. As I understand, there's currently an effort to standardize the already existing device trees and give direction to the newcomers. In a recent interaction with Krzysztof Kozlowski, I learned the already existing device trees are likely not to respond to these regulations. Sooner or later, each upstream device tree will need to be adjusted, and the currently upstreamed quartz64-a DTS is one of these. I understand you are working on the u-boot side, possibly the EDK2. They alone are more than 80% of all the firmware running at this moment, but there's still a non-neglectable number of boots that use something else. All these words to say: * Krzysztof confirmed the upstreamed device tree for the quartz64 needs to be adjusted to meet the device trees node name regulation. * The work needed to add the SATA support is minimal. * Having this SATA DTS is not completely useless since numerous SATA configurations work smoothly. I am willing to work on this patch to make it suitable to be upstreamed. Regards Alessandro > > Very Respectfully, > Peter > > > > > Heiko > > > > > > > > Signed-off-by: Alessandro Carminati <alessandro.carminati@gmail.com> > > > > --- > > > > arch/arm64/boot/dts/rockchip/Makefile | 1 + > > > > arch/arm64/boot/dts/rockchip/rk3566-quartz64-a-sata.dts | 9 +++++++++ > > > > 2 files changed, 10 insertions(+) > > > > create mode 100644 arch/arm64/boot/dts/rockchip/rk3566-quartz64-a-sata.dts > > > > > > > > diff --git a/arch/arm64/boot/dts/rockchip/Makefile b/arch/arm64/boot/dts/rockchip/Makefile > > > > index 8c843f6fc3cc..1d5dd91d1a34 100644 > > > > --- a/arch/arm64/boot/dts/rockchip/Makefile > > > > +++ b/arch/arm64/boot/dts/rockchip/Makefile > > > > @@ -60,6 +60,7 @@ dtb-$(CONFIG_ARCH_ROCKCHIP) += rk3399pro-rock-pi-n10.dtb > > > > dtb-$(CONFIG_ARCH_ROCKCHIP) += rk3566-pinenote-v1.1.dtb > > > > dtb-$(CONFIG_ARCH_ROCKCHIP) += rk3566-pinenote-v1.2.dtb > > > > dtb-$(CONFIG_ARCH_ROCKCHIP) += rk3566-quartz64-a-usb3.dts > > > > +dtb-$(CONFIG_ARCH_ROCKCHIP) += rk3566-quartz64-a-sata.dts > > > > dtb-$(CONFIG_ARCH_ROCKCHIP) += rk3566-quartz64-b.dtb > > > > dtb-$(CONFIG_ARCH_ROCKCHIP) += rk3566-roc-pc.dtb > > > > dtb-$(CONFIG_ARCH_ROCKCHIP) += rk3566-soquartz-cm4.dtb > > > > diff --git a/arch/arm64/boot/dts/rockchip/rk3566-quartz64-a-sata.dts b/arch/arm64/boot/dts/rockchip/rk3566-quartz64-a-sata.dts > > > > new file mode 100644 > > > > index 000000000000..8620df7ec01e > > > > --- /dev/null > > > > +++ b/arch/arm64/boot/dts/rockchip/rk3566-quartz64-a-sata.dts > > > > @@ -0,0 +1,9 @@ > > > > +// SPDX-License-Identifier: (GPL-2.0+ OR MIT) > > > > + > > > > +/dts-v1/; > > > > + > > > > +#include "rk3566-quartz64-a.dtsi" > > > > + > > > > +&sata1 { > > > > + status = "okay"; > > > > +}; > > > > -- > > > > 2.34.1 > > > > > > > > > > > > _______________________________________________ > > > > Linux-rockchip mailing list > > > > Linux-rockchip@lists.infradead.org > > > > http://lists.infradead.org/mailman/listinfo/linux-rockchip > > > > > > > > > > > _______________________________________________ Linux-rockchip mailing list Linux-rockchip@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-rockchip ^ permalink raw reply [flat|nested] 18+ messages in thread
* Re: [PATCH v3 3/3] arm64: dts: rockchip: k3566-quartz64-a: adds sata variant 2022-09-17 13:50 ` Alessandro Carminati (?) @ 2022-09-18 12:40 ` Peter Geis -1 siblings, 0 replies; 18+ messages in thread From: Peter Geis @ 2022-09-18 12:40 UTC (permalink / raw) To: Alessandro Carminati Cc: Heiko Stuebner, Rob Herring, Krzysztof Kozlowski, devicetree, linux-arm-kernel, linux-rockchip, linux-kernel On Sat, Sep 17, 2022 at 9:54 AM Alessandro Carminati <alessandro.carminati@gmail.com> wrote: > > Hello Peter, Good Morning, > > Thank you for the valuable details you added to this thread. > > If I understand correctly, the SATA controller has hardware-related > issues if some electrical conditions are met by devices connected to > the two SoC ports. No, the issue stems from having both ports electrically connected in parallel. This creates impedance and capacitance issues that cause ringing on the data lines when either is used. This sort of design would work perfectly fine if a gpio controlled high speed data switch was installed to isolate the ports. The USB3 port can be restored to near full functionality by permanently destroying the SATA port, but there's no way to easily do something similar to the USB3 port in order to fix SATA. > > Here an example of a faulty device layout would be helpful. > > But I guess this is not such common situation if you just connect a > SATA device to the board. > > > > On Sat, Sep 17, 2022 at 07:23:39AM -0400, Peter Geis wrote: > > On Sat, Sep 17, 2022 at 2:42 AM Heiko Stuebner <heiko@sntech.de> wrote: > > > > Good Morning Heiko, > > > > > > > > > > Hi Peter, > > > > > > Am Samstag, 17. September 2022, 03:40:07 CEST schrieb Peter Geis: > > > > On Fri, Sep 16, 2022 at 12:06 PM Alessandro Carminati > > > > <alessandro.carminati@gmail.com> wrote: > > > > > > > > > > The Quartz64 board is built upon Rockchip RK3566. > > > > > Rockchip RK3566 has two combo phys. > > > > > The first connects USB3 and SATA ctrl1, and the second PCIe lane and SATA > > > > > ctrl2. > > > > > The second combo phy is hardwired to the PCIe slot, where for the first, > > > > > the hardware on the board provides both the USB3 connector and the SATA > > > > > connector. > > > > > This DT allows the users to switch the combo phy to the SATA connector. > > > > > > > > Good Evening, > > > > > > > > NACK to this whole series. Neither works correctly in the hardware as > > > > is, > > > > > > Just for my understanding for the future, sata not working is that a bug > > > in the soc or the board? > > > > This is a board level problem. Attempting to build a device that had > > both ports electrically connected without a switch chip created a > > device where neither worked correctly. The SATA controllers themselves > > are amazing. I've used both nvme and sata m2 drives on the model b for > > example. > > > > > > > > > and USB3 was decided to be left enabled as the SATA port will be > > > > removed completely in the next revision. > > > > > > That is good to know. Thanks for the heads up :-) > > > > In regards to this sort of stuff in the future, we're working on > > fragment overlay support in U-Boot to work around the kernel's lack of > > support. If I remember correctly EDK2 will be implementing the switch > > in firmware as well. Devices that support both (at least ones I > > maintain) will have both in the dts, with the less likely use case > > left disabled. End users can simply switch which one is enabled if > > they want. > > Reading through your message, I have the impression that you are trying > to solve the problem on the firmware side. > I want to express my admiration for this effort. > I think that this is the right approach to solve this kind of problem, > and that the more appropriate place to be for device trees is on the > firmware and not in the kernel. > Currently, the kernel includes a considerable amount of device trees, > and the Quarttz64-a device tree is already upstream. The problem we are trying to solve in firmware is the end user switching modes problem, not a device specific problem. The issue with the SATA/USB port is a board level design problem that cannot be solved in software. > > As I understand, there's currently an effort to standardize the already > existing device trees and give direction to the newcomers. > In a recent interaction with Krzysztof Kozlowski, I learned the already > existing device trees are likely not to respond to these regulations. This is simply not true. As the dt-bindings are improved blanket updates are being applied to the existing trees. As of 5.19, Quartz64-A had no dt-bindings check failures that didn't stem from legacy dt-bindings that haven't been updated to yaml. I am aware of the fixed-regulator not enforcing _regulator as a tail, as well as the other deficiencies with the Quartz64-A dts. They are on my long list of things to work on once I'm done moving. > > Sooner or later, each upstream device tree will need to be adjusted, > and the currently upstreamed quartz64-a DTS is one of these. > > I understand you are working on the u-boot side, possibly the EDK2. > They alone are more than 80% of all the firmware running at this moment, > but there's still a non-neglectable number of boots that use something > else. Aside from EDK2, pretty much every other boot mechanism for rk356x is either U-Boot or something based on U-Boot. Any fixes we apply to U-Boot can easily trickle down to the others. EDK2 on the other hand renders the dts moot as for the time being everything will be using acpi and generic drivers. > > All these words to say: > > * Krzysztof confirmed the upstreamed device tree for the quartz64 needs > to be adjusted to meet the device trees node name regulation. The dt-bindings for fixed-regulator should be fixed as part of this. I certainly wouldn't complain about that being done standalone. > > * The work needed to add the SATA support is minimal. This adds a second dts configuration to an already growing folder, and will force a third configuration when the final hardware revision that *isn't fundamentally broken* lands. If you want to add the SATA port disabled to both phys I would be alright with that, as they work great with simple conversion cables on every device I've tested. For the record, my main problem with this series is the physical SATA port will not exist in the final hardware revision of the Quartz64-A. > > * Having this SATA DTS is not completely useless since numerous SATA > configurations work smoothly. There are many configurations that do not downshift correctly as well and require manual intervention to work. Unfortunately you won't be the one answering the angry end users that don't understand this situation. > > I am willing to work on this patch to make it suitable to be upstreamed. There is significantly more work that needs to happen in order for rk356x (and rk3588 when the time comes) to be first class citizens. For example, almost everyone booting rk356x is either using some variation of the awful bsp u-boot or my hacked together mainline u-boot. The edk2 port shows a lot of promise, but it's not even close to fully functional. I won't complain about assistance with these endeavors either. Very Respectfully, Peter > > Regards > Alessandro > > > > > Very Respectfully, > > Peter > > > > > > > > Heiko > > > > > > > > > > > Signed-off-by: Alessandro Carminati <alessandro.carminati@gmail.com> > > > > > --- > > > > > arch/arm64/boot/dts/rockchip/Makefile | 1 + > > > > > arch/arm64/boot/dts/rockchip/rk3566-quartz64-a-sata.dts | 9 +++++++++ > > > > > 2 files changed, 10 insertions(+) > > > > > create mode 100644 arch/arm64/boot/dts/rockchip/rk3566-quartz64-a-sata.dts > > > > > > > > > > diff --git a/arch/arm64/boot/dts/rockchip/Makefile b/arch/arm64/boot/dts/rockchip/Makefile > > > > > index 8c843f6fc3cc..1d5dd91d1a34 100644 > > > > > --- a/arch/arm64/boot/dts/rockchip/Makefile > > > > > +++ b/arch/arm64/boot/dts/rockchip/Makefile > > > > > @@ -60,6 +60,7 @@ dtb-$(CONFIG_ARCH_ROCKCHIP) += rk3399pro-rock-pi-n10.dtb > > > > > dtb-$(CONFIG_ARCH_ROCKCHIP) += rk3566-pinenote-v1.1.dtb > > > > > dtb-$(CONFIG_ARCH_ROCKCHIP) += rk3566-pinenote-v1.2.dtb > > > > > dtb-$(CONFIG_ARCH_ROCKCHIP) += rk3566-quartz64-a-usb3.dts > > > > > +dtb-$(CONFIG_ARCH_ROCKCHIP) += rk3566-quartz64-a-sata.dts > > > > > dtb-$(CONFIG_ARCH_ROCKCHIP) += rk3566-quartz64-b.dtb > > > > > dtb-$(CONFIG_ARCH_ROCKCHIP) += rk3566-roc-pc.dtb > > > > > dtb-$(CONFIG_ARCH_ROCKCHIP) += rk3566-soquartz-cm4.dtb > > > > > diff --git a/arch/arm64/boot/dts/rockchip/rk3566-quartz64-a-sata.dts b/arch/arm64/boot/dts/rockchip/rk3566-quartz64-a-sata.dts > > > > > new file mode 100644 > > > > > index 000000000000..8620df7ec01e > > > > > --- /dev/null > > > > > +++ b/arch/arm64/boot/dts/rockchip/rk3566-quartz64-a-sata.dts > > > > > @@ -0,0 +1,9 @@ > > > > > +// SPDX-License-Identifier: (GPL-2.0+ OR MIT) > > > > > + > > > > > +/dts-v1/; > > > > > + > > > > > +#include "rk3566-quartz64-a.dtsi" > > > > > + > > > > > +&sata1 { > > > > > + status = "okay"; > > > > > +}; > > > > > -- > > > > > 2.34.1 > > > > > > > > > > > > > > > _______________________________________________ > > > > > Linux-rockchip mailing list > > > > > Linux-rockchip@lists.infradead.org > > > > > http://lists.infradead.org/mailman/listinfo/linux-rockchip > > > > > > > > > > > > > > > > ^ permalink raw reply [flat|nested] 18+ messages in thread
* Re: [PATCH v3 3/3] arm64: dts: rockchip: k3566-quartz64-a: adds sata variant @ 2022-09-18 12:40 ` Peter Geis 0 siblings, 0 replies; 18+ messages in thread From: Peter Geis @ 2022-09-18 12:40 UTC (permalink / raw) To: Alessandro Carminati Cc: Heiko Stuebner, Rob Herring, Krzysztof Kozlowski, devicetree, linux-arm-kernel, linux-rockchip, linux-kernel On Sat, Sep 17, 2022 at 9:54 AM Alessandro Carminati <alessandro.carminati@gmail.com> wrote: > > Hello Peter, Good Morning, > > Thank you for the valuable details you added to this thread. > > If I understand correctly, the SATA controller has hardware-related > issues if some electrical conditions are met by devices connected to > the two SoC ports. No, the issue stems from having both ports electrically connected in parallel. This creates impedance and capacitance issues that cause ringing on the data lines when either is used. This sort of design would work perfectly fine if a gpio controlled high speed data switch was installed to isolate the ports. The USB3 port can be restored to near full functionality by permanently destroying the SATA port, but there's no way to easily do something similar to the USB3 port in order to fix SATA. > > Here an example of a faulty device layout would be helpful. > > But I guess this is not such common situation if you just connect a > SATA device to the board. > > > > On Sat, Sep 17, 2022 at 07:23:39AM -0400, Peter Geis wrote: > > On Sat, Sep 17, 2022 at 2:42 AM Heiko Stuebner <heiko@sntech.de> wrote: > > > > Good Morning Heiko, > > > > > > > > > > Hi Peter, > > > > > > Am Samstag, 17. September 2022, 03:40:07 CEST schrieb Peter Geis: > > > > On Fri, Sep 16, 2022 at 12:06 PM Alessandro Carminati > > > > <alessandro.carminati@gmail.com> wrote: > > > > > > > > > > The Quartz64 board is built upon Rockchip RK3566. > > > > > Rockchip RK3566 has two combo phys. > > > > > The first connects USB3 and SATA ctrl1, and the second PCIe lane and SATA > > > > > ctrl2. > > > > > The second combo phy is hardwired to the PCIe slot, where for the first, > > > > > the hardware on the board provides both the USB3 connector and the SATA > > > > > connector. > > > > > This DT allows the users to switch the combo phy to the SATA connector. > > > > > > > > Good Evening, > > > > > > > > NACK to this whole series. Neither works correctly in the hardware as > > > > is, > > > > > > Just for my understanding for the future, sata not working is that a bug > > > in the soc or the board? > > > > This is a board level problem. Attempting to build a device that had > > both ports electrically connected without a switch chip created a > > device where neither worked correctly. The SATA controllers themselves > > are amazing. I've used both nvme and sata m2 drives on the model b for > > example. > > > > > > > > > and USB3 was decided to be left enabled as the SATA port will be > > > > removed completely in the next revision. > > > > > > That is good to know. Thanks for the heads up :-) > > > > In regards to this sort of stuff in the future, we're working on > > fragment overlay support in U-Boot to work around the kernel's lack of > > support. If I remember correctly EDK2 will be implementing the switch > > in firmware as well. Devices that support both (at least ones I > > maintain) will have both in the dts, with the less likely use case > > left disabled. End users can simply switch which one is enabled if > > they want. > > Reading through your message, I have the impression that you are trying > to solve the problem on the firmware side. > I want to express my admiration for this effort. > I think that this is the right approach to solve this kind of problem, > and that the more appropriate place to be for device trees is on the > firmware and not in the kernel. > Currently, the kernel includes a considerable amount of device trees, > and the Quarttz64-a device tree is already upstream. The problem we are trying to solve in firmware is the end user switching modes problem, not a device specific problem. The issue with the SATA/USB port is a board level design problem that cannot be solved in software. > > As I understand, there's currently an effort to standardize the already > existing device trees and give direction to the newcomers. > In a recent interaction with Krzysztof Kozlowski, I learned the already > existing device trees are likely not to respond to these regulations. This is simply not true. As the dt-bindings are improved blanket updates are being applied to the existing trees. As of 5.19, Quartz64-A had no dt-bindings check failures that didn't stem from legacy dt-bindings that haven't been updated to yaml. I am aware of the fixed-regulator not enforcing _regulator as a tail, as well as the other deficiencies with the Quartz64-A dts. They are on my long list of things to work on once I'm done moving. > > Sooner or later, each upstream device tree will need to be adjusted, > and the currently upstreamed quartz64-a DTS is one of these. > > I understand you are working on the u-boot side, possibly the EDK2. > They alone are more than 80% of all the firmware running at this moment, > but there's still a non-neglectable number of boots that use something > else. Aside from EDK2, pretty much every other boot mechanism for rk356x is either U-Boot or something based on U-Boot. Any fixes we apply to U-Boot can easily trickle down to the others. EDK2 on the other hand renders the dts moot as for the time being everything will be using acpi and generic drivers. > > All these words to say: > > * Krzysztof confirmed the upstreamed device tree for the quartz64 needs > to be adjusted to meet the device trees node name regulation. The dt-bindings for fixed-regulator should be fixed as part of this. I certainly wouldn't complain about that being done standalone. > > * The work needed to add the SATA support is minimal. This adds a second dts configuration to an already growing folder, and will force a third configuration when the final hardware revision that *isn't fundamentally broken* lands. If you want to add the SATA port disabled to both phys I would be alright with that, as they work great with simple conversion cables on every device I've tested. For the record, my main problem with this series is the physical SATA port will not exist in the final hardware revision of the Quartz64-A. > > * Having this SATA DTS is not completely useless since numerous SATA > configurations work smoothly. There are many configurations that do not downshift correctly as well and require manual intervention to work. Unfortunately you won't be the one answering the angry end users that don't understand this situation. > > I am willing to work on this patch to make it suitable to be upstreamed. There is significantly more work that needs to happen in order for rk356x (and rk3588 when the time comes) to be first class citizens. For example, almost everyone booting rk356x is either using some variation of the awful bsp u-boot or my hacked together mainline u-boot. The edk2 port shows a lot of promise, but it's not even close to fully functional. I won't complain about assistance with these endeavors either. Very Respectfully, Peter > > Regards > Alessandro > > > > > Very Respectfully, > > Peter > > > > > > > > Heiko > > > > > > > > > > > Signed-off-by: Alessandro Carminati <alessandro.carminati@gmail.com> > > > > > --- > > > > > arch/arm64/boot/dts/rockchip/Makefile | 1 + > > > > > arch/arm64/boot/dts/rockchip/rk3566-quartz64-a-sata.dts | 9 +++++++++ > > > > > 2 files changed, 10 insertions(+) > > > > > create mode 100644 arch/arm64/boot/dts/rockchip/rk3566-quartz64-a-sata.dts > > > > > > > > > > diff --git a/arch/arm64/boot/dts/rockchip/Makefile b/arch/arm64/boot/dts/rockchip/Makefile > > > > > index 8c843f6fc3cc..1d5dd91d1a34 100644 > > > > > --- a/arch/arm64/boot/dts/rockchip/Makefile > > > > > +++ b/arch/arm64/boot/dts/rockchip/Makefile > > > > > @@ -60,6 +60,7 @@ dtb-$(CONFIG_ARCH_ROCKCHIP) += rk3399pro-rock-pi-n10.dtb > > > > > dtb-$(CONFIG_ARCH_ROCKCHIP) += rk3566-pinenote-v1.1.dtb > > > > > dtb-$(CONFIG_ARCH_ROCKCHIP) += rk3566-pinenote-v1.2.dtb > > > > > dtb-$(CONFIG_ARCH_ROCKCHIP) += rk3566-quartz64-a-usb3.dts > > > > > +dtb-$(CONFIG_ARCH_ROCKCHIP) += rk3566-quartz64-a-sata.dts > > > > > dtb-$(CONFIG_ARCH_ROCKCHIP) += rk3566-quartz64-b.dtb > > > > > dtb-$(CONFIG_ARCH_ROCKCHIP) += rk3566-roc-pc.dtb > > > > > dtb-$(CONFIG_ARCH_ROCKCHIP) += rk3566-soquartz-cm4.dtb > > > > > diff --git a/arch/arm64/boot/dts/rockchip/rk3566-quartz64-a-sata.dts b/arch/arm64/boot/dts/rockchip/rk3566-quartz64-a-sata.dts > > > > > new file mode 100644 > > > > > index 000000000000..8620df7ec01e > > > > > --- /dev/null > > > > > +++ b/arch/arm64/boot/dts/rockchip/rk3566-quartz64-a-sata.dts > > > > > @@ -0,0 +1,9 @@ > > > > > +// SPDX-License-Identifier: (GPL-2.0+ OR MIT) > > > > > + > > > > > +/dts-v1/; > > > > > + > > > > > +#include "rk3566-quartz64-a.dtsi" > > > > > + > > > > > +&sata1 { > > > > > + status = "okay"; > > > > > +}; > > > > > -- > > > > > 2.34.1 > > > > > > > > > > > > > > > _______________________________________________ > > > > > Linux-rockchip mailing list > > > > > Linux-rockchip@lists.infradead.org > > > > > http://lists.infradead.org/mailman/listinfo/linux-rockchip > > > > > > > > > > > > > > > > _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel ^ permalink raw reply [flat|nested] 18+ messages in thread
* Re: [PATCH v3 3/3] arm64: dts: rockchip: k3566-quartz64-a: adds sata variant @ 2022-09-18 12:40 ` Peter Geis 0 siblings, 0 replies; 18+ messages in thread From: Peter Geis @ 2022-09-18 12:40 UTC (permalink / raw) To: Alessandro Carminati Cc: Heiko Stuebner, Rob Herring, Krzysztof Kozlowski, devicetree, linux-arm-kernel, linux-rockchip, linux-kernel On Sat, Sep 17, 2022 at 9:54 AM Alessandro Carminati <alessandro.carminati@gmail.com> wrote: > > Hello Peter, Good Morning, > > Thank you for the valuable details you added to this thread. > > If I understand correctly, the SATA controller has hardware-related > issues if some electrical conditions are met by devices connected to > the two SoC ports. No, the issue stems from having both ports electrically connected in parallel. This creates impedance and capacitance issues that cause ringing on the data lines when either is used. This sort of design would work perfectly fine if a gpio controlled high speed data switch was installed to isolate the ports. The USB3 port can be restored to near full functionality by permanently destroying the SATA port, but there's no way to easily do something similar to the USB3 port in order to fix SATA. > > Here an example of a faulty device layout would be helpful. > > But I guess this is not such common situation if you just connect a > SATA device to the board. > > > > On Sat, Sep 17, 2022 at 07:23:39AM -0400, Peter Geis wrote: > > On Sat, Sep 17, 2022 at 2:42 AM Heiko Stuebner <heiko@sntech.de> wrote: > > > > Good Morning Heiko, > > > > > > > > > > Hi Peter, > > > > > > Am Samstag, 17. September 2022, 03:40:07 CEST schrieb Peter Geis: > > > > On Fri, Sep 16, 2022 at 12:06 PM Alessandro Carminati > > > > <alessandro.carminati@gmail.com> wrote: > > > > > > > > > > The Quartz64 board is built upon Rockchip RK3566. > > > > > Rockchip RK3566 has two combo phys. > > > > > The first connects USB3 and SATA ctrl1, and the second PCIe lane and SATA > > > > > ctrl2. > > > > > The second combo phy is hardwired to the PCIe slot, where for the first, > > > > > the hardware on the board provides both the USB3 connector and the SATA > > > > > connector. > > > > > This DT allows the users to switch the combo phy to the SATA connector. > > > > > > > > Good Evening, > > > > > > > > NACK to this whole series. Neither works correctly in the hardware as > > > > is, > > > > > > Just for my understanding for the future, sata not working is that a bug > > > in the soc or the board? > > > > This is a board level problem. Attempting to build a device that had > > both ports electrically connected without a switch chip created a > > device where neither worked correctly. The SATA controllers themselves > > are amazing. I've used both nvme and sata m2 drives on the model b for > > example. > > > > > > > > > and USB3 was decided to be left enabled as the SATA port will be > > > > removed completely in the next revision. > > > > > > That is good to know. Thanks for the heads up :-) > > > > In regards to this sort of stuff in the future, we're working on > > fragment overlay support in U-Boot to work around the kernel's lack of > > support. If I remember correctly EDK2 will be implementing the switch > > in firmware as well. Devices that support both (at least ones I > > maintain) will have both in the dts, with the less likely use case > > left disabled. End users can simply switch which one is enabled if > > they want. > > Reading through your message, I have the impression that you are trying > to solve the problem on the firmware side. > I want to express my admiration for this effort. > I think that this is the right approach to solve this kind of problem, > and that the more appropriate place to be for device trees is on the > firmware and not in the kernel. > Currently, the kernel includes a considerable amount of device trees, > and the Quarttz64-a device tree is already upstream. The problem we are trying to solve in firmware is the end user switching modes problem, not a device specific problem. The issue with the SATA/USB port is a board level design problem that cannot be solved in software. > > As I understand, there's currently an effort to standardize the already > existing device trees and give direction to the newcomers. > In a recent interaction with Krzysztof Kozlowski, I learned the already > existing device trees are likely not to respond to these regulations. This is simply not true. As the dt-bindings are improved blanket updates are being applied to the existing trees. As of 5.19, Quartz64-A had no dt-bindings check failures that didn't stem from legacy dt-bindings that haven't been updated to yaml. I am aware of the fixed-regulator not enforcing _regulator as a tail, as well as the other deficiencies with the Quartz64-A dts. They are on my long list of things to work on once I'm done moving. > > Sooner or later, each upstream device tree will need to be adjusted, > and the currently upstreamed quartz64-a DTS is one of these. > > I understand you are working on the u-boot side, possibly the EDK2. > They alone are more than 80% of all the firmware running at this moment, > but there's still a non-neglectable number of boots that use something > else. Aside from EDK2, pretty much every other boot mechanism for rk356x is either U-Boot or something based on U-Boot. Any fixes we apply to U-Boot can easily trickle down to the others. EDK2 on the other hand renders the dts moot as for the time being everything will be using acpi and generic drivers. > > All these words to say: > > * Krzysztof confirmed the upstreamed device tree for the quartz64 needs > to be adjusted to meet the device trees node name regulation. The dt-bindings for fixed-regulator should be fixed as part of this. I certainly wouldn't complain about that being done standalone. > > * The work needed to add the SATA support is minimal. This adds a second dts configuration to an already growing folder, and will force a third configuration when the final hardware revision that *isn't fundamentally broken* lands. If you want to add the SATA port disabled to both phys I would be alright with that, as they work great with simple conversion cables on every device I've tested. For the record, my main problem with this series is the physical SATA port will not exist in the final hardware revision of the Quartz64-A. > > * Having this SATA DTS is not completely useless since numerous SATA > configurations work smoothly. There are many configurations that do not downshift correctly as well and require manual intervention to work. Unfortunately you won't be the one answering the angry end users that don't understand this situation. > > I am willing to work on this patch to make it suitable to be upstreamed. There is significantly more work that needs to happen in order for rk356x (and rk3588 when the time comes) to be first class citizens. For example, almost everyone booting rk356x is either using some variation of the awful bsp u-boot or my hacked together mainline u-boot. The edk2 port shows a lot of promise, but it's not even close to fully functional. I won't complain about assistance with these endeavors either. Very Respectfully, Peter > > Regards > Alessandro > > > > > Very Respectfully, > > Peter > > > > > > > > Heiko > > > > > > > > > > > Signed-off-by: Alessandro Carminati <alessandro.carminati@gmail.com> > > > > > --- > > > > > arch/arm64/boot/dts/rockchip/Makefile | 1 + > > > > > arch/arm64/boot/dts/rockchip/rk3566-quartz64-a-sata.dts | 9 +++++++++ > > > > > 2 files changed, 10 insertions(+) > > > > > create mode 100644 arch/arm64/boot/dts/rockchip/rk3566-quartz64-a-sata.dts > > > > > > > > > > diff --git a/arch/arm64/boot/dts/rockchip/Makefile b/arch/arm64/boot/dts/rockchip/Makefile > > > > > index 8c843f6fc3cc..1d5dd91d1a34 100644 > > > > > --- a/arch/arm64/boot/dts/rockchip/Makefile > > > > > +++ b/arch/arm64/boot/dts/rockchip/Makefile > > > > > @@ -60,6 +60,7 @@ dtb-$(CONFIG_ARCH_ROCKCHIP) += rk3399pro-rock-pi-n10.dtb > > > > > dtb-$(CONFIG_ARCH_ROCKCHIP) += rk3566-pinenote-v1.1.dtb > > > > > dtb-$(CONFIG_ARCH_ROCKCHIP) += rk3566-pinenote-v1.2.dtb > > > > > dtb-$(CONFIG_ARCH_ROCKCHIP) += rk3566-quartz64-a-usb3.dts > > > > > +dtb-$(CONFIG_ARCH_ROCKCHIP) += rk3566-quartz64-a-sata.dts > > > > > dtb-$(CONFIG_ARCH_ROCKCHIP) += rk3566-quartz64-b.dtb > > > > > dtb-$(CONFIG_ARCH_ROCKCHIP) += rk3566-roc-pc.dtb > > > > > dtb-$(CONFIG_ARCH_ROCKCHIP) += rk3566-soquartz-cm4.dtb > > > > > diff --git a/arch/arm64/boot/dts/rockchip/rk3566-quartz64-a-sata.dts b/arch/arm64/boot/dts/rockchip/rk3566-quartz64-a-sata.dts > > > > > new file mode 100644 > > > > > index 000000000000..8620df7ec01e > > > > > --- /dev/null > > > > > +++ b/arch/arm64/boot/dts/rockchip/rk3566-quartz64-a-sata.dts > > > > > @@ -0,0 +1,9 @@ > > > > > +// SPDX-License-Identifier: (GPL-2.0+ OR MIT) > > > > > + > > > > > +/dts-v1/; > > > > > + > > > > > +#include "rk3566-quartz64-a.dtsi" > > > > > + > > > > > +&sata1 { > > > > > + status = "okay"; > > > > > +}; > > > > > -- > > > > > 2.34.1 > > > > > > > > > > > > > > > _______________________________________________ > > > > > Linux-rockchip mailing list > > > > > Linux-rockchip@lists.infradead.org > > > > > http://lists.infradead.org/mailman/listinfo/linux-rockchip > > > > > > > > > > > > > > > > _______________________________________________ Linux-rockchip mailing list Linux-rockchip@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-rockchip ^ permalink raw reply [flat|nested] 18+ messages in thread
end of thread, other threads:[~2022-09-18 12:42 UTC | newest] Thread overview: 18+ messages (download: mbox.gz / follow: Atom feed) -- links below jump to the message on this page -- 2022-09-16 16:00 [PATCH v3 3/3] arm64: dts: rockchip: k3566-quartz64-a: adds sata variant Alessandro Carminati 2022-09-16 16:00 ` Alessandro Carminati 2022-09-16 16:00 ` Alessandro Carminati 2022-09-17 1:40 ` Peter Geis 2022-09-17 1:40 ` Peter Geis 2022-09-17 1:40 ` Peter Geis 2022-09-17 6:42 ` Heiko Stuebner 2022-09-17 6:42 ` Heiko Stuebner 2022-09-17 6:42 ` Heiko Stuebner 2022-09-17 11:23 ` Peter Geis 2022-09-17 11:23 ` Peter Geis 2022-09-17 11:23 ` Peter Geis 2022-09-17 13:50 ` Alessandro Carminati 2022-09-17 13:50 ` Alessandro Carminati 2022-09-17 13:50 ` Alessandro Carminati 2022-09-18 12:40 ` Peter Geis 2022-09-18 12:40 ` Peter Geis 2022-09-18 12:40 ` Peter Geis
This is an external index of several public inboxes, see mirroring instructions on how to clone and mirror all data and code used by this external index.