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=-2.6 required=3.0 tests=DKIM_SIGNED,DKIM_VALID, DKIM_VALID_AU,FREEMAIL_FORGED_FROMDOMAIN,FREEMAIL_FROM, HEADER_FROM_DIFFERENT_DOMAINS,MAILING_LIST_MULTI,SPF_HELO_NONE,SPF_PASS, USER_AGENT_SANE_2 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 C133FC433E0 for ; Fri, 26 Jun 2020 09:00:15 +0000 (UTC) Received: from vger.kernel.org (vger.kernel.org [23.128.96.18]) by mail.kernel.org (Postfix) with ESMTP id 93A2F206A1 for ; Fri, 26 Jun 2020 09:00:15 +0000 (UTC) Authentication-Results: mail.kernel.org; dkim=pass (2048-bit key) header.d=gmail.com header.i=@gmail.com header.b="R3A3XX5h" Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1725864AbgFZJAP (ORCPT ); Fri, 26 Jun 2020 05:00:15 -0400 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:49118 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1725820AbgFZJAP (ORCPT ); Fri, 26 Jun 2020 05:00:15 -0400 Received: from mail-lj1-x243.google.com (mail-lj1-x243.google.com [IPv6:2a00:1450:4864:20::243]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id A2BF6C08C5C1 for ; Fri, 26 Jun 2020 02:00:14 -0700 (PDT) Received: by mail-lj1-x243.google.com with SMTP id h22so2356086lji.9 for ; Fri, 26 Jun 2020 02:00:14 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20161025; h=date:from:to:cc:subject:message-id:in-reply-to:references :mime-version; bh=Ao9YTadEp0s2FWKTOdd0MJFofVwfH7gY8WnZItIgTv8=; b=R3A3XX5hvcZ5c3Uo7PL8SHK588RpcT+z26LsvfxNumtko2h6GBOgJ/+3lUSc+eY405 DppNdteVLEyvmFhtDyf3tBAh5e+Xed5hKYYrStIU4WO4K75/w3Gr1LBrEQwTx38Ow1eE DPe5fMiMYJ9eULdK1EYRt9tSq4sLCNqnkkuWjK8w89DRzRlW0xoYz69LCtGr2olX0XTH Cx7EmOf7pJKagozsJbq+NN6VRJtygd5K3eii2O021jZdfsG3S++o6fQwU3a8g6Tj2qCr JX3Snnqk7FKhck1SPRHNvE0KXxKaND2qiSvSTSKAR11IW+r1ZMVmgbBnqifG5Pw1ybxO ffAQ== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:date:from:to:cc:subject:message-id:in-reply-to :references:mime-version; bh=Ao9YTadEp0s2FWKTOdd0MJFofVwfH7gY8WnZItIgTv8=; b=oaeOSj7X6OjPjB+mQLTkqEJKVh2Vl3SsDewPySNw4QqDwc3jvw7Zypxni5nT5A1bHh esQt2Iyq7pSr9yp0F6gt+z8DttiF3L2e53K0o6xi7EK04Ro2IyxpAAXizJSsoVP+z0WB Hks8Gh6LjS7dj+OJ+3K1SVkS7yZlBnot76G/IHX4ZHPIjlUoBZxNsqAPKDRC3lvx81vp DNe3NPBPUtVHKhvel8cZMbWA+E/lsJJc23E6hXZ4ec+kTEAXpdHslva3UvB9QS96kwY7 jviQj0RhCdrS7M9y1OvbTJpHdKo4YiGGyB7IOO8gfxEYp4vAxek2bnwVZ1esPfDQ5NEE O3jA== X-Gm-Message-State: AOAM531Wsj7Z2Z819d8NB7NY26lO1A9d7Kg4efZd959TEGxnJ1tX46AX jv2jXkbOLa++7wVGpSPrSPo= X-Google-Smtp-Source: ABdhPJzs9zYgb0haM+Ksfy8ux78Nnvp/+fMdG2RUAF4IySl/oHXbP7C7C4kvcyZD3TJfpItm13bJlQ== X-Received: by 2002:a2e:9d99:: with SMTP id c25mr920225ljj.404.1593162012963; Fri, 26 Jun 2020 02:00:12 -0700 (PDT) Received: from eldfell ([194.136.85.206]) by smtp.gmail.com with ESMTPSA id d20sm6636423lfi.22.2020.06.26.02.00.11 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Fri, 26 Jun 2020 02:00:12 -0700 (PDT) Date: Fri, 26 Jun 2020 11:59:57 +0300 From: Pekka Paalanen To: Daniel Vetter Cc: Alex Deucher , Simon Ser , Laurent Pinchart , Jernej Skrabec , Laurent Pinchart , Jonas Karlman , Neil Armstrong , Kieran Bingham , Maling list - DRI developers , "open list:DRM DRIVERS FOR RENESAS" , Andrzej Hajda , Thomas Zimmermann , Sam Ravnborg Subject: Re: [PATCH 27/27] drm: Add default modes for connectors in unknown state Message-ID: <20200626115957.566efa5b@eldfell> In-Reply-To: References: <20200526011505.31884-1-laurent.pinchart+renesas@ideasonboard.com> <20200526011505.31884-28-laurent.pinchart+renesas@ideasonboard.com> <20200621084000.GM74146@ravnborg.org> <20200624011209.GU5870@pendragon.ideasonboard.com> <20200624072304.GT20149@phenom.ffwll.local> <20200625075655.GC3278063@phenom.ffwll.local> <20200625133157.0e749602@eldfell> X-Mailer: Claws Mail 3.17.3 (GTK+ 2.24.32; x86_64-pc-linux-gnu) MIME-Version: 1.0 Content-Type: multipart/signed; micalg=pgp-sha256; boundary="Sig_/F4=b=qzv72TRNlRwlpgbuD_"; protocol="application/pgp-signature" Sender: linux-renesas-soc-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-renesas-soc@vger.kernel.org --Sig_/F4=b=qzv72TRNlRwlpgbuD_ Content-Type: text/plain; charset=US-ASCII Content-Transfer-Encoding: quoted-printable On Thu, 25 Jun 2020 12:44:36 +0200 Daniel Vetter wrote: > On Thu, Jun 25, 2020 at 12:32 PM Pekka Paalanen wro= te: > > > > On Thu, 25 Jun 2020 09:57:44 +0200 > > Daniel Vetter wrote: > > =20 > > > On Thu, Jun 25, 2020 at 9:56 AM Daniel Vetter wrote= : =20 > > > > > > > > On Wed, Jun 24, 2020 at 03:40:42PM -0400, Alex Deucher wrote: =20 ... > > > > > No, you are right; you will have the EDID so this shouldn't be an > > > > > issue. I was mis-remembering the original issue. We originally > > > > > always reported connected for LVDS in radeon if the panel was pre= sent, > > > > > but then we got flack because some userspace expected unknown in > > > > > certain cases (e.g., lid or muxed displays). Either way the EDID= info > > > > > is still there. =20 > > > > > > > > Yeah I think i915 started that habit, but I guess people realized i= t's > > > > unreliable enough that they should have their own lid handler in the > > > > desktop enviromnent doing whatever they want to do on lid close. > > > > > > > > Should we perhaps document that somewhere, that panels are always m= arked > > > > as connected? Not even sure where to put that in the docs ... > > > > > > > > Maybe adding a few of the usual suspects from the compositor side, = Simon, > > > > Pekka? =20 > > > > > > Actually adding Simon and Pekka this time around ... =20 > > > > I don't know what anyone else does, but Weston (is not a DE) does not > > look at any lid switch, and assumes that if connector status is not > > DRM_MODE_CONNECTED, then it is disconnected. So, if a driver switched > > to "Unknown" status, it would be taken as disconnected. =20 >=20 > Maybe an aside, but the guideline is for autoconfiguration: > - Light up everything that has connector status connected. > - If nothing has that status, try to light up the connectors with > status "unknown". >=20 > This is only really relevant on older platforms, mostly for VGA and > somewhat for dvi outputs. >=20 > Maybe another thing we should put down somewhere in the uapi docs ... As I had no idea what "unknown" means or when it can happen, I assumed that it must mean "the hardware cannot know". If the hardware cannot know, then I certainly will not be trying to enable that, unless explicitly configured to do so. Having a phantom output is worse than having a real output that does not light up, because it's not obvious at first with phantom output that anything is wrong. You may just be wondering where your windows disappear, or where did you mouse cursor go, or why you see a wallpaper but no login dialog, etc. I certainly do hope no-one uses this quirk of Weston to get their lid do what they want... it's possible, OTOH the desktop user base of Weston according to what I've heard is around one person. It's not me. Thanks, pq --Sig_/F4=b=qzv72TRNlRwlpgbuD_ Content-Type: application/pgp-signature Content-Description: OpenPGP digital signature -----BEGIN PGP SIGNATURE----- iQIzBAEBCAAdFiEEJQjwWQChkWOYOIONI1/ltBGqqqcFAl71uQ0ACgkQI1/ltBGq qqeY1Q/+MhwP6N1Y9lobFwLXKaEM3tJONA3OP40VDs+2S+ZeD181qptSKPPHySb1 lHr+0f/QGr3nWPSEA7hRG+bsKDdcqGXOwyTK1it/YrtrevYp21aG0mzre2QZsSBl xiWxKSvC+3BAU5YiVN5u3FJEzUtp7gwr1OPvQdFhvq4+QZYM+hi8x5+O5ci6GSBT xi9W2nt4XdAImCfRqVxZhDuTO9NncyY0+6/GSTMoeMf3Xw+3qi3HCthF8kBl/Fz3 dd0Elo2Q860lHny42YosSOzKW8CYBLykv24PctKcZjpEZMNWAHjJximHz68MqEvT DvK1PcFwAOJOJFTn+mCI9UW/dy2uczxtXexbcdw/dvBTClrQ+t7Wz9umog+Xz084 GJZjbjrZf8PUUNMkX89+XluETS+SRj4CK0CiJ9jAOaHrW+pdTfOI6OZMDJgDFdK3 jmIrMhgJeOs/uuxc4KEjag6eDR7A2SFcQEau3Bet5zJgrImCElFRsKn5hFAFldme ttxBmegyobK5G3R4olUa0LHUUBCg2tCTfaBMQdzNcQtOiEyllwoMnB7NlV475gOo GOTsczaSMlp0uLc1ckqZWa/m0ybFKVGaC7GXQ9QWZw4x+4uB48HDoVylG3CZQnn2 YskecrYToTJYyu7U1B8gpwThUmsvhd0InOuy1PyK7fpBGss6Idg= =y3r3 -----END PGP SIGNATURE----- --Sig_/F4=b=qzv72TRNlRwlpgbuD_--