* [PATCH v7 1/3] MIPS: DTS: CI20: add DT node for IR sensor
2020-03-06 21:06 [PATCH v7 0/3] MIPS: Fixes and improvements for CI20 board (JZ4780) H. Nikolaus Schaller
@ 2020-03-06 21:06 ` H. Nikolaus Schaller
2020-03-06 22:09 ` Paul Cercueil
2020-03-06 21:06 ` [PATCH v7 2/3] MIPS: DTS: CI20: multiple DTS improvements H. Nikolaus Schaller
` (2 subsequent siblings)
3 siblings, 1 reply; 6+ messages in thread
From: H. Nikolaus Schaller @ 2020-03-06 21:06 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..fc4e64200c3d 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 {
+ 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] 6+ messages in thread* Re: [PATCH v7 1/3] MIPS: DTS: CI20: add DT node for IR sensor
2020-03-06 21:06 ` [PATCH v7 1/3] MIPS: DTS: CI20: add DT node for IR sensor H. Nikolaus Schaller
@ 2020-03-06 22:09 ` Paul Cercueil
0 siblings, 0 replies; 6+ messages in thread
From: Paul Cercueil @ 2020-03-06 22:09 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 22:06, 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>
Reviewed-by: Paul Cercueil <paul@crapouillou.net>
Cheers,
-Paul
> ---
> 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..fc4e64200c3d 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 {
> + 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] 6+ messages in thread
* [PATCH v7 2/3] MIPS: DTS: CI20: multiple DTS improvements
2020-03-06 21:06 [PATCH v7 0/3] MIPS: Fixes and improvements for CI20 board (JZ4780) H. Nikolaus Schaller
2020-03-06 21:06 ` [PATCH v7 1/3] MIPS: DTS: CI20: add DT node for IR sensor H. Nikolaus Schaller
@ 2020-03-06 21:06 ` H. Nikolaus Schaller
2020-03-06 21:06 ` [PATCH v7 3/3] MIPS: CI20: defconfig: multiple improvements H. Nikolaus Schaller
2020-03-23 14:46 ` [PATCH v7 0/3] MIPS: Fixes and improvements for CI20 board (JZ4780) Thomas Bogendoerfer
3 siblings, 0 replies; 6+ messages in thread
From: H. Nikolaus Schaller @ 2020-03-06 21:06 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 fc4e64200c3d..db0ca250bd1a 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] 6+ messages in thread* [PATCH v7 3/3] MIPS: CI20: defconfig: multiple improvements
2020-03-06 21:06 [PATCH v7 0/3] MIPS: Fixes and improvements for CI20 board (JZ4780) H. Nikolaus Schaller
2020-03-06 21:06 ` [PATCH v7 1/3] MIPS: DTS: CI20: add DT node for IR sensor H. Nikolaus Schaller
2020-03-06 21:06 ` [PATCH v7 2/3] MIPS: DTS: CI20: multiple DTS improvements H. Nikolaus Schaller
@ 2020-03-06 21:06 ` H. Nikolaus Schaller
2020-03-23 14:46 ` [PATCH v7 0/3] MIPS: Fixes and improvements for CI20 board (JZ4780) Thomas Bogendoerfer
3 siblings, 0 replies; 6+ messages in thread
From: H. Nikolaus Schaller @ 2020-03-06 21:06 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] 6+ messages in thread
* Re: [PATCH v7 0/3] MIPS: Fixes and improvements for CI20 board (JZ4780)
2020-03-06 21:06 [PATCH v7 0/3] MIPS: Fixes and improvements for CI20 board (JZ4780) H. Nikolaus Schaller
` (2 preceding siblings ...)
2020-03-06 21:06 ` [PATCH v7 3/3] MIPS: CI20: defconfig: multiple improvements H. Nikolaus Schaller
@ 2020-03-23 14:46 ` Thomas Bogendoerfer
3 siblings, 0 replies; 6+ messages in thread
From: Thomas Bogendoerfer @ 2020-03-23 14:46 UTC (permalink / raw)
To: H. Nikolaus Schaller
Cc: Paul Cercueil, Rob Herring, Mark Rutland, Geert Uytterhoeven,
Kees Cook, Eric W. Biederman, Miquel Raynal, devicetree,
linux-mips, linux-kernel, letux-kernel, Paul Boddie
On Fri, Mar 06, 2020 at 10:06:30PM +0100, H. Nikolaus Schaller wrote:
>
> Signed-off-by: Paul Boddie <paul@boddie.org.uk>
> Signed-off-by: H. Nikolaus Schaller <hns@goldelico.com>
>
>
> Alex Smith (1):
> MIPS: DTS: CI20: add DT node for IR sensor
>
> H. Nikolaus Schaller (2):
> MIPS: DTS: CI20: multiple DTS improvements
> MIPS: CI20: defconfig: multiple improvements
>
> arch/mips/boot/dts/ingenic/ci20.dts | 19 +++++++++++++++++++
> arch/mips/configs/ci20_defconfig | 21 +++++++++++++++++++++
> 2 files changed, 40 insertions(+)
series applied to mips-next.
Thomas.
--
Crap can work. Given enough thrust pigs will fly, but it's not necessarily a
good idea. [ RFC1925, 2.3 ]
^ permalink raw reply [flat|nested] 6+ messages in thread