All of lore.kernel.org
 help / color / mirror / Atom feed
* [meta-fsl-arm-extra][PATCH] cfa10058: fix touchscreen calibration
@ 2013-12-28 10:31 Alexandre Belloni
  2013-12-28 10:33 ` Alexandre Belloni
  2013-12-30  5:11 ` Otavio Salvador
  0 siblings, 2 replies; 6+ messages in thread
From: Alexandre Belloni @ 2013-12-28 10:31 UTC (permalink / raw)
  To: meta-freescale; +Cc: Maxime Ripard, jimwall

Probably a copy paste error, the y_min and x_max got inverted.

Signed-off-by: Alexandre Belloni <alexandre.belloni@free-electrons.com>
---
 .../xinput-calibrator/pointercal-xinput/cfa10058/pointercal.xinput      | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/recipes-graphics/xinput-calibrator/pointercal-xinput/cfa10058/pointercal.xinput b/recipes-graphics/xinput-calibrator/pointercal-xinput/cfa10058/pointercal.xinput
index 445863a87099..230619b3e9a6 100644
--- a/recipes-graphics/xinput-calibrator/pointercal-xinput/cfa10058/pointercal.xinput
+++ b/recipes-graphics/xinput-calibrator/pointercal-xinput/cfa10058/pointercal.xinput
@@ -1,2 +1,2 @@
-xinput set-int-prop "mxs-lradc" "Evdev Axis Calibration" 32 1373 1381 3391 3348
+xinput set-int-prop "mxs-lradc" "Evdev Axis Calibration" 32 206 3985 3937 219
 xinput set-int-prop "mxs-lradc" "Evdev Axes Swap" 8 0
-- 
1.8.3.2



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

* Re: [meta-fsl-arm-extra][PATCH] cfa10058: fix touchscreen calibration
  2013-12-28 10:31 [meta-fsl-arm-extra][PATCH] cfa10058: fix touchscreen calibration Alexandre Belloni
@ 2013-12-28 10:33 ` Alexandre Belloni
  2013-12-30  5:11 ` Otavio Salvador
  1 sibling, 0 replies; 6+ messages in thread
From: Alexandre Belloni @ 2013-12-28 10:33 UTC (permalink / raw)
  To: meta-freescale; +Cc: Maxime Ripard, jimwall

I forgot to add, this is probably worth adding in dora.

On 28/12/2013 11:31, Alexandre Belloni wrote:
> Probably a copy paste error, the y_min and x_max got inverted.
>
> Signed-off-by: Alexandre Belloni <alexandre.belloni@free-electrons.com>
> ---
>  .../xinput-calibrator/pointercal-xinput/cfa10058/pointercal.xinput      | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/recipes-graphics/xinput-calibrator/pointercal-xinput/cfa10058/pointercal.xinput b/recipes-graphics/xinput-calibrator/pointercal-xinput/cfa10058/pointercal.xinput
> index 445863a87099..230619b3e9a6 100644
> --- a/recipes-graphics/xinput-calibrator/pointercal-xinput/cfa10058/pointercal.xinput
> +++ b/recipes-graphics/xinput-calibrator/pointercal-xinput/cfa10058/pointercal.xinput
> @@ -1,2 +1,2 @@
> -xinput set-int-prop "mxs-lradc" "Evdev Axis Calibration" 32 1373 1381 3391 3348
> +xinput set-int-prop "mxs-lradc" "Evdev Axis Calibration" 32 206 3985 3937 219
>  xinput set-int-prop "mxs-lradc" "Evdev Axes Swap" 8 0


-- 
Alexandre Belloni, Free Electrons
Embedded Linux, Kernel and Android engineering
http://free-electrons.com



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

* Re: [meta-fsl-arm-extra][PATCH] cfa10058: fix touchscreen calibration
  2013-12-28 10:31 [meta-fsl-arm-extra][PATCH] cfa10058: fix touchscreen calibration Alexandre Belloni
  2013-12-28 10:33 ` Alexandre Belloni
@ 2013-12-30  5:11 ` Otavio Salvador
  2014-01-06 15:39   ` Otavio Salvador
  1 sibling, 1 reply; 6+ messages in thread
From: Otavio Salvador @ 2013-12-30  5:11 UTC (permalink / raw)
  To: Alexandre Belloni; +Cc: meta-freescale@yoctoproject.org, Maxime Ripard, jimwall

On Sat, Dec 28, 2013 at 8:31 AM, Alexandre Belloni
<alexandre.belloni@free-electrons.com> wrote:
> Probably a copy paste error, the y_min and x_max got inverted.
>
> Signed-off-by: Alexandre Belloni <alexandre.belloni@free-electrons.com>

The values does not match as inverted; I am OK with fixing the
calibration but the commit log message ought to be accurate.

-- 
Otavio Salvador                             O.S. Systems
http://www.ossystems.com.br        http://code.ossystems.com.br
Mobile: +55 (53) 9981-7854            Mobile: +1 (347) 903-9750


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

* Re: [meta-fsl-arm-extra][PATCH] cfa10058: fix touchscreen calibration
  2013-12-30  5:11 ` Otavio Salvador
@ 2014-01-06 15:39   ` Otavio Salvador
  2014-01-06 16:46     ` Alexandre Belloni
  0 siblings, 1 reply; 6+ messages in thread
From: Otavio Salvador @ 2014-01-06 15:39 UTC (permalink / raw)
  To: Alexandre Belloni; +Cc: meta-freescale@yoctoproject.org, Maxime Ripard, jimwall

Hello Alexandre,

On Mon, Dec 30, 2013 at 3:11 AM, Otavio Salvador
<otavio@ossystems.com.br> wrote:
> On Sat, Dec 28, 2013 at 8:31 AM, Alexandre Belloni
> <alexandre.belloni@free-electrons.com> wrote:
>> Probably a copy paste error, the y_min and x_max got inverted.
>>
>> Signed-off-by: Alexandre Belloni <alexandre.belloni@free-electrons.com>
>
> The values does not match as inverted; I am OK with fixing the
> calibration but the commit log message ought to be accurate.

ping?

-- 
Otavio Salvador                             O.S. Systems
http://www.ossystems.com.br        http://code.ossystems.com.br
Mobile: +55 (53) 9981-7854            Mobile: +1 (347) 903-9750


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

* Re: [meta-fsl-arm-extra][PATCH] cfa10058: fix touchscreen calibration
  2014-01-06 15:39   ` Otavio Salvador
@ 2014-01-06 16:46     ` Alexandre Belloni
  2014-01-06 17:05       ` Otavio Salvador
  0 siblings, 1 reply; 6+ messages in thread
From: Alexandre Belloni @ 2014-01-06 16:46 UTC (permalink / raw)
  To: Otavio Salvador; +Cc: meta-freescale@yoctoproject.org, Maxime Ripard, jimwall

Hi,

On 06/01/2014 16:39, Otavio Salvador wrote:
> Hello Alexandre,
> 
> On Mon, Dec 30, 2013 at 3:11 AM, Otavio Salvador
> <otavio@ossystems.com.br> wrote:
>> On Sat, Dec 28, 2013 at 8:31 AM, Alexandre Belloni
>> <alexandre.belloni@free-electrons.com> wrote:
>>> Probably a copy paste error, the y_min and x_max got inverted.
>>>
>>> Signed-off-by: Alexandre Belloni <alexandre.belloni@free-electrons.com>
>>
>> The values does not match as inverted; I am OK with fixing the
>> calibration but the commit log message ought to be accurate.
> 
> ping?
> 

Actually, after bumping the 3.10 kernel, the calibration files won't be
necessary anymore. I backported
1eb70a97452cacbe7aae752879b782e1e97a3484 which fixes the first
calibration issue.

I'll send a patch removing those but I was waiting for the kernel to be
bumped first.


-- 
Alexandre Belloni, Free Electrons
Embedded Linux, Kernel and Android engineering
http://free-electrons.com


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

* Re: [meta-fsl-arm-extra][PATCH] cfa10058: fix touchscreen calibration
  2014-01-06 16:46     ` Alexandre Belloni
@ 2014-01-06 17:05       ` Otavio Salvador
  0 siblings, 0 replies; 6+ messages in thread
From: Otavio Salvador @ 2014-01-06 17:05 UTC (permalink / raw)
  To: Alexandre Belloni; +Cc: meta-freescale@yoctoproject.org, Maxime Ripard, jimwall

Hello Alexandre,

On Mon, Jan 6, 2014 at 2:46 PM, Alexandre Belloni
<alexandre.belloni@free-electrons.com> wrote:
> On 06/01/2014 16:39, Otavio Salvador wrote:
>> Hello Alexandre,
>>
>> On Mon, Dec 30, 2013 at 3:11 AM, Otavio Salvador
>> <otavio@ossystems.com.br> wrote:
>>> On Sat, Dec 28, 2013 at 8:31 AM, Alexandre Belloni
>>> <alexandre.belloni@free-electrons.com> wrote:
>>>> Probably a copy paste error, the y_min and x_max got inverted.
>>>>
>>>> Signed-off-by: Alexandre Belloni <alexandre.belloni@free-electrons.com>
>>>
>>> The values does not match as inverted; I am OK with fixing the
>>> calibration but the commit log message ought to be accurate.
>>
>> ping?
>>
>
> Actually, after bumping the 3.10 kernel, the calibration files won't be
> necessary anymore. I backported
> 1eb70a97452cacbe7aae752879b782e1e97a3484 which fixes the first
> calibration issue.
>
> I'll send a patch removing those but I was waiting for the kernel to be
> bumped first.

Great!

-- 
Otavio Salvador                             O.S. Systems
http://www.ossystems.com.br        http://code.ossystems.com.br
Mobile: +55 (53) 9981-7854            Mobile: +1 (347) 903-9750


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

end of thread, other threads:[~2014-01-06 17:05 UTC | newest]

Thread overview: 6+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2013-12-28 10:31 [meta-fsl-arm-extra][PATCH] cfa10058: fix touchscreen calibration Alexandre Belloni
2013-12-28 10:33 ` Alexandre Belloni
2013-12-30  5:11 ` Otavio Salvador
2014-01-06 15:39   ` Otavio Salvador
2014-01-06 16:46     ` Alexandre Belloni
2014-01-06 17:05       ` 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.