Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]


Groups > linux.kernel > #1498653 > unrolled thread

[PATCH] n900 device tree: cleanup, add camera flash

Started byPavel Machek <pavel@ucw.cz>
First post2016-10-11 10:10 +0200
Last post2016-10-11 10:20 +0200
Articles 4 — 2 participants

Back to article view | Back to linux.kernel


Contents

  [PATCH] n900 device tree: cleanup, add camera flash Pavel Machek <pavel@ucw.cz> - 2016-10-11 10:10 +0200
    Re: [PATCH] n900 device tree: cleanup, add camera flash Pali Rohár <pali.rohar@gmail.com> - 2016-10-11 10:10 +0200
      Re: [PATCH] n900 device tree: cleanup, add camera flash Pavel Machek <pavel@ucw.cz> - 2016-10-11 10:20 +0200
    [PATCH] n900 device tree: cleanup Pavel Machek <pavel@ucw.cz> - 2016-10-11 10:20 +0200

#1498653 — [PATCH] n900 device tree: cleanup, add camera flash

FromPavel Machek <pavel@ucw.cz>
Date2016-10-11 10:10 +0200
Subject[PATCH] n900 device tree: cleanup, add camera flash
Message-ID<sr0il-49R-13@gated-at.bofh.it>

[Multipart message — attachments visible in raw view] — view raw

Fix GPIO comment to be consistent with rest of file, add comment what
tpa6130 is, and addd support for adp1653 camera flash.

Signed-off-by: Pavel Machek <pavel@ucw.cz>

diff --git a/arch/arm/boot/dts/omap3-n900.dts b/arch/arm/boot/dts/omap3-n900.dts
index bfffd6c..ca9fe8c 100644
--- a/arch/arm/boot/dts/omap3-n900.dts
+++ b/arch/arm/boot/dts/omap3-n900.dts
@@ -47,7 +47,7 @@
 		compatible = "gpio-leds";
 		heartbeat {
 			label = "debug::sleep";
-			gpios = <&gpio6 2 GPIO_ACTIVE_HIGH>;  /* gpio162 */
+			gpios = <&gpio6 2 GPIO_ACTIVE_HIGH>;  /* 162 */
 			linux,default-trigger = "default-on";
 			pinctrl-names = "default";
 			pinctrl-0 = <&debug_leds>;
@@ -637,6 +637,7 @@
 		reg = <0x55>;
 	};
 
+	/* Stereo headphone amplifier */
 	tpa6130a2: tpa6130a2@60 {
 		compatible = "ti,tpa6130a2";
 		reg = <0x60>;
@@ -669,6 +670,22 @@
 
 		ti,usb-charger-detection = <&isp1707>;
 	};
+
+	adp1653: led-controller@30 {
+		compatible = "adi,adp1653";
+		reg = <0x30>;
+		gpios = <&gpio3 24 GPIO_ACTIVE_HIGH>; /* 88 */
+
+		flash {
+			flash-timeout-us = <500000>;
+			flash-max-microamp = <320000>;
+			max-microamp = <50000>;
+		};
+
+		indicator {
+			max-microamp = <17500>;
+		};
+	};
 };
 
 &i2c3 {

-- 
(english) http://www.livejournal.com/~pavelmachek
(cesky, pictures) http://atrey.karlin.mff.cuni.cz/~pavel/picture/horses/blog.html

[toc] | [next] | [standalone]


#1498654

FromPali Rohár <pali.rohar@gmail.com>
Date2016-10-11 10:10 +0200
Message-ID<sr0im-49R-21@gated-at.bofh.it>
In reply to#1498653
On Tuesday 11 October 2016 09:54:17 Pavel Machek wrote:
> @@ -669,6 +670,22 @@
>  
>  		ti,usb-charger-detection = <&isp1707>;
>  	};
> +
> +	adp1653: led-controller@30 {
> +		compatible = "adi,adp1653";
> +		reg = <0x30>;
> +		gpios = <&gpio3 24 GPIO_ACTIVE_HIGH>; /* 88 */
> +
> +		flash {
> +			flash-timeout-us = <500000>;
> +			flash-max-microamp = <320000>;
> +			max-microamp = <50000>;
> +		};
> +
> +		indicator {
> +			max-microamp = <17500>;
> +		};
> +	};
>  };
>  
>  &i2c3 {
> 

This part of patch is already in mainline kernel, isn't?

-- 
Pali Rohár
pali.rohar@gmail.com

[toc] | [prev] | [next] | [standalone]


#1498655

FromPavel Machek <pavel@ucw.cz>
Date2016-10-11 10:20 +0200
Message-ID<sr0s2-4d1-9@gated-at.bofh.it>
In reply to#1498654

[Multipart message — attachments visible in raw view] — view raw

On Tue 2016-10-11 10:01:39, Pali Rohár wrote:
> On Tuesday 11 October 2016 09:54:17 Pavel Machek wrote:
> > @@ -669,6 +670,22 @@
> >  
> >  		ti,usb-charger-detection = <&isp1707>;
> >  	};
> > +
> > +	adp1653: led-controller@30 {
> > +		compatible = "adi,adp1653";
> > +		reg = <0x30>;
> > +		gpios = <&gpio3 24 GPIO_ACTIVE_HIGH>; /* 88 */
> > +
> > +		flash {
> > +			flash-timeout-us = <500000>;
> > +			flash-max-microamp = <320000>;
> > +			max-microamp = <50000>;
> > +		};
> > +
> > +		indicator {
> > +			max-microamp = <17500>;
> > +		};
> > +	};
> >  };
> >  
> >  &i2c3 {
> > 
> 
> This part of patch is already in mainline kernel, isn't?

Oops, sorry about that. Disregard the patch.
								Pavel
-- 
(english) http://www.livejournal.com/~pavelmachek
(cesky, pictures) http://atrey.karlin.mff.cuni.cz/~pavel/picture/horses/blog.html

[toc] | [prev] | [next] | [standalone]


#1498656 — [PATCH] n900 device tree: cleanup

FromPavel Machek <pavel@ucw.cz>
Date2016-10-11 10:20 +0200
Subject[PATCH] n900 device tree: cleanup
Message-ID<sr0s2-4d1-17@gated-at.bofh.it>
In reply to#1498653

[Multipart message — attachments visible in raw view] — view raw

Fix GPIO comment to be consistent with rest of file and add comment what
tpa6130 is.

Signed-off-by: Pavel Machek <pavel@ucw.cz>

diff --git a/arch/arm/boot/dts/omap3-n900.dts b/arch/arm/boot/dts/omap3-n900.dts
index bfffd6c..ca9fe8c 100644
--- a/arch/arm/boot/dts/omap3-n900.dts
+++ b/arch/arm/boot/dts/omap3-n900.dts
@@ -47,7 +47,7 @@
 		compatible = "gpio-leds";
 		heartbeat {
 			label = "debug::sleep";
-			gpios = <&gpio6 2 GPIO_ACTIVE_HIGH>;  /* gpio162 */
+			gpios = <&gpio6 2 GPIO_ACTIVE_HIGH>;  /* 162 */
 			linux,default-trigger = "default-on";
 			pinctrl-names = "default";
 			pinctrl-0 = <&debug_leds>;
@@ -637,6 +637,7 @@
 		reg = <0x55>;
 	};
 
+	/* Stereo headphone amplifier */
 	tpa6130a2: tpa6130a2@60 {
 		compatible = "ti,tpa6130a2";
 		reg = <0x60>;

-- 
(english) http://www.livejournal.com/~pavelmachek
(cesky, pictures) http://atrey.karlin.mff.cuni.cz/~pavel/picture/horses/blog.html

[toc] | [prev] | [standalone]


Back to top | Article view | linux.kernel


csiph-web