[PATCH v2 5/5] ARM: dts: sun6i: Add support for Sinlinx A31s SDK board

[Chen-Yu Tsai]

On Mon, Oct 19, 2015 at 2:15 PM, Maxime Ripard
<maxime.ripard at free-electrons.com> wrote:
> On Fri, Oct 16, 2015 at 03:25:13PM +0800, Chen-Yu Tsai wrote:
>> On Fri, Oct 16, 2015 at 3:20 PM, Maxime Ripard
>> <maxime.ripard at free-electrons.com> wrote:
>> > Hi,
>> >
>> > On Mon, Oct 12, 2015 at 05:42:10PM +0800, Chen-Yu Tsai wrote:
>> >> On Mon, Oct 12, 2015 at 5:30 PM, Maxime Ripard
>> >> <maxime.ripard at free-electrons.com> wrote:
>> >> > Hi,
>> >> >
>> >> > On Sat, Oct 10, 2015 at 10:49:00PM +0800, Chen-Yu Tsai wrote:
>> >> >> +&i2c0 {
>> >> >> +     /* Available on camera header */
>> >> >> +     pinctrl-names = "default";
>> >> >> +     pinctrl-0 = <&i2c0_pins_a>;
>> >> >> +     status = "okay";
>> >> >> +};
>> >> >> +
>> >> >> +&i2c1 {
>> >> >> +     /* Available on LCD header */
>> >> >> +     pinctrl-names = "default";
>> >> >> +     pinctrl-0 = <&i2c1_pins_a>;
>> >> >> +     status = "okay";
>> >> >> +};
>> >> >
>> >> > Are those two actually used anywhere on the board itself, or just
>> >> > exposed on the pin headers?
>> >>
>> >> They are exposed as grouped headers. 1 header has all the camera
>> >> related stuff, like the CSI pins, and various voltage sources
>> >> from the PMIC. The other has the LCD pins, resistive touchpanel,
>> >> I2C for capacitive touchpanel, and of course voltage sources.
>> >>
>> >> The LCD header is an FPC connector, while the camera header is
>> >> the common (common for sunxi devboards) 2.0mm pins. I'm not sure
>> >> if there is some common pinout amongst vendors, but at least
>> >> Sinlinx sells their own 7" LCD display w/ CTP, VGA converter,
>> >> and camera modules. I have both the LCD and VGA converter.
>> >>
>> >> Note that these I2C pins have valid external pull-ups. I doubt
>> >> they can be easily re-purposed.
>> >
>> > Still, the policy we've had so far is that if the user can use it for
>> > something else (like a GPIO, which is the case here), we don't do
>> > enforce any default, and the user will make the right choice.
>>
>> To clarify, unless an actual device is connected, forcing a choice,
>> we won't make that choice for the user, even if the connector is
>> specifically designed for one type of application.
>>
>> Did I get that right?
>
> Yep.

Cool. I already sent an updated version of this patch yesterday.

ChenYu