MPC5200: How to share GPIO and watchdog

MPC5200: How to share GPIO and watchdog

[Juergen Beisert]

Hi list,

the timer0 on the MPC5200B contains one GPIO and a timer/watchdog.

If I'm using this node:

timer@600 {	/* General Purpose Timer */
    compatible =3D "fsl,mpc5200b-gpt","fsl,mpc5200-gpt";
    cell-index =3D <0>;
    reg =3D <0x600 0x10>;
    interrupts =3D <0x1 0x9 0x0>;
    interrupt-parent =3D <&mpc5200_pic>;
   fsl,has-wdt;
};

I will get watchdog support.

If I'm using this node:

timer@600 {	/* General Purpose Timer */
    compatible =3D "fsl,mpc5200b-gpt","fsl,mpc5200-gpt","fsl,mpc5200b-gpt-g=
pio","fsl,mpc5200-gpt-gpio";
    cell-index =3D <0>;
    reg =3D <0x600 0x10>;
    interrupts =3D <0x1 0x9 0x0>;
    interrupt-parent =3D <&mpc5200_pic>;
    gpio-controller;
    #gpio-cells =3D <2>;
    fsl,has-wdt;
};

I will get GPIO support, but no longer watchdog support (watchdogs probe
function will never be called).

Is there a way to get both features supported?

Regards,
Juergen

=2D-=20
Dipl.-Ing. Juergen Beisert | http://www.pengutronix.de
=A0Pengutronix - Linux Solutions for Science and Industry
=A0   Handelsregister: Amtsgericht Hildesheim, HRA 2686
=A0 =A0 =A0    Vertretung Sued/Muenchen, Germany
   Phone: +49-8766-939 228 |  Fax: +49-5121-206917-9

Re: MPC5200: How to share GPIO and watchdog

[Grant Likely]

On Wed, Oct 15, 2008 at 04:42:04PM +0200, Juergen Beisert wrote:
> Hi list,
> 
> the timer0 on the MPC5200B contains one GPIO and a timer/watchdog.
> 
> If I'm using this node:
> 
> timer@600 {	/* General Purpose Timer */
>     compatible = "fsl,mpc5200b-gpt","fsl,mpc5200-gpt";
>     cell-index = <0>;
>     reg = <0x600 0x10>;
>     interrupts = <0x1 0x9 0x0>;
>     interrupt-parent = <&mpc5200_pic>;
>    fsl,has-wdt;
> };
> 
> I will get watchdog support.
> 
> If I'm using this node:
> 
> timer@600 {	/* General Purpose Timer */
>     compatible = "fsl,mpc5200b-gpt","fsl,mpc5200-gpt","fsl,mpc5200b-gpt-gpio","fsl,mpc5200-gpt-gpio";
>     cell-index = <0>;
>     reg = <0x600 0x10>;
>     interrupts = <0x1 0x9 0x0>;
>     interrupt-parent = <&mpc5200_pic>;
>     gpio-controller;
>     #gpio-cells = <2>;
>     fsl,has-wdt;
> };
> 
> I will get GPIO support, but no longer watchdog support (watchdogs probe
> function will never be called).
> 
> Is there a way to get both features supported?

I don't believe so.  In order to make this work you would need to merge
the two drivers into one I think (which probably isn't a bad idea).

g.