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 786CDC54EAA for ; Thu, 26 Jan 2023 19:01:59 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S232416AbjAZTB6 (ORCPT ); Thu, 26 Jan 2023 14:01:58 -0500 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:46406 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S232380AbjAZTBs (ORCPT ); Thu, 26 Jan 2023 14:01:48 -0500 Received: from bombadil.infradead.org (bombadil.infradead.org [IPv6:2607:7c80:54:3::133]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id 4BACF69B22; Thu, 26 Jan 2023 11:01:37 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=infradead.org; s=bombadil.20210309; h=Sender:In-Reply-To:Content-Type: MIME-Version:References:Message-ID:Subject:Cc:To:From:Date:Reply-To: Content-Transfer-Encoding:Content-ID:Content-Description; bh=n6OTwXbqkvWKaUiJIlak8SUyFhFuvaI0XUfB/XFglaY=; b=0yivqRtx8oA+bkclJMFmY7I0Z2 LcN/cI0mJpodqwqtpjSO5x61IvhuqVioDgtHeDtVl5IaC9s9YLk1yRk3JriGWnM/k5dIV8RK6sf27 PfLYz/ZXqq8JHoSw5+3uRoIy6dMdM2rlzce699zEZ9CNrWyLffGYmge4vbLrfquJcjk4MNwoIA+F2 6qHIQlfP8sXmUR8+/y5USlXmM5T1KIY1dvY0ZgjbfEkRfUmf/MlR4uWKtjzqSnufa/iUNATHVqnjw NCCG30zQnq4Vh0N41UQERypl1mhigEo4AsE8PljYAoTJ8hXgLRg6HXVF+vX1eHcq/b+KITanherFQ q+cC7f/A==; Received: from mcgrof by bombadil.infradead.org with local (Exim 4.94.2 #2 (Red Hat Linux)) id 1pL7V9-00CEvd-Pq; Thu, 26 Jan 2023 19:01:35 +0000 Date: Thu, 26 Jan 2023 11:01:35 -0800 From: Luis Chamberlain To: Douglas Anderson Cc: ath11k@lists.infradead.org, linux-wireless@vger.kernel.org, Nick Kossifidis , Youghandhar Chintala , junyuu@chromium.org, Kalle Valo , Jiri Slaby , "David S. Miller" , Eric Dumazet , Jakub Kicinski , Manikanta Pubbisetty , Paolo Abeni , linux-kernel@vger.kernel.org, netdev@vger.kernel.org Subject: Re: [PATCH 1/2] wifi: ath11k: Use platform_get_irq() to get the interrupt Message-ID: References: <20230124110057.1.I69cf3d56c97098287fe3a70084ee515098390b70@changeid> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20230124110057.1.I69cf3d56c97098287fe3a70084ee515098390b70@changeid> Sender: Luis Chamberlain Precedence: bulk List-ID: X-Mailing-List: linux-wireless@vger.kernel.org On Tue, Jan 24, 2023 at 11:01:00AM -0800, Douglas Anderson wrote: > For the same reasons talked about in commit 9503a1fc123d ("ath9k: Use > platform_get_irq() to get the interrupt"), we should be using > platform_get_irq() in ath11k. Let's make the switch. The commit log is rather weak, it is better to re-state what the commit log in 9503a1fc123d states as it is stronger, and very clear. To that end. Why not write an SmPL Coccinelle grammer patch for this and put it on scripts/coccinelle/api ? Then hunt / convert things which will use DT as well and where this is actually useful / likely buggy. Luis