From mboxrd@z Thu Jan 1 00:00:00 1970 From: Brian Norris Subject: Re: [RESEND PATCH v3 1/3] drm/panel: refactor INNOLUX P079ZCA panel driver Date: Wed, 13 Dec 2017 12:46:29 -0800 Message-ID: <20171213204628.GA237314@google.com> References: <1512371870-15350-1-git-send-email-hl@rock-chips.com> Mime-Version: 1.0 Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: base64 Return-path: Received: from mail-it0-x242.google.com (mail-it0-x242.google.com [IPv6:2607:f8b0:4001:c0b::242]) by gabe.freedesktop.org (Postfix) with ESMTPS id 4DFF76E5CD for ; Wed, 13 Dec 2017 20:46:34 +0000 (UTC) Received: by mail-it0-x242.google.com with SMTP id d137so5927405itc.2 for ; Wed, 13 Dec 2017 12:46:34 -0800 (PST) Content-Disposition: inline In-Reply-To: <1512371870-15350-1-git-send-email-hl@rock-chips.com> List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: dri-devel-bounces@lists.freedesktop.org Sender: "dri-devel" To: Lin Huang Cc: airlied@linux.ie, emil.l.velikov@gmail.com, linux-kernel@vger.kernel.org, robh+dt@kernel.org, nickey.yang@rock-chips.com, thierry.reding@gmail.com, dri-devel@lists.freedesktop.org, zyw@rock-chips.com, Guenter Roeck List-Id: dri-devel@lists.freedesktop.org SGkgSEwsCgpPbiBNb24sIERlYyAwNCwgMjAxNyBhdCAwMzoxNzo0OFBNICswODAwLCBMaW4gSHVh bmcgd3JvdGU6Cj4gUmVmYWN0b3IgSW5ub2x1eCBQMDc5WkNBIHBhbmVsIGRyaXZlciwgbGV0IGl0 IHN1cHBvcnQKPiBtdWx0aSBwYW5lbC4KPiAKPiBTaWduZWQtb2ZmLWJ5OiBMaW4gSHVhbmcgPGhs QHJvY2stY2hpcHMuY29tPgo+IC0tLQo+IENoYW5nZXMgaW4gdjI6Cj4gLSBDaGFuZ2UgcmVndWxh dG9yIHByb3BlcnR5IG5hbWUgdG8gbWVldCB0aGUgcGFuZWwgZGF0YXNoZWV0IAo+IENoYW5nZXMg aW4gdjM6Cj4gLSB0aGlzIHBhdGNoIG9ubHkgcmVmYWN0b3IgUDA3OVpDQSBwYW5lbCB0byBzdXBw b3J0IG11bHRpIHBhbmVsLCBzdXBwb3J0IFAwOTdQRkcgcGFuZWwgaW4gYW5vdGhlciBwYXRjaCAK PiAKPiAgZHJpdmVycy9ncHUvZHJtL3BhbmVsL3BhbmVsLWlubm9sdXgtcDA3OXpjYS5jIHwgMTQ3 ICsrKysrKysrKysrKysrKysrKy0tLS0tLS0tCj4gIDEgZmlsZSBjaGFuZ2VkLCAxMDUgaW5zZXJ0 aW9ucygrKSwgNDIgZGVsZXRpb25zKC0pCj4gCj4gZGlmZiAtLWdpdCBhL2RyaXZlcnMvZ3B1L2Ry bS9wYW5lbC9wYW5lbC1pbm5vbHV4LXAwNzl6Y2EuYyBiL2RyaXZlcnMvZ3B1L2RybS9wYW5lbC9w YW5lbC1pbm5vbHV4LXAwNzl6Y2EuYwo+IGluZGV4IDZiYTkzNDQuLjE1OTc3NDQgMTAwNjQ0Cj4g LS0tIGEvZHJpdmVycy9ncHUvZHJtL3BhbmVsL3BhbmVsLWlubm9sdXgtcDA3OXpjYS5jCj4gKysr IGIvZHJpdmVycy9ncHUvZHJtL3BhbmVsL3BhbmVsLWlubm9sdXgtcDA3OXpjYS5jCi4uLgoKPiBA QCAtMTM0LDEyICsxNjIsMTMgQEAgc3RhdGljIGludCBpbm5vbHV4X3BhbmVsX3ByZXBhcmUoc3Ry dWN0IGRybV9wYW5lbCAqcGFuZWwpCj4gIAlyZXR1cm4gMDsKPiAgCj4gIHBvd2Vyb2ZmOgo+IC0J cmVndWxhdG9yX2VyciA9IHJlZ3VsYXRvcl9kaXNhYmxlKGlubm9sdXgtPnN1cHBseSk7CgpJIHRo aW5rICdyZWd1bGF0b3JfZXJyJyBpcyB1bnVzZWQgYWZ0ZXIgdGhpcyBwYXRjaC4gUGxlYXNlIHJl bW92ZSBpdC4KCj4gLQlpZiAocmVndWxhdG9yX2VycikKPiAtCQlEUk1fREVWX0VSUk9SKHBhbmVs LT5kZXYsICJmYWlsZWQgdG8gZGlzYWJsZSByZWd1bGF0b3I6ICVkXG4iLAo+IC0JCQkgICAgICBy ZWd1bGF0b3JfZXJyKTsKPiAtCj4gIAlncGlvZF9zZXRfdmFsdWVfY2Fuc2xlZXAoaW5ub2x1eC0+ ZW5hYmxlX2dwaW8sIDApOwo+ICsJcmVndWxhdG9yX2Rpc2FibGUoaW5ub2x1eC0+YXZlZSk7Cj4g K2Rpc2FibGVfYXZkZDoKPiArCXJlZ3VsYXRvcl9kaXNhYmxlKGlubm9sdXgtPmF2ZGQpOwo+ICtk aXNhYmxlX3ZkZGk6Cj4gKwlyZWd1bGF0b3JfZGlzYWJsZShpbm5vbHV4LT52ZGRpKTsKPiArCj4g IAlyZXR1cm4gZXJyOwo+ICB9Cj4gIAoKLi4uCgo+IEBAIC0yMDksMjAgKzI1NiwzNiBAQCBzdGF0 aWMgY29uc3Qgc3RydWN0IGRybV9wYW5lbF9mdW5jcyBpbm5vbHV4X3BhbmVsX2Z1bmNzID0gewo+ ICB9Owo+ICAKPiAgc3RhdGljIGNvbnN0IHN0cnVjdCBvZl9kZXZpY2VfaWQgaW5ub2x1eF9vZl9t YXRjaFtdID0gewo+IC0JeyAuY29tcGF0aWJsZSA9ICJpbm5vbHV4LHAwNzl6Y2EiLCB9LAo+IC0J eyB9CgpZb3UncmUgZGVsZXRpbmcgdGhpcyB0ZXJtaW5hdG9yLiBQbGVhc2UgZG9uJ3QuCgo+ICsJ eyAuY29tcGF0aWJsZSA9ICJpbm5vbHV4LHAwNzl6Y2EiLAo+ICsJICAuZGF0YSA9ICZpbm5vbHV4 X3AwNzl6Y2FfcGFuZWxfZGVzYwo+ICsJfSwKPiAgfTsKPiAgTU9EVUxFX0RFVklDRV9UQUJMRShv ZiwgaW5ub2x1eF9vZl9tYXRjaCk7Cj4gIAoKLi4uCgpCcmlhbgpfX19fX19fX19fX19fX19fX19f X19fX19fX19fX19fX19fX19fX19fX19fX19fXwpkcmktZGV2ZWwgbWFpbGluZyBsaXN0CmRyaS1k ZXZlbEBsaXN0cy5mcmVlZGVza3RvcC5vcmcKaHR0cHM6Ly9saXN0cy5mcmVlZGVza3RvcC5vcmcv bWFpbG1hbi9saXN0aW5mby9kcmktZGV2ZWwK From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1753892AbdLMUql (ORCPT ); Wed, 13 Dec 2017 15:46:41 -0500 Received: from mail-it0-f65.google.com ([209.85.214.65]:38538 "EHLO mail-it0-f65.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752932AbdLMUqe (ORCPT ); Wed, 13 Dec 2017 15:46:34 -0500 X-Google-Smtp-Source: ACJfBos+KKYxo9QBXBD75UbRw/yH/K2CHLxfeKWemTXEfNb0QRB+xicBYzT2kyGgy7D7Aj2Du5mmgg== Date: Wed, 13 Dec 2017 12:46:29 -0800 From: Brian Norris To: Lin Huang Cc: emil.l.velikov@gmail.com, thierry.reding@gmail.com, robh+dt@kernel.org, zyw@rock-chips.com, seanpaul@chromium.org, airlied@linux.ie, linux-kernel@vger.kernel.org, dri-devel@lists.freedesktop.org, nickey.yang@rock-chips.com, Guenter Roeck Subject: Re: [RESEND PATCH v3 1/3] drm/panel: refactor INNOLUX P079ZCA panel driver Message-ID: <20171213204628.GA237314@google.com> References: <1512371870-15350-1-git-send-email-hl@rock-chips.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <1512371870-15350-1-git-send-email-hl@rock-chips.com> User-Agent: Mutt/1.5.21 (2010-09-15) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Hi HL, On Mon, Dec 04, 2017 at 03:17:48PM +0800, Lin Huang wrote: > Refactor Innolux P079ZCA panel driver, let it support > multi panel. > > Signed-off-by: Lin Huang > --- > Changes in v2: > - Change regulator property name to meet the panel datasheet > Changes in v3: > - this patch only refactor P079ZCA panel to support multi panel, support P097PFG panel in another patch > > drivers/gpu/drm/panel/panel-innolux-p079zca.c | 147 ++++++++++++++++++-------- > 1 file changed, 105 insertions(+), 42 deletions(-) > > diff --git a/drivers/gpu/drm/panel/panel-innolux-p079zca.c b/drivers/gpu/drm/panel/panel-innolux-p079zca.c > index 6ba9344..1597744 100644 > --- a/drivers/gpu/drm/panel/panel-innolux-p079zca.c > +++ b/drivers/gpu/drm/panel/panel-innolux-p079zca.c ... > @@ -134,12 +162,13 @@ static int innolux_panel_prepare(struct drm_panel *panel) > return 0; > > poweroff: > - regulator_err = regulator_disable(innolux->supply); I think 'regulator_err' is unused after this patch. Please remove it. > - if (regulator_err) > - DRM_DEV_ERROR(panel->dev, "failed to disable regulator: %d\n", > - regulator_err); > - > gpiod_set_value_cansleep(innolux->enable_gpio, 0); > + regulator_disable(innolux->avee); > +disable_avdd: > + regulator_disable(innolux->avdd); > +disable_vddi: > + regulator_disable(innolux->vddi); > + > return err; > } > ... > @@ -209,20 +256,36 @@ static const struct drm_panel_funcs innolux_panel_funcs = { > }; > > static const struct of_device_id innolux_of_match[] = { > - { .compatible = "innolux,p079zca", }, > - { } You're deleting this terminator. Please don't. > + { .compatible = "innolux,p079zca", > + .data = &innolux_p079zca_panel_desc > + }, > }; > MODULE_DEVICE_TABLE(of, innolux_of_match); > ... Brian