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=-1.0 required=3.0 tests=DKIM_SIGNED,DKIM_VALID, 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 440F9C433DF for ; Fri, 26 Jun 2020 09:26:02 +0000 (UTC) Received: from vger.kernel.org (vger.kernel.org [23.128.96.18]) by mail.kernel.org (Postfix) with ESMTP id 2085B21532 for ; Fri, 26 Jun 2020 09:26:02 +0000 (UTC) Authentication-Results: mail.kernel.org; dkim=pass (2048-bit key) header.d=fooishbar-org.20150623.gappssmtp.com header.i=@fooishbar-org.20150623.gappssmtp.com header.b="Artw0zul" Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1726527AbgFZJZ7 (ORCPT ); Fri, 26 Jun 2020 05:25:59 -0400 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:53048 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1725983AbgFZJZ6 (ORCPT ); Fri, 26 Jun 2020 05:25:58 -0400 Received: from mail-wr1-x441.google.com (mail-wr1-x441.google.com [IPv6:2a00:1450:4864:20::441]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id F40E8C08C5C1 for ; Fri, 26 Jun 2020 02:25:57 -0700 (PDT) Received: by mail-wr1-x441.google.com with SMTP id z13so8792099wrw.5 for ; Fri, 26 Jun 2020 02:25:57 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=fooishbar-org.20150623.gappssmtp.com; s=20150623; h=mime-version:references:in-reply-to:from:date:message-id:subject:to :cc; bh=W4QboZ+2BYaxwFLVRZE8pYUcmIgl+vR3O91iE5KRbMc=; b=Artw0zulwtDI/tGmL95uMhiddF93m595rGy03Zk1iMnIh1widOXNePpRNBDDNkusI6 nQBKrbeTvVBSwkhtq99Dgb6AGC7YIoQwcCtSB1oaFIUrZXfM8GsP4n2tUfQdzLK67PdG Bo8EXMjiI9nMmGS3v5cdpELPJKh1akyh837vaf6BddHTUNIAMmZ5ewdKN2JyenXDuUwd 1GbEqrtZScuDZknjwv+MQikYlwO1AK9U4Z7PVk2eMiSxdXtu5bfwnM6Gh+x1HLEhrs4K 0ag+n2DyGzeWjz1eeELP2TlQ6fuY1HJ7c289GSjvbK24jEghcCmeKmk5FGEK1uR7ceQi 2a5Q== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:mime-version:references:in-reply-to:from:date :message-id:subject:to:cc; bh=W4QboZ+2BYaxwFLVRZE8pYUcmIgl+vR3O91iE5KRbMc=; b=K4JdatjCD0k792KmiZYkCcw9Erx/Xd5Fa620IAsRHjFVzYdBYemak2MSKW+dLcrd5r lSTfmDUqr5z8ZGmi+VfgnF/osrA8A7J/XvxTXPsdsbD50l89AmS8xMoQcCJBV+Xq+7xg DqvoqUhuiqCZ07WcZ2LJI9WuYqw25ez4bNuHnSaw2k5EskP1F3bcgdXgldGOb/lLVqdi Cfeco3L+RcVQAwWgJvUV/cPK94Cz+HPhrsYT80IPClrmdD4YyI8mzQMLRuLpNNmU73eQ ZaJoHY0jTqYllBCax4CwvbZBtCvl1MqQxeyH4FFtg47v0ZGqyOjmz9oWS0tE+4291M8E yTlg== X-Gm-Message-State: AOAM533XDkBtpMQKcufEJZBJ1zVKvtQuxH7JKoq4jZMHoGCX2Sk2pdxQ 0E2J8sEzMXkLxF9oOsI9y4sfIqkXVZ2gRfEkCm+Gvg== X-Google-Smtp-Source: ABdhPJziZBr18mhrggpfS2x3HdfY+CYO0A70NiaVkBhAoksbbX17oyoZOYgWqaoO72WmViYHknTtYHeVMn6O87BVprg= X-Received: by 2002:adf:8091:: with SMTP id 17mr2688885wrl.244.1593163556649; Fri, 26 Jun 2020 02:25:56 -0700 (PDT) MIME-Version: 1.0 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> <20200626115957.566efa5b@eldfell> In-Reply-To: <20200626115957.566efa5b@eldfell> From: Daniel Stone Date: Fri, 26 Jun 2020 10:25:45 +0100 Message-ID: Subject: Re: [PATCH 27/27] drm: Add default modes for connectors in unknown state To: Pekka Paalanen Cc: Daniel Vetter , Jernej Skrabec , Laurent Pinchart , Jonas Karlman , Neil Armstrong , Kieran Bingham , Maling list - DRI developers , "open list:DRM DRIVERS FOR RENESAS" , Andrzej Hajda , Laurent Pinchart , Thomas Zimmermann , Sam Ravnborg Content-Type: text/plain; charset="UTF-8" Sender: linux-renesas-soc-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-renesas-soc@vger.kernel.org Hi, On Fri, 26 Jun 2020 at 10:00, Pekka Paalanen wrote: > On Thu, 25 Jun 2020 12:44:36 +0200 Daniel Vetter wrote: > > 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". > > > > This is only really relevant on older platforms, mostly for VGA and > > somewhat for dvi outputs. > > > > 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. How about a refinement of Dan's suggestion, proceeding down this logical order and breaking if true: - ignore all disconnected outputs - if any outputs are connected, ignore all unknown outputs - if only one output is unknown, use only that output (with default mode if need be) - if any outputs are unknown but have EDID present, use only those outputs - at this point, we have multiple unknown outputs with no EDID - break and demand explicit user configuration Cheers, Daniel