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=-8.6 required=3.0 tests=BAYES_00,DKIM_INVALID, DKIM_SIGNED,HEADER_FROM_DIFFERENT_DOMAINS,INCLUDES_CR_TRAILER, MAILING_LIST_MULTI,SPF_HELO_NONE,SPF_PASS,URIBL_BLOCKED autolearn=ham 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 5C003C63793 for ; Thu, 22 Jul 2021 07:31:53 +0000 (UTC) Received: from gabe.freedesktop.org (gabe.freedesktop.org [131.252.210.177]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by mail.kernel.org (Postfix) with ESMTPS id 1F7CB61260 for ; Thu, 22 Jul 2021 07:31:53 +0000 (UTC) DMARC-Filter: OpenDMARC Filter v1.3.2 mail.kernel.org 1F7CB61260 Authentication-Results: mail.kernel.org; dmarc=fail (p=none dis=none) header.from=cerno.tech Authentication-Results: mail.kernel.org; spf=none smtp.mailfrom=dri-devel-bounces@lists.freedesktop.org Received: from gabe.freedesktop.org (localhost [127.0.0.1]) by gabe.freedesktop.org (Postfix) with ESMTP id A8A6E6ED06; Thu, 22 Jul 2021 07:31:52 +0000 (UTC) Received: from new2-smtp.messagingengine.com (new2-smtp.messagingengine.com [66.111.4.224]) by gabe.freedesktop.org (Postfix) with ESMTPS id 39E6E6ED06 for ; Thu, 22 Jul 2021 07:31:52 +0000 (UTC) Received: from compute4.internal (compute4.nyi.internal [10.202.2.44]) by mailnew.nyi.internal (Postfix) with ESMTP id 6A3305816CC; Thu, 22 Jul 2021 03:31:51 -0400 (EDT) Received: from mailfrontend1 ([10.202.2.162]) by compute4.internal (MEProxy); Thu, 22 Jul 2021 03:31:51 -0400 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=cerno.tech; h= date:from:to:cc:subject:message-id:references:mime-version :content-type:content-transfer-encoding:in-reply-to; s=fm3; bh=/ I4etZeEnDA+034NbHZPFsmfe1xdUqAONXI0N+Ofdbw=; b=YjaVhF5u4EalTcQYp XlPYecVWnReNLESjPxrTCjYsnol4ZeRqrKS5P38PSBDXdoHhF+4COp3FV/g06IOZ MmLDQEwcUMFHQcpKnDyfcuJEAJH9GC4PzHAuyTwLHiUaunpp/cTB9ZVf/AACyeyp ZCXcTbPWf2s0brf0XpJGU9UWi5TYyYvpG4zQ4Hf8v/+/akL6J+o7GCWpCO4n8v/y qsaO15WyXkWK/vMZeCZpGEmfJe/AIHgtQn/XtBFW8dyLo9dwRnd5lLJsSj8PK3lG kV6OBPTDhp9EwAqRZthIFtQCANlMS6N4jTZ8MPx1iPEWDYLVPLJJkmJJkod1Lxsy MOBvA== DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d= messagingengine.com; h=cc:content-transfer-encoding:content-type :date:from:in-reply-to:message-id:mime-version:references :subject:to:x-me-proxy:x-me-proxy:x-me-sender:x-me-sender :x-sasl-enc; s=fm3; bh=/I4etZeEnDA+034NbHZPFsmfe1xdUqAONXI0N+Ofd bw=; b=lhxmBOXhDcB7NXHcrBa4kzZTu6EMGfRieJNn0yCuwVLvyOZc4gk5/BPRe BGyVHpnstMUbAmeBxnxfIFo2qgKYpIcvCiq5DN+E2B8btE3EUX8dREB4PGkXBdmk W3j4y0AIJXgetK6OSMS561pNqUiMQoQ5Cy7328r00yUToHzRTxLpgsNpgTJ8cCm6 8fIzfAbGz7jEvzS40MVr2NWVS4Nc7ycJ0A7HzJt73o2hVO07XKp28hyirCTr6mVE Ve40+jylhLivequ5ooJR0D47r0UF07mDeAt/8NKt6hAc6h6UPfHg6nrig6OEIaRN /Ce9R+JYMl0lJBVsxE1tc7GmmKWNw== X-ME-Sender: X-ME-Received: X-ME-Proxy-Cause: gggruggvucftvghtrhhoucdtuddrgedvtddrfeehgdduudelucetufdoteggodetrfdotf fvucfrrhhofhhilhgvmecuhfgrshhtofgrihhlpdfqfgfvpdfurfetoffkrfgpnffqhgen uceurghilhhouhhtmecufedttdenucesvcftvggtihhpihgvnhhtshculddquddttddmne cujfgurhepfffhvffukfhfgggtugfgjgesthhqredttddtvdenucfhrhhomhepofgrgihi mhgvucftihhprghrugcuoehmrgigihhmvgestggvrhhnohdrthgvtghhqeenucggtffrrg htthgvrhhnpefgjeettdejgffgffdvteeutdehtdehgeehueetkeefgefhtdetjeekledu gedvudenucevlhhushhtvghrufhiiigvpedtnecurfgrrhgrmhepmhgrihhlfhhrohhmpe hmrgigihhmvgestggvrhhnohdrthgvtghh X-ME-Proxy: Received: by mail.messagingengine.com (Postfix) with ESMTPA; Thu, 22 Jul 2021 03:31:49 -0400 (EDT) Date: Thu, 22 Jul 2021 09:31:48 +0200 From: Maxime Ripard To: Sam Ravnborg Subject: Re: [PATCH v1 5/7] drm/mediatek: Drop chain_mode_fixup call in mode_valid() Message-ID: <20210722073148.rkgsvam37ytafykf@gilmour> References: <20210722062246.2512666-1-sam@ravnborg.org> <20210722062246.2512666-6-sam@ravnborg.org> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline Content-Transfer-Encoding: quoted-printable In-Reply-To: <20210722062246.2512666-6-sam@ravnborg.org> X-BeenThere: dri-devel@lists.freedesktop.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: Direct Rendering Infrastructure - Development List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Cc: Chun-Kuang Hu , Dafna Hirschfeld , Thomas Zimmermann , Jonas Karlman , David Airlie , Robert Foss , Neil Armstrong , dri-devel@lists.freedesktop.org, Andrzej Hajda , linux-mediatek@lists.infradead.org, Jernej Skrabec , Matthias Brugger , linux-arm-kernel@lists.infradead.org, Laurent Pinchart Errors-To: dri-devel-bounces@lists.freedesktop.org Sender: "dri-devel" On Thu, Jul 22, 2021 at 08:22:44AM +0200, Sam Ravnborg wrote: > The mode_valid implementation had a call to > drm_bridge_chain_mode_fixup() which would be wrong as the mode_valid is > not allowed to change anything - only to validate the mode. >=20 > As the next bridge is often/always a connector the call had no effect > anyway. So drop it. >=20 > From the git history I could see this call was included in the original > version of the driver so there was no help there to find out why it was > added in the first place. But a lot has changed since the initial driver > were added and is seems safe to remove the call now. >=20 > Signed-off-by: Sam Ravnborg > Cc: Chun-Kuang Hu > Cc: Philipp Zabel > Cc: Matthias Brugger > Cc: Dafna Hirschfeld > Cc: linux-mediatek@lists.infradead.org > Cc: linux-arm-kernel@lists.infradead.org Reviewed-by: Maxime Ripard Maxime