Re: [PATCH v2 2/3] ARM: dts: stm32: enable l3gd20 on stm32429-disco board

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 





On 5/14/20 9:07 AM, dillon min wrote:
Hi Alexandre,

Alexandre Torgue <alexandre.torgue@xxxxxx> 于2020年5月14日周四 下午10:10写道:

Hi Dillon

On 5/12/20 9:36 AM, dillon.minfei@xxxxxxxxx wrote:
From: dillon min <dillon.minfei@xxxxxxxxx>

L3gd20, st mems motion sensor, 3-axis digital output gyroscope,
connect to stm32f429 via spi5

Signed-off-by: dillon min <dillon.minfei@xxxxxxxxx>
---

Hi Alexandre,

V2:
      1, insert blank line at stm32f420-disco.dts line 143
      2, add more description about l3gd20 in commit message

V1:
      enable l3gd20 dts binding on stm32f429-disco

thanks.

dillon,

   arch/arm/boot/dts/stm32f429-disco.dts | 25 +++++++++++++++++++++++++
   1 file changed, 25 insertions(+)

diff --git a/arch/arm/boot/dts/stm32f429-disco.dts b/arch/arm/boot/dts/stm32f429-disco.dts
index 30c0f67..1bfb903 100644
--- a/arch/arm/boot/dts/stm32f429-disco.dts
+++ b/arch/arm/boot/dts/stm32f429-disco.dts
@@ -49,6 +49,8 @@
   #include "stm32f429.dtsi"
   #include "stm32f429-pinctrl.dtsi"
   #include <dt-bindings/input/input.h>
+#include <dt-bindings/interrupt-controller/irq.h>
+#include <dt-bindings/gpio/gpio.h>

   / {
       model = "STMicroelectronics STM32F429i-DISCO board";
@@ -127,3 +129,26 @@
       pinctrl-names = "default";
       status = "okay";
   };
+
+&spi5 {
+     status = "okay";
+     pinctrl-0 = <&spi5_pins>;
+     pinctrl-names = "default";
+     #address-cells = <1>;
+     #size-cells = <0>;
+     cs-gpios = <&gpioc 1 GPIO_ACTIVE_LOW>;
+     dmas = <&dma2 3 2 0x400 0x0>,
+            <&dma2 4 2 0x400 0x0>;
+     dma-names = "rx", "tx";
+

You added this spi5 node in this patch but also in the display series. I
will have issue to merge. Even if I could fix it easily, as you are
going to resend, the good practice could be to have several patches in
one series: one patch for spi5 controller, another for gyro and another
for display.

And also same remark than Linus did in display series move DMA to soc
dtsi file please.

Sure, how about this patch sequence:
1 add spi5 dma to soc (stm32f429.dtsi)
2 add pin map for spi5 (stm32f4-pinctrl.dtsi)
3 add spi5 controller with gyro (stm32f429-disco.dts)
4 add spi modification to support gyro (spi-stm32.c)

5 add ltdc pin map for stm32f429-disco board (stm32f4-pinctrl.dtsi)
6 add ilitek-ili9341 dts bindings for disco (stm32f429-disco.dts,
depends on above step 3)
7 add yaml document about ilitek-ili9341 (ilitek,ili9341.yaml)
8 add clk changes to support ltdc driver (clk-stm32f4.c)

so, i will combine gyro and display two patches to one patchset next
time. right ?


looks good

thanks.


+     l3gd20: l3gd20@0 {
+             compatible = "st,l3gd20-gyro";
+             spi-max-frequency = <10000000>;
+             st,drdy-int-pin = <2>;
+             interrupt-parent = <&gpioa>;
+             interrupts = <1 IRQ_TYPE_EDGE_RISING>,
+                             <2 IRQ_TYPE_EDGE_RISING>;
+             reg = <0>;
+             status = "okay";
+     };
+};




[Index of Archives]     [Device Tree Compilter]     [Device Tree Spec]     [Linux Driver Backports]     [Video for Linux]     [Linux USB Devel]     [Linux PCI Devel]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [XFree86]     [Yosemite Backpacking]


  Powered by Linux