Re: [PATCH v3] arm: dts: sun7i-a20-bananapi: name the GPIO lines

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 




Am 10.05.2017 um 14:47 schrieb Maxime Ripard:
> On Wed, May 10, 2017 at 06:47:58AM +0200, Oleksij Rempel wrote:
>>>> +		"", "", "", "", "SCL", "SDA", "", "",
>>>
>>> I'm not sure how it's called in the schematics / documentation, but
>>> having a way to get which bus it's from would be helpful (like you did
>>> for the PMIC I2C bus).
>>
>> I use only names provided by schematics. Using something different IMO
>> makes no sense and confusing.
> 
> Indeed.
> 
>>>> +		"", "", "", "", "", "", "", "",
>>>> +		/* PC */
>>>> +		"", "", "", "", "", "", "", "",
>>>> +		"", "", "", "", "", "", "", "",
>>>> +		"", "", "", "", "", "", "", "",
>>>> +		"", "", "", "", "", "", "", "",
>>>> +		/* PD */
>>>> +		"", "", "", "", "", "", "", "",
>>>> +		"", "", "", "", "", "", "", "",
>>>> +		"", "", "", "", "", "", "", "",
>>>> +		"", "", "", "", "", "", "", "",
>>>> +		/* PE */
>>>> +		"", "", "", "", "", "", "", "",
>>>> +		"", "", "", "", "", "", "", "",
>>>> +		"", "", "", "", "", "", "", "",
>>>> +		"", "", "", "", "", "", "", "",
>>>> +		/* PF */
>>>> +		"SD0-D1", "SD0-D0", "SD0-CLK", "SD0-CMD", "SD0-D3",
>>>> +			"SD0-D2", "", "",
>>>
>>> Why did you change the wrapping and indentation on those two lines?
>>
>> i use 8 entry per line to keep counting easier. If line is too wide, i
>> wrap it with changed indentation.
> 
> Ack.
> 
>>>> +		"", "", "", "", "", "", "", "",
>>>> +		"", "", "", "", "", "", "", "",
>>>> +		"", "", "", "", "", "", "", "",
>>>> +		/* PG */
>>>> +		"", "", "", "", "", "", "", "",
>>>> +		"", "", "", "", "", "", "", "",
>>>> +		"", "", "", "", "", "", "", "",
>>>> +		"", "", "", "", "", "", "", "",
>>>> +		/* PH */
>>>> +		"TXD0", "RXD0", "IO-1", "PH3", "USB0-IDDET", "PH5", "", "",
>>>
>>> What are PH3 and PH5 used for on the board? Can't we provide a more
>>> explicit name?
>>
>> Only names used by schematic :) If we will use something different users
>> will hate us even more.
> 
> Those schematics suck then :)
> 
>> I can image to use some kind of combination of connector name and pin
>> name. For example: "JP1-RXD0", or some thing like this.
> 
> There's no mark on the PCB either?

correct, no mark on the PCB.

> RXD0 and the other are fine, the
> only ones that concern me are PH3 and PH5, which are basically only
> the SoC pin name, and nothing else.
> 
> I guess we don't really have the choice if the schematic uses that,
> and there's nothing on the PCB..

ack.

-- 
Regards,
Oleksij

Attachment: signature.asc
Description: OpenPGP digital signature


[Index of Archives]     [Device Tree Compilter]     [Device Tree Spec]     [Linux Driver Backports]     [Video for Linux]     [Linux USB Devel]     [Linux PCI Devel]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [XFree86]     [Yosemite Backpacking]
  Powered by Linux