All of lore.kernel.org
 help / color / mirror / Atom feed
* [PATCH 1/2] sound: tegra_wm8903: Adjust to of_get_named_gpio() change
@ 2012-06-18 16:42 ` Roland Stigge
  0 siblings, 0 replies; 24+ messages in thread
From: Roland Stigge @ 2012-06-18 16:42 UTC (permalink / raw)
  To: swarren, cjb, grant.likely, rob.herring, rmk+kernel, ulf.hansson,
	linus.walleij, linux-mmc, linux-kernel, aletes.xgr,
	linux-arm-kernel, broonie, lrg, perex, tiwai, alsa-devel
  Cc: Roland Stigge

of_get_named_gpio() was changed to return -EPROBE_DEFER in case of
gpios not probed yet. This patch adjusts tegra_wm8903 to this.

Signed-off-by: Roland Stigge <stigge@antcom.de>

---
Applies to git://git.kernel.org/pub/scm/linux/kernel/git/broonie/sound.git,
branch for-next

 sound/soc/tegra/tegra_wm8903.c |   10 +++++-----
 1 file changed, 5 insertions(+), 5 deletions(-)

diff --git a/sound/soc/tegra/tegra_wm8903.c b/sound/soc/tegra/tegra_wm8903.c
index 0c5bb33..d4f14e4 100644
--- a/sound/soc/tegra/tegra_wm8903.c
+++ b/sound/soc/tegra/tegra_wm8903.c
@@ -284,27 +284,27 @@ static __devinit int tegra_wm8903_driver_probe(struct platform_device *pdev)
 	} else if (np) {
 		pdata->gpio_spkr_en = of_get_named_gpio(np,
 						"nvidia,spkr-en-gpios", 0);
-		if (pdata->gpio_spkr_en == -ENODEV)
+		if (pdata->gpio_spkr_en == -EPROBE_DEFER)
 			return -EPROBE_DEFER;
 
 		pdata->gpio_hp_mute = of_get_named_gpio(np,
 						"nvidia,hp-mute-gpios", 0);
-		if (pdata->gpio_hp_mute == -ENODEV)
+		if (pdata->gpio_hp_mute == -EPROBE_DEFER)
 			return -EPROBE_DEFER;
 
 		pdata->gpio_hp_det = of_get_named_gpio(np,
 						"nvidia,hp-det-gpios", 0);
-		if (pdata->gpio_hp_det == -ENODEV)
+		if (pdata->gpio_hp_det == -EPROBE_DEFER)
 			return -EPROBE_DEFER;
 
 		pdata->gpio_int_mic_en = of_get_named_gpio(np,
 						"nvidia,int-mic-en-gpios", 0);
-		if (pdata->gpio_int_mic_en == -ENODEV)
+		if (pdata->gpio_int_mic_en == -EPROBE_DEFER)
 			return -EPROBE_DEFER;
 
 		pdata->gpio_ext_mic_en = of_get_named_gpio(np,
 						"nvidia,ext-mic-en-gpios", 0);
-		if (pdata->gpio_ext_mic_en == -ENODEV)
+		if (pdata->gpio_ext_mic_en == -EPROBE_DEFER)
 			return -EPROBE_DEFER;
 	}
 
-- 
1.7.10


^ permalink raw reply related	[flat|nested] 24+ messages in thread

* [PATCH 1/2] sound: tegra_wm8903: Adjust to of_get_named_gpio() change
@ 2012-06-18 16:42 ` Roland Stigge
  0 siblings, 0 replies; 24+ messages in thread
From: Roland Stigge @ 2012-06-18 16:42 UTC (permalink / raw)
  To: linux-arm-kernel

of_get_named_gpio() was changed to return -EPROBE_DEFER in case of
gpios not probed yet. This patch adjusts tegra_wm8903 to this.

Signed-off-by: Roland Stigge <stigge@antcom.de>

---
Applies to git://git.kernel.org/pub/scm/linux/kernel/git/broonie/sound.git,
branch for-next

 sound/soc/tegra/tegra_wm8903.c |   10 +++++-----
 1 file changed, 5 insertions(+), 5 deletions(-)

diff --git a/sound/soc/tegra/tegra_wm8903.c b/sound/soc/tegra/tegra_wm8903.c
index 0c5bb33..d4f14e4 100644
--- a/sound/soc/tegra/tegra_wm8903.c
+++ b/sound/soc/tegra/tegra_wm8903.c
@@ -284,27 +284,27 @@ static __devinit int tegra_wm8903_driver_probe(struct platform_device *pdev)
 	} else if (np) {
 		pdata->gpio_spkr_en = of_get_named_gpio(np,
 						"nvidia,spkr-en-gpios", 0);
-		if (pdata->gpio_spkr_en == -ENODEV)
+		if (pdata->gpio_spkr_en == -EPROBE_DEFER)
 			return -EPROBE_DEFER;
 
 		pdata->gpio_hp_mute = of_get_named_gpio(np,
 						"nvidia,hp-mute-gpios", 0);
-		if (pdata->gpio_hp_mute == -ENODEV)
+		if (pdata->gpio_hp_mute == -EPROBE_DEFER)
 			return -EPROBE_DEFER;
 
 		pdata->gpio_hp_det = of_get_named_gpio(np,
 						"nvidia,hp-det-gpios", 0);
-		if (pdata->gpio_hp_det == -ENODEV)
+		if (pdata->gpio_hp_det == -EPROBE_DEFER)
 			return -EPROBE_DEFER;
 
 		pdata->gpio_int_mic_en = of_get_named_gpio(np,
 						"nvidia,int-mic-en-gpios", 0);
-		if (pdata->gpio_int_mic_en == -ENODEV)
+		if (pdata->gpio_int_mic_en == -EPROBE_DEFER)
 			return -EPROBE_DEFER;
 
 		pdata->gpio_ext_mic_en = of_get_named_gpio(np,
 						"nvidia,ext-mic-en-gpios", 0);
-		if (pdata->gpio_ext_mic_en == -ENODEV)
+		if (pdata->gpio_ext_mic_en == -EPROBE_DEFER)
 			return -EPROBE_DEFER;
 	}
 
-- 
1.7.10

^ permalink raw reply related	[flat|nested] 24+ messages in thread

* [PATCH 2/2] sound: tegra_alc5632: Adjust to of_get_named_gpio() change
  2012-06-18 16:42 ` Roland Stigge
@ 2012-06-18 16:42   ` Roland Stigge
  -1 siblings, 0 replies; 24+ messages in thread
From: Roland Stigge @ 2012-06-18 16:42 UTC (permalink / raw)
  To: swarren, cjb, grant.likely, rob.herring, rmk+kernel, ulf.hansson,
	linus.walleij, linux-mmc, linux-kernel, aletes.xgr,
	linux-arm-kernel, broonie, lrg, perex, tiwai, alsa-devel
  Cc: Roland Stigge

of_get_named_gpio() was changed to return -EPROBE_DEFER in case of
gpios not probed yet. This patch adjusts tegra_alc5632 to this.

Signed-off-by: Roland Stigge <stigge@antcom.de>

---
Applies to git://git.kernel.org/pub/scm/linux/kernel/git/broonie/sound.git,
branch for-next

 sound/soc/tegra/tegra_alc5632.c |    2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/sound/soc/tegra/tegra_alc5632.c b/sound/soc/tegra/tegra_alc5632.c
index d684df2..e463529 100644
--- a/sound/soc/tegra/tegra_alc5632.c
+++ b/sound/soc/tegra/tegra_alc5632.c
@@ -177,7 +177,7 @@ static __devinit int tegra_alc5632_probe(struct platform_device *pdev)
 	}
 
 	alc5632->gpio_hp_det = of_get_named_gpio(np, "nvidia,hp-det-gpios", 0);
-	if (alc5632->gpio_hp_det == -ENODEV)
+	if (alc5632->gpio_hp_det == -EPROBE_DEFER)
 		return -EPROBE_DEFER;
 
 	ret = snd_soc_of_parse_card_name(card, "nvidia,model");
-- 
1.7.10


^ permalink raw reply related	[flat|nested] 24+ messages in thread

* [PATCH 2/2] sound: tegra_alc5632: Adjust to of_get_named_gpio() change
@ 2012-06-18 16:42   ` Roland Stigge
  0 siblings, 0 replies; 24+ messages in thread
From: Roland Stigge @ 2012-06-18 16:42 UTC (permalink / raw)
  To: linux-arm-kernel

of_get_named_gpio() was changed to return -EPROBE_DEFER in case of
gpios not probed yet. This patch adjusts tegra_alc5632 to this.

Signed-off-by: Roland Stigge <stigge@antcom.de>

---
Applies to git://git.kernel.org/pub/scm/linux/kernel/git/broonie/sound.git,
branch for-next

 sound/soc/tegra/tegra_alc5632.c |    2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/sound/soc/tegra/tegra_alc5632.c b/sound/soc/tegra/tegra_alc5632.c
index d684df2..e463529 100644
--- a/sound/soc/tegra/tegra_alc5632.c
+++ b/sound/soc/tegra/tegra_alc5632.c
@@ -177,7 +177,7 @@ static __devinit int tegra_alc5632_probe(struct platform_device *pdev)
 	}
 
 	alc5632->gpio_hp_det = of_get_named_gpio(np, "nvidia,hp-det-gpios", 0);
-	if (alc5632->gpio_hp_det == -ENODEV)
+	if (alc5632->gpio_hp_det == -EPROBE_DEFER)
 		return -EPROBE_DEFER;
 
 	ret = snd_soc_of_parse_card_name(card, "nvidia,model");
-- 
1.7.10

^ permalink raw reply related	[flat|nested] 24+ messages in thread

* Re: [PATCH 1/2] sound: tegra_wm8903: Adjust to of_get_named_gpio() change
  2012-06-18 16:42 ` Roland Stigge
@ 2012-06-18 16:46   ` Mark Brown
  -1 siblings, 0 replies; 24+ messages in thread
From: Mark Brown @ 2012-06-18 16:46 UTC (permalink / raw)
  To: Roland Stigge
  Cc: swarren, cjb, grant.likely, rob.herring, rmk+kernel, ulf.hansson,
	linus.walleij, linux-mmc, linux-kernel, aletes.xgr,
	linux-arm-kernel, lrg, perex, tiwai, alsa-devel

[-- Attachment #1: Type: text/plain, Size: 423 bytes --]

On Mon, Jun 18, 2012 at 06:42:21PM +0200, Roland Stigge wrote:
> of_get_named_gpio() was changed to return -EPROBE_DEFER in case of
> gpios not probed yet. This patch adjusts tegra_wm8903 to this.

So, I saw from the other thread that this didn't actually happen yet - I
guess the best thing is to merge this along with the changes for
introducing -EPROBE_DEFER.

Acked-by: Mark Brown <broonie@opensource.wolfsonmicro.com>

[-- Attachment #2: Digital signature --]
[-- Type: application/pgp-signature, Size: 836 bytes --]

^ permalink raw reply	[flat|nested] 24+ messages in thread

* [PATCH 1/2] sound: tegra_wm8903: Adjust to of_get_named_gpio() change
@ 2012-06-18 16:46   ` Mark Brown
  0 siblings, 0 replies; 24+ messages in thread
From: Mark Brown @ 2012-06-18 16:46 UTC (permalink / raw)
  To: linux-arm-kernel

On Mon, Jun 18, 2012 at 06:42:21PM +0200, Roland Stigge wrote:
> of_get_named_gpio() was changed to return -EPROBE_DEFER in case of
> gpios not probed yet. This patch adjusts tegra_wm8903 to this.

So, I saw from the other thread that this didn't actually happen yet - I
guess the best thing is to merge this along with the changes for
introducing -EPROBE_DEFER.

Acked-by: Mark Brown <broonie@opensource.wolfsonmicro.com>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 836 bytes
Desc: Digital signature
URL: <http://lists.infradead.org/pipermail/linux-arm-kernel/attachments/20120618/4d0d66e5/attachment-0001.sig>

^ permalink raw reply	[flat|nested] 24+ messages in thread

* Re: [PATCH 1/2] sound: tegra_wm8903: Adjust to of_get_named_gpio() change
  2012-06-18 16:46   ` Mark Brown
@ 2012-06-18 17:00     ` Roland Stigge
  -1 siblings, 0 replies; 24+ messages in thread
From: Roland Stigge @ 2012-06-18 17:00 UTC (permalink / raw)
  To: Mark Brown
  Cc: swarren, cjb, grant.likely, rob.herring, rmk+kernel, ulf.hansson,
	linus.walleij, linux-mmc, linux-kernel, aletes.xgr,
	linux-arm-kernel, lrg, perex, tiwai, alsa-devel

On 06/18/2012 06:46 PM, Mark Brown wrote:
> On Mon, Jun 18, 2012 at 06:42:21PM +0200, Roland Stigge wrote:
>> of_get_named_gpio() was changed to return -EPROBE_DEFER in case of
>> gpios not probed yet. This patch adjusts tegra_wm8903 to this.
> 
> So, I saw from the other thread that this didn't actually happen yet - I
> guess the best thing is to merge this along with the changes for
> introducing -EPROBE_DEFER.

Basically, yes. But this would mean that doing via gpio, we would need
to pull other stuff from the sound tree over to gpio to build upon
because "my" tegra_* sound patches won't apply to the gpio branch currently.

Any hint welcome.

Thanks in advance,

Roland

^ permalink raw reply	[flat|nested] 24+ messages in thread

* [PATCH 1/2] sound: tegra_wm8903: Adjust to of_get_named_gpio() change
@ 2012-06-18 17:00     ` Roland Stigge
  0 siblings, 0 replies; 24+ messages in thread
From: Roland Stigge @ 2012-06-18 17:00 UTC (permalink / raw)
  To: linux-arm-kernel

On 06/18/2012 06:46 PM, Mark Brown wrote:
> On Mon, Jun 18, 2012 at 06:42:21PM +0200, Roland Stigge wrote:
>> of_get_named_gpio() was changed to return -EPROBE_DEFER in case of
>> gpios not probed yet. This patch adjusts tegra_wm8903 to this.
> 
> So, I saw from the other thread that this didn't actually happen yet - I
> guess the best thing is to merge this along with the changes for
> introducing -EPROBE_DEFER.

Basically, yes. But this would mean that doing via gpio, we would need
to pull other stuff from the sound tree over to gpio to build upon
because "my" tegra_* sound patches won't apply to the gpio branch currently.

Any hint welcome.

Thanks in advance,

Roland

^ permalink raw reply	[flat|nested] 24+ messages in thread

* Re: [PATCH 1/2] sound: tegra_wm8903: Adjust to of_get_named_gpio() change
  2012-06-18 17:00     ` Roland Stigge
@ 2012-06-18 17:04       ` Mark Brown
  -1 siblings, 0 replies; 24+ messages in thread
From: Mark Brown @ 2012-06-18 17:04 UTC (permalink / raw)
  To: Roland Stigge
  Cc: swarren, cjb, grant.likely, rob.herring, rmk+kernel, ulf.hansson,
	linus.walleij, linux-mmc, linux-kernel, aletes.xgr,
	linux-arm-kernel, lrg, perex, tiwai, alsa-devel

[-- Attachment #1: Type: text/plain, Size: 548 bytes --]

On Mon, Jun 18, 2012 at 07:00:14PM +0200, Roland Stigge wrote:

> Basically, yes. But this would mean that doing via gpio, we would need
> to pull other stuff from the sound tree over to gpio to build upon
> because "my" tegra_* sound patches won't apply to the gpio branch currently.

Oh, yes.  Stephen's gone and reorganised all that code which means it's
not a trivial merge.  Ho hum.  I'm happy to apply them if folks are
comfortable with it, I guess ideally I'd be able to pull a branch with
the gpio changes in and then apply on top of that.

[-- Attachment #2: Digital signature --]
[-- Type: application/pgp-signature, Size: 836 bytes --]

^ permalink raw reply	[flat|nested] 24+ messages in thread

* [PATCH 1/2] sound: tegra_wm8903: Adjust to of_get_named_gpio() change
@ 2012-06-18 17:04       ` Mark Brown
  0 siblings, 0 replies; 24+ messages in thread
From: Mark Brown @ 2012-06-18 17:04 UTC (permalink / raw)
  To: linux-arm-kernel

On Mon, Jun 18, 2012 at 07:00:14PM +0200, Roland Stigge wrote:

> Basically, yes. But this would mean that doing via gpio, we would need
> to pull other stuff from the sound tree over to gpio to build upon
> because "my" tegra_* sound patches won't apply to the gpio branch currently.

Oh, yes.  Stephen's gone and reorganised all that code which means it's
not a trivial merge.  Ho hum.  I'm happy to apply them if folks are
comfortable with it, I guess ideally I'd be able to pull a branch with
the gpio changes in and then apply on top of that.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 836 bytes
Desc: Digital signature
URL: <http://lists.infradead.org/pipermail/linux-arm-kernel/attachments/20120618/135b3d89/attachment.sig>

^ permalink raw reply	[flat|nested] 24+ messages in thread

* Re: [PATCH 1/2] sound: tegra_wm8903: Adjust to of_get_named_gpio() change
  2012-06-18 17:04       ` Mark Brown
@ 2012-06-18 17:15         ` Roland Stigge
  -1 siblings, 0 replies; 24+ messages in thread
From: Roland Stigge @ 2012-06-18 17:15 UTC (permalink / raw)
  To: Mark Brown
  Cc: swarren, cjb, grant.likely, rob.herring, rmk+kernel, ulf.hansson,
	linus.walleij, linux-mmc, linux-kernel, aletes.xgr,
	linux-arm-kernel, lrg, perex, tiwai, alsa-devel

On 06/18/2012 07:04 PM, Mark Brown wrote:
> On Mon, Jun 18, 2012 at 07:00:14PM +0200, Roland Stigge wrote:
> 
>> Basically, yes. But this would mean that doing via gpio, we would
>> need to pull other stuff from the sound tree over to gpio to
>> build upon because "my" tegra_* sound patches won't apply to the
>> gpio branch currently.
> 
> Oh, yes.  Stephen's gone and reorganised all that code which means
> it's not a trivial merge.  Ho hum.  I'm happy to apply them if
> folks are comfortable with it, I guess ideally I'd be able to pull
> a branch with the gpio changes in and then apply on top of that.

Depending on what else changes will come via gpio for 3.6, maybe you
don't even need to merge gpio to sound?

Can you please coordinate between sound and gpio?

Thanks in advance!

Roland

PS: I'm posting a combined patch suitable for the sound subsystem.
Please tell me if you also need the gpiolib-of patch singled out.

^ permalink raw reply	[flat|nested] 24+ messages in thread

* [PATCH 1/2] sound: tegra_wm8903: Adjust to of_get_named_gpio() change
@ 2012-06-18 17:15         ` Roland Stigge
  0 siblings, 0 replies; 24+ messages in thread
From: Roland Stigge @ 2012-06-18 17:15 UTC (permalink / raw)
  To: linux-arm-kernel

On 06/18/2012 07:04 PM, Mark Brown wrote:
> On Mon, Jun 18, 2012 at 07:00:14PM +0200, Roland Stigge wrote:
> 
>> Basically, yes. But this would mean that doing via gpio, we would
>> need to pull other stuff from the sound tree over to gpio to
>> build upon because "my" tegra_* sound patches won't apply to the
>> gpio branch currently.
> 
> Oh, yes.  Stephen's gone and reorganised all that code which means
> it's not a trivial merge.  Ho hum.  I'm happy to apply them if
> folks are comfortable with it, I guess ideally I'd be able to pull
> a branch with the gpio changes in and then apply on top of that.

Depending on what else changes will come via gpio for 3.6, maybe you
don't even need to merge gpio to sound?

Can you please coordinate between sound and gpio?

Thanks in advance!

Roland

PS: I'm posting a combined patch suitable for the sound subsystem.
Please tell me if you also need the gpiolib-of patch singled out.

^ permalink raw reply	[flat|nested] 24+ messages in thread

* Re: [PATCH 1/2] sound: tegra_wm8903: Adjust to of_get_named_gpio() change
  2012-06-18 16:42 ` Roland Stigge
@ 2012-06-18 17:28   ` Alexandre Pereira da Silva
  -1 siblings, 0 replies; 24+ messages in thread
From: Alexandre Pereira da Silva @ 2012-06-18 17:28 UTC (permalink / raw)
  To: Roland Stigge
  Cc: swarren, cjb, grant.likely, rob.herring, rmk+kernel, ulf.hansson,
	linus.walleij, linux-mmc, linux-kernel, linux-arm-kernel,
	broonie, lrg, perex, tiwai, alsa-devel

On Mon, Jun 18, 2012 at 1:42 PM, Roland Stigge <stigge@antcom.de> wrote:
> of_get_named_gpio() was changed to return -EPROBE_DEFER in case of
> gpios not probed yet. This patch adjusts tegra_wm8903 to this.
>
> Signed-off-by: Roland Stigge <stigge@antcom.de>
Acked-by: Alexandre Pereira da Silva <aletes.xgr@gmail.com>

^ permalink raw reply	[flat|nested] 24+ messages in thread

* [PATCH 1/2] sound: tegra_wm8903: Adjust to of_get_named_gpio() change
@ 2012-06-18 17:28   ` Alexandre Pereira da Silva
  0 siblings, 0 replies; 24+ messages in thread
From: Alexandre Pereira da Silva @ 2012-06-18 17:28 UTC (permalink / raw)
  To: linux-arm-kernel

On Mon, Jun 18, 2012 at 1:42 PM, Roland Stigge <stigge@antcom.de> wrote:
> of_get_named_gpio() was changed to return -EPROBE_DEFER in case of
> gpios not probed yet. This patch adjusts tegra_wm8903 to this.
>
> Signed-off-by: Roland Stigge <stigge@antcom.de>
Acked-by: Alexandre Pereira da Silva <aletes.xgr@gmail.com>

^ permalink raw reply	[flat|nested] 24+ messages in thread

* Re: [PATCH 2/2] sound: tegra_alc5632: Adjust to of_get_named_gpio() change
  2012-06-18 16:42   ` Roland Stigge
@ 2012-06-18 17:28     ` Alexandre Pereira da Silva
  -1 siblings, 0 replies; 24+ messages in thread
From: Alexandre Pereira da Silva @ 2012-06-18 17:28 UTC (permalink / raw)
  To: Roland Stigge
  Cc: swarren, cjb, grant.likely, rob.herring, rmk+kernel, ulf.hansson,
	linus.walleij, linux-mmc, linux-kernel, linux-arm-kernel,
	broonie, lrg, perex, tiwai, alsa-devel

On Mon, Jun 18, 2012 at 1:42 PM, Roland Stigge <stigge@antcom.de> wrote:
> of_get_named_gpio() was changed to return -EPROBE_DEFER in case of
> gpios not probed yet. This patch adjusts tegra_alc5632 to this.
>
> Signed-off-by: Roland Stigge <stigge@antcom.de>
Acked-by: Alexandre Pereira da Silva <aletes.xgr@gmail.com>

^ permalink raw reply	[flat|nested] 24+ messages in thread

* [PATCH 2/2] sound: tegra_alc5632: Adjust to of_get_named_gpio() change
@ 2012-06-18 17:28     ` Alexandre Pereira da Silva
  0 siblings, 0 replies; 24+ messages in thread
From: Alexandre Pereira da Silva @ 2012-06-18 17:28 UTC (permalink / raw)
  To: linux-arm-kernel

On Mon, Jun 18, 2012 at 1:42 PM, Roland Stigge <stigge@antcom.de> wrote:
> of_get_named_gpio() was changed to return -EPROBE_DEFER in case of
> gpios not probed yet. This patch adjusts tegra_alc5632 to this.
>
> Signed-off-by: Roland Stigge <stigge@antcom.de>
Acked-by: Alexandre Pereira da Silva <aletes.xgr@gmail.com>

^ permalink raw reply	[flat|nested] 24+ messages in thread

* Re: [PATCH 2/2] sound: tegra_alc5632: Adjust to of_get_named_gpio() change
  2012-06-18 17:28     ` Alexandre Pereira da Silva
@ 2012-06-19  6:29       ` Leon Romanovsky
  -1 siblings, 0 replies; 24+ messages in thread
From: Leon Romanovsky @ 2012-06-19  6:29 UTC (permalink / raw)
  To: Alexandre Pereira da Silva
  Cc: Roland Stigge, swarren, cjb, grant.likely, rob.herring,
	rmk+kernel, ulf.hansson, linus.walleij, linux-mmc, linux-kernel,
	linux-arm-kernel, broonie, lrg, perex, tiwai, alsa-devel

On Mon, Jun 18, 2012 at 8:28 PM, Alexandre Pereira da Silva
<aletes.xgr@gmail.com> wrote:
> On Mon, Jun 18, 2012 at 1:42 PM, Roland Stigge <stigge@antcom.de> wrote:
>> of_get_named_gpio() was changed to return -EPROBE_DEFER in case of
>> gpios not probed yet. This patch adjusts tegra_alc5632 to this.
>>
>> Signed-off-by: Roland Stigge <stigge@antcom.de>
> Acked-by: Alexandre Pereira da Silva <aletes.xgr@gmail.com>
Hi Alexandre,
Sorry to be rude, but can you explain me why did you "Acked-by" on this patch ?
I don't see your name in git log for tegra_alc5632.c file:
http://git.kernel.org/?p=linux/kernel/git/broonie/sound.git;a=history;f=sound/soc/tegra/tegra_alc5632.c;h=d684df294c0c5c80458b6de591c16965027db14b;hb=refs/heads/for-next

Thanks.

-- 
Leon Romanovsky | Independent Linux Consultant
        www.leon.nu | leon@leon.nu

^ permalink raw reply	[flat|nested] 24+ messages in thread

* [PATCH 2/2] sound: tegra_alc5632: Adjust to of_get_named_gpio() change
@ 2012-06-19  6:29       ` Leon Romanovsky
  0 siblings, 0 replies; 24+ messages in thread
From: Leon Romanovsky @ 2012-06-19  6:29 UTC (permalink / raw)
  To: linux-arm-kernel

On Mon, Jun 18, 2012 at 8:28 PM, Alexandre Pereira da Silva
<aletes.xgr@gmail.com> wrote:
> On Mon, Jun 18, 2012 at 1:42 PM, Roland Stigge <stigge@antcom.de> wrote:
>> of_get_named_gpio() was changed to return -EPROBE_DEFER in case of
>> gpios not probed yet. This patch adjusts tegra_alc5632 to this.
>>
>> Signed-off-by: Roland Stigge <stigge@antcom.de>
> Acked-by: Alexandre Pereira da Silva <aletes.xgr@gmail.com>
Hi Alexandre,
Sorry to be rude, but can you explain me why did you "Acked-by" on this patch ?
I don't see your name in git log for tegra_alc5632.c file:
http://git.kernel.org/?p=linux/kernel/git/broonie/sound.git;a=history;f=sound/soc/tegra/tegra_alc5632.c;h=d684df294c0c5c80458b6de591c16965027db14b;hb=refs/heads/for-next

Thanks.

-- 
Leon Romanovsky | Independent Linux Consultant
? ? ? ??www.leon.nu?| leon at leon.nu

^ permalink raw reply	[flat|nested] 24+ messages in thread

* Re: [PATCH 2/2] sound: tegra_alc5632: Adjust to of_get_named_gpio() change
  2012-06-19  6:29       ` Leon Romanovsky
@ 2012-06-19 10:03         ` Alexandre Pereira da Silva
  -1 siblings, 0 replies; 24+ messages in thread
From: Alexandre Pereira da Silva @ 2012-06-19 10:03 UTC (permalink / raw)
  To: Leon Romanovsky
  Cc: Roland Stigge, swarren, cjb, grant.likely, rob.herring,
	rmk+kernel, ulf.hansson, linus.walleij, linux-mmc, linux-kernel,
	linux-arm-kernel, broonie, lrg, perex, tiwai, alsa-devel

On Tue, Jun 19, 2012 at 3:29 AM, Leon Romanovsky <leon@leon.nu> wrote:
> On Mon, Jun 18, 2012 at 8:28 PM, Alexandre Pereira da Silva
> <aletes.xgr@gmail.com> wrote:
>> On Mon, Jun 18, 2012 at 1:42 PM, Roland Stigge <stigge@antcom.de> wrote:
>>> of_get_named_gpio() was changed to return -EPROBE_DEFER in case of
>>> gpios not probed yet. This patch adjusts tegra_alc5632 to this.
>>>
>>> Signed-off-by: Roland Stigge <stigge@antcom.de>
>> Acked-by: Alexandre Pereira da Silva <aletes.xgr@gmail.com>
> Hi Alexandre,
> Sorry to be rude, but can you explain me why did you "Acked-by" on this patch ?
> I don't see your name in git log for tegra_alc5632.c file:
> http://git.kernel.org/?p=linux/kernel/git/broonie/sound.git;a=history;f=sound/soc/tegra/tegra_alc5632.c;h=d684df294c0c5c80458b6de591c16965027db14b;hb=refs/heads/for-next

I'm tracking Roland's work on Probe defer.

^ permalink raw reply	[flat|nested] 24+ messages in thread

* [PATCH 2/2] sound: tegra_alc5632: Adjust to of_get_named_gpio() change
@ 2012-06-19 10:03         ` Alexandre Pereira da Silva
  0 siblings, 0 replies; 24+ messages in thread
From: Alexandre Pereira da Silva @ 2012-06-19 10:03 UTC (permalink / raw)
  To: linux-arm-kernel

On Tue, Jun 19, 2012 at 3:29 AM, Leon Romanovsky <leon@leon.nu> wrote:
> On Mon, Jun 18, 2012 at 8:28 PM, Alexandre Pereira da Silva
> <aletes.xgr@gmail.com> wrote:
>> On Mon, Jun 18, 2012 at 1:42 PM, Roland Stigge <stigge@antcom.de> wrote:
>>> of_get_named_gpio() was changed to return -EPROBE_DEFER in case of
>>> gpios not probed yet. This patch adjusts tegra_alc5632 to this.
>>>
>>> Signed-off-by: Roland Stigge <stigge@antcom.de>
>> Acked-by: Alexandre Pereira da Silva <aletes.xgr@gmail.com>
> Hi Alexandre,
> Sorry to be rude, but can you explain me why did you "Acked-by" on this patch ?
> I don't see your name in git log for tegra_alc5632.c file:
> http://git.kernel.org/?p=linux/kernel/git/broonie/sound.git;a=history;f=sound/soc/tegra/tegra_alc5632.c;h=d684df294c0c5c80458b6de591c16965027db14b;hb=refs/heads/for-next

I'm tracking Roland's work on Probe defer.

^ permalink raw reply	[flat|nested] 24+ messages in thread

* Re: [PATCH 2/2] sound: tegra_alc5632: Adjust to of_get_named_gpio() change
  2012-06-19 10:03         ` Alexandre Pereira da Silva
@ 2012-06-19 11:03           ` Roland Stigge
  -1 siblings, 0 replies; 24+ messages in thread
From: Roland Stigge @ 2012-06-19 11:03 UTC (permalink / raw)
  To: Alexandre Pereira da Silva
  Cc: Leon Romanovsky, swarren, cjb, grant.likely, rob.herring,
	rmk+kernel, ulf.hansson, linus.walleij, linux-mmc, linux-kernel,
	linux-arm-kernel, broonie, lrg, perex, tiwai, alsa-devel

On 06/19/2012 12:03 PM, Alexandre Pereira da Silva wrote:
> On Tue, Jun 19, 2012 at 3:29 AM, Leon Romanovsky <leon@leon.nu> wrote:
>> Hi Alexandre,
>> Sorry to be rude, but can you explain me why did you "Acked-by" on this patch ?
>> I don't see your name in git log for tegra_alc5632.c file:
>> http://git.kernel.org/?p=linux/kernel/git/broonie/sound.git;a=history;f=sound/soc/tegra/tegra_alc5632.c;h=d684df294c0c5c80458b6de591c16965027db14b;hb=refs/heads/for-next
> 
> I'm tracking Roland's work on Probe defer.

Yes, that's valuable. Thanks!

Roland

^ permalink raw reply	[flat|nested] 24+ messages in thread

* [PATCH 2/2] sound: tegra_alc5632: Adjust to of_get_named_gpio() change
@ 2012-06-19 11:03           ` Roland Stigge
  0 siblings, 0 replies; 24+ messages in thread
From: Roland Stigge @ 2012-06-19 11:03 UTC (permalink / raw)
  To: linux-arm-kernel

On 06/19/2012 12:03 PM, Alexandre Pereira da Silva wrote:
> On Tue, Jun 19, 2012 at 3:29 AM, Leon Romanovsky <leon@leon.nu> wrote:
>> Hi Alexandre,
>> Sorry to be rude, but can you explain me why did you "Acked-by" on this patch ?
>> I don't see your name in git log for tegra_alc5632.c file:
>> http://git.kernel.org/?p=linux/kernel/git/broonie/sound.git;a=history;f=sound/soc/tegra/tegra_alc5632.c;h=d684df294c0c5c80458b6de591c16965027db14b;hb=refs/heads/for-next
> 
> I'm tracking Roland's work on Probe defer.

Yes, that's valuable. Thanks!

Roland

^ permalink raw reply	[flat|nested] 24+ messages in thread

* Re: [PATCH 1/2] sound: tegra_wm8903: Adjust to of_get_named_gpio() change
  2012-06-18 16:42 ` Roland Stigge
@ 2012-07-26 21:25   ` Mark Brown
  -1 siblings, 0 replies; 24+ messages in thread
From: Mark Brown @ 2012-07-26 21:25 UTC (permalink / raw)
  To: Roland Stigge
  Cc: swarren, cjb, grant.likely, rob.herring, rmk+kernel, ulf.hansson,
	linus.walleij, linux-mmc, linux-kernel, aletes.xgr,
	linux-arm-kernel, lrg, perex, tiwai, alsa-devel

[-- Attachment #1: Type: text/plain, Size: 220 bytes --]

On Mon, Jun 18, 2012 at 06:42:21PM +0200, Roland Stigge wrote:
> of_get_named_gpio() was changed to return -EPROBE_DEFER in case of
> gpios not probed yet. This patch adjusts tegra_wm8903 to this.

Applied both, thanks.

[-- Attachment #2: Digital signature --]
[-- Type: application/pgp-signature, Size: 836 bytes --]

^ permalink raw reply	[flat|nested] 24+ messages in thread

* [PATCH 1/2] sound: tegra_wm8903: Adjust to of_get_named_gpio() change
@ 2012-07-26 21:25   ` Mark Brown
  0 siblings, 0 replies; 24+ messages in thread
From: Mark Brown @ 2012-07-26 21:25 UTC (permalink / raw)
  To: linux-arm-kernel

On Mon, Jun 18, 2012 at 06:42:21PM +0200, Roland Stigge wrote:
> of_get_named_gpio() was changed to return -EPROBE_DEFER in case of
> gpios not probed yet. This patch adjusts tegra_wm8903 to this.

Applied both, thanks.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 836 bytes
Desc: Digital signature
URL: <http://lists.infradead.org/pipermail/linux-arm-kernel/attachments/20120726/7742a608/attachment.sig>

^ permalink raw reply	[flat|nested] 24+ messages in thread

end of thread, other threads:[~2012-07-26 21:26 UTC | newest]

Thread overview: 24+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2012-06-18 16:42 [PATCH 1/2] sound: tegra_wm8903: Adjust to of_get_named_gpio() change Roland Stigge
2012-06-18 16:42 ` Roland Stigge
2012-06-18 16:42 ` [PATCH 2/2] sound: tegra_alc5632: " Roland Stigge
2012-06-18 16:42   ` Roland Stigge
2012-06-18 17:28   ` Alexandre Pereira da Silva
2012-06-18 17:28     ` Alexandre Pereira da Silva
2012-06-19  6:29     ` Leon Romanovsky
2012-06-19  6:29       ` Leon Romanovsky
2012-06-19 10:03       ` Alexandre Pereira da Silva
2012-06-19 10:03         ` Alexandre Pereira da Silva
2012-06-19 11:03         ` Roland Stigge
2012-06-19 11:03           ` Roland Stigge
2012-06-18 16:46 ` [PATCH 1/2] sound: tegra_wm8903: " Mark Brown
2012-06-18 16:46   ` Mark Brown
2012-06-18 17:00   ` Roland Stigge
2012-06-18 17:00     ` Roland Stigge
2012-06-18 17:04     ` Mark Brown
2012-06-18 17:04       ` Mark Brown
2012-06-18 17:15       ` Roland Stigge
2012-06-18 17:15         ` Roland Stigge
2012-06-18 17:28 ` Alexandre Pereira da Silva
2012-06-18 17:28   ` Alexandre Pereira da Silva
2012-07-26 21:25 ` Mark Brown
2012-07-26 21:25   ` Mark Brown

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.