Hi! Dne petek, 19. maj 2017 ob 19:49:58 CEST je Icenowy Zheng napisal(a): > 于 2017年5月20日 GMT+08:00 上午1:47:29, Maxime Ripard <maxime.ripard@free- electrons.com> 写到: > >On Thu, May 18, 2017 at 12:43:45AM +0800, Icenowy Zheng wrote: > >> From: Icenowy Zheng <icenowy@xxxxxxxx> > >> > >> Allwinner H3 SoC has two mixers, one has 1 VI channel and 3 UI > > > >channels, > > > >> and the other has 1 VI and 1 UI. > >> > >> Add support for these two variants. > >> > >> Signed-off-by: Icenowy Zheng <icenowy@xxxxxxxx> > >> --- > >> > >> drivers/gpu/drm/sun4i/sun8i_mixer.c | 18 ++++++++++++++++++ > >> 1 file changed, 18 insertions(+) > >> > >> diff --git a/drivers/gpu/drm/sun4i/sun8i_mixer.c > > > >b/drivers/gpu/drm/sun4i/sun8i_mixer.c > > > >> index cb193c5f1686..d658a3a8159a 100644 > >> --- a/drivers/gpu/drm/sun4i/sun8i_mixer.c > >> +++ b/drivers/gpu/drm/sun4i/sun8i_mixer.c > >> @@ -390,11 +390,29 @@ static const struct sun8i_mixer_cfg > > > >sun8i_v3s_mixer_cfg = { > > > >> .ui_num = 1, > >> > >> }; > >> > >> +static const struct sun8i_mixer_cfg sun8i_h3_mixer0_cfg = { > >> + .vi_num = 1, > >> + .ui_num = 3, > >> +}; > >> + > >> +static const struct sun8i_mixer_cfg sun8i_h3_mixer1_cfg = { > >> + .vi_num = 1, > >> + .ui_num = 1, > >> +}; > >> + > >> > >> static const struct of_device_id sun8i_mixer_of_table[] = { > >> > >> { > >> > >> .compatible = "allwinner,sun8i-v3s-de2-mixer", > >> .data = &sun8i_v3s_mixer_cfg, > >> > >> }, > >> > >> + { > >> + .compatible = "allwinner,sun8i-h3-de2-mixer0", > >> + .data = &sun8i_h3_mixer0_cfg > >> + }, > >> + { > >> + .compatible = "allwinner,sun8i-h3-de2-mixer1", > >> + .data = &sun8i_h3_mixer1_cfg > >> + }, > > > >So the only difference between the two is the number of ui planes? > > Not only., but currently we only implemented this. > > More functions differ, but we still don't support them... > As far as I can tell, they only differ in ui & vi number of planes and between different SoCs, max plane size. Icenowy, Do you know any other property they differ? I think everything else is based mostly on ui & vi number of planes. Best regards, Jernej > >Why not create a property to give the number then, instead of a > >compatible? > > > >Maxime > > -- > You received this message because you are subscribed to the Google Groups > "linux-sunxi" group. To unsubscribe from this group and stop receiving > emails from it, send an email to linux-sunxi+unsubscribe@xxxxxxxxxxxxxxxx. > For more options, visit https://groups.google.com/d/optout. -- To unsubscribe from this list: send the line "unsubscribe devicetree" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html