[PATCH] mtd: nand: fix bug writing 1 byte less than page size

2016-07-18 Thread Hector Palacios
overed in U-Boot where the issue can be reproduced by writing any size that is 1 byte less than a page-size multiple. For example, on a NAND with 2K page (0x800): => nand erase.part => nand write $loadaddr 7ff Signed-off-by: Hector Palacios <hector.palac...@digi.com>

[PATCH] mtd: nand: fix bug writing 1 byte less than page size

2016-07-18 Thread Hector Palacios
overed in U-Boot where the issue can be reproduced by writing any size that is 1 byte less than a page-size multiple. For example, on a NAND with 2K page (0x800): => nand erase.part => nand write $loadaddr 7ff Signed-off-by: Hector Palacios --- drivers/mtd/nand/nand_base

Re: [PATCH v3 2/5] ARM: dts: add reference voltage property for MXS LRADC

2013-09-23 Thread Hector Palacios
cribe linux-iio" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Best regards, -- Hector Palacios -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to major

Re: [PATCH v3 2/5] ARM: dts: add reference voltage property for MXS LRADC

2013-09-23 Thread Hector Palacios
linux-iio in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Best regards, -- Hector Palacios -- To unsubscribe from this list: send the line unsubscribe linux-kernel in the body of a message to majord...@vger.kernel.org

Re: [PATCH v3 2/5] ARM: dts: add reference voltage property for MXS LRADC

2013-08-22 Thread Hector Palacios
Dear Alexandre, On 08/22/2013 12:13 AM, Alexandre Belloni wrote: Hi Pawel, On 14/08/2013 16:44, Pawel Moll wrote: On Tue, 2013-08-13 at 22:23 +0100, Jonathan Cameron wrote: On 07/22/13 15:04, Hector Palacios wrote: Some LRADC channels have fixed pre-dividers so they can measure different

Re: [PATCH v3 2/5] ARM: dts: add reference voltage property for MXS LRADC

2013-08-22 Thread Hector Palacios
Dear Alexandre, On 08/22/2013 12:13 AM, Alexandre Belloni wrote: Hi Pawel, On 14/08/2013 16:44, Pawel Moll wrote: On Tue, 2013-08-13 at 22:23 +0100, Jonathan Cameron wrote: On 07/22/13 15:04, Hector Palacios wrote: Some LRADC channels have fixed pre-dividers so they can measure different

Re: [PATCH] bluetooth: remove wrong dependency for BT_ATH3K

2013-07-30 Thread Hector Palacios
Hi Marcel, On 07/29/2013 06:43 PM, Marcel Holtmann wrote: Hi Hector, The Atheros firmware download driver is not dependent on HCI USB driver. This driver is needed to complement any Atheros driver like BT_HCIUART_ATH3K. Signed-off-by: Hector Palacios CC: Suraj Sumangala CC: Vikram

Re: [PATCH] bluetooth: remove wrong dependency for BT_ATH3K

2013-07-30 Thread Hector Palacios
Hi Marcel, On 07/29/2013 06:43 PM, Marcel Holtmann wrote: Hi Hector, The Atheros firmware download driver is not dependent on HCI USB driver. This driver is needed to complement any Atheros driver like BT_HCIUART_ATH3K. Signed-off-by: Hector Palacios hector.palac...@digi.com CC: Suraj

[PATCH] bluetooth: remove wrong dependency for BT_ATH3K

2013-07-29 Thread Hector Palacios
The Atheros firmware download driver is not dependent on HCI USB driver. This driver is needed to complement any Atheros driver like BT_HCIUART_ATH3K. Signed-off-by: Hector Palacios CC: Suraj Sumangala CC: Vikram Kandukuri --- drivers/bluetooth/Kconfig | 1 - 1 file changed, 1 deletion

[PATCH] bluetooth: remove wrong dependency for BT_ATH3K

2013-07-29 Thread Hector Palacios
The Atheros firmware download driver is not dependent on HCI USB driver. This driver is needed to complement any Atheros driver like BT_HCIUART_ATH3K. Signed-off-by: Hector Palacios hector.palac...@digi.com CC: Suraj Sumangala su...@atheros.com CC: Vikram Kandukuri vikram.kanduk...@atheros.com

Re: [PATCH v3 4/5] iio: mxs-lradc: add scale_available file to channels

2013-07-23 Thread Hector Palacios
Dear Lars, On 07/23/2013 10:46 AM, Lars-Peter Clausen wrote: On 07/22/2013 04:04 PM, Hector Palacios wrote: [...] +static ssize_t mxs_lradc_show_scale_available_ch(struct device *dev, + struct device_attribute *attr, + char *buf, + int ch

Re: [PATCH v3 4/5] iio: mxs-lradc: add scale_available file to channels

2013-07-23 Thread Hector Palacios
Hello Marek, On 07/23/2013 12:36 AM, Marek Vasut wrote: Dear Hector Palacios, Adds in_voltageX_scale_available file for every channel to read the different available scales. There are two scales per channel: [0] = divider_by_two disabled (default) [1] = divider_by_two enabled The scale

Re: [PATCH v3 4/5] iio: mxs-lradc: add scale_available file to channels

2013-07-23 Thread Hector Palacios
Hello Marek, On 07/23/2013 12:36 AM, Marek Vasut wrote: Dear Hector Palacios, Adds in_voltageX_scale_available file for every channel to read the different available scales. There are two scales per channel: [0] = divider_by_two disabled (default) [1] = divider_by_two enabled The scale

Re: [PATCH v3 4/5] iio: mxs-lradc: add scale_available file to channels

2013-07-23 Thread Hector Palacios
Dear Lars, On 07/23/2013 10:46 AM, Lars-Peter Clausen wrote: On 07/22/2013 04:04 PM, Hector Palacios wrote: [...] +static ssize_t mxs_lradc_show_scale_available_ch(struct device *dev, + struct device_attribute *attr, + char *buf, + int ch

[PATCH v3 4/5] iio: mxs-lradc: add scale_available file to channels

2013-07-22 Thread Hector Palacios
-by: Hector Palacios --- drivers/staging/iio/adc/mxs-lradc.c | 112 +++- 1 file changed, 111 insertions(+), 1 deletion(-) diff --git a/drivers/staging/iio/adc/mxs-lradc.c b/drivers/staging/iio/adc/mxs-lradc.c index 5037577..5d332b1 100644 --- a/drivers/staging/iio

[PATCH v3 0/5] iio: mxs-lradc: add support to optional divider_by_two

2013-07-22 Thread Hector Palacios
platform. Could someone please test on an i.MX23? Hector Palacios (5): iio: mxs-lradc: change the realbits to 12 ARM: dts: add reference voltage property for MXS LRADC iio: mxs-lradc: add scale attribute to channels iio: mxs-lradc: add scale_available file to channels iio: mxs-lradc: add

[PATCH v3 3/5] iio: mxs-lradc: add scale attribute to channels

2013-07-22 Thread Hector Palacios
) - allows to read the scaling attribute (computed from the Vref) Signed-off-by: Hector Palacios . Acked-by: Marek Vasut --- drivers/staging/iio/adc/mxs-lradc.c | 77 + 1 file changed, 53 insertions(+), 24 deletions(-) diff --git a/drivers/staging/iio/adc/mxs-lradc.c

[PATCH v3 2/5] ARM: dts: add reference voltage property for MXS LRADC

2013-07-22 Thread Hector Palacios
Some LRADC channels have fixed pre-dividers so they can measure different voltages at full scale. The reference voltage allows to expose a scaling attribute through the IIO sysfs so that a user can compute the real voltage out of a measured sample value. Signed-off-by: Hector Palacios Acked

[PATCH v3 1/5] iio: mxs-lradc: change the realbits to 12

2013-07-22 Thread Hector Palacios
. Besides, the driver was using an 18 bit mask (LRADC_CH_VALUE_MASK) to report touch coordinates to userland. A 12 bit mask should be used instead or else the touch libraries will expect a coordinates range between 0 and 0x3 (18 bits), instead of between 0 and 0xfff (12 bits). Signed-off-by: Hector

[PATCH v3 5/5] iio: mxs-lradc: add write_raw function to modify scale

2013-07-22 Thread Hector Palacios
Added write_raw function to manipulate the optional divider_by_two through the scaling attribute out of the available scales. Signed-off-by: Hector Palacios --- drivers/staging/iio/adc/mxs-lradc.c | 56 - 1 file changed, 55 insertions(+), 1 deletion(-) diff

Re: [PATCH v2 5/5] iio: mxs-lradc: add write_raw function to modify scale

2013-07-22 Thread Hector Palacios
I'm happy either way. I prefer the struct approach, it removes one dimension to the array and I find it cleaner. Best regards, -- Hector Palacios -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majord...@vger.kernel.org More

Re: [PATCH v2 3/5] iio: mxs-lradc: add scale attribute to channels

2013-07-22 Thread Hector Palacios
Hi Marek, On 07/22/2013 09:42 AM, Marek Vasut wrote: Dear Hector Palacios, Hi Marek, On 07/19/2013 06:14 PM, Marek Vasut wrote: Dear Hector Palacios, Dear Marek, On 07/19/2013 04:30 PM, Marek Vasut wrote: @@ -228,39 +230,12 @@ struct mxs_lradc { #define LRADC_RESOLUTION

Re: [PATCH v2 3/5] iio: mxs-lradc: add scale attribute to channels

2013-07-22 Thread Hector Palacios
Hi Alexandre, On 07/19/2013 07:06 PM, Alexandre Belloni wrote: Hi Hector, On 19/07/2013 11:13, Hector Palacios wrote: Some LRADC channels have fixed pre-dividers and all have an optional divider by two which allows a maximum input voltage of VDDIO - 50mV. This patch - adds the scaling info

Re: [PATCH v2 3/5] iio: mxs-lradc: add scale attribute to channels

2013-07-22 Thread Hector Palacios
Hi Marek, On 07/19/2013 06:14 PM, Marek Vasut wrote: Dear Hector Palacios, Dear Marek, On 07/19/2013 04:30 PM, Marek Vasut wrote: @@ -228,39 +230,12 @@ struct mxs_lradc { #define LRADC_RESOLUTION 12 #define LRADC_SINGLE_SAMPLE_MASK ((1

Re: [PATCH v2 3/5] iio: mxs-lradc: add scale attribute to channels

2013-07-22 Thread Hector Palacios
Hi Marek, On 07/19/2013 06:14 PM, Marek Vasut wrote: Dear Hector Palacios, Dear Marek, On 07/19/2013 04:30 PM, Marek Vasut wrote: @@ -228,39 +230,12 @@ struct mxs_lradc { #define LRADC_RESOLUTION 12 #define LRADC_SINGLE_SAMPLE_MASK ((1

Re: [PATCH v2 3/5] iio: mxs-lradc: add scale attribute to channels

2013-07-22 Thread Hector Palacios
Hi Alexandre, On 07/19/2013 07:06 PM, Alexandre Belloni wrote: Hi Hector, On 19/07/2013 11:13, Hector Palacios wrote: Some LRADC channels have fixed pre-dividers and all have an optional divider by two which allows a maximum input voltage of VDDIO - 50mV. This patch - adds the scaling info

Re: [PATCH v2 3/5] iio: mxs-lradc: add scale attribute to channels

2013-07-22 Thread Hector Palacios
Hi Marek, On 07/22/2013 09:42 AM, Marek Vasut wrote: Dear Hector Palacios, Hi Marek, On 07/19/2013 06:14 PM, Marek Vasut wrote: Dear Hector Palacios, Dear Marek, On 07/19/2013 04:30 PM, Marek Vasut wrote: @@ -228,39 +230,12 @@ struct mxs_lradc { #define LRADC_RESOLUTION

Re: [PATCH v2 5/5] iio: mxs-lradc: add write_raw function to modify scale

2013-07-22 Thread Hector Palacios
the struct approach, it removes one dimension to the array and I find it cleaner. Best regards, -- Hector Palacios -- To unsubscribe from this list: send the line unsubscribe linux-kernel in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo

[PATCH v3 1/5] iio: mxs-lradc: change the realbits to 12

2013-07-22 Thread Hector Palacios
. Besides, the driver was using an 18 bit mask (LRADC_CH_VALUE_MASK) to report touch coordinates to userland. A 12 bit mask should be used instead or else the touch libraries will expect a coordinates range between 0 and 0x3 (18 bits), instead of between 0 and 0xfff (12 bits). Signed-off-by: Hector

[PATCH v3 5/5] iio: mxs-lradc: add write_raw function to modify scale

2013-07-22 Thread Hector Palacios
Added write_raw function to manipulate the optional divider_by_two through the scaling attribute out of the available scales. Signed-off-by: Hector Palacios hector.palac...@digi.com --- drivers/staging/iio/adc/mxs-lradc.c | 56 - 1 file changed, 55 insertions

[PATCH v3 3/5] iio: mxs-lradc: add scale attribute to channels

2013-07-22 Thread Hector Palacios
) - allows to read the scaling attribute (computed from the Vref) Signed-off-by: Hector Palacios hector.palac...@digi.com. Acked-by: Marek Vasut ma...@denx.de --- drivers/staging/iio/adc/mxs-lradc.c | 77 + 1 file changed, 53 insertions(+), 24 deletions(-) diff --git

[PATCH v3 2/5] ARM: dts: add reference voltage property for MXS LRADC

2013-07-22 Thread Hector Palacios
Some LRADC channels have fixed pre-dividers so they can measure different voltages at full scale. The reference voltage allows to expose a scaling attribute through the IIO sysfs so that a user can compute the real voltage out of a measured sample value. Signed-off-by: Hector Palacios

[PATCH v3 4/5] iio: mxs-lradc: add scale_available file to channels

2013-07-22 Thread Hector Palacios
-by: Hector Palacios hector.palac...@digi.com --- drivers/staging/iio/adc/mxs-lradc.c | 112 +++- 1 file changed, 111 insertions(+), 1 deletion(-) diff --git a/drivers/staging/iio/adc/mxs-lradc.c b/drivers/staging/iio/adc/mxs-lradc.c index 5037577..5d332b1 100644

[PATCH v3 0/5] iio: mxs-lradc: add support to optional divider_by_two

2013-07-22 Thread Hector Palacios
platform. Could someone please test on an i.MX23? Hector Palacios (5): iio: mxs-lradc: change the realbits to 12 ARM: dts: add reference voltage property for MXS LRADC iio: mxs-lradc: add scale attribute to channels iio: mxs-lradc: add scale_available file to channels iio: mxs-lradc: add

Re: [PATCH v2 3/5] iio: mxs-lradc: add scale attribute to channels

2013-07-19 Thread Hector Palacios
mask', right? Yeah, I think I had resolved that one before submitting, but looks like I didn't. The other check is not resolved afaik. We agreed to remove it, but on a different patch. Otherwise: Acked-by: Marek Vasut Marek Vasut Best regards, -- Hector Palacios -- To unsubscribe from thi

Re: [PATCH v2 5/5] iio: mxs-lradc: add write_raw function to modify scale

2013-07-19 Thread Hector Palacios
was rejected and I was told to use the scale_available descriptor instead, which is the common interface the rest of drivers use. Best regards, -- Hector Palacios -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majord...@vger.ke

Re: [PATCH v2 5/5] iio: mxs-lradc: add write_raw function to modify scale

2013-07-19 Thread Hector Palacios
Dear Marek, On 07/19/2013 04:39 PM, Marek Vasut wrote: Dear Hector Palacios, Added write_raw function to manipulate the optional divider_by_two through the scaling attribute out of the available scales. Signed-off-by: Hector Palacios --- drivers/staging/iio/adc/mxs-lradc.c | 55

[PATCH v2 5/5] iio: mxs-lradc: add write_raw function to modify scale

2013-07-19 Thread Hector Palacios
Added write_raw function to manipulate the optional divider_by_two through the scaling attribute out of the available scales. Signed-off-by: Hector Palacios --- drivers/staging/iio/adc/mxs-lradc.c | 55 - 1 file changed, 54 insertions(+), 1 deletion(-) diff

[PATCH v2 0/5] iio: mxs-lradc: add support to optional divider_by_two

2013-07-19 Thread Hector Palacios
. The second adds the channels reference voltages to the DT. The following add the scale read operation, scale_available read operation, and scale write operation. This was tested on a custom i.MX28 platform. Could someone please test on an i.MX23? Hector Palacios (5): iio: mxs-lradc: change

[PATCH v2 4/5] iio: mxs-lradc: add scale_available file to channels

2013-07-19 Thread Hector Palacios
Adds in_voltageX_scale_available file for every channel to read the different available scales. There are two scales per channel: [0] = divider_by_two disabled (default) [1] = divider_by_two enabled Signed-off-by: Hector Palacios --- drivers/staging/iio/adc/mxs-lradc.c | 103

[PATCH v2 3/5] iio: mxs-lradc: add scale attribute to channels

2013-07-19 Thread Hector Palacios
) - allows to read the scaling attribute (computed from the Vref) Signed-off-by: Hector Palacios . --- drivers/staging/iio/adc/mxs-lradc.c | 81 - 1 file changed, 52 insertions(+), 29 deletions(-) diff --git a/drivers/staging/iio/adc/mxs-lradc.c b/drivers/staging/iio

[PATCH v2 1/5] iio: mxs-lradc: change the realbits to 12

2013-07-19 Thread Hector Palacios
. Besides, the driver was using an 18 bit mask (LRADC_CH_VALUE_MASK) to report touch coordinates to userland. A 12 bit mask should be used instead or else the touch libraries will expect a coordinates range between 0 and 0x3 (18 bits), instead of between 0 and 0xfff (12 bits). Signed-off-by: Hector

[PATCH v2 2/5] ARM: dts: add reference voltage property for MXS LRADC

2013-07-19 Thread Hector Palacios
Some LRADC channels have fixed pre-dividers so they can measure different voltages at full scale. The reference voltage allows to expose a scaling attribute through the IIO sysfs so that a user can compute the real voltage out of a measured sample value. Signed-off-by: Hector Palacios

[PATCH v2 2/5] ARM: dts: add reference voltage property for MXS LRADC

2013-07-19 Thread Hector Palacios
Some LRADC channels have fixed pre-dividers so they can measure different voltages at full scale. The reference voltage allows to expose a scaling attribute through the IIO sysfs so that a user can compute the real voltage out of a measured sample value. Signed-off-by: Hector Palacios

[PATCH v2 4/5] iio: mxs-lradc: add scale_available file to channels

2013-07-19 Thread Hector Palacios
Adds in_voltageX_scale_available file for every channel to read the different available scales. There are two scales per channel: [0] = divider_by_two disabled (default) [1] = divider_by_two enabled Signed-off-by: Hector Palacios hector.palac...@digi.com --- drivers/staging/iio/adc/mxs-lradc.c

[PATCH v2 3/5] iio: mxs-lradc: add scale attribute to channels

2013-07-19 Thread Hector Palacios
) - allows to read the scaling attribute (computed from the Vref) Signed-off-by: Hector Palacios hector.palac...@digi.com. --- drivers/staging/iio/adc/mxs-lradc.c | 81 - 1 file changed, 52 insertions(+), 29 deletions(-) diff --git a/drivers/staging/iio/adc/mxs-lradc.c

[PATCH v2 1/5] iio: mxs-lradc: change the realbits to 12

2013-07-19 Thread Hector Palacios
. Besides, the driver was using an 18 bit mask (LRADC_CH_VALUE_MASK) to report touch coordinates to userland. A 12 bit mask should be used instead or else the touch libraries will expect a coordinates range between 0 and 0x3 (18 bits), instead of between 0 and 0xfff (12 bits). Signed-off-by: Hector

[PATCH v2 0/5] iio: mxs-lradc: add support to optional divider_by_two

2013-07-19 Thread Hector Palacios
. The second adds the channels reference voltages to the DT. The following add the scale read operation, scale_available read operation, and scale write operation. This was tested on a custom i.MX28 platform. Could someone please test on an i.MX23? Hector Palacios (5): iio: mxs-lradc: change

[PATCH v2 5/5] iio: mxs-lradc: add write_raw function to modify scale

2013-07-19 Thread Hector Palacios
Added write_raw function to manipulate the optional divider_by_two through the scaling attribute out of the available scales. Signed-off-by: Hector Palacios hector.palac...@digi.com --- drivers/staging/iio/adc/mxs-lradc.c | 55 - 1 file changed, 54 insertions

Re: [PATCH v2 5/5] iio: mxs-lradc: add write_raw function to modify scale

2013-07-19 Thread Hector Palacios
Dear Marek, On 07/19/2013 04:39 PM, Marek Vasut wrote: Dear Hector Palacios, Added write_raw function to manipulate the optional divider_by_two through the scaling attribute out of the available scales. Signed-off-by: Hector Palacios hector.palac...@digi.com --- drivers/staging/iio/adc/mxs

Re: [PATCH v2 5/5] iio: mxs-lradc: add write_raw function to modify scale

2013-07-19 Thread Hector Palacios
was told to use the scale_available descriptor instead, which is the common interface the rest of drivers use. Best regards, -- Hector Palacios -- To unsubscribe from this list: send the line unsubscribe linux-kernel in the body of a message to majord...@vger.kernel.org More majordomo info at http

Re: [PATCH v2 3/5] iio: mxs-lradc: add scale attribute to channels

2013-07-19 Thread Hector Palacios
, I think I had resolved that one before submitting, but looks like I didn't. The other check is not resolved afaik. We agreed to remove it, but on a different patch. Otherwise: Acked-by: Marek Vasut ma...@denx.de Marek Vasut Best regards, -- Hector Palacios -- To unsubscribe from

Re: MXS persistent bits driver

2013-07-11 Thread Hector Palacios
Dear Maxime, On 07/11/2013 10:24 AM, maxime.rip...@free-electrons.com wrote: Hi Hector, On Thu, Jul 11, 2013 at 09:30:34AM +0200, Hector Palacios wrote: Greetings, Linux 2.6.35 had a driver for sysfs access to the MXS persistent bits (drivers/misc/mxs-persistent.c). Freescale's 2.6.35 I

Re: MXS persistent bits driver

2013-07-11 Thread Hector Palacios
On 07/11/2013 09:30 AM, Hector Palacios wrote: Greetings, Linux 2.6.35 had a driver for sysfs access to the MXS persistent bits (drivers/misc/mxs-persistent.c). I haven't seen this is supported upstream. Is anybody working on that? Where would such a driver fit? Adding Steffen Trumtrar, as I

MXS persistent bits driver

2013-07-11 Thread Hector Palacios
Greetings, Linux 2.6.35 had a driver for sysfs access to the MXS persistent bits (drivers/misc/mxs-persistent.c). I haven't seen this is supported upstream. Is anybody working on that? Where would such a driver fit? Best regards, -- Hector Palacios -- To unsubscribe from this list: send

MXS persistent bits driver

2013-07-11 Thread Hector Palacios
Greetings, Linux 2.6.35 had a driver for sysfs access to the MXS persistent bits (drivers/misc/mxs-persistent.c). I haven't seen this is supported upstream. Is anybody working on that? Where would such a driver fit? Best regards, -- Hector Palacios -- To unsubscribe from this list: send

Re: MXS persistent bits driver

2013-07-11 Thread Hector Palacios
On 07/11/2013 09:30 AM, Hector Palacios wrote: Greetings, Linux 2.6.35 had a driver for sysfs access to the MXS persistent bits (drivers/misc/mxs-persistent.c). I haven't seen this is supported upstream. Is anybody working on that? Where would such a driver fit? Adding Steffen Trumtrar, as I

Re: MXS persistent bits driver

2013-07-11 Thread Hector Palacios
Dear Maxime, On 07/11/2013 10:24 AM, maxime.rip...@free-electrons.com wrote: Hi Hector, On Thu, Jul 11, 2013 at 09:30:34AM +0200, Hector Palacios wrote: Greetings, Linux 2.6.35 had a driver for sysfs access to the MXS persistent bits (drivers/misc/mxs-persistent.c). Freescale's 2.6.35 I

Re: [PATCH] video: mxsfb: fix color settings for 18bit data bus and 32bpp

2013-06-18 Thread Hector Palacios
On 06/18/2013 10:32 AM, maxime.rip...@free-electrons.com wrote: Hi Hector, On Fri, Jun 07, 2013 at 11:02:28AM +0200, maxime.rip...@free-electrons.com wrote: On Fri, Jun 07, 2013 at 10:10:39AM +0200, Hector Palacios wrote: For a combination of 18bit LCD data bus width and a color mode

[PATCH RESEND] video: mxsfb: fix color settings for 18bit data bus and 32bpp

2013-06-18 Thread Hector Palacios
://lkml.org/lkml/2013/5/23/220 Signed-off-by: Hector Palacios Acked-by: Juergen Beisert Acked-by: Maxime Ripard --- drivers/video/mxsfb.c | 26 -- 1 file changed, 26 deletions(-) diff --git a/drivers/video/mxsfb.c b/drivers/video/mxsfb.c index 21223d4..d2c5105 100644

[PATCH RESEND] video: mxsfb: fix color settings for 18bit data bus and 32bpp

2013-06-18 Thread Hector Palacios
://lkml.org/lkml/2013/5/23/220 Signed-off-by: Hector Palacios hector.palac...@digi.com Acked-by: Juergen Beisert j...@pengutronix.de Acked-by: Maxime Ripard maxime.rip...@free-electrons.com --- drivers/video/mxsfb.c | 26 -- 1 file changed, 26 deletions(-) diff --git a/drivers

Re: [PATCH] video: mxsfb: fix color settings for 18bit data bus and 32bpp

2013-06-18 Thread Hector Palacios
On 06/18/2013 10:32 AM, maxime.rip...@free-electrons.com wrote: Hi Hector, On Fri, Jun 07, 2013 at 11:02:28AM +0200, maxime.rip...@free-electrons.com wrote: On Fri, Jun 07, 2013 at 10:10:39AM +0200, Hector Palacios wrote: For a combination of 18bit LCD data bus width and a color mode

[PATCH] video: mxsfb: fix color settings for 18bit data bus and 32bpp

2013-06-07 Thread Hector Palacios
://lkml.org/lkml/2013/5/23/220 Signed-off-by: Hector Palacios Acked-by: Juergen Beisert --- drivers/video/mxsfb.c | 26 -- 1 file changed, 26 deletions(-) diff --git a/drivers/video/mxsfb.c b/drivers/video/mxsfb.c index 21223d4..d2c5105 100644 --- a/drivers/video/mxsfb.c

Re: mxsfb: DATA_FORMAT_24_BIT flag outputs invalid colours

2013-06-07 Thread Hector Palacios
, -- Hector Palacios -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/

Re: mxsfb: DATA_FORMAT_24_BIT flag outputs invalid colours

2013-06-07 Thread Hector Palacios
, -- Hector Palacios -- To unsubscribe from this list: send the line unsubscribe linux-kernel in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/

[PATCH] video: mxsfb: fix color settings for 18bit data bus and 32bpp

2013-06-07 Thread Hector Palacios
://lkml.org/lkml/2013/5/23/220 Signed-off-by: Hector Palacios hector.palac...@digi.com Acked-by: Juergen Beisert j...@pengutronix.de --- drivers/video/mxsfb.c | 26 -- 1 file changed, 26 deletions(-) diff --git a/drivers/video/mxsfb.c b/drivers/video/mxsfb.c index 21223d4

MXS USB otg device support

2013-05-28 Thread Hector Palacios
Hello, Am I right in assuming that the MXS USB on-the-go port does not currently support the device (gadget) functionality? Anybody out there working on that? Regards, -- Héctor Palacios -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to

MXS USB otg device support

2013-05-28 Thread Hector Palacios
Hello, Am I right in assuming that the MXS USB on-the-go port does not currently support the device (gadget) functionality? Anybody out there working on that? Regards, -- Héctor Palacios -- To unsubscribe from this list: send the line unsubscribe linux-kernel in the body of a message to

Re: mxsfb: DATA_FORMAT_24_BIT flag outputs invalid colours

2013-05-24 Thread Hector Palacios
Hi Juergen, On 05/24/2013 12:28 PM, Juergen Beisert wrote: Hector Palacios wrote: Hi Juergen, On 05/23/2013 03:31 PM, Juergen Beisert wrote: Hi Maxime, maxime.rip...@free-electrons.com wrote: On Thu, May 23, 2013 at 01:55:28PM +0200, Hector Palacios wrote: I'm using an i.MX28 based board

Re: mxsfb: DATA_FORMAT_24_BIT flag outputs invalid colours

2013-05-24 Thread Hector Palacios
Hi Juergen, On 05/24/2013 12:28 PM, Juergen Beisert wrote: Hector Palacios wrote: Hi Juergen, On 05/23/2013 03:31 PM, Juergen Beisert wrote: Hi Maxime, maxime.rip...@free-electrons.com wrote: On Thu, May 23, 2013 at 01:55:28PM +0200, Hector Palacios wrote: I'm using an i.MX28 based board

Re: mxsfb: DATA_FORMAT_24_BIT flag outputs invalid colours

2013-05-23 Thread Hector Palacios
Hi Juergen, On 05/23/2013 03:31 PM, Juergen Beisert wrote: Hi Maxime, maxime.rip...@free-electrons.com wrote: On Thu, May 23, 2013 at 01:55:28PM +0200, Hector Palacios wrote: I'm using an i.MX28 based board with lcd connected with 18bits data bus. My platform uses 32 bits per pixel

mxsfb: DATA_FORMAT_24_BIT flag outputs invalid colours

2013-05-23 Thread Hector Palacios
Hello, I'm using an i.MX28 based board with lcd connected with 18bits data bus. My platform uses 32 bits per pixel: mxsfb_pdata.default_bpp = 32; mxsfb_pdata.ld_intf_width = STMLCDIF_18BIT; With these settings the mxsfb.c driver sets flag DATA_FORMAT_24_BIT at HW_LCDIF_CTRL

mxsfb: DATA_FORMAT_24_BIT flag outputs invalid colours

2013-05-23 Thread Hector Palacios
Hello, I'm using an i.MX28 based board with lcd connected with 18bits data bus. My platform uses 32 bits per pixel: mxsfb_pdata.default_bpp = 32; mxsfb_pdata.ld_intf_width = STMLCDIF_18BIT; With these settings the mxsfb.c driver sets flag DATA_FORMAT_24_BIT at HW_LCDIF_CTRL

Re: mxsfb: DATA_FORMAT_24_BIT flag outputs invalid colours

2013-05-23 Thread Hector Palacios
Hi Juergen, On 05/23/2013 03:31 PM, Juergen Beisert wrote: Hi Maxime, maxime.rip...@free-electrons.com wrote: On Thu, May 23, 2013 at 01:55:28PM +0200, Hector Palacios wrote: I'm using an i.MX28 based board with lcd connected with 18bits data bus. My platform uses 32 bits per pixel

Re: mxsfb on i.MX28 uses bypassed ref_xtal 24MHz clock for LCD

2013-05-20 Thread Hector Palacios
Dear Fabio, On 05/17/2013 04:07 PM, Fabio Estevam wrote: On 05/17/2013 10:57 AM, Hector Palacios wrote: On the MX28EVK board with the Seiko 4.3" LCD, I do not see the penguin or any image in v3.8 or v3.9 though the framebuffer device is present and correctly loaded and backlight on. Thi

Re: mxsfb on i.MX28 uses bypassed ref_xtal 24MHz clock for LCD

2013-05-20 Thread Hector Palacios
Dear Fabio, On 05/17/2013 04:07 PM, Fabio Estevam wrote: On 05/17/2013 10:57 AM, Hector Palacios wrote: On the MX28EVK board with the Seiko 4.3 LCD, I do not see the penguin or any image in v3.8 or v3.9 though the framebuffer device is present and correctly loaded and backlight on. This LCD

mxsfb on i.MX28 uses bypassed ref_xtal 24MHz clock for LCD

2013-05-17 Thread Hector Palacios
Hello, I was testing the framebuffer on an i.MX28 based platform and found out that the lcdif clock is actually bypassed to use the ref_xtal of 24MHz, which makes the clk_set_rate() call in the driver useless (as well as the pixclock settings in fb_videomode variables in mach-mxs.c): static

mxsfb on i.MX28 uses bypassed ref_xtal 24MHz clock for LCD

2013-05-17 Thread Hector Palacios
Hello, I was testing the framebuffer on an i.MX28 based platform and found out that the lcdif clock is actually bypassed to use the ref_xtal of 24MHz, which makes the clk_set_rate() call in the driver useless (as well as the pixclock settings in fb_videomode variables in mach-mxs.c): static

How usable is spi-mxs driver?

2013-04-22 Thread Hector Palacios
Hello, I had the feeling that the MXS SPI driver was rather useless due to the SPI being only half-duplex on i.MX28, but it looks like people's using it so I was wondering what kind of SPI devices can be driven with it, or if there is any trick that allows bi-directional communication.

How usable is spi-mxs driver?

2013-04-22 Thread Hector Palacios
Hello, I had the feeling that the MXS SPI driver was rather useless due to the SPI being only half-duplex on i.MX28, but it looks like people's using it so I was wondering what kind of SPI devices can be driven with it, or if there is any trick that allows bi-directional communication.

Re: [PATCH RFC] ARM: dts: mxs: leave card detect out of common mmc pins config

2013-04-10 Thread Hector Palacios
On 04/09/2013 12:51 PM, Marek Vasut wrote: Hi Hector, Dear Marek Vasut, On 04/09/2013 10:15 AM, Marek Vasut wrote: Dear Hector Palacios, Dear Marek Vasut, On 04/08/2013 06:28 PM, Marek Vasut wrote: Dear Shawn Guo, On Mon, Apr 08, 2013 at 03:58:05PM +0200, Hector Palacios wrote: On 04

Re: [PATCH v2 2/2] mmc: mxs-mmc: add non-removable property

2013-04-10 Thread Hector Palacios
On Tue 09 Apr 2013 06:34:10 PM CEST, Marc Kleine-Budde wrote: On 04/09/2013 05:21 PM, Hector Palacios wrote: On 04/09/2013 05:14 PM, Hector Palacios wrote: @@ -95,7 +97,8 @@ static int mxs_mmc_get_cd(struct mmc_host *mmc) struct mxs_mmc_host *host = mmc_priv(mmc); struct

Re: [PATCH v2 2/2] mmc: mxs-mmc: add non-removable property

2013-04-10 Thread Hector Palacios
On Tue 09 Apr 2013 06:34:10 PM CEST, Marc Kleine-Budde wrote: On 04/09/2013 05:21 PM, Hector Palacios wrote: On 04/09/2013 05:14 PM, Hector Palacios wrote: @@ -95,7 +97,8 @@ static int mxs_mmc_get_cd(struct mmc_host *mmc) struct mxs_mmc_host *host = mmc_priv(mmc); struct

Re: [PATCH RFC] ARM: dts: mxs: leave card detect out of common mmc pins config

2013-04-10 Thread Hector Palacios
On 04/09/2013 12:51 PM, Marek Vasut wrote: Hi Hector, Dear Marek Vasut, On 04/09/2013 10:15 AM, Marek Vasut wrote: Dear Hector Palacios, Dear Marek Vasut, On 04/08/2013 06:28 PM, Marek Vasut wrote: Dear Shawn Guo, On Mon, Apr 08, 2013 at 03:58:05PM +0200, Hector Palacios wrote: On 04

Re: [PATCH v2 2/2] mmc: mxs-mmc: add non-removable property

2013-04-09 Thread Hector Palacios
On 04/09/2013 05:14 PM, Hector Palacios wrote: @@ -95,7 +97,8 @@ static int mxs_mmc_get_cd(struct mmc_host *mmc) struct mxs_mmc_host *host = mmc_priv(mmc); struct mxs_ssp *ssp = >ssp; - return !(readl(ssp->base + HW_SSP_STATUS(ssp)) & + return host->

Re: [PATCH v2 2/2] mmc: mxs-mmc: add non-removable property

2013-04-09 Thread Hector Palacios
Dear Marc Kleine-Budde, On 04/05/2013 02:47 PM, Marc Kleine-Budde wrote: Some boards have non removable cards like eMMC. Handle such case. Acked-by: Shawn Guo Signed-off-by: Marc Kleine-Budde --- changes since v1: - fix removeable typo drivers/mmc/host/mxs-mmc.c | 10 +++--- 1 file

Re: [PATCH RFC] ARM: dts: mxs: leave card detect out of common mmc pins config

2013-04-09 Thread Hector Palacios
Dear Marek Vasut, On 04/09/2013 10:15 AM, Marek Vasut wrote: Dear Hector Palacios, Dear Marek Vasut, On 04/08/2013 06:28 PM, Marek Vasut wrote: Dear Shawn Guo, On Mon, Apr 08, 2013 at 03:58:05PM +0200, Hector Palacios wrote: On 04/08/2013 02:48 PM, Shawn Guo wrote: On Mon, Apr 08, 2013

Re: [PATCH RFC] ARM: dts: mxs: leave card detect out of common mmc pins config

2013-04-09 Thread Hector Palacios
Dear Marek Vasut, On 04/08/2013 06:28 PM, Marek Vasut wrote: Dear Shawn Guo, On Mon, Apr 08, 2013 at 03:58:05PM +0200, Hector Palacios wrote: On 04/08/2013 02:48 PM, Shawn Guo wrote: On Mon, Apr 08, 2013 at 12:12:20PM +0200, Hector Palacios wrote: MicroSD card sockets don't usually have

Re: [PATCH RFC] ARM: dts: mxs: leave card detect out of common mmc pins config

2013-04-09 Thread Hector Palacios
Dear Marek Vasut, On 04/08/2013 06:28 PM, Marek Vasut wrote: Dear Shawn Guo, On Mon, Apr 08, 2013 at 03:58:05PM +0200, Hector Palacios wrote: On 04/08/2013 02:48 PM, Shawn Guo wrote: On Mon, Apr 08, 2013 at 12:12:20PM +0200, Hector Palacios wrote: MicroSD card sockets don't usually have

Re: [PATCH RFC] ARM: dts: mxs: leave card detect out of common mmc pins config

2013-04-09 Thread Hector Palacios
Dear Marek Vasut, On 04/09/2013 10:15 AM, Marek Vasut wrote: Dear Hector Palacios, Dear Marek Vasut, On 04/08/2013 06:28 PM, Marek Vasut wrote: Dear Shawn Guo, On Mon, Apr 08, 2013 at 03:58:05PM +0200, Hector Palacios wrote: On 04/08/2013 02:48 PM, Shawn Guo wrote: On Mon, Apr 08, 2013

Re: [PATCH v2 2/2] mmc: mxs-mmc: add non-removable property

2013-04-09 Thread Hector Palacios
Dear Marc Kleine-Budde, On 04/05/2013 02:47 PM, Marc Kleine-Budde wrote: Some boards have non removable cards like eMMC. Handle such case. Acked-by: Shawn Guo shawn@linaro.org Signed-off-by: Marc Kleine-Budde m...@pengutronix.de --- changes since v1: - fix removeable typo

Re: [PATCH v2 2/2] mmc: mxs-mmc: add non-removable property

2013-04-09 Thread Hector Palacios
On 04/09/2013 05:14 PM, Hector Palacios wrote: @@ -95,7 +97,8 @@ static int mxs_mmc_get_cd(struct mmc_host *mmc) struct mxs_mmc_host *host = mmc_priv(mmc); struct mxs_ssp *ssp = host-ssp; - return !(readl(ssp-base + HW_SSP_STATUS(ssp)) + return host-non_removable

[PATCH v3] watchdog: core: don't try to stop device if not running

2013-04-08 Thread Hector Palacios
ot;/dev/watchdog", O_RDWR); flags = WDIOS_DISABLECARD; ioctl(wd_fd, WDIOC_SETOPTIONS, ); close(wd_fd); Signed-off-by: Hector Palacios --- Changes from v1: - Make it less intrusive with 'if/else if' clauses. - Change the logic of 'err' variable for better readabilit

Re: [PATCH v2] watchdog: core: don't try to stop device if not running

2013-04-08 Thread Hector Palacios
On 04/08/2013 03:41 PM, Guenter Roeck wrote: On Mon, Apr 08, 2013 at 11:10:58AM +0200, Hector Palacios wrote: A watchdog device may be stopped from userspace using WDIOC_SETOPTIONS ioctl and flag WDIOS_DISABLECARD. If the device is closed after this operation, watchdog_release() is called

Re: [PATCH RFC] ARM: dts: mxs: leave card detect out of common mmc pins config

2013-04-08 Thread Hector Palacios
On 04/08/2013 02:48 PM, Shawn Guo wrote: On Mon, Apr 08, 2013 at 12:12:20PM +0200, Hector Palacios wrote: MicroSD card sockets don't usually have card detect line. This pin is actually not needed for the MMC to work and it is more of a platform design decission to have it. The card detect pin

[PATCH RFC] ARM: dts: mxs: leave card detect out of common mmc pins config

2013-04-08 Thread Hector Palacios
and let platforms to include it or not depending on whether the card detect line is routed to the SD socket. Signed-off-by: Hector Palacios --- Hello, All imx28 based platforms except 'bluegiga,apx4devkit' and 'schulercontrol,imx28-sps1', use 'mmc0_cd_cfg' in their mmc configuration so please check

[PATCH v2] watchdog: core: don't try to stop device if not running

2013-04-08 Thread Hector Palacios
ot;/dev/watchdog", O_RDWR); flags = WDIOS_DISABLECARD; ioctl(wd_fd, WDIOC_SETOPTIONS, ); close(wd_fd); Signed-off-by: Hector Palacios --- Changes from v1: - Make it less intrusive with 'if/else if' clauses - Change the logic of 'err' variable for better readabil

Re: [PATCH RFC] watchdog: core: don't try to stop device if not running

2013-04-08 Thread Hector Palacios
On 04/08/2013 10:16 AM, Guenter Roeck wrote: On Mon, Apr 08, 2013 at 09:48:57AM +0200, Hector Palacios wrote: On 04/05/2013 08:34 PM, Guenter Roeck wrote: On Fri, Apr 05, 2013 at 06:09:44PM +0200, Hector Palacios wrote: A watchdog device may be stopped from userspace using WDIOC_SETOPTIONS

Re: [PATCH RFC] watchdog: core: don't try to stop device if not running

2013-04-08 Thread Hector Palacios
On 04/05/2013 08:34 PM, Guenter Roeck wrote: On Fri, Apr 05, 2013 at 06:09:44PM +0200, Hector Palacios wrote: A watchdog device may be stopped from userspace using WDIOC_SETOPTIONS ioctl and flag WDIOS_DISABLECARD. If the device is closed after this operation, watchdog_release() is called

Re: [PATCH RFC] watchdog: core: don't try to stop device if not running

2013-04-08 Thread Hector Palacios
On 04/05/2013 08:34 PM, Guenter Roeck wrote: On Fri, Apr 05, 2013 at 06:09:44PM +0200, Hector Palacios wrote: A watchdog device may be stopped from userspace using WDIOC_SETOPTIONS ioctl and flag WDIOS_DISABLECARD. If the device is closed after this operation, watchdog_release() is called

  1   2   >