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

Groups > linux.kernel > #1240558 > unrolled thread

Re: [PATCH v4 00/26] ARM: pxa: magician: Rework HTC Magician support

Started byPavel Machek <pavel@ucw.cz>
First post2015-10-06 17:30 +0200
Last post2015-10-12 06:30 +0200
Articles 5 — 3 participants

Back to article view | Back to linux.kernel

This discussion starts older than the indexed window; earlier articles aren't shown. The article labeled Started by below is the oldest one visible, not the original post.

Contents

  Re: [PATCH v4 00/26] ARM: pxa: magician: Rework HTC Magician support Pavel Machek <pavel@ucw.cz> - 2015-10-06 17:30 +0200
    Re: [PATCH v4 00/26] ARM: pxa: magician: Rework HTC Magician support Petr Cvek <petr.cvek@tul.cz> - 2015-10-09 05:10 +0200
      Re: [PATCH v4 00/26] ARM: pxa: magician: Rework HTC Magician support Robert Jarzmik <robert.jarzmik@free.fr> - 2015-10-11 10:30 +0200
        Re: [PATCH v4 00/26] ARM: pxa: magician: Rework HTC Magician support Robert Jarzmik <robert.jarzmik@free.fr> - 2015-10-11 22:10 +0200
        Re: [PATCH v4 00/26] ARM: pxa: magician: Rework HTC Magician support Petr Cvek <petr.cvek@tul.cz> - 2015-10-12 06:30 +0200

#1240558 — Re: [PATCH v4 00/26] ARM: pxa: magician: Rework HTC Magician support

FromPavel Machek <pavel@ucw.cz>
Date2015-10-06 17:30 +0200
SubjectRe: [PATCH v4 00/26] ARM: pxa: magician: Rework HTC Magician support
Message-ID<qgClJ-1De-37@gated-at.bofh.it>
Hi!

> This patchset will clean, fix, optimize and add multiple devices,
> which can be found on the HTC Magician machine.
> 
> Platform data for the pasic3-leds driver has been removed, as this
> driver do not exist in the vanilla. And it is expected to be recreated
> in the future with an incompatible platform data structure.
> 
> Default setting of the Samsung LCD pixel clock period has been
> increased, so resulting frequency is 50Hz (original 117Hz). Period
> can be possibly incresed even more. At this moment there is no video
> player with IWMMXT support and software only versions cannot make it
> to 30fps. Frequency decreasing has a good impact to memory troughtput
> as LCD controller does not do as many memory bursts as at original
> speed. This was verified with the lat_mem_rd from the lmbench3.
> 
> The Omnivision OV9640 camera has a problem with running on 400kHz
> of the I2C clock. It seems it freezes the bus until PXA I2C driver
> reload. Datasheet says it OK on 400kHz though. There may be some
> compatibility bug in the PXA I2C driver as Omnivision cameras often
> uses special I2C protocol (SCCB).
> 
> Using the CPU frequency scaling (with help of an MAX1587A regulator)
> sometimes causes an SoC failure. It is probably related to the PXA27x
> erratum E37, E36, E80 or E89. Infreqent changes mostly works.

Nice to see work done on a cellphone...

Seeing all the GPIO name changes... would it be possible / make sense
to make to device tree one day?

What userspace do you run on the phone? Can it do calls?

[I have N900 here, kernel is in pretty good state, and I have Mate desktop
running with python application for calls, but...]

Thanks (a pozdrav :-),

								Pavel
> STUART removal is a prepare for:
> 
> 	net: irda: pxaficp_ir: dmaengine conversion
> 
> Function was tested on the board_id 0x3a version (specifically on
> the T-Mobile MDA compact).
> 
> Changes from v3:
>  * Nontrivial rebase for indentation fixup (EGPIO and PXA UDC)
> 
> Petr Cvek (26):
>   ARM: pxa: magician: Fix indentation in machine files
>   ARM: pxa: magician: Change comments to be more informative
>   ARM: pxa: magician: Print more specific error message for global
>     GPIOs
>   ARM: pxa: magician: Optimize debug messages for LCD power
>   ARM: pxa: magician: Change description of LCD power GPIO
>   ARM: pxa: magician: Add new discovered EGPIO pins
>   ARM: pxa: magician: Fix HTC Magician pin mux definitions
>   ARM: pxa: magician: Rename abstract LCD GPIOs
>   ARM: pxa: magician: Optimize powerup delays for Samsung LCD
>   ARM: pxa: magician: Optimize Samsung LCD refresh to 50Hz
>   ARM: pxa: magician: Optimize EGPIO initial values
>   ARM: pxa: magician: Rename charger cable detection EGPIOs
>   ARM: pxa: magician: Fix and add charging detection functions
>   ARM: pxa: magician: Fix platform data for both PXA27x I2C controllers
>   ARM: pxa: magician: Fix redundant GPIO request for pxaficp_ir
>   ARM: pxa: magician: Fix support for Intel Strata NOR Flash
>   ARM: pxa: magician: Fix wrongly enabled USB host ports
>   ARM: pxa: magician: Add support for ADS7846
>   ARM: pxa: magician: Add support for Omnivision OV9640 camera
>   ARM: pxa: magician: Add support for MAX1587A Vcore regulator
>   ARM: pxa: magician: Add support for PXA27x UDC
>   ARM: pxa: magician: Remove pdata for pasic3-leds
>   ARM: pxa: magician: Remove definition of the STUART port
>   ARM: pxa: magician: Add debug message for backlight brightness
>     function
>   ARM: pxa: magician: Add missing regulator for PWM backlight
>   ARM: pxa: magician: Move platform_add_devices() to the end of
>     magician_init()
> 
>  arch/arm/mach-pxa/include/mach/magician.h |  70 ++-
>  arch/arm/mach-pxa/magician.c              | 907 +++++++++++++++++++++---------
>  2 files changed, 678 insertions(+), 299 deletions(-)
> 
> -- 
> 1.7.12.1
> 
> 
> _______________________________________________
> linux-arm-kernel mailing list
> linux-arm-kernel@lists.infradead.org
> http://lists.infradead.org/mailman/listinfo/linux-arm-kernel

-- 
(english) http://www.livejournal.com/~pavelmachek
(cesky, pictures) http://atrey.karlin.mff.cuni.cz/~pavel/picture/horses/blog.html
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

[toc] | [next] | [standalone]

#1243010

FromPetr Cvek <petr.cvek@tul.cz>
Date2015-10-09 05:10 +0200
Message-ID<qhwed-6pz-1@gated-at.bofh.it>
In reply to#1240558
Dne 6.10.2015 v 17:25 Pavel Machek napsal(a):
> Hi!

Ahoj :-),

> 
>> This patchset will clean, fix, optimize and add multiple devices,
>> which can be found on the HTC Magician machine.
>>
>> Platform data for the pasic3-leds driver has been removed, as this
>> driver do not exist in the vanilla. And it is expected to be recreated
>> in the future with an incompatible platform data structure.
>>
>> Default setting of the Samsung LCD pixel clock period has been
>> increased, so resulting frequency is 50Hz (original 117Hz). Period
>> can be possibly incresed even more. At this moment there is no video
>> player with IWMMXT support and software only versions cannot make it
>> to 30fps. Frequency decreasing has a good impact to memory troughtput
>> as LCD controller does not do as many memory bursts as at original
>> speed. This was verified with the lat_mem_rd from the lmbench3.
>>
>> The Omnivision OV9640 camera has a problem with running on 400kHz
>> of the I2C clock. It seems it freezes the bus until PXA I2C driver
>> reload. Datasheet says it OK on 400kHz though. There may be some
>> compatibility bug in the PXA I2C driver as Omnivision cameras often
>> uses special I2C protocol (SCCB).
>>
>> Using the CPU frequency scaling (with help of an MAX1587A regulator)
>> sometimes causes an SoC failure. It is probably related to the PXA27x
>> erratum E37, E36, E80 or E89. Infreqent changes mostly works.
> 
> Nice to see work done on a cellphone...
> 
> Seeing all the GPIO name changes... would it be possible / make sense
> to make to device tree one day?
Probably, but there are still some missing parts (camera, usb webcam gadget,
sound, leds, gsm GPIOs)

> 
> What userspace do you run on the phone? Can it do calls?

I have found it is faster and funny to create from scratch (and openmoko
image is too old), so I've built it by Buildroot:

Busybox, Xorg and Matchbox, but as GTK apps are not too good for 240x320 resolution
I have made my own "GUI" (so I can control Magician even without any USB/IrDA connection).

Calls are planned but by now I'm receiving SMS by hand by writing AT commands :-D.

> 
> [I have N900 here, kernel is in pretty good state, and I have Mate desktop
> running with python application for calls, but...]

Magician is very limited by RAM (64MB + 64MB Flash), so I can only have C
applications (or shell :-D).

> 
> Thanks (a pozdrav :-),
> 
> 								Pavel
>> STUART removal is a prepare for:
>>
>> 	net: irda: pxaficp_ir: dmaengine conversion
>>
>> Function was tested on the board_id 0x3a version (specifically on
>> the T-Mobile MDA compact).
>>
>> Changes from v3:
>>  * Nontrivial rebase for indentation fixup (EGPIO and PXA UDC)
>>
>> Petr Cvek (26):
>>   ARM: pxa: magician: Fix indentation in machine files
>>   ARM: pxa: magician: Change comments to be more informative
>>   ARM: pxa: magician: Print more specific error message for global
>>     GPIOs
>>   ARM: pxa: magician: Optimize debug messages for LCD power
>>   ARM: pxa: magician: Change description of LCD power GPIO
>>   ARM: pxa: magician: Add new discovered EGPIO pins
>>   ARM: pxa: magician: Fix HTC Magician pin mux definitions
>>   ARM: pxa: magician: Rename abstract LCD GPIOs
>>   ARM: pxa: magician: Optimize powerup delays for Samsung LCD
>>   ARM: pxa: magician: Optimize Samsung LCD refresh to 50Hz
>>   ARM: pxa: magician: Optimize EGPIO initial values
>>   ARM: pxa: magician: Rename charger cable detection EGPIOs
>>   ARM: pxa: magician: Fix and add charging detection functions
>>   ARM: pxa: magician: Fix platform data for both PXA27x I2C controllers
>>   ARM: pxa: magician: Fix redundant GPIO request for pxaficp_ir
>>   ARM: pxa: magician: Fix support for Intel Strata NOR Flash
>>   ARM: pxa: magician: Fix wrongly enabled USB host ports
>>   ARM: pxa: magician: Add support for ADS7846
>>   ARM: pxa: magician: Add support for Omnivision OV9640 camera
>>   ARM: pxa: magician: Add support for MAX1587A Vcore regulator
>>   ARM: pxa: magician: Add support for PXA27x UDC
>>   ARM: pxa: magician: Remove pdata for pasic3-leds
>>   ARM: pxa: magician: Remove definition of the STUART port
>>   ARM: pxa: magician: Add debug message for backlight brightness
>>     function
>>   ARM: pxa: magician: Add missing regulator for PWM backlight
>>   ARM: pxa: magician: Move platform_add_devices() to the end of
>>     magician_init()
>>
>>  arch/arm/mach-pxa/include/mach/magician.h |  70 ++-
>>  arch/arm/mach-pxa/magician.c              | 907 +++++++++++++++++++++---------
>>  2 files changed, 678 insertions(+), 299 deletions(-)
>>
>> -- 
>> 1.7.12.1
>>
>>
>> _______________________________________________
>> linux-arm-kernel mailing list
>> linux-arm-kernel@lists.infradead.org
>> http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
> 

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

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

#1244072

FromRobert Jarzmik <robert.jarzmik@free.fr>
Date2015-10-11 10:30 +0200
Message-ID<qikb0-2Me-11@gated-at.bofh.it>
In reply to#1243010
Petr Cvek <petr.cvek@tul.cz> writes:

>> Thanks (a pozdrav :-),
>> 
>> 								Pavel
Hi Petr,

I'll start applying part of this serie today evening (my evening).
All Philipp's acked patches for a start.

For the minor changes required by Philipp (comments, ...), if you just state by
mail that you agree I'll make them inline before applying.

Cheers.

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

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

#1244204

FromRobert Jarzmik <robert.jarzmik@free.fr>
Date2015-10-11 22:10 +0200
Message-ID<qiv6p-1UM-11@gated-at.bofh.it>
In reply to#1244072
Robert Jarzmik <robert.jarzmik@free.fr> writes:

> Petr Cvek <petr.cvek@tul.cz> writes:
>
>>> Thanks (a pozdrav :-),
>>> 
>>> 								Pavel
> Hi Petr,
>
> I'll start applying part of this serie today evening (my evening).
> All Philipp's acked patches for a start.
>
> For the minor changes required by Philipp (comments, ...), if you just state by
> mail that you agree I'll make them inline before applying.
>
> Cheers.
>
> --
> Robert
Ok, applied and pushed out in pxa/for-next
    https://github.com/rjarzmik/linux.git pxa/for-next :

425c97c3c937 ARM: pxa: magician: Move platform_add_devices() to the end of magician_init()
3b1c0c4ae8fd ARM: pxa: magician: Add missing regulator for PWM backlight
452f1945c9b7 ARM: pxa: magician: Add debug message for backlight brightness function
20c1c7bb35c2 ARM: pxa: magician: Remove definition of the STUART port
3181b4a3815d ARM: pxa: magician: Fix wrongly enabled USB host ports
ba79d845999d ARM: pxa: magician: Fix support for Intel Strata NOR Flash
d62591e4e5d1 ARM: pxa: magician: Fix redundant GPIO request for pxaficp_ir
d73219de5868 ARM: pxa: magician: Fix platform data for both PXA27x I2C controllers
48e112eb54eb ARM: pxa: magician: Fix and add charging detection functions
65cd6cc24910 ARM: pxa: magician: Optimize Samsung LCD refresh to 50Hz
c85c66dbc212 ARM: pxa: magician: Rename charger cable detection EGPIOs
6fd9d402d8c4 ARM: pxa: magician: Optimize powerup delays for Samsung LCD
e391a4d95dd4 ARM: pxa: magician: Rename abstract LCD GPIOs
32bed563d51f ARM: pxa: magician: Add new discovered EGPIO pins
ab558b6c5834 ARM: pxa: magician: Change description of LCD power GPIO
c1ed9f57e853 ARM: pxa: magician: Optimize debug messages for LCD power
33e910e79b77 ARM: pxa: magician: Print more specific error message for global GPIOs
4c006b908a07 ARM: pxa: magician: Fix indentation in machine files

That leaves out 8 patches, for you to :
 - either resubmit (rebased on top of the pxa/for-next tree)
 - or drop (mostly because of Philipp's NACK or the camera reset)

This way your next submission should be quite shorter and easy to manage.

Cheers.

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

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

#1244377

FromPetr Cvek <petr.cvek@tul.cz>
Date2015-10-12 06:30 +0200
Message-ID<qiCUh-4NC-1@gated-at.bofh.it>
In reply to#1244072
Dne 11.10.2015 v 10:22 Robert Jarzmik napsal(a):
> Petr Cvek <petr.cvek@tul.cz> writes:
> 
>>> Thanks (a pozdrav :-),
>>>
>>> 								Pavel
> Hi Petr,
> 
> I'll start applying part of this serie today evening (my evening).
> All Philipp's acked patches for a start.
> 
> For the minor changes required by Philipp (comments, ...), if you just state by
> mail that you agree I'll make them inline before applying.

Yes they are OK with me.

Petr Cvek

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

[toc] | [prev] | [standalone]

Back to top | Article view | linux.kernel

csiph-web