Re: [PATCH 1/8] ARM: dts: bcm283x: Define standard pinctrl groups in the gpio node.

From: Eric Anholt
Date: Tue Sep 13 2016 - 11:32:15 EST


Dave Stevenson <linux-rpi-kernel@xxxxxxxxxxxxxxxxxxxxxxxxxxx> writes:

> On 09/09/16 22:20, Eric Anholt wrote:
>> e<#secure method=pgpmime mode=sign>
>> Gerd Hoffmann <kraxel@xxxxxxxxxx> writes:
>>
>>> Hi,
>>>
>>>> According to this page [1] the pinctrl group for parallel display interface is
>>>> missing. Is it intended?
>>>>
>>>> [1] - http://elinux.org/RPi_BCM2835_GPIOs
>>> Just an oversight I guess. Eric?
>>>
>>> Does this look correct?
>>>
>>> + dpi_gpio4: dpi_gpio4 {
>>> + brcm,pins = <4 5 6 7 8 9 10 11 12 13
>>> + 14 15 16 17 18 19 20 21
>>> + 22 23 24 25 26 27>;
>>> + brcm,function = <BCM2835_FSEL_ALT2>;
>>> + };
>> For DPI, you also need pins 0-3 in there for clock and syncs.
>>
>> That set of data pins would be for a 24-bit mode, which is what we
>> should be using for the Adafruit kippah + 7" panel combo.
> The Kippah is only 18bit, RGB666.
> https://www.adafruit.com/products/2454
> "The pins used are GPIO 2 through 21 inclusive. That means you don't get
> the UART RX/TX pins (no console cable) and you don't get the standard
> user I2C pins, the EEPROM I2C pins, or hardware SPI pins. You do get to
> use pins #22, #23, #24, #25, #26 and #27, and the USB ports are fine to
> use too."

I was confused because my pinctrl was set up for 18, but the panel
driver for it is set up for 24. It looks like 18 is right, so I should
probably correct the panel driver.

Attachment: signature.asc
Description: PGP signature