Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1752177AbdI2KXK (ORCPT ); Fri, 29 Sep 2017 06:23:10 -0400 Received: from smtp.csie.ntu.edu.tw ([140.112.30.61]:41244 "EHLO smtp.csie.ntu.edu.tw" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751027AbdI2KXI (ORCPT ); Fri, 29 Sep 2017 06:23:08 -0400 X-Google-Smtp-Source: AOwi7QAloKMPrh719GBf3YXKhZLVi4l1T5O1J7+srGBcXQdo/EpYI0YJzKkvPRNjWUGjzjHuIJt2vvHD/0oEo3VtwGw= MIME-Version: 1.0 In-Reply-To: <20170929102006.2wz7wrkpplxwynoh@flea> References: <20170929082306.16193-1-wens@csie.org> <20170929082306.16193-5-wens@csie.org> <20170929102006.2wz7wrkpplxwynoh@flea> From: Chen-Yu Tsai Date: Fri, 29 Sep 2017 18:22:42 +0800 X-Gmail-Original-Message-ID: Message-ID: Subject: Re: [PATCH v3 04/14] drm/sun4i: tcon: Add support for demuxing TCON output on A31 To: Maxime Ripard Cc: Chen-Yu Tsai , Mark Brown , David Airlie , Michael Turquette , Stephen Boyd , Rob Herring , Mark Rutland , dri-devel , linux-arm-kernel , linux-clk , devicetree , linux-kernel , linux-sunxi Content-Type: text/plain; charset="UTF-8" Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 1276 Lines: 32 On Fri, Sep 29, 2017 at 6:20 PM, Maxime Ripard wrote: > On Fri, Sep 29, 2017 at 08:22:56AM +0000, Chen-Yu Tsai wrote: >> On systems with 2 TCONs such as the A31, it is possible to demux the >> output of the TCONs to one encoder. >> >> Add support for this for the A31. >> >> Signed-off-by: Chen-Yu Tsai >> --- >> drivers/gpu/drm/sun4i/sun4i_tcon.c | 38 ++++++++++++++++++++++++++++++++++++++ >> 1 file changed, 38 insertions(+) >> >> diff --git a/drivers/gpu/drm/sun4i/sun4i_tcon.c b/drivers/gpu/drm/sun4i/sun4i_tcon.c >> index 7bf51abaee97..c949309d4285 100644 >> --- a/drivers/gpu/drm/sun4i/sun4i_tcon.c >> +++ b/drivers/gpu/drm/sun4i/sun4i_tcon.c >> @@ -112,6 +112,21 @@ void sun4i_tcon_enable_vblank(struct sun4i_tcon *tcon, bool enable) >> } >> EXPORT_SYMBOL(sun4i_tcon_enable_vblank); >> >> +static struct sun4i_tcon *sun4i_get_first_tcon(struct drm_device *drm) > > Would that make sense to make it a bit more generic, and pass the id > to look for as an argument? The reason to look for TCON0 explicitly is to access the muxing registers, which are only available in TCON0. Other than that, there's nothing else shared between the two TCONs. So there's no particular reason to look for TCON1 explicitly. ChenYu