Re: MPC5200: How to share GPIO and watchdog

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.