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 vger.kernel.org (vger.kernel.org [23.128.96.18]) by smtp.lore.kernel.org (Postfix) with ESMTP id 5C91FC54EE9 for ; Tue, 13 Sep 2022 07:53:51 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S231273AbiIMHxu (ORCPT ); Tue, 13 Sep 2022 03:53:50 -0400 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:37294 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S231131AbiIMHxs (ORCPT ); Tue, 13 Sep 2022 03:53:48 -0400 Received: from dfw.source.kernel.org (dfw.source.kernel.org [IPv6:2604:1380:4641:c500::1]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id 5323554651; Tue, 13 Sep 2022 00:53:47 -0700 (PDT) Received: from smtp.kernel.org (relay.kernel.org [52.25.139.140]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by dfw.source.kernel.org (Postfix) with ESMTPS id D74A6612E7; Tue, 13 Sep 2022 07:53:46 +0000 (UTC) Received: by smtp.kernel.org (Postfix) with ESMTPSA id 39DA7C433D7; Tue, 13 Sep 2022 07:53:46 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org; s=k20201202; t=1663055626; bh=rFPbnv2DxX3rBK1a7Jmm1zE+iq04DfP9/i/2IqvALRc=; h=Date:From:To:Cc:Subject:References:In-Reply-To:From; b=hlGrmphSQgEThIAQsJUpshNn2BE0nacqeqs8Un/E4gMkyv1wHHJevTQNjLxpzPwPO 79tm1goievGKLE8mqyJYBRtHSP2cbkez1QlJfEtWCp67OAaS1wkzYC+Hsv+r++ibRf +8s88EmcISfP1CLtcVK9YQKe2RNDEMYIHwq0SB+zhJrR30JQm34isT9nd2jzII2ddg Gh+XeLDhPYWBdpk2sp1AlTHVOv4V363IpEu8P2J+90YBc5ZBH+w7iKtiVuOS45et7I eTtpsdSjQrXqo2fLL3nixe2uzN8mejPBbKLMNWXIFmeQidRgiWQmzL0fBwOiOhT7XR fA5hXas4pgW8w== Received: from johan by xi.lan with local (Exim 4.94.2) (envelope-from ) id 1oY0jo-00010R-LD; Tue, 13 Sep 2022 09:53:44 +0200 Date: Tue, 13 Sep 2022 09:53:44 +0200 From: Johan Hovold To: Dmitry Baryshkov Cc: Johan Hovold , Douglas Anderson , Rob Clark , Andrzej Hajda , Neil Armstrong , Robert Foss , Laurent Pinchart , Jonas Karlman , Jernej Skrabec , David Airlie , Daniel Vetter , Sean Paul , Stephen Boyd , Bjorn Andersson , Manivannan Sadhasivam , dri-devel@lists.freedesktop.org, linux-arm-msm@vger.kernel.org, freedreno@lists.freedesktop.org, linux-kernel@vger.kernel.org Subject: Re: [PATCH 7/7] drm/msm: drop modeset sanity checks Message-ID: References: <20220912154046.12900-1-johan+linaro@kernel.org> <20220912154046.12900-8-johan+linaro@kernel.org> <39cbf0cb-9f25-6a0d-6e56-3f9e0aaa0296@linaro.org> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <39cbf0cb-9f25-6a0d-6e56-3f9e0aaa0296@linaro.org> Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Mon, Sep 12, 2022 at 09:06:28PM +0300, Dmitry Baryshkov wrote: > On 12/09/2022 18:40, Johan Hovold wrote: > > Drop the overly defensive modeset sanity checks of function parameters > > which have already been checked or used by the callers. > > > > Signed-off-by: Johan Hovold > > Again, please split into dp and dsi patches. After that: Sure, if you prefer. But without the stable-tree argument I think there's little point in splitting. > Reviewed-by: Dmitry Baryshkov > > > --- > > drivers/gpu/drm/msm/dp/dp_display.c | 7 +------ > > drivers/gpu/drm/msm/dsi/dsi.c | 7 +------ > > 2 files changed, 2 insertions(+), 12 deletions(-) Johan