Re: MPC5200: How to share GPIO and watchdog

[Grant Likely <grant.likely@secretlab.ca>]

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.