* [PATCH v6 1/3] MIPS: DTS: CI20: add DT node for IR sensor
2020-03-06 17:29 [PATCH v6 0/3] MIPS: Fixes and improvements for CI20 board (JZ4780) H. Nikolaus Schaller
@ 2020-03-06 17:29 ` H. Nikolaus Schaller
2020-03-06 18:26 ` Paul Cercueil
2020-03-06 17:29 ` [PATCH v6 2/3] MIPS: DTS: CI20: multiple DTS improvements H. Nikolaus Schaller
2020-03-06 17:29 ` [PATCH v6 3/3] MIPS: CI20: defconfig: multiple improvements H. Nikolaus Schaller
2 siblings, 1 reply; 5+ messages in thread
From: H. Nikolaus Schaller @ 2020-03-06 17:29 UTC (permalink / raw)
To: Paul Cercueil, Rob Herring, Mark Rutland, Thomas Bogendoerfer,
H. Nikolaus Schaller, Geert Uytterhoeven, Kees Cook,
Eric W. Biederman, Miquel Raynal
Cc: devicetree, linux-mips, linux-kernel, letux-kernel, Alex Smith
From: Alex Smith <alex.smith@imgtec.com>
The infrared sensor on the CI20 board is connected to a GPIO and can
be operated by using the gpio-ir-recv driver. Add a DT node for the
sensor to allow that driver to be used.
Signed-off-by: Alex Smith <alex.smith@imgtec.com>
Signed-off-by: H. Nikolaus Schaller <hns@goldelico.com>
---
arch/mips/boot/dts/ingenic/ci20.dts | 5 +++++
1 file changed, 5 insertions(+)
diff --git a/arch/mips/boot/dts/ingenic/ci20.dts b/arch/mips/boot/dts/ingenic/ci20.dts
index c340f947baa0..e6ae08c979ea 100644
--- a/arch/mips/boot/dts/ingenic/ci20.dts
+++ b/arch/mips/boot/dts/ingenic/ci20.dts
@@ -62,6 +62,11 @@
enable-active-high;
};
+ ir: ir-receiver {
+ compatible = "gpio-ir-receiver";
+ gpios = <&gpe 3 GPIO_ACTIVE_LOW>;
+ };
+
wlan0_power: fixedregulator@1 {
compatible = "regulator-fixed";
regulator-name = "wlan0_power";
--
2.23.0
^ permalink raw reply related [flat|nested] 5+ messages in thread
* Re: [PATCH v6 1/3] MIPS: DTS: CI20: add DT node for IR sensor
2020-03-06 17:29 ` [PATCH v6 1/3] MIPS: DTS: CI20: add DT node for IR sensor H. Nikolaus Schaller
@ 2020-03-06 18:26 ` Paul Cercueil
0 siblings, 0 replies; 5+ messages in thread
From: Paul Cercueil @ 2020-03-06 18:26 UTC (permalink / raw)
To: H. Nikolaus Schaller
Cc: Rob Herring, Mark Rutland, Thomas Bogendoerfer,
Geert Uytterhoeven, Kees Cook, Eric W. Biederman, Miquel Raynal,
devicetree, linux-mips, linux-kernel, letux-kernel, Alex Smith
Hi Nikolaus,
Le ven., mars 6, 2020 at 18:29, H. Nikolaus Schaller
<hns@goldelico.com> a écrit :
> From: Alex Smith <alex.smith@imgtec.com>
>
> The infrared sensor on the CI20 board is connected to a GPIO and can
> be operated by using the gpio-ir-recv driver. Add a DT node for the
> sensor to allow that driver to be used.
>
> Signed-off-by: Alex Smith <alex.smith@imgtec.com>
> Signed-off-by: H. Nikolaus Schaller <hns@goldelico.com>
> ---
> arch/mips/boot/dts/ingenic/ci20.dts | 5 +++++
> 1 file changed, 5 insertions(+)
>
> diff --git a/arch/mips/boot/dts/ingenic/ci20.dts
> b/arch/mips/boot/dts/ingenic/ci20.dts
> index c340f947baa0..e6ae08c979ea 100644
> --- a/arch/mips/boot/dts/ingenic/ci20.dts
> +++ b/arch/mips/boot/dts/ingenic/ci20.dts
> @@ -62,6 +62,11 @@
> enable-active-high;
> };
>
> + ir: ir-receiver {
According to Documentation/devicetree/bindings/media/rc.yaml, the node
should be named just "ir".
Cheers,
-Paul
> + compatible = "gpio-ir-receiver";
> + gpios = <&gpe 3 GPIO_ACTIVE_LOW>;
> + };
> +
> wlan0_power: fixedregulator@1 {
> compatible = "regulator-fixed";
> regulator-name = "wlan0_power";
> --
> 2.23.0
>
^ permalink raw reply [flat|nested] 5+ messages in thread
* [PATCH v6 2/3] MIPS: DTS: CI20: multiple DTS improvements
2020-03-06 17:29 [PATCH v6 0/3] MIPS: Fixes and improvements for CI20 board (JZ4780) H. Nikolaus Schaller
2020-03-06 17:29 ` [PATCH v6 1/3] MIPS: DTS: CI20: add DT node for IR sensor H. Nikolaus Schaller
@ 2020-03-06 17:29 ` H. Nikolaus Schaller
2020-03-06 17:29 ` [PATCH v6 3/3] MIPS: CI20: defconfig: multiple improvements H. Nikolaus Schaller
2 siblings, 0 replies; 5+ messages in thread
From: H. Nikolaus Schaller @ 2020-03-06 17:29 UTC (permalink / raw)
To: Paul Cercueil, Rob Herring, Mark Rutland, Thomas Bogendoerfer,
H. Nikolaus Schaller, Geert Uytterhoeven, Kees Cook,
Eric W. Biederman, Miquel Raynal
Cc: devicetree, linux-mips, linux-kernel, letux-kernel
a) add DT node for SW1 as Enter button
The SW1 button can be used as a simple one-button keyboard
and is connected to PD17.
Note: SW1 has a second meaning to change the boot sequence
when pressed while powering on.
b) give eth0_power a defined voltage.
This is a 3.3V power switch (DVNET3.3V).
Signed-off-by: H. Nikolaus Schaller <hns@goldelico.com>
Reviewed-by: Paul Cercueil <paul@crapouillou.net>
---
arch/mips/boot/dts/ingenic/ci20.dts | 14 ++++++++++++++
1 file changed, 14 insertions(+)
diff --git a/arch/mips/boot/dts/ingenic/ci20.dts b/arch/mips/boot/dts/ingenic/ci20.dts
index e6ae08c979ea..a18c7650c489 100644
--- a/arch/mips/boot/dts/ingenic/ci20.dts
+++ b/arch/mips/boot/dts/ingenic/ci20.dts
@@ -4,6 +4,7 @@
#include "jz4780.dtsi"
#include <dt-bindings/clock/ingenic,tcu.h>
#include <dt-bindings/gpio/gpio.h>
+#include <dt-bindings/input/input.h>
#include <dt-bindings/interrupt-controller/irq.h>
#include <dt-bindings/regulator/active-semi,8865-regulator.h>
@@ -27,6 +28,17 @@
0x30000000 0x30000000>;
};
+ gpio-keys {
+ compatible = "gpio-keys";
+
+ sw1 {
+ label = "ci20:sw1";
+ linux,code = <KEY_F13>;
+ gpios = <&gpd 17 GPIO_ACTIVE_HIGH>;
+ wakeup-source;
+ };
+ };
+
leds {
compatible = "gpio-leds";
@@ -58,6 +70,8 @@
eth0_power: fixedregulator@0 {
compatible = "regulator-fixed";
regulator-name = "eth0_power";
+ regulator-min-microvolt = <3300000>;
+ regulator-max-microvolt = <3300000>;
gpio = <&gpb 25 GPIO_ACTIVE_LOW>;
enable-active-high;
};
--
2.23.0
^ permalink raw reply related [flat|nested] 5+ messages in thread
* [PATCH v6 3/3] MIPS: CI20: defconfig: multiple improvements
2020-03-06 17:29 [PATCH v6 0/3] MIPS: Fixes and improvements for CI20 board (JZ4780) H. Nikolaus Schaller
2020-03-06 17:29 ` [PATCH v6 1/3] MIPS: DTS: CI20: add DT node for IR sensor H. Nikolaus Schaller
2020-03-06 17:29 ` [PATCH v6 2/3] MIPS: DTS: CI20: multiple DTS improvements H. Nikolaus Schaller
@ 2020-03-06 17:29 ` H. Nikolaus Schaller
2 siblings, 0 replies; 5+ messages in thread
From: H. Nikolaus Schaller @ 2020-03-06 17:29 UTC (permalink / raw)
To: Paul Cercueil, Rob Herring, Mark Rutland, Thomas Bogendoerfer,
H. Nikolaus Schaller, Geert Uytterhoeven, Kees Cook,
Eric W. Biederman, Miquel Raynal
Cc: devicetree, linux-mips, linux-kernel, letux-kernel
a) configure for supporting modules
Not all drivers need to be compiled into the kernel.
Support building and loading of kernel modules.
b) compile leds-gpio driver into the kernel and configure for LED triggers
DTS has been augmented to add some gpio-leds. We need the leds-gpio driver
and enable the triggers.
c) configure CONFIG_REGULATOR_ACT8865 for PMU
The PMU on the CI20 board is an ACT8600 using the ACT8865 driver.
Since it is not compiled, the PMU and the CI20 board is running in
power-on reset state of the PMU.
d) compile gpio-ir driver
The CI20 board has a gpio based IR receiver.
e) configure for CONFIG_KEYBOARD_GPIO=m
The SW1 button is hooked up to send input events.
Signed-off-by: H. Nikolaus Schaller <hns@goldelico.com>
Reviewed-by: Paul Cercueil <paul@crapouillou.net>
---
arch/mips/configs/ci20_defconfig | 21 +++++++++++++++++++++
1 file changed, 21 insertions(+)
diff --git a/arch/mips/configs/ci20_defconfig b/arch/mips/configs/ci20_defconfig
index be41df2a81fb..0db0088bbc1c 100644
--- a/arch/mips/configs/ci20_defconfig
+++ b/arch/mips/configs/ci20_defconfig
@@ -1,4 +1,5 @@
# CONFIG_LOCALVERSION_AUTO is not set
+CONFIG_MODULES=y
CONFIG_KERNEL_XZ=y
CONFIG_SYSVIPC=y
CONFIG_POSIX_MQUEUE=y
@@ -88,12 +89,14 @@ CONFIG_I2C_JZ4780=y
CONFIG_SPI=y
CONFIG_SPI_GPIO=y
CONFIG_GPIO_SYSFS=y
+CONFIG_KEYBOARD_GPIO=m
# CONFIG_HWMON is not set
CONFIG_WATCHDOG=y
CONFIG_JZ4740_WDT=y
CONFIG_REGULATOR=y
CONFIG_REGULATOR_DEBUG=y
CONFIG_REGULATOR_FIXED_VOLTAGE=y
+CONFIG_REGULATOR_ACT8865=y
# CONFIG_VGA_CONSOLE is not set
# CONFIG_HID is not set
# CONFIG_USB_SUPPORT is not set
@@ -166,3 +169,21 @@ CONFIG_STACKTRACE=y
# CONFIG_FTRACE is not set
CONFIG_CMDLINE_BOOL=y
CONFIG_CMDLINE="earlycon console=ttyS4,115200 clk_ignore_unused"
+CONFIG_LEDS_CLASS=y
+CONFIG_LEDS_GPIO=y
+CONFIG_LEDS_TRIGGERS=y
+CONFIG_LEDS_TRIGGER_MTD=y
+CONFIG_LEDS_TRIGGER_TIMER=y
+CONFIG_LEDS_TRIGGER_ONESHOT=y
+CONFIG_LEDS_TRIGGER_ONESHOT=y
+CONFIG_LEDS_TRIGGER_HEARTBEAT=y
+CONFIG_LEDS_TRIGGER_BACKLIGHT=m
+CONFIG_LEDS_TRIGGER_CPU=y
+CONFIG_LEDS_TRIGGER_DEFAULT_ON=y
+CONFIG_LEDS_TRIGGER_TRANSIENT=y
+CONFIG_LEDS_TRIGGER_CAMERA=m
+CONFIG_LIRC=y
+CONFIG_MEDIA_SUPPORT=m
+CONFIG_RC_DEVICES=y
+CONFIG_IR_GPIO_CIR=m
+CONFIG_IR_GPIO_TX=m
--
2.23.0
^ permalink raw reply related [flat|nested] 5+ messages in thread