mbox series

[v8,00/26] Ingenic TCU patchset v8

Message ID 20181212220922.18759-1-paul@crapouillou.net
Headers show
Series Ingenic TCU patchset v8 | expand

Message

Paul Cercueil Dec. 12, 2018, 10:08 p.m. UTC
Hi,

Here's the version 8 and hopefully final version of my patchset, which
adds support for the Timer/Counter Unit found in JZ47xx SoCs from
Ingenic.

The big change is that the timer driver has been simplified. The code to
dynamically update the system timer or clocksource to a new channel has
been removed. Now, the system timer and clocksource are provided as
children nodes in the devicetree, and the TCU channel to use for these
is deduced from their respective memory resource. The PWM driver will
also deduce from its memory resources whether a given PWM channel can be
used, or is reserved for the system timers.

Kind regards,
- Paul Cercueil

Comments

Mathieu Malaterre Jan. 24, 2019, 9:26 p.m. UTC | #1
Paul,

On Wed, Dec 12, 2018 at 11:09 PM Paul Cercueil <paul@crapouillou.net> wrote:
>
> Hi,
>
> Here's the version 8 and hopefully final version of my patchset, which
> adds support for the Timer/Counter Unit found in JZ47xx SoCs from
> Ingenic.

I can no longer boot my MIPS Creator CI20 with this series (merged
opendingux/for-upstream-timer-v8).

Using screen+ttyUSB, I can see messages stopping at:

...
[  OK  ] Started Cgroup management daemon.
         Starting Regular background program processing daemon...
[  OK  ] Started Regular background program processing daemon.
         Starting System Logging Service...
         Starting Provide limited super user privileges to specific users...
         Starting Restore /etc/resolv.conf if the system cras...s shut down....
         Starting WPA supplicant...
         Starting D-Bus System Message Bus...
[  OK  ] Started D-Bus System Message Bus.

Nothing really stands out in the error messages. Could you suggest
things to try out to get into a bootable state ?


> The big change is that the timer driver has been simplified. The code to
> dynamically update the system timer or clocksource to a new channel has
> been removed. Now, the system timer and clocksource are provided as
> children nodes in the devicetree, and the TCU channel to use for these
> is deduced from their respective memory resource. The PWM driver will
> also deduce from its memory resources whether a given PWM channel can be
> used, or is reserved for the system timers.
>
> Kind regards,
> - Paul Cercueil
>
Paul Cercueil Jan. 24, 2019, 9:41 p.m. UTC | #2
Hi Mathieu,

Le jeu. 24 janv. 2019 à 18:26, Mathieu Malaterre <malat@debian.org> a 
écrit :
> Paul,
> 
> On Wed, Dec 12, 2018 at 11:09 PM Paul Cercueil <paul@crapouillou.net> 
> wrote:
>> 
>>  Hi,
>> 
>>  Here's the version 8 and hopefully final version of my patchset, 
>> which
>>  adds support for the Timer/Counter Unit found in JZ47xx SoCs from
>>  Ingenic.
> 
> I can no longer boot my MIPS Creator CI20 with this series (merged
> opendingux/for-upstream-timer-v8).
> 
> Using screen+ttyUSB, I can see messages stopping at:
> 
> ...
> [  OK  ] Started Cgroup management daemon.
>          Starting Regular background program processing daemon...
> [  OK  ] Started Regular background program processing daemon.
>          Starting System Logging Service...
>          Starting Provide limited super user privileges to specific 
> users...
>          Starting Restore /etc/resolv.conf if the system cras...s 
> shut down....
>          Starting WPA supplicant...
>          Starting D-Bus System Message Bus...
> [  OK  ] Started D-Bus System Message Bus.
> 
> Nothing really stands out in the error messages. Could you suggest
> things to try out to get into a bootable state ?

I'm debugging it right now on jz4740, it seems to happen when the 
clocksource
from the ingenic-timer driver is used. Is it your case? It should not 
happen
if you have CONFIG_INGENIC_OST set.

>>  The big change is that the timer driver has been simplified. The 
>> code to
>>  dynamically update the system timer or clocksource to a new channel 
>> has
>>  been removed. Now, the system timer and clocksource are provided as
>>  children nodes in the devicetree, and the TCU channel to use for 
>> these
>>  is deduced from their respective memory resource. The PWM driver 
>> will
>>  also deduce from its memory resources whether a given PWM channel 
>> can be
>>  used, or is reserved for the system timers.
>> 
>>  Kind regards,
>>  - Paul Cercueil
>>