* [PATCH 1/5] ARM: dts: imx23-evk: enable USB PHY and controller
@ 2013-07-15 14:22 ` Otavio Salvador
0 siblings, 0 replies; 32+ messages in thread
From: Otavio Salvador @ 2013-07-15 14:22 UTC (permalink / raw)
To: Jonathan Cameron, linux-kernel
Cc: Otavio Salvador, Russell King, Shawn Guo, Fabio Estevam,
linux-arm-kernel
The i.MX23EVK board provides a USB port so the USB PHY and controller
need to be enabled for it to be usable.
Signed-off-by: Otavio Salvador <otavio@ossystems.com.br>
---
arch/arm/boot/dts/imx23-evk.dts | 10 ++++++++++
1 file changed, 10 insertions(+)
diff --git a/arch/arm/boot/dts/imx23-evk.dts b/arch/arm/boot/dts/imx23-evk.dts
index da0588a..dc3ab45 100644
--- a/arch/arm/boot/dts/imx23-evk.dts
+++ b/arch/arm/boot/dts/imx23-evk.dts
@@ -107,6 +107,16 @@
pinctrl-0 = <&duart_pins_a>;
status = "okay";
};
+
+ usbphy0: usbphy@8007c000 {
+ status = "okay";
+ };
+ };
+ };
+
+ ahb@80080000 {
+ usb0: usb@80080000 {
+ status = "okay";
};
};
--
1.8.3.2
^ permalink raw reply related [flat|nested] 32+ messages in thread
* [PATCH 1/5] ARM: dts: imx23-evk: enable USB PHY and controller
@ 2013-07-15 14:22 ` Otavio Salvador
0 siblings, 0 replies; 32+ messages in thread
From: Otavio Salvador @ 2013-07-15 14:22 UTC (permalink / raw)
To: linux-arm-kernel
The i.MX23EVK board provides a USB port so the USB PHY and controller
need to be enabled for it to be usable.
Signed-off-by: Otavio Salvador <otavio@ossystems.com.br>
---
arch/arm/boot/dts/imx23-evk.dts | 10 ++++++++++
1 file changed, 10 insertions(+)
diff --git a/arch/arm/boot/dts/imx23-evk.dts b/arch/arm/boot/dts/imx23-evk.dts
index da0588a..dc3ab45 100644
--- a/arch/arm/boot/dts/imx23-evk.dts
+++ b/arch/arm/boot/dts/imx23-evk.dts
@@ -107,6 +107,16 @@
pinctrl-0 = <&duart_pins_a>;
status = "okay";
};
+
+ usbphy0: usbphy at 8007c000 {
+ status = "okay";
+ };
+ };
+ };
+
+ ahb at 80080000 {
+ usb0: usb at 80080000 {
+ status = "okay";
};
};
--
1.8.3.2
^ permalink raw reply related [flat|nested] 32+ messages in thread
* [PATCH 2/5] ARM: mxs-lradc: Simplify mxs_lradc_trigger_init error handle
2013-07-15 14:22 ` Otavio Salvador
(?)
@ 2013-07-15 14:22 ` Otavio Salvador
2013-07-15 14:24 ` Marek Vasut
-1 siblings, 1 reply; 32+ messages in thread
From: Otavio Salvador @ 2013-07-15 14:22 UTC (permalink / raw)
To: Jonathan Cameron, linux-kernel
Cc: Otavio Salvador, Jonathan Cameron, Greg Kroah-Hartman,
Marek Vasut, Bill Pemberton, Lars-Peter Clausen, linux-iio,
devel
As we have a 'ret' variable with the iio_trigger_register return, this
can be used as result.
Signed-off-by: Otavio Salvador <otavio@ossystems.com.br>
---
drivers/staging/iio/adc/mxs-lradc.c | 6 ++----
1 file changed, 2 insertions(+), 4 deletions(-)
diff --git a/drivers/staging/iio/adc/mxs-lradc.c b/drivers/staging/iio/adc/mxs-lradc.c
index d92c97a..6d41846 100644
--- a/drivers/staging/iio/adc/mxs-lradc.c
+++ b/drivers/staging/iio/adc/mxs-lradc.c
@@ -671,14 +671,12 @@ static int mxs_lradc_trigger_init(struct iio_dev *iio)
trig->ops = &mxs_lradc_trigger_ops;
ret = iio_trigger_register(trig);
- if (ret) {
+ if (ret)
iio_trigger_free(trig);
- return ret;
- }
iio->trig = trig;
- return 0;
+ return ret;
}
static void mxs_lradc_trigger_remove(struct iio_dev *iio)
--
1.8.3.2
^ permalink raw reply related [flat|nested] 32+ messages in thread
* [PATCH 3/5] ARM: dts: imx23-evk: enable Low Resolution ADC
2013-07-15 14:22 ` Otavio Salvador
@ 2013-07-15 14:22 ` Otavio Salvador
-1 siblings, 0 replies; 32+ messages in thread
From: Otavio Salvador @ 2013-07-15 14:22 UTC (permalink / raw)
To: Jonathan Cameron, linux-kernel
Cc: Otavio Salvador, Russell King, Shawn Guo, Fabio Estevam,
linux-arm-kernel
The i.XM23 has a internal Low Resolution ADC; this enables the support
for this device.
Signed-off-by: Otavio Salvador <otavio@ossystems.com.br>
---
arch/arm/boot/dts/imx23-evk.dts | 5 +++++
1 file changed, 5 insertions(+)
diff --git a/arch/arm/boot/dts/imx23-evk.dts b/arch/arm/boot/dts/imx23-evk.dts
index dc3ab45..185c7c0 100644
--- a/arch/arm/boot/dts/imx23-evk.dts
+++ b/arch/arm/boot/dts/imx23-evk.dts
@@ -90,6 +90,11 @@
};
apbx@80040000 {
+ lradc@80050000 {
+ status = "okay";
+ fsl,lradc-touchscreen-wires = <4>;
+ };
+
pwm: pwm@80064000 {
pinctrl-names = "default";
pinctrl-0 = <&pwm2_pins_a>;
--
1.8.3.2
^ permalink raw reply related [flat|nested] 32+ messages in thread
* [PATCH 3/5] ARM: dts: imx23-evk: enable Low Resolution ADC
@ 2013-07-15 14:22 ` Otavio Salvador
0 siblings, 0 replies; 32+ messages in thread
From: Otavio Salvador @ 2013-07-15 14:22 UTC (permalink / raw)
To: linux-arm-kernel
The i.XM23 has a internal Low Resolution ADC; this enables the support
for this device.
Signed-off-by: Otavio Salvador <otavio@ossystems.com.br>
---
arch/arm/boot/dts/imx23-evk.dts | 5 +++++
1 file changed, 5 insertions(+)
diff --git a/arch/arm/boot/dts/imx23-evk.dts b/arch/arm/boot/dts/imx23-evk.dts
index dc3ab45..185c7c0 100644
--- a/arch/arm/boot/dts/imx23-evk.dts
+++ b/arch/arm/boot/dts/imx23-evk.dts
@@ -90,6 +90,11 @@
};
apbx at 80040000 {
+ lradc at 80050000 {
+ status = "okay";
+ fsl,lradc-touchscreen-wires = <4>;
+ };
+
pwm: pwm at 80064000 {
pinctrl-names = "default";
pinctrl-0 = <&pwm2_pins_a>;
--
1.8.3.2
^ permalink raw reply related [flat|nested] 32+ messages in thread
* [PATCH 4/5] ARM: dts: imx23-olinuxino: enable Low Resolution ADC
2013-07-15 14:22 ` Otavio Salvador
@ 2013-07-15 14:22 ` Otavio Salvador
-1 siblings, 0 replies; 32+ messages in thread
From: Otavio Salvador @ 2013-07-15 14:22 UTC (permalink / raw)
To: Jonathan Cameron, linux-kernel
Cc: Otavio Salvador, Russell King, Shawn Guo, Fabio Estevam,
Alexandre Pereira da Silva, Fadil Berisha, linux-arm-kernel
The i.XM23 has a internal Low Resolution ADC; this enables the support
for this device.
Signed-off-by: Otavio Salvador <otavio@ossystems.com.br>
---
arch/arm/boot/dts/imx23-olinuxino.dts | 4 ++++
1 file changed, 4 insertions(+)
diff --git a/arch/arm/boot/dts/imx23-olinuxino.dts b/arch/arm/boot/dts/imx23-olinuxino.dts
index d107c4a..fc766ae 100644
--- a/arch/arm/boot/dts/imx23-olinuxino.dts
+++ b/arch/arm/boot/dts/imx23-olinuxino.dts
@@ -69,6 +69,10 @@
};
apbx@80040000 {
+ lradc@80050000 {
+ status = "okay";
+ };
+
duart: serial@80070000 {
pinctrl-names = "default";
pinctrl-0 = <&duart_pins_a>;
--
1.8.3.2
^ permalink raw reply related [flat|nested] 32+ messages in thread
* [PATCH 4/5] ARM: dts: imx23-olinuxino: enable Low Resolution ADC
@ 2013-07-15 14:22 ` Otavio Salvador
0 siblings, 0 replies; 32+ messages in thread
From: Otavio Salvador @ 2013-07-15 14:22 UTC (permalink / raw)
To: linux-arm-kernel
The i.XM23 has a internal Low Resolution ADC; this enables the support
for this device.
Signed-off-by: Otavio Salvador <otavio@ossystems.com.br>
---
arch/arm/boot/dts/imx23-olinuxino.dts | 4 ++++
1 file changed, 4 insertions(+)
diff --git a/arch/arm/boot/dts/imx23-olinuxino.dts b/arch/arm/boot/dts/imx23-olinuxino.dts
index d107c4a..fc766ae 100644
--- a/arch/arm/boot/dts/imx23-olinuxino.dts
+++ b/arch/arm/boot/dts/imx23-olinuxino.dts
@@ -69,6 +69,10 @@
};
apbx at 80040000 {
+ lradc at 80050000 {
+ status = "okay";
+ };
+
duart: serial at 80070000 {
pinctrl-names = "default";
pinctrl-0 = <&duart_pins_a>;
--
1.8.3.2
^ permalink raw reply related [flat|nested] 32+ messages in thread
* [PATCH 5/5] iio: add configuration option for debug support
2013-07-15 14:22 ` Otavio Salvador
` (3 preceding siblings ...)
(?)
@ 2013-07-15 14:22 ` Otavio Salvador
2013-07-16 9:58 ` Jonathan Cameron
-1 siblings, 1 reply; 32+ messages in thread
From: Otavio Salvador @ 2013-07-15 14:22 UTC (permalink / raw)
To: Jonathan Cameron, linux-kernel
Cc: Otavio Salvador, Jonathan Cameron, linux-iio
Signed-off-by: Otavio Salvador <otavio@ossystems.com.br>
---
drivers/iio/Kconfig | 7 +++++++
drivers/iio/Makefile | 2 ++
2 files changed, 9 insertions(+)
diff --git a/drivers/iio/Kconfig b/drivers/iio/Kconfig
index 9af763a..d1f06f2 100644
--- a/drivers/iio/Kconfig
+++ b/drivers/iio/Kconfig
@@ -12,6 +12,13 @@ menuconfig IIO
if IIO
+config IIO_DEBUG
+ boolean "Debug support for Industrial I/O"
+ depends on DEBUG_KERNEL
+ help
+ Say "yes" to enable debug messaging (like dev_dbg and pr_debug),
+ sysfs, and debugfs support in Industrial I/O subsystem and drivers.
+
config IIO_BUFFER
bool "Enable buffer support within IIO"
help
diff --git a/drivers/iio/Makefile b/drivers/iio/Makefile
index 7a3866c..6e43e5b 100644
--- a/drivers/iio/Makefile
+++ b/drivers/iio/Makefile
@@ -2,6 +2,8 @@
# Makefile for the industrial I/O core.
#
+ccflags-$(CONFIG_IIO_DEBUG) := -DDEBUG
+
obj-$(CONFIG_IIO) += industrialio.o
industrialio-y := industrialio-core.o industrialio-event.o inkern.o
industrialio-$(CONFIG_IIO_BUFFER) += industrialio-buffer.o
--
1.8.3.2
^ permalink raw reply related [flat|nested] 32+ messages in thread
* Re: [PATCH 2/5] ARM: mxs-lradc: Simplify mxs_lradc_trigger_init error handle
2013-07-15 14:22 ` [PATCH 2/5] ARM: mxs-lradc: Simplify mxs_lradc_trigger_init error handle Otavio Salvador
@ 2013-07-15 14:24 ` Marek Vasut
2013-07-15 14:38 ` Otavio Salvador
0 siblings, 1 reply; 32+ messages in thread
From: Marek Vasut @ 2013-07-15 14:24 UTC (permalink / raw)
To: Otavio Salvador
Cc: Jonathan Cameron, linux-kernel, Jonathan Cameron,
Greg Kroah-Hartman, Bill Pemberton, Lars-Peter Clausen,
linux-iio, devel
Dear Otavio Salvador,
> As we have a 'ret' variable with the iio_trigger_register return, this
> can be used as result.
>
> Signed-off-by: Otavio Salvador <otavio@ossystems.com.br>
> ---
> drivers/staging/iio/adc/mxs-lradc.c | 6 ++----
> 1 file changed, 2 insertions(+), 4 deletions(-)
>
> diff --git a/drivers/staging/iio/adc/mxs-lradc.c
> b/drivers/staging/iio/adc/mxs-lradc.c index d92c97a..6d41846 100644
> --- a/drivers/staging/iio/adc/mxs-lradc.c
> +++ b/drivers/staging/iio/adc/mxs-lradc.c
> @@ -671,14 +671,12 @@ static int mxs_lradc_trigger_init(struct iio_dev
> *iio) trig->ops = &mxs_lradc_trigger_ops;
>
> ret = iio_trigger_register(trig);
> - if (ret) {
> + if (ret)
> iio_trigger_free(trig);
> - return ret;
> - }
>
> iio->trig = trig;
You are still setting iio->trig , you dont want to do that in case of failure
> - return 0;
> + return ret;
> }
>
> static void mxs_lradc_trigger_remove(struct iio_dev *iio)
Best regards,
Marek Vasut
^ permalink raw reply [flat|nested] 32+ messages in thread
* Re: [PATCH 2/5] ARM: mxs-lradc: Simplify mxs_lradc_trigger_init error handle
2013-07-15 14:24 ` Marek Vasut
@ 2013-07-15 14:38 ` Otavio Salvador
2013-07-15 23:00 ` Marek Vasut
0 siblings, 1 reply; 32+ messages in thread
From: Otavio Salvador @ 2013-07-15 14:38 UTC (permalink / raw)
To: Marek Vasut
Cc: Jonathan Cameron, Kernel development list, Jonathan Cameron,
Greg Kroah-Hartman, Bill Pemberton, Lars-Peter Clausen,
linux-iio, devel
On Mon, Jul 15, 2013 at 11:24 AM, Marek Vasut <marek.vasut@gmail.com> wrote:
> Dear Otavio Salvador,
>
>> As we have a 'ret' variable with the iio_trigger_register return, this
>> can be used as result.
>>
>> Signed-off-by: Otavio Salvador <otavio@ossystems.com.br>
>> ---
>> drivers/staging/iio/adc/mxs-lradc.c | 6 ++----
>> 1 file changed, 2 insertions(+), 4 deletions(-)
>>
>> diff --git a/drivers/staging/iio/adc/mxs-lradc.c
>> b/drivers/staging/iio/adc/mxs-lradc.c index d92c97a..6d41846 100644
>> --- a/drivers/staging/iio/adc/mxs-lradc.c
>> +++ b/drivers/staging/iio/adc/mxs-lradc.c
>> @@ -671,14 +671,12 @@ static int mxs_lradc_trigger_init(struct iio_dev
>> *iio) trig->ops = &mxs_lradc_trigger_ops;
>>
>> ret = iio_trigger_register(trig);
>> - if (ret) {
>> + if (ret)
>> iio_trigger_free(trig);
>> - return ret;
>> - }
>>
>> iio->trig = trig;
>
>
> You are still setting iio->trig , you dont want to do that in case of failure
In this case it'd end being NULL I think, wouldn't it?
--
Otavio Salvador O.S. Systems
http://www.ossystems.com.br http://projetos.ossystems.com.br
Mobile: +55 (53) 9981-7854 Mobile: +1 (347) 903-9750
^ permalink raw reply [flat|nested] 32+ messages in thread
* Re: [PATCH 1/5] ARM: dts: imx23-evk: enable USB PHY and controller
2013-07-15 14:22 ` Otavio Salvador
@ 2013-07-15 15:39 ` Fabio Estevam
-1 siblings, 0 replies; 32+ messages in thread
From: Fabio Estevam @ 2013-07-15 15:39 UTC (permalink / raw)
To: Otavio Salvador
Cc: Jonathan Cameron, linux-kernel, Fabio Estevam, Shawn Guo,
Russell King, linux-arm-kernel
Hi Otavio,
On Mon, Jul 15, 2013 at 11:22 AM, Otavio Salvador
<otavio@ossystems.com.br> wrote:
> The i.MX23EVK board provides a USB port so the USB PHY and controller
> need to be enabled for it to be usable.
>
> Signed-off-by: Otavio Salvador <otavio@ossystems.com.br>
> ---
You should have Cc'd Shawn Guo on this series.
^ permalink raw reply [flat|nested] 32+ messages in thread
* [PATCH 1/5] ARM: dts: imx23-evk: enable USB PHY and controller
@ 2013-07-15 15:39 ` Fabio Estevam
0 siblings, 0 replies; 32+ messages in thread
From: Fabio Estevam @ 2013-07-15 15:39 UTC (permalink / raw)
To: linux-arm-kernel
Hi Otavio,
On Mon, Jul 15, 2013 at 11:22 AM, Otavio Salvador
<otavio@ossystems.com.br> wrote:
> The i.MX23EVK board provides a USB port so the USB PHY and controller
> need to be enabled for it to be usable.
>
> Signed-off-by: Otavio Salvador <otavio@ossystems.com.br>
> ---
You should have Cc'd Shawn Guo on this series.
^ permalink raw reply [flat|nested] 32+ messages in thread
* Re: [PATCH 4/5] ARM: dts: imx23-olinuxino: enable Low Resolution ADC
2013-07-15 14:22 ` Otavio Salvador
@ 2013-07-15 16:11 ` Alexandre Pereira da Silva
-1 siblings, 0 replies; 32+ messages in thread
From: Alexandre Pereira da Silva @ 2013-07-15 16:11 UTC (permalink / raw)
To: Otavio Salvador
Cc: Jonathan Cameron, LKML, Russell King, Shawn Guo, Fabio Estevam,
Fadil Berisha, linux-arm-kernel
On Mon, Jul 15, 2013 at 11:22 AM, Otavio Salvador
<otavio@ossystems.com.br> wrote:
> The i.XM23 has a internal Low Resolution ADC; this enables the support
> for this device.
>
> Signed-off-by: Otavio Salvador <otavio@ossystems.com.br>
Acked-by: Alexandre Pereira da Silva <aletes.xgr@gmail.com>
^ permalink raw reply [flat|nested] 32+ messages in thread
* [PATCH 4/5] ARM: dts: imx23-olinuxino: enable Low Resolution ADC
@ 2013-07-15 16:11 ` Alexandre Pereira da Silva
0 siblings, 0 replies; 32+ messages in thread
From: Alexandre Pereira da Silva @ 2013-07-15 16:11 UTC (permalink / raw)
To: linux-arm-kernel
On Mon, Jul 15, 2013 at 11:22 AM, Otavio Salvador
<otavio@ossystems.com.br> wrote:
> The i.XM23 has a internal Low Resolution ADC; this enables the support
> for this device.
>
> Signed-off-by: Otavio Salvador <otavio@ossystems.com.br>
Acked-by: Alexandre Pereira da Silva <aletes.xgr@gmail.com>
^ permalink raw reply [flat|nested] 32+ messages in thread
* Re: [PATCH 1/5] ARM: dts: imx23-evk: enable USB PHY and controller
2013-07-15 15:39 ` Fabio Estevam
@ 2013-07-15 16:39 ` Otavio Salvador
-1 siblings, 0 replies; 32+ messages in thread
From: Otavio Salvador @ 2013-07-15 16:39 UTC (permalink / raw)
To: Fabio Estevam
Cc: Jonathan Cameron, Kernel development list, Fabio Estevam,
Shawn Guo, Russell King, linux-arm-kernel
On Mon, Jul 15, 2013 at 12:39 PM, Fabio Estevam <festevam@gmail.com> wrote:
> Hi Otavio,
>
> On Mon, Jul 15, 2013 at 11:22 AM, Otavio Salvador
> <otavio@ossystems.com.br> wrote:
>> The i.MX23EVK board provides a USB port so the USB PHY and controller
>> need to be enabled for it to be usable.
>>
>> Signed-off-by: Otavio Salvador <otavio@ossystems.com.br>
>> ---
>
> You should have Cc'd Shawn Guo on this series.
Oh; thanks for fix my mistake. I am new to kernel development :-)
--
Otavio Salvador O.S. Systems
http://www.ossystems.com.br http://projetos.ossystems.com.br
Mobile: +55 (53) 9981-7854 Mobile: +1 (347) 903-9750
^ permalink raw reply [flat|nested] 32+ messages in thread
* [PATCH 1/5] ARM: dts: imx23-evk: enable USB PHY and controller
@ 2013-07-15 16:39 ` Otavio Salvador
0 siblings, 0 replies; 32+ messages in thread
From: Otavio Salvador @ 2013-07-15 16:39 UTC (permalink / raw)
To: linux-arm-kernel
On Mon, Jul 15, 2013 at 12:39 PM, Fabio Estevam <festevam@gmail.com> wrote:
> Hi Otavio,
>
> On Mon, Jul 15, 2013 at 11:22 AM, Otavio Salvador
> <otavio@ossystems.com.br> wrote:
>> The i.MX23EVK board provides a USB port so the USB PHY and controller
>> need to be enabled for it to be usable.
>>
>> Signed-off-by: Otavio Salvador <otavio@ossystems.com.br>
>> ---
>
> You should have Cc'd Shawn Guo on this series.
Oh; thanks for fix my mistake. I am new to kernel development :-)
--
Otavio Salvador O.S. Systems
http://www.ossystems.com.br http://projetos.ossystems.com.br
Mobile: +55 (53) 9981-7854 Mobile: +1 (347) 903-9750
^ permalink raw reply [flat|nested] 32+ messages in thread
* Re: [PATCH 2/5] ARM: mxs-lradc: Simplify mxs_lradc_trigger_init error handle
2013-07-15 14:38 ` Otavio Salvador
@ 2013-07-15 23:00 ` Marek Vasut
2013-07-16 12:38 ` Otavio Salvador
0 siblings, 1 reply; 32+ messages in thread
From: Marek Vasut @ 2013-07-15 23:00 UTC (permalink / raw)
To: Otavio Salvador
Cc: Jonathan Cameron, Kernel development list, Jonathan Cameron,
Greg Kroah-Hartman, Bill Pemberton, Lars-Peter Clausen,
linux-iio, devel
Dear Otavio Salvador,
> On Mon, Jul 15, 2013 at 11:24 AM, Marek Vasut <marek.vasut@gmail.com> wrote:
> > Dear Otavio Salvador,
> >
> >> As we have a 'ret' variable with the iio_trigger_register return, this
> >> can be used as result.
> >>
> >> Signed-off-by: Otavio Salvador <otavio@ossystems.com.br>
> >> ---
> >>
> >> drivers/staging/iio/adc/mxs-lradc.c | 6 ++----
> >> 1 file changed, 2 insertions(+), 4 deletions(-)
> >>
> >> diff --git a/drivers/staging/iio/adc/mxs-lradc.c
> >> b/drivers/staging/iio/adc/mxs-lradc.c index d92c97a..6d41846 100644
> >> --- a/drivers/staging/iio/adc/mxs-lradc.c
> >> +++ b/drivers/staging/iio/adc/mxs-lradc.c
> >> @@ -671,14 +671,12 @@ static int mxs_lradc_trigger_init(struct iio_dev
> >> *iio) trig->ops = &mxs_lradc_trigger_ops;
> >>
> >> ret = iio_trigger_register(trig);
> >>
> >> - if (ret) {
> >> + if (ret)
> >>
> >> iio_trigger_free(trig);
> >>
> >> - return ret;
> >> - }
> >>
> >> iio->trig = trig;
> >
> > You are still setting iio->trig , you dont want to do that in case of
> > failure
>
> In this case it'd end being NULL I think, wouldn't it?
It won't, see the implementation in:
drivers/iio/industrialio-trigger.c
Best regards,
Marek Vasut
^ permalink raw reply [flat|nested] 32+ messages in thread
* Re: [PATCH 1/5] ARM: dts: imx23-evk: enable USB PHY and controller
2013-07-15 16:39 ` Otavio Salvador
@ 2013-07-16 1:30 ` Shawn Guo
-1 siblings, 0 replies; 32+ messages in thread
From: Shawn Guo @ 2013-07-16 1:30 UTC (permalink / raw)
To: Otavio Salvador
Cc: Fabio Estevam, Jonathan Cameron, Kernel development list,
Fabio Estevam, Russell King, linux-arm-kernel
On Mon, Jul 15, 2013 at 01:39:13PM -0300, Otavio Salvador wrote:
> On Mon, Jul 15, 2013 at 12:39 PM, Fabio Estevam <festevam@gmail.com> wrote:
> > Hi Otavio,
> >
> > On Mon, Jul 15, 2013 at 11:22 AM, Otavio Salvador
> > <otavio@ossystems.com.br> wrote:
> >> The i.MX23EVK board provides a USB port so the USB PHY and controller
> >> need to be enabled for it to be usable.
> >>
> >> Signed-off-by: Otavio Salvador <otavio@ossystems.com.br>
> >> ---
> >
> > You should have Cc'd Shawn Guo on this series.
>
> Oh; thanks for fix my mistake. I am new to kernel development :-)
I'm on copy of patches 1, 3 and 4. 2 and 5 are missing?
Shawn
^ permalink raw reply [flat|nested] 32+ messages in thread
* [PATCH 1/5] ARM: dts: imx23-evk: enable USB PHY and controller
@ 2013-07-16 1:30 ` Shawn Guo
0 siblings, 0 replies; 32+ messages in thread
From: Shawn Guo @ 2013-07-16 1:30 UTC (permalink / raw)
To: linux-arm-kernel
On Mon, Jul 15, 2013 at 01:39:13PM -0300, Otavio Salvador wrote:
> On Mon, Jul 15, 2013 at 12:39 PM, Fabio Estevam <festevam@gmail.com> wrote:
> > Hi Otavio,
> >
> > On Mon, Jul 15, 2013 at 11:22 AM, Otavio Salvador
> > <otavio@ossystems.com.br> wrote:
> >> The i.MX23EVK board provides a USB port so the USB PHY and controller
> >> need to be enabled for it to be usable.
> >>
> >> Signed-off-by: Otavio Salvador <otavio@ossystems.com.br>
> >> ---
> >
> > You should have Cc'd Shawn Guo on this series.
>
> Oh; thanks for fix my mistake. I am new to kernel development :-)
I'm on copy of patches 1, 3 and 4. 2 and 5 are missing?
Shawn
^ permalink raw reply [flat|nested] 32+ messages in thread
* Re: [PATCH 1/5] ARM: dts: imx23-evk: enable USB PHY and controller
2013-07-15 14:22 ` Otavio Salvador
@ 2013-07-16 3:42 ` Fabio Estevam
-1 siblings, 0 replies; 32+ messages in thread
From: Fabio Estevam @ 2013-07-16 3:42 UTC (permalink / raw)
To: Otavio Salvador
Cc: Jonathan Cameron, linux-kernel, Fabio Estevam, Shawn Guo,
Russell King, linux-arm-kernel
On Mon, Jul 15, 2013 at 11:22 AM, Otavio Salvador
<otavio@ossystems.com.br> wrote:
> The i.MX23EVK board provides a USB port so the USB PHY and controller
> need to be enabled for it to be usable.
Has this been tested? I don't see in this patch any code to enable USB power.
^ permalink raw reply [flat|nested] 32+ messages in thread
* [PATCH 1/5] ARM: dts: imx23-evk: enable USB PHY and controller
@ 2013-07-16 3:42 ` Fabio Estevam
0 siblings, 0 replies; 32+ messages in thread
From: Fabio Estevam @ 2013-07-16 3:42 UTC (permalink / raw)
To: linux-arm-kernel
On Mon, Jul 15, 2013 at 11:22 AM, Otavio Salvador
<otavio@ossystems.com.br> wrote:
> The i.MX23EVK board provides a USB port so the USB PHY and controller
> need to be enabled for it to be usable.
Has this been tested? I don't see in this patch any code to enable USB power.
^ permalink raw reply [flat|nested] 32+ messages in thread
* Re: [PATCH 5/5] iio: add configuration option for debug support
2013-07-15 14:22 ` [PATCH 5/5] iio: add configuration option for debug support Otavio Salvador
@ 2013-07-16 9:58 ` Jonathan Cameron
2013-07-16 10:26 ` Lars-Peter Clausen
0 siblings, 1 reply; 32+ messages in thread
From: Jonathan Cameron @ 2013-07-16 9:58 UTC (permalink / raw)
To: Otavio Salvador; +Cc: linux-kernel, Jonathan Cameron, linux-iio
On 07/15/2013 03:22 PM, Otavio Salvador wrote:
> Signed-off-by: Otavio Salvador <otavio@ossystems.com.br>
seems a sensible facility to have available.
Applied to the togreg branch of iio.git
> ---
> drivers/iio/Kconfig | 7 +++++++
> drivers/iio/Makefile | 2 ++
> 2 files changed, 9 insertions(+)
>
> diff --git a/drivers/iio/Kconfig b/drivers/iio/Kconfig
> index 9af763a..d1f06f2 100644
> --- a/drivers/iio/Kconfig
> +++ b/drivers/iio/Kconfig
> @@ -12,6 +12,13 @@ menuconfig IIO
>
> if IIO
>
> +config IIO_DEBUG
> + boolean "Debug support for Industrial I/O"
> + depends on DEBUG_KERNEL
> + help
> + Say "yes" to enable debug messaging (like dev_dbg and pr_debug),
> + sysfs, and debugfs support in Industrial I/O subsystem and drivers.
> +
> config IIO_BUFFER
> bool "Enable buffer support within IIO"
> help
> diff --git a/drivers/iio/Makefile b/drivers/iio/Makefile
> index 7a3866c..6e43e5b 100644
> --- a/drivers/iio/Makefile
> +++ b/drivers/iio/Makefile
> @@ -2,6 +2,8 @@
> # Makefile for the industrial I/O core.
> #
>
> +ccflags-$(CONFIG_IIO_DEBUG) := -DDEBUG
> +
> obj-$(CONFIG_IIO) += industrialio.o
> industrialio-y := industrialio-core.o industrialio-event.o inkern.o
> industrialio-$(CONFIG_IIO_BUFFER) += industrialio-buffer.o
>
^ permalink raw reply [flat|nested] 32+ messages in thread
* Re: [PATCH 5/5] iio: add configuration option for debug support
2013-07-16 9:58 ` Jonathan Cameron
@ 2013-07-16 10:26 ` Lars-Peter Clausen
2013-07-16 12:01 ` Otavio Salvador
2013-07-16 16:06 ` Jonathan Cameron
0 siblings, 2 replies; 32+ messages in thread
From: Lars-Peter Clausen @ 2013-07-16 10:26 UTC (permalink / raw)
To: Jonathan Cameron
Cc: Otavio Salvador, linux-kernel, Jonathan Cameron, linux-iio
On 07/16/2013 11:58 AM, Jonathan Cameron wrote:
> On 07/15/2013 03:22 PM, Otavio Salvador wrote:
>> Signed-off-by: Otavio Salvador <otavio@ossystems.com.br>
> seems a sensible facility to have available.
>
> Applied to the togreg branch of iio.git
Can you update the Kconfig help text?
>> ---
>> drivers/iio/Kconfig | 7 +++++++
>> drivers/iio/Makefile | 2 ++
>> 2 files changed, 9 insertions(+)
>>
>> diff --git a/drivers/iio/Kconfig b/drivers/iio/Kconfig
>> index 9af763a..d1f06f2 100644
>> --- a/drivers/iio/Kconfig
>> +++ b/drivers/iio/Kconfig
>> @@ -12,6 +12,13 @@ menuconfig IIO
>>
>> if IIO
>>
>> +config IIO_DEBUG
>> + boolean "Debug support for Industrial I/O"
>> + depends on DEBUG_KERNEL
>> + help
>> + Say "yes" to enable debug messaging (like dev_dbg and pr_debug),
>> + sysfs, and debugfs support in Industrial I/O subsystem and drivers.
This has nothing to do with either sysfs or debugfs support.
>> +
>> config IIO_BUFFER
>> bool "Enable buffer support within IIO"
>> help
>> diff --git a/drivers/iio/Makefile b/drivers/iio/Makefile
>> index 7a3866c..6e43e5b 100644
>> --- a/drivers/iio/Makefile
>> +++ b/drivers/iio/Makefile
>> @@ -2,6 +2,8 @@
>> # Makefile for the industrial I/O core.
>> #
>>
>> +ccflags-$(CONFIG_IIO_DEBUG) := -DDEBUG
>> +
>> obj-$(CONFIG_IIO) += industrialio.o
>> industrialio-y := industrialio-core.o industrialio-event.o inkern.o
>> industrialio-$(CONFIG_IIO_BUFFER) += industrialio-buffer.o
>>
> --
> To unsubscribe from this list: send the line "unsubscribe linux-iio" in
> the body of a message to majordomo@vger.kernel.org
> More majordomo info at http://vger.kernel.org/majordomo-info.html
>
^ permalink raw reply [flat|nested] 32+ messages in thread
* Re: [PATCH 5/5] iio: add configuration option for debug support
2013-07-16 10:26 ` Lars-Peter Clausen
@ 2013-07-16 12:01 ` Otavio Salvador
2013-07-16 12:09 ` Lars-Peter Clausen
2013-07-16 16:06 ` Jonathan Cameron
1 sibling, 1 reply; 32+ messages in thread
From: Otavio Salvador @ 2013-07-16 12:01 UTC (permalink / raw)
To: Lars-Peter Clausen
Cc: Jonathan Cameron, Kernel development list, Jonathan Cameron, linux-iio
On Tue, Jul 16, 2013 at 7:26 AM, Lars-Peter Clausen <lars@metafoo.de> wrote:
> On 07/16/2013 11:58 AM, Jonathan Cameron wrote:
>> On 07/15/2013 03:22 PM, Otavio Salvador wrote:
>>> Signed-off-by: Otavio Salvador <otavio@ossystems.com.br>
>> seems a sensible facility to have available.
>>
>> Applied to the togreg branch of iio.git
>
> Can you update the Kconfig help text?
Sure I can. Jonathan seem to have applied it so should I send a v2 or
a new patch?
>>> ---
>>> drivers/iio/Kconfig | 7 +++++++
>>> drivers/iio/Makefile | 2 ++
>>> 2 files changed, 9 insertions(+)
>>>
>>> diff --git a/drivers/iio/Kconfig b/drivers/iio/Kconfig
>>> index 9af763a..d1f06f2 100644
>>> --- a/drivers/iio/Kconfig
>>> +++ b/drivers/iio/Kconfig
>>> @@ -12,6 +12,13 @@ menuconfig IIO
>>>
>>> if IIO
>>>
>>> +config IIO_DEBUG
>>> + boolean "Debug support for Industrial I/O"
>>> + depends on DEBUG_KERNEL
>>> + help
>>> + Say "yes" to enable debug messaging (like dev_dbg and pr_debug),
>>> + sysfs, and debugfs support in Industrial I/O subsystem and drivers.
>
> This has nothing to do with either sysfs or debugfs support.
Say "yes" to enable debug messaging (like dev_dbg and pr_debug)
support in Industrial I/O subsystem and drivers.
?
--
Otavio Salvador O.S. Systems
http://www.ossystems.com.br http://projetos.ossystems.com.br
Mobile: +55 (53) 9981-7854 Mobile: +1 (347) 903-9750
^ permalink raw reply [flat|nested] 32+ messages in thread
* Re: [PATCH 5/5] iio: add configuration option for debug support
2013-07-16 12:01 ` Otavio Salvador
@ 2013-07-16 12:09 ` Lars-Peter Clausen
2013-07-16 16:11 ` Jonathan Cameron
0 siblings, 1 reply; 32+ messages in thread
From: Lars-Peter Clausen @ 2013-07-16 12:09 UTC (permalink / raw)
To: Otavio Salvador
Cc: Jonathan Cameron, Kernel development list, Jonathan Cameron, linux-iio
On 07/16/2013 02:01 PM, Otavio Salvador wrote:
> On Tue, Jul 16, 2013 at 7:26 AM, Lars-Peter Clausen <lars@metafoo.de> wrote:
>> On 07/16/2013 11:58 AM, Jonathan Cameron wrote:
>>> On 07/15/2013 03:22 PM, Otavio Salvador wrote:
>>>> Signed-off-by: Otavio Salvador <otavio@ossystems.com.br>
>>> seems a sensible facility to have available.
>>>
>>> Applied to the togreg branch of iio.git
>>
>> Can you update the Kconfig help text?
>
> Sure I can. Jonathan seem to have applied it so should I send a v2 or
> a new patch?
>
He hasn't pushed things out yet, I think a v2 will be fine.
>>>> ---
>>>> drivers/iio/Kconfig | 7 +++++++
>>>> drivers/iio/Makefile | 2 ++
>>>> 2 files changed, 9 insertions(+)
>>>>
>>>> diff --git a/drivers/iio/Kconfig b/drivers/iio/Kconfig
>>>> index 9af763a..d1f06f2 100644
>>>> --- a/drivers/iio/Kconfig
>>>> +++ b/drivers/iio/Kconfig
>>>> @@ -12,6 +12,13 @@ menuconfig IIO
>>>>
>>>> if IIO
>>>>
>>>> +config IIO_DEBUG
>>>> + boolean "Debug support for Industrial I/O"
>>>> + depends on DEBUG_KERNEL
>>>> + help
>>>> + Say "yes" to enable debug messaging (like dev_dbg and pr_debug),
>>>> + sysfs, and debugfs support in Industrial I/O subsystem and drivers.
>>
>> This has nothing to do with either sysfs or debugfs support.
>
> Say "yes" to enable debug messaging (like dev_dbg and pr_debug)
> support in Industrial I/O subsystem and drivers.
>
> ?
>
> --
> Otavio Salvador O.S. Systems
> http://www.ossystems.com.br http://projetos.ossystems.com.br
> Mobile: +55 (53) 9981-7854 Mobile: +1 (347) 903-9750
>
^ permalink raw reply [flat|nested] 32+ messages in thread
* Re: [PATCH 1/5] ARM: dts: imx23-evk: enable USB PHY and controller
2013-07-16 3:42 ` Fabio Estevam
@ 2013-07-16 12:24 ` Otavio Salvador
-1 siblings, 0 replies; 32+ messages in thread
From: Otavio Salvador @ 2013-07-16 12:24 UTC (permalink / raw)
To: Fabio Estevam
Cc: Jonathan Cameron, Kernel development list, Fabio Estevam,
Shawn Guo, Russell King, linux-arm-kernel
On Tue, Jul 16, 2013 at 12:42 AM, Fabio Estevam <festevam@gmail.com> wrote:
> On Mon, Jul 15, 2013 at 11:22 AM, Otavio Salvador
> <otavio@ossystems.com.br> wrote:
>> The i.MX23EVK board provides a USB port so the USB PHY and controller
>> need to be enabled for it to be usable.
>
> Has this been tested? I don't see in this patch any code to enable USB power.
It does, I am using it.
--
Otavio Salvador O.S. Systems
http://www.ossystems.com.br http://projetos.ossystems.com.br
Mobile: +55 (53) 9981-7854 Mobile: +1 (347) 903-9750
^ permalink raw reply [flat|nested] 32+ messages in thread
* [PATCH 1/5] ARM: dts: imx23-evk: enable USB PHY and controller
@ 2013-07-16 12:24 ` Otavio Salvador
0 siblings, 0 replies; 32+ messages in thread
From: Otavio Salvador @ 2013-07-16 12:24 UTC (permalink / raw)
To: linux-arm-kernel
On Tue, Jul 16, 2013 at 12:42 AM, Fabio Estevam <festevam@gmail.com> wrote:
> On Mon, Jul 15, 2013 at 11:22 AM, Otavio Salvador
> <otavio@ossystems.com.br> wrote:
>> The i.MX23EVK board provides a USB port so the USB PHY and controller
>> need to be enabled for it to be usable.
>
> Has this been tested? I don't see in this patch any code to enable USB power.
It does, I am using it.
--
Otavio Salvador O.S. Systems
http://www.ossystems.com.br http://projetos.ossystems.com.br
Mobile: +55 (53) 9981-7854 Mobile: +1 (347) 903-9750
^ permalink raw reply [flat|nested] 32+ messages in thread
* Re: [PATCH 1/5] ARM: dts: imx23-evk: enable USB PHY and controller
2013-07-16 1:30 ` Shawn Guo
@ 2013-07-16 12:25 ` Otavio Salvador
-1 siblings, 0 replies; 32+ messages in thread
From: Otavio Salvador @ 2013-07-16 12:25 UTC (permalink / raw)
To: Shawn Guo
Cc: Fabio Estevam, Jonathan Cameron, Kernel development list,
Fabio Estevam, Russell King, linux-arm-kernel
On Mon, Jul 15, 2013 at 10:30 PM, Shawn Guo <shawn.guo@linaro.org> wrote:
> On Mon, Jul 15, 2013 at 01:39:13PM -0300, Otavio Salvador wrote:
>> On Mon, Jul 15, 2013 at 12:39 PM, Fabio Estevam <festevam@gmail.com> wrote:
>> > Hi Otavio,
>> >
>> > On Mon, Jul 15, 2013 at 11:22 AM, Otavio Salvador
>> > <otavio@ossystems.com.br> wrote:
>> >> The i.MX23EVK board provides a USB port so the USB PHY and controller
>> >> need to be enabled for it to be usable.
>> >>
>> >> Signed-off-by: Otavio Salvador <otavio@ossystems.com.br>
>> >> ---
>> >
>> > You should have Cc'd Shawn Guo on this series.
>>
>> Oh; thanks for fix my mistake. I am new to kernel development :-)
>
> I'm on copy of patches 1, 3 and 4. 2 and 5 are missing?
I will prepare a v2, as per Jonathan feedback and will add you in all them.
--
Otavio Salvador O.S. Systems
http://www.ossystems.com.br http://projetos.ossystems.com.br
Mobile: +55 (53) 9981-7854 Mobile: +1 (347) 903-9750
^ permalink raw reply [flat|nested] 32+ messages in thread
* [PATCH 1/5] ARM: dts: imx23-evk: enable USB PHY and controller
@ 2013-07-16 12:25 ` Otavio Salvador
0 siblings, 0 replies; 32+ messages in thread
From: Otavio Salvador @ 2013-07-16 12:25 UTC (permalink / raw)
To: linux-arm-kernel
On Mon, Jul 15, 2013 at 10:30 PM, Shawn Guo <shawn.guo@linaro.org> wrote:
> On Mon, Jul 15, 2013 at 01:39:13PM -0300, Otavio Salvador wrote:
>> On Mon, Jul 15, 2013 at 12:39 PM, Fabio Estevam <festevam@gmail.com> wrote:
>> > Hi Otavio,
>> >
>> > On Mon, Jul 15, 2013 at 11:22 AM, Otavio Salvador
>> > <otavio@ossystems.com.br> wrote:
>> >> The i.MX23EVK board provides a USB port so the USB PHY and controller
>> >> need to be enabled for it to be usable.
>> >>
>> >> Signed-off-by: Otavio Salvador <otavio@ossystems.com.br>
>> >> ---
>> >
>> > You should have Cc'd Shawn Guo on this series.
>>
>> Oh; thanks for fix my mistake. I am new to kernel development :-)
>
> I'm on copy of patches 1, 3 and 4. 2 and 5 are missing?
I will prepare a v2, as per Jonathan feedback and will add you in all them.
--
Otavio Salvador O.S. Systems
http://www.ossystems.com.br http://projetos.ossystems.com.br
Mobile: +55 (53) 9981-7854 Mobile: +1 (347) 903-9750
^ permalink raw reply [flat|nested] 32+ messages in thread
* Re: [PATCH 2/5] ARM: mxs-lradc: Simplify mxs_lradc_trigger_init error handle
2013-07-15 23:00 ` Marek Vasut
@ 2013-07-16 12:38 ` Otavio Salvador
0 siblings, 0 replies; 32+ messages in thread
From: Otavio Salvador @ 2013-07-16 12:38 UTC (permalink / raw)
To: Marek Vasut
Cc: Jonathan Cameron, Kernel development list, Jonathan Cameron,
Greg Kroah-Hartman, Bill Pemberton, Lars-Peter Clausen,
linux-iio, devel
On Mon, Jul 15, 2013 at 8:00 PM, Marek Vasut <marex@denx.de> wrote:
> Dear Otavio Salvador,
>
>> On Mon, Jul 15, 2013 at 11:24 AM, Marek Vasut <marek.vasut@gmail.com> wrote:
>> > Dear Otavio Salvador,
>> >
>> >> As we have a 'ret' variable with the iio_trigger_register return, this
>> >> can be used as result.
>> >>
>> >> Signed-off-by: Otavio Salvador <otavio@ossystems.com.br>
>> >> ---
>> >>
>> >> drivers/staging/iio/adc/mxs-lradc.c | 6 ++----
>> >> 1 file changed, 2 insertions(+), 4 deletions(-)
>> >>
>> >> diff --git a/drivers/staging/iio/adc/mxs-lradc.c
>> >> b/drivers/staging/iio/adc/mxs-lradc.c index d92c97a..6d41846 100644
>> >> --- a/drivers/staging/iio/adc/mxs-lradc.c
>> >> +++ b/drivers/staging/iio/adc/mxs-lradc.c
>> >> @@ -671,14 +671,12 @@ static int mxs_lradc_trigger_init(struct iio_dev
>> >> *iio) trig->ops = &mxs_lradc_trigger_ops;
>> >>
>> >> ret = iio_trigger_register(trig);
>> >>
>> >> - if (ret) {
>> >> + if (ret)
>> >>
>> >> iio_trigger_free(trig);
>> >>
>> >> - return ret;
>> >> - }
>> >>
>> >> iio->trig = trig;
>> >
>> > You are still setting iio->trig , you dont want to do that in case of
>> > failure
>>
>> In this case it'd end being NULL I think, wouldn't it?
>
> It won't, see the implementation in:
>
> drivers/iio/industrialio-trigger.c
Agreed; will drop this from v2.
--
Otavio Salvador O.S. Systems
http://www.ossystems.com.br http://projetos.ossystems.com.br
Mobile: +55 (53) 9981-7854 Mobile: +1 (347) 903-9750
^ permalink raw reply [flat|nested] 32+ messages in thread
* Re: [PATCH 5/5] iio: add configuration option for debug support
2013-07-16 10:26 ` Lars-Peter Clausen
2013-07-16 12:01 ` Otavio Salvador
@ 2013-07-16 16:06 ` Jonathan Cameron
1 sibling, 0 replies; 32+ messages in thread
From: Jonathan Cameron @ 2013-07-16 16:06 UTC (permalink / raw)
To: Lars-Peter Clausen
Cc: Otavio Salvador, linux-kernel, Jonathan Cameron, linux-iio
Lars-Peter Clausen <lars@metafoo.de> wrote:
>On 07/16/2013 11:58 AM, Jonathan Cameron wrote:
>> On 07/15/2013 03:22 PM, Otavio Salvador wrote:
>>> Signed-off-by: Otavio Salvador <otavio@ossystems.com.br>
>> seems a sensible facility to have available.
>>
>> Applied to the togreg branch of iio.git
>
>Can you update the Kconfig help text?
Good point. Will fix!
>
>>> ---
>>> drivers/iio/Kconfig | 7 +++++++
>>> drivers/iio/Makefile | 2 ++
>>> 2 files changed, 9 insertions(+)
>>>
>>> diff --git a/drivers/iio/Kconfig b/drivers/iio/Kconfig
>>> index 9af763a..d1f06f2 100644
>>> --- a/drivers/iio/Kconfig
>>> +++ b/drivers/iio/Kconfig
>>> @@ -12,6 +12,13 @@ menuconfig IIO
>>>
>>> if IIO
>>>
>>> +config IIO_DEBUG
>>> + boolean "Debug support for Industrial I/O"
>>> + depends on DEBUG_KERNEL
>>> + help
>>> + Say "yes" to enable debug messaging (like dev_dbg and pr_debug),
>>> + sysfs, and debugfs support in Industrial I/O subsystem and
>drivers.
>
>This has nothing to do with either sysfs or debugfs support.
>
>>> +
>>> config IIO_BUFFER
>>> bool "Enable buffer support within IIO"
>>> help
>>> diff --git a/drivers/iio/Makefile b/drivers/iio/Makefile
>>> index 7a3866c..6e43e5b 100644
>>> --- a/drivers/iio/Makefile
>>> +++ b/drivers/iio/Makefile
>>> @@ -2,6 +2,8 @@
>>> # Makefile for the industrial I/O core.
>>> #
>>>
>>> +ccflags-$(CONFIG_IIO_DEBUG) := -DDEBUG
>>> +
>>> obj-$(CONFIG_IIO) += industrialio.o
>>> industrialio-y := industrialio-core.o industrialio-event.o inkern.o
>>> industrialio-$(CONFIG_IIO_BUFFER) += industrialio-buffer.o
>>>
>> --
>> To unsubscribe from this list: send the line "unsubscribe linux-iio"
>in
>> the body of a message to majordomo@vger.kernel.org
>> More majordomo info at http://vger.kernel.org/majordomo-info.html
>>
--
Sent from my Android phone with K-9 Mail. Please excuse my brevity.
^ permalink raw reply [flat|nested] 32+ messages in thread
* Re: [PATCH 5/5] iio: add configuration option for debug support
2013-07-16 12:09 ` Lars-Peter Clausen
@ 2013-07-16 16:11 ` Jonathan Cameron
0 siblings, 0 replies; 32+ messages in thread
From: Jonathan Cameron @ 2013-07-16 16:11 UTC (permalink / raw)
To: Lars-Peter Clausen, Otavio Salvador
Cc: Kernel development list, Jonathan Cameron, linux-iio
Lars-Peter Clausen <lars@metafoo.de> wrote:
>On 07/16/2013 02:01 PM, Otavio Salvador wrote:
>> On Tue, Jul 16, 2013 at 7:26 AM, Lars-Peter Clausen <lars@metafoo.de>
>wrote:
>>> On 07/16/2013 11:58 AM, Jonathan Cameron wrote:
>>>> On 07/15/2013 03:22 PM, Otavio Salvador wrote:
>>>>> Signed-off-by: Otavio Salvador <otavio@ossystems.com.br>
>>>> seems a sensible facility to have available.
>>>>
>>>> Applied to the togreg branch of iio.git
>>>
>>> Can you update the Kconfig help text?
>>
>> Sure I can. Jonathan seem to have applied it so should I send a v2 or
>> a new patch?
>>
>
>He hasn't pushed things out yet, I think a v2 will be fine.
>
Yup a v2 is fine. Actually I regularly rebase this tree and it may eat babies! All work more or less should be against Greg's staging/staging-next tree as that is nice and non debasing.
>>>>> ---
>>>>> drivers/iio/Kconfig | 7 +++++++
>>>>> drivers/iio/Makefile | 2 ++
>>>>> 2 files changed, 9 insertions(+)
>>>>>
>>>>> diff --git a/drivers/iio/Kconfig b/drivers/iio/Kconfig
>>>>> index 9af763a..d1f06f2 100644
>>>>> --- a/drivers/iio/Kconfig
>>>>> +++ b/drivers/iio/Kconfig
>>>>> @@ -12,6 +12,13 @@ menuconfig IIO
>>>>>
>>>>> if IIO
>>>>>
>>>>> +config IIO_DEBUG
>>>>> + boolean "Debug support for Industrial I/O"
>>>>> + depends on DEBUG_KERNEL
>>>>> + help
>>>>> + Say "yes" to enable debug messaging (like dev_dbg and
>pr_debug),
>>>>> + sysfs, and debugfs support in Industrial I/O subsystem and
>drivers.
>>>
>>> This has nothing to do with either sysfs or debugfs support.
>>
>> Say "yes" to enable debug messaging (like dev_dbg and pr_debug)
>> support in Industrial I/O subsystem and drivers.
>>
>> ?
>>
>> --
>> Otavio Salvador O.S. Systems
>> http://www.ossystems.com.br http://projetos.ossystems.com.br
>> Mobile: +55 (53) 9981-7854 Mobile: +1 (347) 903-9750
>>
>
>--
>To unsubscribe from this list: send the line "unsubscribe linux-iio" in
>the body of a message to majordomo@vger.kernel.org
>More majordomo info at http://vger.kernel.org/majordomo-info.html
--
Sent from my Android phone with K-9 Mail. Please excuse my brevity.
^ permalink raw reply [flat|nested] 32+ messages in thread
end of thread, other threads:[~2013-07-16 16:11 UTC | newest]
Thread overview: 32+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2013-07-15 14:22 [PATCH 1/5] ARM: dts: imx23-evk: enable USB PHY and controller Otavio Salvador
2013-07-15 14:22 ` Otavio Salvador
2013-07-15 14:22 ` [PATCH 2/5] ARM: mxs-lradc: Simplify mxs_lradc_trigger_init error handle Otavio Salvador
2013-07-15 14:24 ` Marek Vasut
2013-07-15 14:38 ` Otavio Salvador
2013-07-15 23:00 ` Marek Vasut
2013-07-16 12:38 ` Otavio Salvador
2013-07-15 14:22 ` [PATCH 3/5] ARM: dts: imx23-evk: enable Low Resolution ADC Otavio Salvador
2013-07-15 14:22 ` Otavio Salvador
2013-07-15 14:22 ` [PATCH 4/5] ARM: dts: imx23-olinuxino: " Otavio Salvador
2013-07-15 14:22 ` Otavio Salvador
2013-07-15 16:11 ` Alexandre Pereira da Silva
2013-07-15 16:11 ` Alexandre Pereira da Silva
2013-07-15 14:22 ` [PATCH 5/5] iio: add configuration option for debug support Otavio Salvador
2013-07-16 9:58 ` Jonathan Cameron
2013-07-16 10:26 ` Lars-Peter Clausen
2013-07-16 12:01 ` Otavio Salvador
2013-07-16 12:09 ` Lars-Peter Clausen
2013-07-16 16:11 ` Jonathan Cameron
2013-07-16 16:06 ` Jonathan Cameron
2013-07-15 15:39 ` [PATCH 1/5] ARM: dts: imx23-evk: enable USB PHY and controller Fabio Estevam
2013-07-15 15:39 ` Fabio Estevam
2013-07-15 16:39 ` Otavio Salvador
2013-07-15 16:39 ` Otavio Salvador
2013-07-16 1:30 ` Shawn Guo
2013-07-16 1:30 ` Shawn Guo
2013-07-16 12:25 ` Otavio Salvador
2013-07-16 12:25 ` Otavio Salvador
2013-07-16 3:42 ` Fabio Estevam
2013-07-16 3:42 ` Fabio Estevam
2013-07-16 12:24 ` Otavio Salvador
2013-07-16 12:24 ` Otavio Salvador
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.