Hi Chris, On Wednesday 05 Oct 2016 13:50:18 Chris Brandt wrote: > > There are 34 compatible strings defined, one per UART + SoC combination, > > plus a set of more generic ones. So far, the driver handles the r7s72100 > > compatible string separately as the UART on that chip is quite peculiar, > > but for all other SoCs we only need to match against > > Just FYI... > There are 2 different types of UARTS (SCI) on r7s72100: SCIF and SCI > > Getting the "SCI" ones to work takes some 'tweaking' of some registers in > the interrupt controller that are outside of the scope of the SCI driver. > Basically, the power on reset values for some IRQ signaling is wrong but > can be adjusted with a register. At the moment, I've been fixing that in > u-boot. > > So in reality, a "renesas,sci-r7s72100" is probably needed too if you want > to throw that one on the pile.... I'm totally fine adding compatible strings when it makes sense, what bothers me is a pile of compat strings to describe IP cores that are identical :-) -- Regards, Laurent Pinchart -- To unsubscribe from this list: send the line "unsubscribe linux-serial" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html