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 Received: from mail.kernel.org (mail.kernel.org [198.145.29.99]) by smtp.lore.kernel.org (Postfix) with ESMTP id 33EFAC433F5 for ; Tue, 28 Sep 2021 17:01:08 +0000 (UTC) Received: from phobos.denx.de (phobos.denx.de [85.214.62.61]) (using TLSv1.2 with cipher ECDHE-RSA-AES128-GCM-SHA256 (128/128 bits)) (No client certificate requested) by mail.kernel.org (Postfix) with ESMTPS id 8A0E56128E for ; Tue, 28 Sep 2021 17:01:07 +0000 (UTC) DMARC-Filter: OpenDMARC Filter v1.4.1 mail.kernel.org 8A0E56128E Authentication-Results: mail.kernel.org; dmarc=none (p=none dis=none) header.from=gateworks.com Authentication-Results: mail.kernel.org; spf=pass smtp.mailfrom=lists.denx.de Received: from h2850616.stratoserver.net (localhost [IPv6:::1]) by phobos.denx.de (Postfix) with ESMTP id 6E5E98312A; Tue, 28 Sep 2021 19:01:05 +0200 (CEST) Authentication-Results: phobos.denx.de; dmarc=none (p=none dis=none) header.from=gateworks.com Authentication-Results: phobos.denx.de; spf=pass smtp.mailfrom=u-boot-bounces@lists.denx.de Authentication-Results: phobos.denx.de; dkim=pass (2048-bit key; unprotected) header.d=gateworks-com.20210112.gappssmtp.com header.i=@gateworks-com.20210112.gappssmtp.com header.b="wSkaZfX1"; dkim-atps=neutral Received: by phobos.denx.de (Postfix, from userid 109) id 777CA80C58; Tue, 28 Sep 2021 19:01:02 +0200 (CEST) Received: from mail-pj1-x1031.google.com (mail-pj1-x1031.google.com [IPv6:2607:f8b0:4864:20::1031]) (using TLSv1.3 with cipher TLS_AES_128_GCM_SHA256 (128/128 bits)) (No client certificate requested) by phobos.denx.de (Postfix) with ESMTPS id 155DD8312A for ; Tue, 28 Sep 2021 19:00:58 +0200 (CEST) Authentication-Results: phobos.denx.de; dmarc=none (p=none dis=none) header.from=gateworks.com Authentication-Results: phobos.denx.de; spf=pass smtp.mailfrom=tharvey@gateworks.com Received: by mail-pj1-x1031.google.com with SMTP id d4-20020a17090ad98400b0019ece228690so2152493pjv.5 for ; Tue, 28 Sep 2021 10:00:58 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gateworks-com.20210112.gappssmtp.com; s=20210112; h=mime-version:references:in-reply-to:from:date:message-id:subject:to :cc; bh=bKjuJXyqKDMhjDIoQtGHVjuZTMSHfeow+Cd4/ccQe5g=; b=wSkaZfX1a0cKX5Syo5Hk77xJN/g/1VLLxMW8lrYghIZVOU763Y20e9F2VIFf8zMAdI ZxUjqGX9uMmUK9kUuSnu3gTM6AKt/a3wJuVlkRYUGWP8TKu5eWdqDvmFdzyIQwDrv1n6 hvpV1pHAG1MdxHpXf4m2gXbd6DkBZO34Geyu8Uw9pML57AkP7GFj5XLWr54viznUNXJd VWuw8cjUud2Q6Mq3K0gGY5ARL1rt9GnDw+Y1aWb1QVIPSOSPwOKlitq70WnXmcBz+92K 5SKEoNPLPiZXrqNRpy1wWTkGU5RrYNEdDU0vZPy9xNryjUZ2dBfnJwLxIqTeTp3yd6tU LY3A== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20210112; h=x-gm-message-state:mime-version:references:in-reply-to:from:date :message-id:subject:to:cc; bh=bKjuJXyqKDMhjDIoQtGHVjuZTMSHfeow+Cd4/ccQe5g=; b=RYohRTvUAxCLtlqeWW1UwCmC2DO83SPcBFEL3AuZRTeddvb9UifteXjQ8VX1YBxKsC waUE+XhA+ICEjX9D+E/8VS6+wgRkPb+JQ7czduadKQkatP027OJpaF/PoOm+oFyPhl2N 5rFkgVW2e2mY2VHWWVJMX6GGnXS6B9Ojy15x672DUlbnL+kWzeDvsvKjZGsiOe39txtM dyyD2F+Ggav5ajdIIXXOn/9psUuHLgdFIt2ktBxX0awPOyos7Y1RQ/gmNgzuMFuoKpNS +PGINLkUVY4Ca8ckDwvevU1o5p+bTZNMyAkqglrt2pD9jzUfFqfi/A54ABdgvsAmMPGe DMjg== X-Gm-Message-State: AOAM533rNZtPAsQ2Y3mOrPSaD72u86w3cuOYv+r4CenahRbXs+Y67lPw mHDrHWJ7d80bEx+mpMtA9F4V88hCT0yf6nS2RXPs4Q== X-Google-Smtp-Source: ABdhPJybbDeo5LiIz7pnRftryO9BQgsAGvm/FJu5CceJdp62JyMRITlNEPbjKs7H0hFdO5UsjGPcX9ePLFvlKEJC39g= X-Received: by 2002:a17:90a:6f44:: with SMTP id d62mr977103pjk.179.1632848456204; Tue, 28 Sep 2021 10:00:56 -0700 (PDT) MIME-Version: 1.0 References: <20210928145519.649314fd@crub> In-Reply-To: <20210928145519.649314fd@crub> From: Tim Harvey Date: Tue, 28 Sep 2021 10:00:45 -0700 Message-ID: Subject: Re: imx6 DM_VIDEObroken To: Anatolij Gustschin Cc: u-boot , Fabio Estevam , Stefano Babic Content-Type: text/plain; charset="UTF-8" X-BeenThere: u-boot@lists.denx.de X-Mailman-Version: 2.1.34 Precedence: list List-Id: U-Boot discussion List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: u-boot-bounces@lists.denx.de Sender: "U-Boot" X-Virus-Scanned: clamav-milter 0.103.2 at phobos.denx.de X-Virus-Status: Clean On Tue, Sep 28, 2021 at 5:55 AM Anatolij Gustschin wrote: > > Hey Tim, > > On Mon, 27 Sep 2021 17:25:58 -0700 > Tim Harvey tharvey@gateworks.com wrote: > > > Anatolij, > > > > Since commit d37618d18d49 ("imx: convert gwventana board to DM_VIDEO") > > video support for IMX6 based Ventana boards has been broken. > > Back then I've tested similar DM_VIDEO conversion changes on i.mx6q > nitrogen6q board and on i.mx6d/i.mx6s wandboards, it was okay if > the board configuration uses the video console output during the > boot sequence (i.e. configured to show splash screen or to output > strings on vidconsole). > > > I find that while the bind function for fsl_imx6q_ipu is called the > > probe never is (ipuv3_video_probe). Do you know why this is? > > Is video console output used during the boot? If not, then it > might be the reason. With DM, the devices are probed when a subsystem > actually tries to use them. You can try to trigger probing via > switching to the video console output, i.e.: > > => setenv stdout vidconsole Yes, this calls ipuv3_video_probe. I guess I expected the display to just work by default as it did before. I looked over doc/README.console. It may be out of date as it refers to 'video' instead of 'vidconsole'. What is the difference? How do I get back to the state where a splash-screen is shown on the display by default? > > > > I see that with commit 57f065fee2a4 ("video: ipuv3: add DM_VIDEO > > support") you mention that DTS files must include > > 'u-boot,dm-pre-reloc' for soc/ipu nodes to enable driver binding to > > ipu device but I haven't been able to get that to make a difference > > nor have I found a board that does this. You did add those props to > > imx6qdl.dtsi at one point but they are no longer there. > > Does your board dts include imx6qdl-u-boot.dtsi ? > > The u-boot,dm-pre-reloc properties were moved to this U-Boot specific > dtsi in 7932b1c9fdb73393aa110249c89bd426533c0649 > (imx: imx6qdl: dtsi: move U-Boot specific change to u-boot.dtsi) > No, I'm not including that but that is the right place for it. Strangely, I found that if I don't have 'u-boot,dm-pre-reloc' for soc/ipu nodes enabling video via 'setenv stdout serial,vidconsole' works. Is there some reason why the prop is no longer required? > > There must have been several other IMX6 boards with video support that > > were affected by this so perhaps I'm missing something simple. > > I did not test it recently, it looks I'll have to do it before the > new release. I would think the other board maintainers would have raised a red flag if things broke for their boards, I just haven't tested it in quite some time for gwventana_*_defconfig. How do you go about testing this by the way without having all the boards and displays? Best regards, Tim