Received: by 2002:a05:7412:8521:b0:e2:908c:2ebd with SMTP id t33csp2404605rdf; Mon, 6 Nov 2023 13:13:02 -0800 (PST) X-Google-Smtp-Source: AGHT+IEDFNR1LWBJb8JTOptF3wtDGeKZ97M6+IXlISisJw6aOzl5ZCcIyARLGkTqhF867t2A7lqT X-Received: by 2002:a17:902:ea0e:b0:1cc:47a6:12bf with SMTP id s14-20020a170902ea0e00b001cc47a612bfmr21581907plg.47.1699305182109; Mon, 06 Nov 2023 13:13:02 -0800 (PST) ARC-Seal: i=1; a=rsa-sha256; t=1699305182; cv=none; d=google.com; s=arc-20160816; b=hzzETTNqwOq0q2DRqFhEqdGcNRRTLepmyQQdCTSFGBGiDmLBiQnxHuQHcmBfrEmzy/ HKLWihy4eKGjGftcIy/mznswxm2Ft5s1g3jxmF3zVJqeu77HaOxRhvy7VoLzTaVnI1I5 lJpSibCxwlHD+ZpKPSVs+7kvqZ5mYszZ3yZLRSzl04ybj1l6qkWI+iymAsbmmNCbw1M6 XaKjsejFCHyD8lt0A0hLFIqOmswMcxwIN1b3b5/sP0j+nEOFxsWe5iGArjzoYbtNh5RV VqRW6La1FFoW6U1eM8sfrmhNqF0zwfpjK8gQkAG9yeu2s8QxlzOmPWxMBkscSd5/oBqk 0ooQ== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:content-transfer-encoding:cc:to:subject :message-id:date:from:in-reply-to:references:mime-version :dkim-signature; bh=5fUcm+yN6XAMjy25ZogKbU2Yy8ZDqJ+sIhPAHGmb2ZM=; fh=qZrehyY6pGoaVCnv+maK9kSU1l8t0TXDpKkLqbKti0g=; b=KpkiretNTOIIGx9gFovvZJj8vjMWxPLsXQ7/hLrdmN+JeJeou+2UWKpUpVCE4Z82+N XcMomEcO6RFjwYpyV1CwRVHDJ2k++fal3GxmdnCK0l2/Cd872tgLO2wGMNJqu29OPVt1 i/JNqw3yIFefVXe6xYvz7lFDx5Cbk9GxPuYxDfpiHR9SXxEdgkheTQQ7Q2oJteUuaWGa K5nsH2AnF5hRua/qLd18/ApYOsCBX8GSTnFbR7QnUqz+t0kJze+vUSkaiPdPLWT9QKem RT3XWLtTi0w50nZpE30FNd6RY29mBmIXDQVxsJ6uwMnLCmz4YQvd3Z1SV1o1+R7YHUiv qxfg== ARC-Authentication-Results: i=1; mx.google.com; dkim=pass header.i=@chromium.org header.s=google header.b=C1QRDL3f; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.32 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=pass (p=NONE sp=NONE dis=NONE) header.from=chromium.org Return-Path: Received: from agentk.vger.email (agentk.vger.email. [23.128.96.32]) by mx.google.com with ESMTPS id n15-20020a170903110f00b001c4062a0e43si9882347plh.169.2023.11.06.13.13.01 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Mon, 06 Nov 2023 13:13:02 -0800 (PST) Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.32 as permitted sender) client-ip=23.128.96.32; Authentication-Results: mx.google.com; dkim=pass header.i=@chromium.org header.s=google header.b=C1QRDL3f; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.32 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=pass (p=NONE sp=NONE dis=NONE) header.from=chromium.org Received: from out1.vger.email (depot.vger.email [IPv6:2620:137:e000::3:0]) by agentk.vger.email (Postfix) with ESMTP id AEAFC8039A00; Mon, 6 Nov 2023 13:12:59 -0800 (PST) X-Virus-Status: Clean X-Virus-Scanned: clamav-milter 0.103.10 at agentk.vger.email Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S233089AbjKFVMv (ORCPT + 99 others); Mon, 6 Nov 2023 16:12:51 -0500 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:34810 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S232927AbjKFVMu (ORCPT ); Mon, 6 Nov 2023 16:12:50 -0500 Received: from mail-ej1-x631.google.com (mail-ej1-x631.google.com [IPv6:2a00:1450:4864:20::631]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id 71FAFD76 for ; Mon, 6 Nov 2023 13:12:47 -0800 (PST) Received: by mail-ej1-x631.google.com with SMTP id a640c23a62f3a-9dbb3d12aefso714504766b.0 for ; Mon, 06 Nov 2023 13:12:47 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=chromium.org; s=google; t=1699305162; x=1699909962; darn=vger.kernel.org; h=content-transfer-encoding:cc:to:subject:message-id:date:from :in-reply-to:references:mime-version:from:to:cc:subject:date :message-id:reply-to; bh=5fUcm+yN6XAMjy25ZogKbU2Yy8ZDqJ+sIhPAHGmb2ZM=; b=C1QRDL3f+CGiMD3LKE74yCnlqFyDm/nsFS6VQLSJhmPOZ8rwbXuoPJ0eBTfbD4Rcpf GHVyMaEwY5Jsyvmbes5Qpzsd7mVMcCF+y+idEZZ9OEsj4h/hbVMLvKtgtQ+jdRTL5Qka +zpnen0e8ROOStYomE6hMaoOMQ20Ma+PQpNRY= X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20230601; t=1699305162; x=1699909962; h=content-transfer-encoding:cc:to:subject:message-id:date:from :in-reply-to:references:mime-version:x-gm-message-state:from:to:cc :subject:date:message-id:reply-to; bh=5fUcm+yN6XAMjy25ZogKbU2Yy8ZDqJ+sIhPAHGmb2ZM=; b=mESAiZcOGqzbxt2t9L0JKDyQRYThRps+erzU89BB0N9AkV99i7wA50Nvdfq4LfLGyO bdf/lJQVvTFs8JtwrslZv3WthmG4vjGWnTys7y/OSAREzg4JrEq1kV3wM51Lv+JSDxuu DHSZrzrvmZPrgn5rv5ZI0dsmHf2kzuSztIQZE844WhanZ6mTZIuxffbK9vMvj50imYqs x3R1Qh3IuN5GQFn3BqDef6Qmxxwf4V8Hju3jr3/HGdDzXdS2wOzL0DWcXdyM4B36kcTK TfgFCrwR0/25yqaYq0F/anqPzTIVAKXirkyD260Oj4E+bZ/p20na+K4hAjD+Hul4T7CV Jpqg== X-Gm-Message-State: AOJu0YxwfNV3bumeZ3iIsqpVHbmquYCzFYHGM8l6f5wbeIk19VY7wovw hFKWEb2JAEg1N+IYTtEshnh8h2ZI4Ym9XNmVOERJiBCP X-Received: by 2002:a17:907:26c9:b0:9e0:dde:cc9f with SMTP id bp9-20020a17090726c900b009e00ddecc9fmr4053633ejc.22.1699305162351; Mon, 06 Nov 2023 13:12:42 -0800 (PST) Received: from mail-wm1-f51.google.com (mail-wm1-f51.google.com. [209.85.128.51]) by smtp.gmail.com with ESMTPSA id jp27-20020a170906f75b00b009b94c545678sm251117ejb.153.2023.11.06.13.12.41 for (version=TLS1_3 cipher=TLS_AES_128_GCM_SHA256 bits=128/128); Mon, 06 Nov 2023 13:12:41 -0800 (PST) Received: by mail-wm1-f51.google.com with SMTP id 5b1f17b1804b1-40853f2e93eso22915e9.0 for ; Mon, 06 Nov 2023 13:12:41 -0800 (PST) X-Received: by 2002:a05:600c:1f92:b0:407:7f21:878e with SMTP id je18-20020a05600c1f9200b004077f21878emr5139wmb.1.1699305160926; Mon, 06 Nov 2023 13:12:40 -0800 (PST) MIME-Version: 1.0 References: <20231106210337.2900034-1-hsinyi@chromium.org> <20231106210337.2900034-4-hsinyi@chromium.org> In-Reply-To: <20231106210337.2900034-4-hsinyi@chromium.org> From: Doug Anderson Date: Mon, 6 Nov 2023 13:12:23 -0800 X-Gmail-Original-Message-ID: Message-ID: Subject: Re: [PATCH v4 3/3] drm/panel-edp: Avoid adding multiple preferred modes To: Hsin-Yi Wang Cc: Neil Armstrong , Jessica Zhang , Sam Ravnborg , Maarten Lankhorst , Maxime Ripard , Thomas Zimmermann , David Airlie , Daniel Vetter , dri-devel@lists.freedesktop.org, linux-kernel@vger.kernel.org Content-Type: text/plain; charset="UTF-8" Content-Transfer-Encoding: quoted-printable X-Spam-Status: No, score=-0.9 required=5.0 tests=DKIMWL_WL_HIGH,DKIM_SIGNED, DKIM_VALID,DKIM_VALID_AU,HEADER_FROM_DIFFERENT_DOMAINS, MAILING_LIST_MULTI,SPF_HELO_NONE,SPF_PASS,T_SCC_BODY_TEXT_LINE autolearn=unavailable autolearn_force=no version=3.4.6 X-Spam-Checker-Version: SpamAssassin 3.4.6 (2021-04-09) on agentk.vger.email Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org X-Greylist: Sender passed SPF test, not delayed by milter-greylist-4.6.4 (agentk.vger.email [0.0.0.0]); Mon, 06 Nov 2023 13:12:59 -0800 (PST) Hi, On Mon, Nov 6, 2023 at 1:03=E2=80=AFPM Hsin-Yi Wang w= rote: > > @@ -622,7 +627,7 @@ static int panel_edp_get_modes(struct drm_panel *pane= l, > * and no modes (the generic edp-panel case) because it will clob= ber > * the display_info that was already set by drm_add_edid_modes(). > */ > - if (p->desc->num_timings || p->desc->num_modes) > + if (has_hard_coded_modes) nit: the comment above this line is a bit confusing now and probably needs to be updated or removed.