On Wed, Apr 05, 2017 at 12:16:49PM +0200, gregkh@xxxxxxxxxxxxxxxxxxx wrote: > On Tue, Apr 04, 2017 at 07:32:02PM +0000, alexander.levin@xxxxxxxxxxx wrote: > > From: Chen-Yu Tsai <wens@xxxxxxxx> > > > > [ Upstream commit 91ea2f29cba6a7fe035ea232e4f981211a9fce5d ] > > > > We already have some differences between the 2 supported SoCs. > > More will be added as we support other SoCs. To avoid bloating > > the probe function with even more conditionals, move the quirks > > to a separate data structure that's tied to the compatible string. > > > > Signed-off-by: Chen-Yu Tsai <wens@xxxxxxxx> > > Signed-off-by: Maxime Ripard <maxime.ripard@xxxxxxxxxxxxxxxxxx> > > Signed-off-by: Sasha Levin <alexander.levin@xxxxxxxxxxx> > > --- > > drivers/gpu/drm/sun4i/sun4i_tcon.c | 33 ++++++++++++++++++--------------- > > drivers/gpu/drm/sun4i/sun4i_tcon.h | 11 +++++++---- > > 2 files changed, 25 insertions(+), 19 deletions(-) > > What new hardware is this enabling? It looks like this is just code > reorginizing to me, to make things easier for later patches, which I > don't see in this patch series... Uh, right, I pulled it in for: 49c440e87cd ("drm/sun4i: Add compatible strings for A31/A31s display pipelines") 93a5ec14da2 ("drm/sun4i: Add compatible string for A31/A31s TCON (timing controller)") I'm not sure where they went, can you pull them in please? -- Thanks, Sasha