On 02/21/2018 01:18 PM, Drew Fustini wrote:
On Wed, Feb 21, 2018 at 11:01 AM, David Lechner <da...@lechnology.com> wrote:
Are you sure your phandles are correct here? They don't match the comments.
Should it be <&gpio1 16 0> instead?

+                               reset-gpios = <&gpio1 16 0>; /*
reset:60 P9.12 GPIO1_28 */

And then <&gpio1 28 0> here?
Here is the full BB-SPIDEV1-00A0.dts:
https://gist.github.com/pdp7/aee5664598059c9b9a9020f107957f80

I have P9.12 connected to LCD reset pin.  I have P9.15 connected to
LCD dc pin.  For fbtft driver, this translated to GPIO_60 for reset
and GPIO_48 for dc.  This is based on this pinout diagram:
https://elinux.org/File:Cape_expansion_headers.PNG

For this dts file, I was trying to figure out what appropriate gpio
phandle.  I looked at this table for the P9 header:
https://elinux.org/File:H9Pinout.PNG

I believe that gpio1[16] corresponds to P9.15 which is LCD dc pin, and
gpio1[28] corresponds to P9.12 which is LCD reset pin.

My understanding of those mappings may be incorrect, so I'd appreciate
any additional feedback.

You understand correct, but I think you have swapped the values in your device tree overlay. You say "gpio1[16] 
corresponds to P9.15 which is LCD dc pin", but in the overlay, you have dc-gpios = <&gpio1 28 0>;. 
Likewise "gpio1[28] corresponds to P9.12 which is LCD reset pin.", but you have reset-gpios = <&gpio1 
16 0>;.


--
For more options, visit http://beagleboard.org/discuss
--- You received this message because you are subscribed to the Google Groups "BeagleBoard" group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to beagleboard+unsubscr...@googlegroups.com.
To view this discussion on the web visit 
https://groups.google.com/d/msgid/beagleboard/df2f4b67-8a4a-f266-22fa-a3474961d68f%40lechnology.com.
For more options, visit https://groups.google.com/d/optout.

Reply via email to