From mboxrd@z Thu Jan 1 00:00:00 1970 From: Jeykumar Sankaran Subject: Re: [PATCH] drm/msm/disp/dpu: Use proper define for drm_encoder_init() 'encoder_type' Date: Fri, 17 Aug 2018 10:00:02 -0700 Message-ID: <63a08a5c9f73b369c5d579ee340fca72@codeaurora.org> References: <20180816233616.148248-1-swboyd@chromium.org> Mime-Version: 1.0 Content-Type: text/plain; charset="utf-8"; Format="flowed" Content-Transfer-Encoding: base64 Return-path: In-Reply-To: <20180816233616.148248-1-swboyd-F7+t8E8rja9g9hUCZPvPmw@public.gmane.org> List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: freedreno-bounces-PD4FTy7X32lNgt0PjOBp9y5qC8QIuHrW@public.gmane.org Sender: "Freedreno" To: Stephen Boyd Cc: linux-arm-msm-u79uwXL29TY76Z2rM5mHXA@public.gmane.org, Jordan Crouse , dri-devel-PD4FTy7X32lNgt0PjOBp9y5qC8QIuHrW@public.gmane.org, linux-kernel-u79uwXL29TY76Z2rM5mHXA@public.gmane.org, Rob Clark , Sean Paul , freedreno-PD4FTy7X32lNgt0PjOBp9y5qC8QIuHrW@public.gmane.org List-Id: linux-arm-msm@vger.kernel.org T24gMjAxOC0wOC0xNiAxNjozNiwgU3RlcGhlbiBCb3lkIHdyb3RlOgo+IFdlIGdvdCBhIGJ1ZyBy ZXBvcnQgdGhhdCB0aGlzIGZ1bmN0aW9uIG9vcHNlcyB3aGVuIHRyeWluZyB0byBkbyBhCj4ga2Fz cHJpbnRmKCkuCj4gCj4gUEMgaXMgYXQgc3RyaW5nKzB4MmMvMHg2MAo+IExSIGlzIGF0IHZzbnBy aW50ZisweDI4Yy8weDRlYwo+IHBjIDogWzxmZmZmZmY4MDA4OGQzNWQ4Pl0gbHIgOiBbPGZmZmZm ZjgwMDg4ZDVmYzQ+XSBwc3RhdGU6IGEwYzAwMDQ5Cj4gc3AgOiBmZmZmZmY4MDA5NWZiNTQwCj4g eDI5OiBmZmZmZmY4MDA5NWZiNTQwIHgyODogZmZmZmZmODAwOGFkNDJiYwo+IHgyNzogMDAwMDAw MDBmZmZmZmZkOCB4MjY6IDAwMDAwMDAwMDAwMDAwMDAKPiB4MjU6IGZmZmZmZjgwMDhjMjE2Yzgg eDI0OiAwMDAwMDAwMDAwMDAwMDAwCj4geDIzOiAwMDAwMDAwMDAwMDAwMDAwIHgyMjogZmZmZmZm ODAwOTVmYjcyMAo+IHgyMTogMDAwMDAwMDAwMDAwMDAwMCB4MjA6IGZmZmZmZjgwMDk1ZmI3MjAK PiB4MTk6IGZmZmZmZjgwMDk1ZmI2ZjAgeDE4OiAwMDAwMDAwMDAwMDAwMDBhCj4geDE3OiAwMDAw MDAwMGI0MmJhNDczIHgxNjogZmZmZmZmODAwODA1YmJlOAo+IHgxNTogMDAwMDAwMDAwMDBhMTU3 ZCB4MTQ6IDAwMDAwMDAwMDAwMDAwMGMKPiB4MTM6IDAwMDAwMDAwMDAwMDAwMDAgeDEyOiAwMDAw ZmZmZjAwMDAwMDBmCj4geDExOiAwMDAwMDAwMDAwMDAwMDAzIHgxMDogMDAwMDAwMDAwMDAwMDAw MQo+IHg5IDogMDAwMDAwMDAwMDAwMDA0MCB4OCA6IDAwMDAwMDAwMDAwMDAwMWMKPiB4NyA6IGZm ZmZmZmZmZmZmZmZmZmYgeDYgOiAwMDAwMDAwMDAwMDAwMDAwCj4geDUgOiAwMDAwMDAwMDAwMDAw MjI4IHg0IDogMDAwMDAwMDAwMDAwMDAwMAo+IHgzIDogZmZmZjBhMDBmZmZmZmYwNCB4MiA6IDAw MDAwMDAwMDAwMDc5NjEKPiB4MSA6IDAwMDAwMDAwMDAwMDAwMDAgeDAgOiAwMDAwMDAwMDAwMDAw MDAwCj4gUHJvY2VzcyBrd29ya2VyLzM6MSAocGlkOiA2MSwgc3RhY2sgbGltaXQgPSAweGZmZmZm ZjgwMDk1ZjgwMDApCj4gQ2FsbCB0cmFjZToKPiBFeGNlcHRpb24gc3RhY2soMHhmZmZmZmY4MDA5 NWZiNDAwIHRvIDB4ZmZmZmZmODAwOTVmYjU0MCkKPiBiNDAwOiAwMDAwMDAwMDAwMDAwMDAwIDAw MDAwMDAwMDAwMDAwMDAgMDAwMDAwMDAwMDAwNzk2MSAKPiBmZmZmMGEwMGZmZmZmZjA0Cj4gYjQy MDogMDAwMDAwMDAwMDAwMDAwMCAwMDAwMDAwMDAwMDAwMjI4IDAwMDAwMDAwMDAwMDAwMDAgCj4g ZmZmZmZmZmZmZmZmZmZmZgo+IGI0NDA6IDAwMDAwMDAwMDAwMDAwMWMgMDAwMDAwMDAwMDAwMDA0 MCAwMDAwMDAwMDAwMDAwMDAxIAo+IDAwMDAwMDAwMDAwMDAwMDMKPiBiNDYwOiAwMDAwZmZmZjAw MDAwMDBmIDAwMDAwMDAwMDAwMDAwMDAgMDAwMDAwMDAwMDAwMDAwYyAKPiAwMDAwMDAwMDAwMGEx NTdkCj4gYjQ4MDogZmZmZmZmODAwODA1YmJlOCAwMDAwMDAwMGI0MmJhNDczIDAwMDAwMDAwMDAw MDAwMGEgCj4gZmZmZmZmODAwOTVmYjZmMAo+IGI0YTA6IGZmZmZmZjgwMDk1ZmI3MjAgMDAwMDAw MDAwMDAwMDAwMCBmZmZmZmY4MDA5NWZiNzIwIAo+IDAwMDAwMDAwMDAwMDAwMDAKPiBiNGMwOiAw MDAwMDAwMDAwMDAwMDAwIGZmZmZmZjgwMDhjMjE2YzggMDAwMDAwMDAwMDAwMDAwMCAKPiAwMDAw MDAwMGZmZmZmZmQ4Cj4gYjRlMDogZmZmZmZmODAwOGFkNDJiYyBmZmZmZmY4MDA5NWZiNTQwIGZm ZmZmZjgwMDg4ZDVmYzQgCj4gZmZmZmZmODAwOTVmYjU0MAo+IGI1MDA6IGZmZmZmZjgwMDg4ZDM1 ZDggMDAwMDAwMDBhMGMwMDA0OSBmZmZmZmY4MDA5NWZiNTUwIAo+IGZmZmZmZjgwMDgwZDA2YTQK PiBiNTIwOiBmZmZmZmZmZmZmZmZmZmZmIGZmZmZmZjgwMDg4ZDVlMGMgZmZmZmZmODAwOTVmYjU0 MCAKPiBmZmZmZmY4MDA4OGQzNWQ4Cj4gWzxmZmZmZmY4MDA4OGQzNWQ4Pl0gc3RyaW5nKzB4MmMv MHg2MAo+IFs8ZmZmZmZmODAwODhkNWZjND5dIHZzbnByaW50ZisweDI4Yy8weDRlYwo+IFs8ZmZm ZmZmODAwODM5NzNiOD5dIGt2YXNwcmludGYrMHg2OC8weDEwMAo+IFs8ZmZmZmZmODAwODM5NzU1 Yz5dIGthc3ByaW50ZisweDYwLzB4ODAKPiBbPGZmZmZmZjgwMDg0OWNjMjQ+XSBkcm1fZW5jb2Rl cl9pbml0KzB4MTM0LzB4MTY0Cj4gWzxmZmZmZmY4MDA4NGQ5YTdjPl0gZHB1X2VuY29kZXJfaW5p dCsweDYwLzB4OTQKPiBbPGZmZmZmZjgwMDg0ZWNlZDA+XSBfZHB1X2ttc19kcm1fb2JqX2luaXQr MHhhMC8weDQyNAo+IFs8ZmZmZmZmODAwODRlZDg3MD5dIGRwdV9rbXNfaHdfaW5pdCsweDYxYy8w eDZiYwo+IFs8ZmZmZmZmODAwODRmNzYxND5dIG1zbV9kcm1fYmluZCsweDM4MC8weDY3Ywo+IFs8 ZmZmZmZmODAwODUxMTRlND5dIHRyeV90b19icmluZ191cF9tYXN0ZXIrMHgyMjgvMHgyNjQKPiBb PGZmZmZmZjgwMDg1MTE2ZTg+XSBjb21wb25lbnRfbWFzdGVyX2FkZF93aXRoX21hdGNoKzB4OTAv MHhjMAo+IFs8ZmZmZmZmODAwODRmNzIyYz5dIG1zbV9wZGV2X3Byb2JlKzB4MjYwLzB4MmM4Cj4g WzxmZmZmZmY4MDA4NTFhOTEwPl0gcGxhdGZvcm1fZHJ2X3Byb2JlKzB4NTgvMHhhOAo+IFs8ZmZm ZmZmODAwODUxODVjOD5dIGRyaXZlcl9wcm9iZV9kZXZpY2UrMHgyZDgvMHg0MGMKPiBbPGZmZmZm ZjgwMDg1MTg5Mjg+XSBfX2RldmljZV9hdHRhY2hfZHJpdmVyKzB4ZDQvMHgxMGMKPiBbPGZmZmZm ZjgwMDg1MTY0NGM+XSBidXNfZm9yX2VhY2hfZHJ2KzB4YjQvMHhkMAo+IFs8ZmZmZmZmODAwODUx ODIzMD5dIF9fZGV2aWNlX2F0dGFjaCsweGQwLzB4MTYwCj4gWzxmZmZmZmY4MDA4NTE4OTg0Pl0g ZGV2aWNlX2luaXRpYWxfcHJvYmUrMHgyNC8weDMwCj4gWzxmZmZmZmY4MDA4NTE3NDRjPl0gYnVz X3Byb2JlX2RldmljZSsweDM4LzB4OTgKPiBbPGZmZmZmZjgwMDg1MTdhYWM+XSBkZWZlcnJlZF9w cm9iZV93b3JrX2Z1bmMrMHgxNDQvMHgxNDgKPiBbPGZmZmZmZjgwMDgwYzg2NTQ+XSBwcm9jZXNz X29uZV93b3JrKzB4MjE4LzB4M2JjCj4gWzxmZmZmZmY4MDA4MGM4ODNjPl0gcHJvY2Vzc19zY2hl ZHVsZWRfd29ya3MrMHg0NC8weDQ4Cj4gWzxmZmZmZmY4MDA4MGM5NWJjPl0gd29ya2VyX3RocmVh ZCsweDI4OC8weDMyYwo+IFs8ZmZmZmZmODAwODBjZWEzMD5dIGt0aHJlYWQrMHgxMzQvMHgxM2MK PiBbPGZmZmZmZjgwMDgwODQ3NTA+XSByZXRfZnJvbV9mb3JrKzB4MTAvMHgxOAo+IENvZGU6IDkx MDAwM2ZkIDJhMDQwM2U2IGViMDQwMGZmIDU0MDAwMDYwICgzODY0Njg0NSkKPiAKPiBMb29raW5n IGF0IHRoZSBjb2RlIEkgc2VlIHRoYXQgZHJtX2VuY29kZXJfaW5pdCgpIGlzIGNhbGxlZCBmcm9t IHRoZSAKPiBEUFUKPiBjb2RlIHdpdGggJ0RSTV9NT0RFX0NPTk5FQ1RPUl9EU0knIHBhc3NlZCBp biBhcyB0aGUgJ2VuY29kZXJfdHlwZScKPiBhcmd1bWVudCAoZm9sbG93IGZyb20gX2RwdV9rbXNf aW5pdGlhbGl6ZV9kc2koKSkuIFRoYXQgY29ycmVzcG9uZHMgdG8KPiB0aGUgaW50ZWdlciAxNi4g VGhhdCBpcyB0aGVuIGluZGV4ZWQgaW50byBkcm1fZW5jb2Rlcl9lbnVtX2xpc3QgaW4KPiBkcm1f ZW5jb2Rlcl9pbml0KCkgdG8gbG9vayB1cCB0aGUgbmFtZSBvZiB0aGUgZW5jb2Rlci4gSWYgeW91 J3JlIHN0aWxsCj4gZm9sbG93aW5nIGFsb25nLCB0aGF0J3MgYW4gZW5jb2RlciBub3QgYSBjb25u ZWN0b3IhIFdlIHJlYWxseSB3YW50IHRvCj4gdXNlIERSTV9NT0RFX0VOQ09ERVJfRFNJIChpbnRl Z2VyIDYpIGluc3RlYWQgb2YgRFJNX01PREVfQ09OTkVDVE9SX0RTSQo+IGhlcmUsIG9yIHdlJ2xs IGdvIG91dCBvZiBib3VuZHMgb2YgdGhlIGVuY29kZXIgYXJyYXkuIFBhc3MgdGhlIHJpZ2h0Cj4g dGhpbmcgYW5kIGV2ZXJ5dGhpbmcgaXMgZmluZS4KPiAKPiBDYzogSmV5a3VtYXIgU2Fua2FyYW4g PGpzYW5rYUBjb2RlYXVyb3JhLm9yZz4KPiBDYzogSm9yZGFuIENyb3VzZSA8amNyb3VzZUBjb2Rl YXVyb3JhLm9yZz4KPiBDYzogU2VhbiBQYXVsIDxzZWFucGF1bEBjaHJvbWl1bS5vcmc+Cj4gRml4 ZXM6IDI1ZmRkNTkzM2U0YyAoImRybS9tc206IEFkZCBTRE04NDUgRFBVIHN1cHBvcnQiKQo+IFNp Z25lZC1vZmYtYnk6IFN0ZXBoZW4gQm95ZCA8c3dib3lkQGNocm9taXVtLm9yZz4KPiAtLS0KClJl dmlld2VkLWJ5OiBKZXlrdW1hciBTYW5rYXJhbiA8anNhbmthQGNvZGVhdXJvcmEub3JnPgoKPiAg ZHJpdmVycy9ncHUvZHJtL21zbS9kaXNwL2RwdTEvZHB1X2ttcy5jIHwgMiArLQo+ICAxIGZpbGUg Y2hhbmdlZCwgMSBpbnNlcnRpb24oKyksIDEgZGVsZXRpb24oLSkKPiAKPiBkaWZmIC0tZ2l0IGEv ZHJpdmVycy9ncHUvZHJtL21zbS9kaXNwL2RwdTEvZHB1X2ttcy5jCj4gYi9kcml2ZXJzL2dwdS9k cm0vbXNtL2Rpc3AvZHB1MS9kcHVfa21zLmMKPiBpbmRleCA3ZGQ2YmQyZDZkMzcuLjc0Y2MyMDRi MDdlOCAxMDA2NDQKPiAtLS0gYS9kcml2ZXJzL2dwdS9kcm0vbXNtL2Rpc3AvZHB1MS9kcHVfa21z LmMKPiArKysgYi9kcml2ZXJzL2dwdS9kcm0vbXNtL2Rpc3AvZHB1MS9kcHVfa21zLmMKPiBAQCAt NDUwLDcgKzQ1MCw3IEBAIHN0YXRpYyB2b2lkIF9kcHVfa21zX2luaXRpYWxpemVfZHNpKHN0cnVj dCAKPiBkcm1fZGV2aWNlCj4gKmRldiwKPiAgCWludCBpLCByYzsKPiAKPiAgCS8qVE9ETzogU3Vw cG9ydCB0d28gaW5kZXBlbmRlbnQgRFNJIGNvbm5lY3RvcnMgKi8KPiAtCWVuY29kZXIgPSBkcHVf ZW5jb2Rlcl9pbml0KGRldiwgRFJNX01PREVfQ09OTkVDVE9SX0RTSSk7Cj4gKwllbmNvZGVyID0g ZHB1X2VuY29kZXJfaW5pdChkZXYsIERSTV9NT0RFX0VOQ09ERVJfRFNJKTsKPiAgCWlmIChJU19F UlJfT1JfTlVMTChlbmNvZGVyKSkgewo+ICAJCURQVV9FUlJPUigiZW5jb2RlciBpbml0IGZhaWxl ZCBmb3IgZHNpIGRpc3BsYXlcbiIpOwo+ICAJCXJldHVybjsKCi0tIApKZXlrdW1hciBTCl9fX19f X19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fCkZyZWVkcmVubyBtYWls aW5nIGxpc3QKRnJlZWRyZW5vQGxpc3RzLmZyZWVkZXNrdG9wLm9yZwpodHRwczovL2xpc3RzLmZy ZWVkZXNrdG9wLm9yZy9tYWlsbWFuL2xpc3RpbmZvL2ZyZWVkcmVubwo= From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org X-Spam-Level: X-Spam-Status: No, score=-0.9 required=3.0 tests=DKIM_SIGNED, HEADER_FROM_DIFFERENT_DOMAINS,MAILING_LIST_MULTI,SPF_PASS,T_DKIM_INVALID, URIBL_BLOCKED autolearn=ham autolearn_force=no version=3.4.0 Received: from mail.kernel.org (mail.kernel.org [198.145.29.99]) by smtp.lore.kernel.org (Postfix) with ESMTP id B727CC4321D for ; Fri, 17 Aug 2018 17:00:07 +0000 (UTC) Received: from vger.kernel.org (vger.kernel.org [209.132.180.67]) by mail.kernel.org (Postfix) with ESMTP id 5685F208E4 for ; Fri, 17 Aug 2018 17:00:07 +0000 (UTC) Authentication-Results: mail.kernel.org; dkim=fail reason="key not found in DNS" (0-bit key) header.d=codeaurora.org header.i=@codeaurora.org header.b="mtG11CuQ"; dkim=fail reason="key not found in DNS" (0-bit key) header.d=codeaurora.org header.i=@codeaurora.org header.b="TJced6By" DMARC-Filter: OpenDMARC Filter v1.3.2 mail.kernel.org 5685F208E4 Authentication-Results: mail.kernel.org; dmarc=none (p=none dis=none) header.from=codeaurora.org Authentication-Results: mail.kernel.org; spf=none smtp.mailfrom=linux-kernel-owner@vger.kernel.org Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1727880AbeHQUEM (ORCPT ); Fri, 17 Aug 2018 16:04:12 -0400 Received: from smtp.codeaurora.org ([198.145.29.96]:56708 "EHLO smtp.codeaurora.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1727218AbeHQUEM (ORCPT ); Fri, 17 Aug 2018 16:04:12 -0400 Received: by smtp.codeaurora.org (Postfix, from userid 1000) id 3CFD26256D; Fri, 17 Aug 2018 17:00:04 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=codeaurora.org; s=default; t=1534525204; bh=Y5p+/M4nf4/G6LvN8JqsdUsbneXErfuwRD+QR+p+6c4=; h=Date:From:To:Cc:Subject:In-Reply-To:References:From; b=mtG11CuQDg4rtoRHnQsrZp2RIoT85vsvRAP8Oi09m4hzzxYv6PLUZJDxFycXtsxLQ t2neAJI+L+nA5RKcd5irYLestZI7iXcz34yhLNcFaP2kvix0ThYaiRwAmGwgONf/Pk kT6Jkslko8kPDu+9BbV14tf/Y8AzNJj9+FoRvnYA= Received: from mail.codeaurora.org (localhost.localdomain [127.0.0.1]) by smtp.codeaurora.org (Postfix) with ESMTP id AE26A62486; Fri, 17 Aug 2018 17:00:02 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=codeaurora.org; s=default; t=1534525202; bh=Y5p+/M4nf4/G6LvN8JqsdUsbneXErfuwRD+QR+p+6c4=; h=Date:From:To:Cc:Subject:In-Reply-To:References:From; b=TJced6By2J/kCtJTaaaJfa29T4PnpfBZBZaQApoD3P7l9HwSeGX8uDhSFPRTz9G/2 dhss/A+3pLP9tZs+P+oUXl3SVpXtmv+jnnJnTYbpX6WxpRbZTTRfIRK+4Fi6T382Cc 4t1Fb7FgHocPmp1Yr3jUjEBqQDLHvFdpdYkCnjtQ= MIME-Version: 1.0 Content-Type: text/plain; charset=US-ASCII; format=flowed Content-Transfer-Encoding: 7bit Date: Fri, 17 Aug 2018 10:00:02 -0700 From: Jeykumar Sankaran To: Stephen Boyd Cc: Rob Clark , linux-kernel@vger.kernel.org, dri-devel@lists.freedesktop.org, freedreno@lists.freedesktop.org, linux-arm-msm@vger.kernel.org, Jordan Crouse , Sean Paul Subject: Re: [PATCH] drm/msm/disp/dpu: Use proper define for drm_encoder_init() 'encoder_type' In-Reply-To: <20180816233616.148248-1-swboyd@chromium.org> References: <20180816233616.148248-1-swboyd@chromium.org> Message-ID: <63a08a5c9f73b369c5d579ee340fca72@codeaurora.org> X-Sender: jsanka@codeaurora.org User-Agent: Roundcube Webmail/1.2.5 Sender: linux-kernel-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On 2018-08-16 16:36, Stephen Boyd wrote: > We got a bug report that this function oopses when trying to do a > kasprintf(). > > PC is at string+0x2c/0x60 > LR is at vsnprintf+0x28c/0x4ec > pc : [] lr : [] pstate: a0c00049 > sp : ffffff80095fb540 > x29: ffffff80095fb540 x28: ffffff8008ad42bc > x27: 00000000ffffffd8 x26: 0000000000000000 > x25: ffffff8008c216c8 x24: 0000000000000000 > x23: 0000000000000000 x22: ffffff80095fb720 > x21: 0000000000000000 x20: ffffff80095fb720 > x19: ffffff80095fb6f0 x18: 000000000000000a > x17: 00000000b42ba473 x16: ffffff800805bbe8 > x15: 00000000000a157d x14: 000000000000000c > x13: 0000000000000000 x12: 0000ffff0000000f > x11: 0000000000000003 x10: 0000000000000001 > x9 : 0000000000000040 x8 : 000000000000001c > x7 : ffffffffffffffff x6 : 0000000000000000 > x5 : 0000000000000228 x4 : 0000000000000000 > x3 : ffff0a00ffffff04 x2 : 0000000000007961 > x1 : 0000000000000000 x0 : 0000000000000000 > Process kworker/3:1 (pid: 61, stack limit = 0xffffff80095f8000) > Call trace: > Exception stack(0xffffff80095fb400 to 0xffffff80095fb540) > b400: 0000000000000000 0000000000000000 0000000000007961 > ffff0a00ffffff04 > b420: 0000000000000000 0000000000000228 0000000000000000 > ffffffffffffffff > b440: 000000000000001c 0000000000000040 0000000000000001 > 0000000000000003 > b460: 0000ffff0000000f 0000000000000000 000000000000000c > 00000000000a157d > b480: ffffff800805bbe8 00000000b42ba473 000000000000000a > ffffff80095fb6f0 > b4a0: ffffff80095fb720 0000000000000000 ffffff80095fb720 > 0000000000000000 > b4c0: 0000000000000000 ffffff8008c216c8 0000000000000000 > 00000000ffffffd8 > b4e0: ffffff8008ad42bc ffffff80095fb540 ffffff80088d5fc4 > ffffff80095fb540 > b500: ffffff80088d35d8 00000000a0c00049 ffffff80095fb550 > ffffff80080d06a4 > b520: ffffffffffffffff ffffff80088d5e0c ffffff80095fb540 > ffffff80088d35d8 > [] string+0x2c/0x60 > [] vsnprintf+0x28c/0x4ec > [] kvasprintf+0x68/0x100 > [] kasprintf+0x60/0x80 > [] drm_encoder_init+0x134/0x164 > [] dpu_encoder_init+0x60/0x94 > [] _dpu_kms_drm_obj_init+0xa0/0x424 > [] dpu_kms_hw_init+0x61c/0x6bc > [] msm_drm_bind+0x380/0x67c > [] try_to_bring_up_master+0x228/0x264 > [] component_master_add_with_match+0x90/0xc0 > [] msm_pdev_probe+0x260/0x2c8 > [] platform_drv_probe+0x58/0xa8 > [] driver_probe_device+0x2d8/0x40c > [] __device_attach_driver+0xd4/0x10c > [] bus_for_each_drv+0xb4/0xd0 > [] __device_attach+0xd0/0x160 > [] device_initial_probe+0x24/0x30 > [] bus_probe_device+0x38/0x98 > [] deferred_probe_work_func+0x144/0x148 > [] process_one_work+0x218/0x3bc > [] process_scheduled_works+0x44/0x48 > [] worker_thread+0x288/0x32c > [] kthread+0x134/0x13c > [] ret_from_fork+0x10/0x18 > Code: 910003fd 2a0403e6 eb0400ff 54000060 (38646845) > > Looking at the code I see that drm_encoder_init() is called from the > DPU > code with 'DRM_MODE_CONNECTOR_DSI' passed in as the 'encoder_type' > argument (follow from _dpu_kms_initialize_dsi()). That corresponds to > the integer 16. That is then indexed into drm_encoder_enum_list in > drm_encoder_init() to look up the name of the encoder. If you're still > following along, that's an encoder not a connector! We really want to > use DRM_MODE_ENCODER_DSI (integer 6) instead of DRM_MODE_CONNECTOR_DSI > here, or we'll go out of bounds of the encoder array. Pass the right > thing and everything is fine. > > Cc: Jeykumar Sankaran > Cc: Jordan Crouse > Cc: Sean Paul > Fixes: 25fdd5933e4c ("drm/msm: Add SDM845 DPU support") > Signed-off-by: Stephen Boyd > --- Reviewed-by: Jeykumar Sankaran > drivers/gpu/drm/msm/disp/dpu1/dpu_kms.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/drivers/gpu/drm/msm/disp/dpu1/dpu_kms.c > b/drivers/gpu/drm/msm/disp/dpu1/dpu_kms.c > index 7dd6bd2d6d37..74cc204b07e8 100644 > --- a/drivers/gpu/drm/msm/disp/dpu1/dpu_kms.c > +++ b/drivers/gpu/drm/msm/disp/dpu1/dpu_kms.c > @@ -450,7 +450,7 @@ static void _dpu_kms_initialize_dsi(struct > drm_device > *dev, > int i, rc; > > /*TODO: Support two independent DSI connectors */ > - encoder = dpu_encoder_init(dev, DRM_MODE_CONNECTOR_DSI); > + encoder = dpu_encoder_init(dev, DRM_MODE_ENCODER_DSI); > if (IS_ERR_OR_NULL(encoder)) { > DPU_ERROR("encoder init failed for dsi display\n"); > return; -- Jeykumar S