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.8 required=3.0 tests=DKIM_SIGNED,DKIM_VALID, DKIM_VALID_AU,HEADER_FROM_DIFFERENT_DOMAINS,MAILING_LIST_MULTI,SPF_HELO_NONE, SPF_PASS,URIBL_BLOCKED autolearn=no 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 CBB5DC433E0 for ; Tue, 16 Jun 2020 17:58:09 +0000 (UTC) Received: from vger.kernel.org (vger.kernel.org [23.128.96.18]) by mail.kernel.org (Postfix) with ESMTP id 9DF7820776 for ; Tue, 16 Jun 2020 17:58:09 +0000 (UTC) Authentication-Results: mail.kernel.org; dkim=pass (1024-bit key) header.d=ideasonboard.com header.i=@ideasonboard.com header.b="QEkKm2Eo" Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1731186AbgFPR6J (ORCPT ); Tue, 16 Jun 2020 13:58:09 -0400 Received: from perceval.ideasonboard.com ([213.167.242.64]:59058 "EHLO perceval.ideasonboard.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1727819AbgFPR6I (ORCPT ); Tue, 16 Jun 2020 13:58:08 -0400 Received: from pendragon.ideasonboard.com (81-175-216-236.bb.dnainternet.fi [81.175.216.236]) by perceval.ideasonboard.com (Postfix) with ESMTPSA id 53F63F9; Tue, 16 Jun 2020 19:58:06 +0200 (CEST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=ideasonboard.com; s=mail; t=1592330286; bh=g3XeyB2jnsB9xQPVHmobYwY5dlFfO+u30lKf7fZa8qA=; h=Date:From:To:Cc:Subject:References:In-Reply-To:From; b=QEkKm2EoaQ9soywHbcuXld3TIassOqk3LyBXvlSXcSwRsWn3oUAwbQOXNRWIKwevc wn1bwhiY9RADIAKLE426OtfCt7mwy42OZ6Dd9r0ZYd21jnvGryVFNTcpOamQaQXfTl 8PgVmO8UJWQW28bVAVUqlCnO2XbN33AOpCIMmovY= Date: Tue, 16 Jun 2020 20:57:43 +0300 From: Laurent Pinchart To: Adam Ford Cc: "H. Nikolaus Schaller" , Tony Lindgren , Linux-OMAP , Tomi Valkeinen Subject: Re: omap3 dss failues on 5.7 Message-ID: <20200616175743.GC913@pendragon.ideasonboard.com> References: <20200613160242.GW37466@atomide.com> <20200613160601.GB9722@pendragon.ideasonboard.com> <8BE96966-02C2-4AB4-B36F-7F0CBBBF4928@goldelico.com> MIME-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Disposition: inline In-Reply-To: Sender: linux-omap-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-omap@vger.kernel.org Hi Adam, On Tue, Jun 16, 2020 at 09:53:24AM -0500, Adam Ford wrote: > On Tue, Jun 16, 2020 at 9:24 AM H. Nikolaus Schaller wrote: > >> Am 13.06.2020 um 04:26 schrieb Adam Ford : > >> > >> Is anyone else having DSS failures on the 5.7 stable branch using an > >> omap3 board? > > > > The GTA04 omapdss is running on dm3730 and OpenPandora with omap3530. > > > > But I also have a problem on omap5 (not yet analysed in any way...). > > > >> Thanks for any suggestions. I'll bisect when I have more time. > >> Am 13.06.2020 um 18:06 schrieb Laurent Pinchart : > >> > >> Hi Tony, > >> > >> On Sat, Jun 13, 2020 at 09:02:42AM -0700, Tony Lindgren wrote: > >>> * Adam Ford [200613 02:28]: > >>>> Is anyone else having DSS failures on the 5.7 stable branch using an > >>>> omap3 board? > >>>> > >>>> I haven't had time to bisect yet, but before I do I thought I'd ask. > >>> > >>> Sounds this might be caused by some panel related between v5.6 and v5.7, > >>> so adding Laurent to Cc. > >>> > >>> We also started dropping legacy platform data for some SoCs, but that > >>> should not affect omap3 so far. > >> > >> I think this is caused by the panel driver not setting the connector > >> type. Only the panel-simple driver should be affected, > > > > Both omap3 based devices mentioned above use special panel drivers. > > > >> fixing this > >> should just be a matter of setting > >> > >> .connector_type = DRM_MODE_CONNECTOR_DPI > >> > >> for the appropriate panel_desc entry. > > > > I have another panel in panel-simple where .connector_type is not set. > > (Haven't tested yet but would assume it fails as well). > > > > I wonder why it was not a bulk change for all panel_desc there if it is > > that important. > > I tested Tomi's patch and it fixes the issue with one screen. I > updated a different panel that we use at Logic PD to enable the same > flag on a different panel. That also fixed the issue on the second > LCD. > > I don't know if we can do a bulk set or not. I am not sure if all the > screens need the DPI flag. That's exactly why the conversion wasn't done in bulk, we don't know if all the panels that are missing a .connector_type are DPI panel. We need to get information about every single of them individually, and that was more work that I could handle at the time. > >>>> 10.370574] ------------[ cut here ]------------ > >>>> [ 10.375335] WARNING: CPU: 0 PID: 71 at > >>>> drivers/gpu/drm/bridge/panel.c:176 drm_panel_bridge_add+0x40/0x4c > >>>> [drm_kms_helper] > >>>> [ 10.386474] Modules linked in: joydev mousedev evdev leds_gpio > >>>> pwm_bl panel_simple snd_soc_omap_twl4030 pwm_omap_dmtimer led_class > >>>> gpio_keys cpufreq_dt snd_soc_omap_mcbsp bq27xxx_battery_hdq > >>>> snd_soc_ti_sdma bq27xxx_battery omap_wdt omap3_isp > >>>> videobuf2_dma_contig videobuf2_memops omap_hdq videobuf2_v4l2 > >>>> omap_sham videobuf2_common v4l2_fwnode wire cn hwmon mt9p031 > >>>> aptina_pll phy_twl4030_usb omap_aes_driver omap2430 crypto_engine > >>>> omap_crypto musb_hdrc hci_uart omap_rng btbcm wlcore_sdio udc_core > >>>> rng_core videodev omap_mailbox mc usbcore snd_soc_twl4030 twl4030_wdt > >>>> usb_common watchdog rtc_twl snd_soc_core snd_pcm_dmaengine bluetooth > >>>> snd_pcm snd_timer twl4030_pwrbutton snd omapdss pwm_twl_led > >>>> twl4030_charger ecdh_generic soundcore ecc omapdss_base industrialio > >>>> libaes pwm_twl drm_kms_helper tsc2004 at24 tsc200x_core cfbfillrect > >>>> omap_ssi syscopyarea cfbimgblt hsi sysfillrect sysimgblt fb_sys_fops > >>>> cfbcopyarea drm drm_panel_orientation_quirks cec > >>>> [ 10.470123] CPU: 0 PID: 71 Comm: kworker/0:3 Not tainted 5.7.2-dirty #2 > >>>> [ 10.476776] Hardware name: Generic OMAP36xx (Flattened Device Tree) > >>>> [ 10.483093] Workqueue: events deferred_probe_work_func > >>>> [ 10.488281] [] (unwind_backtrace) from [] > >>>> (show_stack+0x10/0x14) > >>>> [ 10.496063] [] (show_stack) from [] > >>>> (dump_stack+0xc0/0xdc) > >>>> [ 10.503326] [] (dump_stack) from [] (__warn+0xc0/0xf8) > >>>> [ 10.510223] [] (__warn) from [] > >>>> (warn_slowpath_fmt+0x5c/0xc0) > >>>> [ 10.517822] [] (warn_slowpath_fmt) from [] > >>>> (drm_panel_bridge_add+0x40/0x4c [drm_kms_helper]) > >>>> [ 10.528106] [] (drm_panel_bridge_add [drm_kms_helper]) > >>>> from [] (omapdss_device_init_output+0x7c/0x780 > >>>> [omapdss_base]) > >>>> [ 10.540618] [] (omapdss_device_init_output > >>>> [omapdss_base]) from [] (dpi_init_port+0x188/0x288 > >>>> [omapdss]) > >>>> [ 10.551971] [] (dpi_init_port [omapdss]) from > >>>> [] (dss_probe+0x29c/0x578 [omapdss]) > >>>> [ 10.561370] [] (dss_probe [omapdss]) from [] > >>>> (platform_drv_probe+0x48/0x98) > >>>> [ 10.570129] [] (platform_drv_probe) from [] > >>>> (really_probe+0x1d8/0x344) > >>>> [ 10.578430] [] (really_probe) from [] > >>>> (driver_probe_device+0x5c/0x164) > >>>> [ 10.586730] [] (driver_probe_device) from [] > >>>> (bus_for_each_drv+0x80/0xcc) > >>>> [ 10.595306] [] (bus_for_each_drv) from [] > >>>> (__device_attach+0xd4/0x148) > >>>> [ 10.603607] [] (__device_attach) from [] > >>>> (bus_probe_device+0x84/0x8c) > >>>> [ 10.611816] [] (bus_probe_device) from [] > >>>> (deferred_probe_work_func+0x60/0x8c) > >>>> [ 10.620819] [] (deferred_probe_work_func) from > >>>> [] (process_one_work+0x1ec/0x530) > >>>> [ 10.630004] [] (process_one_work) from [] > >>>> (worker_thread+0x30/0x558) > >>>> [ 10.638153] [] (worker_thread) from [] > >>>> (kthread+0x134/0x148) > >>>> [ 10.645568] [] (kthread) from [] > >>>> (ret_from_fork+0x14/0x2c) > >>>> [ 10.652832] Exception stack(0xcc84bfb0 to 0xcc84bff8) > >>>> [ 10.657897] bfa0: 00000000 > >>>> 00000000 00000000 00000000 > >>>> [ 10.666137] bfc0: 00000000 00000000 00000000 00000000 00000000 > >>>> 00000000 00000000 00000000 > >>>> [ 10.674346] bfe0: 00000000 00000000 00000000 00000000 00000013 00000000 > >>>> [ 10.681060] ---[ end trace 9571aa0bd994d08b ]--- > >>>> [ 10.967132] cfg80211: Loading compiled-in X.509 certificates for > >>>> regulatory database > >>>> [ 11.284881] cfg80211: Loaded X.509 cert 'sforshee: 00b28ddf47aef9cea7' > >>>> [ 11.867645] ------------[ cut here ]------------ > >>>> [ 11.872619] WARNING: CPU: 0 PID: 15 at > >>>> drivers/gpu/drm/bridge/panel.c:176 drm_panel_bridge_add+0x40/0x4c > >>>> [drm_kms_helper] > >>>> [ 11.883819] Modules linked in: wl12xx wlcore mac80211 libarc4 > >>>> sha256_generic libsha256 sha256_arm cfg80211 joydev mousedev evdev > >>>> leds_gpio pwm_bl panel_simple snd_soc_omap_twl4030 pwm_omap_dmtimer > >>>> led_class gpio_keys cpufreq_dt snd_soc_omap_mcbsp bq27xxx_battery_hdq > >>>> snd_soc_ti_sdma bq27xxx_battery omap_wdt omap3_isp > >>>> videobuf2_dma_contig videobuf2_memops omap_hdq videobuf2_v4l2 > >>>> omap_sham videobuf2_common v4l2_fwnode wire cn hwmon mt9p031 > >>>> aptina_pll phy_twl4030_usb omap_aes_driver omap2430 crypto_engine > >>>> omap_crypto musb_hdrc hci_uart omap_rng btbcm wlcore_sdio udc_core > >>>> rng_core videodev omap_mailbox mc usbcore snd_soc_twl4030 twl4030_wdt > >>>> usb_common watchdog rtc_twl snd_soc_core snd_pcm_dmaengine bluetooth > >>>> snd_pcm snd_timer twl4030_pwrbutton snd omapdss pwm_twl_led > >>>> twl4030_charger ecdh_generic soundcore ecc omapdss_base industrialio > >>>> libaes pwm_twl drm_kms_helper tsc2004 at24 tsc200x_core cfbfillrect > >>>> omap_ssi syscopyarea cfbimgblt hsi sysfillrect sysimgblt fb_sys_fops > >>>> cfbcopyarea drm > >>>> [ 11.884246] drm_panel_orientation_quirks cec > >>>> [ 11.976013] CPU: 0 PID: 15 Comm: kworker/0:1 Tainted: G W > >>>> 5.7.2-dirty #2 > >>>> [ 11.984069] Hardware name: Generic OMAP36xx (Flattened Device Tree) > >>>> [ 11.990417] Workqueue: events deferred_probe_work_func > >>>> [ 11.995635] [] (unwind_backtrace) from [] > >>>> (show_stack+0x10/0x14) > >>>> [ 12.003448] [] (show_stack) from [] > >>>> (dump_stack+0xc0/0xdc) > >>>> [ 12.010742] [] (dump_stack) from [] (__warn+0xc0/0xf8) > >>>> [ 12.017669] [] (__warn) from [] > >>>> (warn_slowpath_fmt+0x5c/0xc0) > >>>> [ 12.025360] [] (warn_slowpath_fmt) from [] > >>>> (drm_panel_bridge_add+0x40/0x4c [drm_kms_helper]) > >>>> [ 12.035766] [] (drm_panel_bridge_add [drm_kms_helper]) > >>>> from [] (omapdss_device_init_output+0x7c/0x780 > >>>> [omapdss_base]) > >>>> [ 12.048400] [] (omapdss_device_init_output > >>>> [omapdss_base]) from [] (dpi_init_port+0x188/0x288 > >>>> [omapdss]) > >>>> [ 12.059906] [] (dpi_init_port [omapdss]) from > >>>> [] (dss_probe+0x29c/0x578 [omapdss]) > >>>> [ 12.069396] [] (dss_probe [omapdss]) from [] > >>>> (platform_drv_probe+0x48/0x98) > >>>> [ 12.078186] [] (platform_drv_probe) from [] > >>>> (really_probe+0x1d8/0x344) > >>>> [ 12.086517] [] (really_probe) from [] > >>>> (driver_probe_device+0x5c/0x164) > >>>> [ 12.094848] [] (driver_probe_device) from [] > >>>> (bus_for_each_drv+0x80/0xcc) > >>>> [ 12.103454] [] (bus_for_each_drv) from [] > >>>> (__device_attach+0xd4/0x148) > >>>> [ 12.111785] [] (__device_attach) from [] > >>>> (bus_probe_device+0x84/0x8c) > >>>> [ 12.120025] [] (bus_probe_device) from [] > >>>> (deferred_probe_work_func+0x60/0x8c) > >>>> [ 12.129058] [] (deferred_probe_work_func) from > >>>> [] (process_one_work+0x1ec/0x530) > >>>> [ 12.138275] [] (process_one_work) from [] > >>>> (worker_thread+0x30/0x558) > >>>> [ 12.146423] [] (worker_thread) from [] > >>>> (kthread+0x134/0x148) > >>>> [ 12.153900] [] (kthread) from [] > >>>> (ret_from_fork+0x14/0x2c) > >>>> [ 12.161163] Exception stack(0xce1d7fb0 to 0xce1d7ff8) > >>>> [ 12.166259] 7fa0: 00000000 > >>>> 00000000 00000000 00000000 > >>>> [ 12.174530] 7fc0: 00000000 00000000 00000000 00000000 00000000 > >>>> 00000000 00000000 00000000 > >>>> [ 12.182769] 7fe0: 00000000 00000000 00000000 00000000 00000013 00000000 > >>>> [ 12.189544] ---[ end trace 9571aa0bd994d08c ]--- > >>>> > >>>> > >>>> Thanks for any suggestions. I'll bisect when I have more time. -- Regards, Laurent Pinchart