Hi David,

>>>> +                  // Port D is LCD exclusive. Don't export as GPIO
>>>> +                  CPM1_PIO: [EMAIL PROTECTED] {
>>>> +                          compatible = "fsl,cpm1-pario";
>>>> +                          reg = <970 180>;
>>>> +                          num-ports = <3>;
>>>> +                          #gpio-cells = <2>;
>>>> +                  };
>>>> +
>>>> +                  [EMAIL PROTECTED] {
>>>> +                          reg = <970 10>;
>>>> +                          compatible = "samsung,ks0713";
>>> Is this representing an LCD controller, or the display itself.  Either
>>> way I'm surprised there's something here in the SoC that has a
>>> compatible string that's not "fsl,something"
>> It's a LCD controller wired to PortD. PortD is used for four 1bit lines
>> and one 8bit bus.
> 
> I'm still kind of confused here.  Does the [EMAIL PROTECTED] node above
> represent the PortD controller?  If the LCD controller is accessed
> solely through PortD, then it should be a child of the PortD node.
> 
> At present, pio and lcd have overlapping reg resources which is
> certainly wrong.
> 

[EMAIL PROTECTED] represent 4 ports (A-D). Ports A-C are used as GPIO lines. 
Port D
is exclusively used for the LCD, but it needs an 8bit accessor. Unfortunately,
the GPIO API doesn't have such an accessor, so the LCD driver needs to access
Port D itself and needs to make sure no other driver can access Port D via the
GPIO API. The representation as child of pio seems reasonable.

Thanks,
Jochen
_______________________________________________
Linuxppc-dev mailing list
Linuxppc-dev@ozlabs.org
https://ozlabs.org/mailman/listinfo/linuxppc-dev

Reply via email to