From mboxrd@z Thu Jan 1 00:00:00 1970 From: Chris Zhong Subject: Re: [PATCH v2 1/4] drm/rockchip/dsi: check phy_cfg_clk only for RK3399 Date: Fri, 17 Mar 2017 09:19:06 +0800 Message-ID: <58CB398A.9050800@rock-chips.com> References: <1489635107-21327-1-git-send-email-zyw@rock-chips.com> <1489635107-21327-2-git-send-email-zyw@rock-chips.com> <20170316105500.675b413f.john@metanate.com> Mime-Version: 1.0 Content-Type: text/plain; charset="utf-8"; Format="flowed" Content-Transfer-Encoding: base64 Return-path: In-Reply-To: <20170316105500.675b413f.john@metanate.com> List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: dri-devel-bounces@lists.freedesktop.org Sender: "dri-devel" To: John Keeping Cc: linux-kernel@vger.kernel.org, dri-devel@lists.freedesktop.org, linux-rockchip@lists.infradead.org, linux-arm-kernel@lists.infradead.org List-Id: linux-rockchip.vger.kernel.org SGkgSm9obgoKT24gMDMvMTYvMjAxNyAwNjo1NSBQTSwgSm9obiBLZWVwaW5nIHdyb3RlOgo+IE9u IFRodSwgMTYgTWFyIDIwMTcgMTE6MzE6NDQgKzA4MDAsIENocmlzIFpob25nIHdyb3RlOgo+Cj4+ IEZvciBSSzMzOTksIHRoZSBwaHlfY2ZnX2NsayBpcyBhIHJlcXVpcmVkIGNsb2NrLCBpZiBwaHlf Y2ZnX2NsayBpcwo+PiBkaXNhYmxlZCwgTUlQSSBwaHkgY2FuIG5vdCB3b3JrLiBMZXQncyByZXR1 cm4gYSBlcnJvciBpZiB0aGVyZSBpcyBubwo+PiBwaHlfY2ZnX2NsayBpbiBkdHMgcHJvcGVydHks IHdoZW4gdGhlIHBkYXRhIG1hdGNoIFJLMzM5OS4KPj4KPj4gU2lnbmVkLW9mZi1ieTogQ2hyaXMg WmhvbmcgPHp5d0Byb2NrLWNoaXBzLmNvbT4KPj4gLS0tCj4+Cj4+IENoYW5nZXMgaW4gdjI6IE5v bmUKPj4KPj4gICBkcml2ZXJzL2dwdS9kcm0vcm9ja2NoaXAvZHctbWlwaS1kc2kuYyB8IDEwICsr KystLS0tLS0KPj4gICAxIGZpbGUgY2hhbmdlZCwgNCBpbnNlcnRpb25zKCspLCA2IGRlbGV0aW9u cygtKQo+Pgo+PiBkaWZmIC0tZ2l0IGEvZHJpdmVycy9ncHUvZHJtL3JvY2tjaGlwL2R3LW1pcGkt ZHNpLmMgYi9kcml2ZXJzL2dwdS9kcm0vcm9ja2NoaXAvZHctbWlwaS1kc2kuYwo+PiBpbmRleCBm ODRmOWFlLi4xMWM0MTY2IDEwMDY0NAo+PiAtLS0gYS9kcml2ZXJzL2dwdS9kcm0vcm9ja2NoaXAv ZHctbWlwaS1kc2kuYwo+PiArKysgYi9kcml2ZXJzL2dwdS9kcm0vcm9ja2NoaXAvZHctbWlwaS1k c2kuYwo+PiBAQCAtMTIyNywxNSArMTIyNywxMyBAQCBzdGF0aWMgaW50IGR3X21pcGlfZHNpX2Jp bmQoc3RydWN0IGRldmljZSAqZGV2LCBzdHJ1Y3QgZGV2aWNlICptYXN0ZXIsCj4+ICAgCQljbGtf ZGlzYWJsZV91bnByZXBhcmUoZHNpLT5wY2xrKTsKPj4gICAJfQo+PiAgIAo+PiAtCWRzaS0+cGh5 X2NmZ19jbGsgPSBkZXZtX2Nsa19nZXQoZGV2LCAicGh5X2NmZyIpOwo+PiAtCWlmIChJU19FUlIo ZHNpLT5waHlfY2ZnX2NsaykpIHsKPj4gLQkJcmV0ID0gUFRSX0VSUihkc2ktPnBoeV9jZmdfY2xr KTsKPj4gLQkJaWYgKHJldCAhPSAtRU5PRU5UKSB7Cj4+ICsJaWYgKHBkYXRhID09ICZyazMzOTlf bWlwaV9kc2lfZHJ2X2RhdGEpIHsKPiBUaGlzIHdpbGwgZ2V0IG1lc3N5IGlmIHRoZSBuZXh0IFNP QyBhbHNvIG5lZWRzIHBoeV9jZmdfY2xrLiAgQ2FuIHdlIGRvCj4gc29tZXRoaW5nIGxpa2U6Cj4K PiAJaWYgKHBkYXRhLT5mbGFncyAmIERXX01JUElfTkVFRFNfUEhZX0NGR19DTEspIHsKPiAJCS4u LgpUaGFua3MsIGdvb2QgaWRlYS4gSSB0aGluayBSSzMzNjggbWlwaS1kc2kgZHJpdmVyIGlzIG9u IHRoZSB3YXkuIDopCgoKPj4gKwkJZHNpLT5waHlfY2ZnX2NsayA9IGRldm1fY2xrX2dldChkZXYs ICJwaHlfY2ZnIik7Cj4+ICsJCWlmIChJU19FUlIoZHNpLT5waHlfY2ZnX2NsaykpIHsKPj4gKwkJ CXJldCA9IFBUUl9FUlIoZHNpLT5waHlfY2ZnX2Nsayk7Cj4+ICAgCQkJZGV2X2VycihkZXYsICJV bmFibGUgdG8gZ2V0IHBoeV9jZmdfY2xrOiAlZFxuIiwgcmV0KTsKPj4gICAJCQlyZXR1cm4gcmV0 Owo+PiAgIAkJfQo+PiAtCQlkc2ktPnBoeV9jZmdfY2xrID0gTlVMTDsKPj4gLQkJZGV2X2RiZyhk ZXYsICJoYXZlIG5vdCBwaHlfY2ZnX2Nsa1xuIik7Cj4+ICAgCX0KPj4gICAKPj4gICAJcmV0ID0g Y2xrX3ByZXBhcmVfZW5hYmxlKGRzaS0+cGxscmVmX2Nsayk7Cj4KPgoKLS0gCkNocmlzIFpob25n CgoKX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX18KZHJpLWRl dmVsIG1haWxpbmcgbGlzdApkcmktZGV2ZWxAbGlzdHMuZnJlZWRlc2t0b3Aub3JnCmh0dHBzOi8v bGlzdHMuZnJlZWRlc2t0b3Aub3JnL21haWxtYW4vbGlzdGluZm8vZHJpLWRldmVsCg== From mboxrd@z Thu Jan 1 00:00:00 1970 From: zyw@rock-chips.com (Chris Zhong) Date: Fri, 17 Mar 2017 09:19:06 +0800 Subject: [PATCH v2 1/4] drm/rockchip/dsi: check phy_cfg_clk only for RK3399 In-Reply-To: <20170316105500.675b413f.john@metanate.com> References: <1489635107-21327-1-git-send-email-zyw@rock-chips.com> <1489635107-21327-2-git-send-email-zyw@rock-chips.com> <20170316105500.675b413f.john@metanate.com> Message-ID: <58CB398A.9050800@rock-chips.com> To: linux-arm-kernel@lists.infradead.org List-Id: linux-arm-kernel.lists.infradead.org Hi John On 03/16/2017 06:55 PM, John Keeping wrote: > On Thu, 16 Mar 2017 11:31:44 +0800, Chris Zhong wrote: > >> For RK3399, the phy_cfg_clk is a required clock, if phy_cfg_clk is >> disabled, MIPI phy can not work. Let's return a error if there is no >> phy_cfg_clk in dts property, when the pdata match RK3399. >> >> Signed-off-by: Chris Zhong >> --- >> >> Changes in v2: None >> >> drivers/gpu/drm/rockchip/dw-mipi-dsi.c | 10 ++++------ >> 1 file changed, 4 insertions(+), 6 deletions(-) >> >> diff --git a/drivers/gpu/drm/rockchip/dw-mipi-dsi.c b/drivers/gpu/drm/rockchip/dw-mipi-dsi.c >> index f84f9ae..11c4166 100644 >> --- a/drivers/gpu/drm/rockchip/dw-mipi-dsi.c >> +++ b/drivers/gpu/drm/rockchip/dw-mipi-dsi.c >> @@ -1227,15 +1227,13 @@ static int dw_mipi_dsi_bind(struct device *dev, struct device *master, >> clk_disable_unprepare(dsi->pclk); >> } >> >> - dsi->phy_cfg_clk = devm_clk_get(dev, "phy_cfg"); >> - if (IS_ERR(dsi->phy_cfg_clk)) { >> - ret = PTR_ERR(dsi->phy_cfg_clk); >> - if (ret != -ENOENT) { >> + if (pdata == &rk3399_mipi_dsi_drv_data) { > This will get messy if the next SOC also needs phy_cfg_clk. Can we do > something like: > > if (pdata->flags & DW_MIPI_NEEDS_PHY_CFG_CLK) { > ... Thanks, good idea. I think RK3368 mipi-dsi driver is on the way. :) >> + dsi->phy_cfg_clk = devm_clk_get(dev, "phy_cfg"); >> + if (IS_ERR(dsi->phy_cfg_clk)) { >> + ret = PTR_ERR(dsi->phy_cfg_clk); >> dev_err(dev, "Unable to get phy_cfg_clk: %d\n", ret); >> return ret; >> } >> - dsi->phy_cfg_clk = NULL; >> - dev_dbg(dev, "have not phy_cfg_clk\n"); >> } >> >> ret = clk_prepare_enable(dsi->pllref_clk); > > -- Chris Zhong From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1752358AbdCQB3Q (ORCPT ); Thu, 16 Mar 2017 21:29:16 -0400 Received: from regular1.263xmail.com ([211.150.99.133]:52102 "EHLO regular1.263xmail.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751098AbdCQB3L (ORCPT ); Thu, 16 Mar 2017 21:29:11 -0400 X-263anti-spam: KSV:0; X-MAIL-GRAY: 0 X-MAIL-DELIVERY: 1 X-KSVirus-check: 0 X-ABS-CHECKED: 4 X-RL-SENDER: zyw@rock-chips.com X-FST-TO: mark.yao@rock-chips.com X-SENDER-IP: 103.29.142.67 X-LOGIN-NAME: zyw@rock-chips.com X-UNIQUE-TAG: <4b672fb4a5a85e2e35217991a81bdfed> X-ATTACHMENT-NUM: 0 X-DNS-TYPE: 0 Subject: Re: [PATCH v2 1/4] drm/rockchip/dsi: check phy_cfg_clk only for RK3399 To: John Keeping References: <1489635107-21327-1-git-send-email-zyw@rock-chips.com> <1489635107-21327-2-git-send-email-zyw@rock-chips.com> <20170316105500.675b413f.john@metanate.com> Cc: linux-rockchip@lists.infradead.org, Heiko Stuebner , David Airlie , linux-kernel@vger.kernel.org, dri-devel@lists.freedesktop.org, linux-arm-kernel@lists.infradead.org, Mark Yao From: Chris Zhong Message-ID: <58CB398A.9050800@rock-chips.com> Date: Fri, 17 Mar 2017 09:19:06 +0800 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:38.0) Gecko/20100101 Thunderbird/38.4.0 MIME-Version: 1.0 In-Reply-To: <20170316105500.675b413f.john@metanate.com> Content-Type: text/plain; charset=windows-1252; format=flowed Content-Transfer-Encoding: 7bit Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Hi John On 03/16/2017 06:55 PM, John Keeping wrote: > On Thu, 16 Mar 2017 11:31:44 +0800, Chris Zhong wrote: > >> For RK3399, the phy_cfg_clk is a required clock, if phy_cfg_clk is >> disabled, MIPI phy can not work. Let's return a error if there is no >> phy_cfg_clk in dts property, when the pdata match RK3399. >> >> Signed-off-by: Chris Zhong >> --- >> >> Changes in v2: None >> >> drivers/gpu/drm/rockchip/dw-mipi-dsi.c | 10 ++++------ >> 1 file changed, 4 insertions(+), 6 deletions(-) >> >> diff --git a/drivers/gpu/drm/rockchip/dw-mipi-dsi.c b/drivers/gpu/drm/rockchip/dw-mipi-dsi.c >> index f84f9ae..11c4166 100644 >> --- a/drivers/gpu/drm/rockchip/dw-mipi-dsi.c >> +++ b/drivers/gpu/drm/rockchip/dw-mipi-dsi.c >> @@ -1227,15 +1227,13 @@ static int dw_mipi_dsi_bind(struct device *dev, struct device *master, >> clk_disable_unprepare(dsi->pclk); >> } >> >> - dsi->phy_cfg_clk = devm_clk_get(dev, "phy_cfg"); >> - if (IS_ERR(dsi->phy_cfg_clk)) { >> - ret = PTR_ERR(dsi->phy_cfg_clk); >> - if (ret != -ENOENT) { >> + if (pdata == &rk3399_mipi_dsi_drv_data) { > This will get messy if the next SOC also needs phy_cfg_clk. Can we do > something like: > > if (pdata->flags & DW_MIPI_NEEDS_PHY_CFG_CLK) { > ... Thanks, good idea. I think RK3368 mipi-dsi driver is on the way. :) >> + dsi->phy_cfg_clk = devm_clk_get(dev, "phy_cfg"); >> + if (IS_ERR(dsi->phy_cfg_clk)) { >> + ret = PTR_ERR(dsi->phy_cfg_clk); >> dev_err(dev, "Unable to get phy_cfg_clk: %d\n", ret); >> return ret; >> } >> - dsi->phy_cfg_clk = NULL; >> - dev_dbg(dev, "have not phy_cfg_clk\n"); >> } >> >> ret = clk_prepare_enable(dsi->pllref_clk); > > -- Chris Zhong