Hi Jacopo, On Tue, Sep 11, 2018 at 9:44 AM jacopo mondi <jacopo@xxxxxxxxxx> wrote: > On Mon, Sep 10, 2018 at 03:01:15PM +0200, Simon Horman wrote: > > On Wed, Sep 05, 2018 at 05:29:42PM +0200, Jacopo Mondi wrote: > > > This patch adds VIN{4,5} pins, groups and functions to the R8A77990 SoC. > Currently there are two open questions on this PFC patch: > 2) VIN5 synchronism signals (V/HSYNC, CLKENB, FIELD) are marked as > "_A" only, while VIN4 ones have not _A or _B extensions and are > shared between _A and _B group. The VIN5_#_A extension is an > indication that synchronism signals for group _B are not > multiplexed but active be default according to Morimoto-san, that > is about to confirm this with HW team. In that case, we need to > decide if to provide an 'vin5_sync_b' group anyway to let user > select it from DTS. Otherwise it won't be possible to select > synchronism pins for VIN5_B group (which is maybe fine if they're > not multiplexed at all). If the a "B" sync group exists, the pins are probably configurable as GPIOs, too, so we probably do need a group for them in the driver. Gr{oetje,eeting}s, Geert -- Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert@xxxxxxxxxxxxxx In personal conversations with technical people, I call myself a hacker. But when I'm talking to journalists I just say "programmer" or something like that. -- Linus Torvalds