Hi Geert,
On Wednesday 05 Oct 2016 11:51:49 Geert Uytterhoeven wrote:
> On Wed, Oct 5, 2016 at 10:33 AM, Niklas Söderlund wrote:
> > On 2016-10-04 21:13:18 +0200, Geert Uytterhoeven wrote:
> >> On Tue, Sep 13, 2016 at 4:03 PM, Niklas Söderlund wrote:
> >> > --- a/drivers/pinctrl/sh-pfc/pfc-r8a7795.c
> >> > +++ b/drivers/pinctrl/sh-pfc/pfc-r8a7795.c
[snip]
> >>> + SH_PFC_PIN_NAMED_CFG(ROW_GROUP_A('E'), 4, AE4,
> >>> SH_PFC_PIN_CFG_DRIVE_STRENGTH), /* QSPI1_IO2 */
> >>> + SH_PFC_PIN_NAMED_CFG(ROW_GROUP_A('E'), 5, AE5,
> >>> SH_PFC_PIN_CFG_DRIVE_STRENGTH), /* QSPI1_MISO_IO1 */
> >>> + SH_PFC_PIN_NAMED_CFG(ROW_GROUP_A('P'), 7, AP7,
> >>> SH_PFC_PIN_CFG_DRIVE_STRENGTH), /* DU_DOTCLKIN0 */
> >>> + SH_PFC_PIN_NAMED_CFG(ROW_GROUP_A('P'), 8, AP8,
> >>> SH_PFC_PIN_CFG_DRIVE_STRENGTH), /* DU_DOTCLKIN1 */
> >>> + SH_PFC_PIN_NAMED_CFG(ROW_GROUP_A('R'), 7, AR7,
> >>> SH_PFC_PIN_CFG_DRIVE_STRENGTH), /* DU_DOTCLKIN2 */
> >>> + SH_PFC_PIN_NAMED_CFG(ROW_GROUP_A('R'), 8, AR8,
> >>> SH_PFC_PIN_CFG_DRIVE_STRENGTH), /* DU_DOTCLKIN3 */
> >>> + SH_PFC_PIN_NAMED_CFG(ROW_GROUP_A('R'), 30, AR30,
> >>> SH_PFC_PIN_CFG_DRIVE_STRENGTH), /* TMS */
> >>> + SH_PFC_PIN_NAMED_CFG(ROW_GROUP_A('T'), 28, AT28,
> >>> SH_PFC_PIN_CFG_DRIVE_STRENGTH), /* TDO */
> >>> + SH_PFC_PIN_NAMED_CFG(ROW_GROUP_A('T'), 30, AT30,
> >>> SH_PFC_PIN_CFG_DRIVE_STRENGTH), /* ASEBRK */>
> >>
> >> All these pin numbers match R-Car H3SiP, while there exists also a plain
> >> R-Car H3, which uses completely different pin numbers.
> >>
> >> How are we gonna distinguish these two variants?
> >> Perhaps we can refer to these pins in some other way, to have consistent
> >> numbering?
> >>
> >> Or don't we have to? Are these numbers visible in userspace (sysfs)?
> >
> > Unfortunately both the number and name are show in sysfs under
> > '/sys/kernel/debug/pinctrl/e6060000.pfc/*', example from the pins node:
> >
> > <snip>
> > pin 1906 (PIN_AP7) sh-pfc
> > pin 1907 (PIN_AP8) sh-pfc
> > pin 1984 (PIN_AR7) sh-pfc
> > pin 1985 (PIN_AR8) sh-pfc
> > pin 2007 (PIN_AR30) sh-pfc
> > pin 2083 (PIN_AT28) sh-pfc
> > pin 2085 (PIN_AT30) sh-pfc
> > </snip>
>
> Thanks for checking!
>
> > So yes a way to present consistent names is needed if this driver should
> > match both H3 variants. But I'm not sure the numbers needs to be
> > correlated to the pin matrix they only need to be unique I think, please
> > correct me if I'm wrong. And if that is the case then maybe a solution
>
> Yes, I also think they just have to be unique.
> Having some system to make it easier to have unique numbers is nice.
>
> > to the problem is to simply change the name of the pins from there pin
> > matrix location to there function:
> >
> > - SH_PFC_PIN_NAMED_CFG(ROW_GROUP_A('T'), 30, AT30,
> > SH_PFC_PIN_CFG_DRIVE_STRENGTH), /* ASEBRK */
> > + SH_PFC_PIN_NAMED_CFG(ROW_GROUP_A('T'), 30, ASEBRK,
> > SH_PFC_PIN_CFG_DRIVE_STRENGTH), /* ASEBRK */
> >
> > That would keep the names and numbers consistent on both H3 varinats.
> > The names would correlate to function and the numbers simply serve as a
> > pin identifier which is unique and derived from the H3SiP pin layout,
> > probably a comment about this in the source is a good idea :-)
>
> So "the system" would be H3SiP pin numbers.
> Looks good to me.
>
> Laurent, do you agree?
I'm fine with that.
--
Regards,
Laurent Pinchart
