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=-2.2 required=3.0 tests=HEADER_FROM_DIFFERENT_DOMAINS, MAILING_LIST_MULTI,SPF_HELO_NONE,SPF_PASS,URIBL_BLOCKED,USER_AGENT_SANE_1 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 DC535C7618B for ; Wed, 24 Jul 2019 06:48:06 +0000 (UTC) Received: from vger.kernel.org (vger.kernel.org [209.132.180.67]) by mail.kernel.org (Postfix) with ESMTP id B033B21743 for ; Wed, 24 Jul 2019 06:48:06 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1726371AbfGXGsF (ORCPT ); Wed, 24 Jul 2019 02:48:05 -0400 Received: from muru.com ([72.249.23.125]:55858 "EHLO muru.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1725878AbfGXGsD (ORCPT ); Wed, 24 Jul 2019 02:48:03 -0400 Received: from atomide.com (localhost [127.0.0.1]) by muru.com (Postfix) with ESMTPS id 6EFF9816A; Wed, 24 Jul 2019 06:48:27 +0000 (UTC) Date: Tue, 23 Jul 2019 23:47:58 -0700 From: Tony Lindgren To: Suman Anna Cc: linux-omap@vger.kernel.org, Dave Gerlach , Faiz Abbas , Greg Kroah-Hartman , Keerthy , Nishanth Menon , Peter Ujfalusi , Roger Quadros , Tero Kristo , linux-kernel@vger.kernel.org, linux-arm-kernel@lists.infradead.org Subject: Re: [PATCH 5/8] ARM: dts: Drop bogus ahclkr clocks for dra7 mcasp 3 to 8 Message-ID: <20190724064758.GU5447@atomide.com> References: <20190723112811.44381-1-tony@atomide.com> <20190723112811.44381-6-tony@atomide.com> <2c750847-700e-c835-ee53-a656b363c36c@ti.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <2c750847-700e-c835-ee53-a656b363c36c@ti.com> User-Agent: Mutt/1.11.4 (2019-03-13) Sender: linux-kernel-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org * Suman Anna [190723 21:02]: > Hi Tony, > > On 7/23/19 6:28 AM, Tony Lindgren wrote: > > The ahclkr clkctrl clock bit 28 only exists for mcasp 1 and 2 on dra7. > > Otherwise we get the following warning on beagle-x15: ... > > @@ -2962,9 +2958,8 @@ > > ; > > /* Domains (P, C): l4per_pwrdm, l4per2_clkdm */ > > clocks = <&l4per2_clkctrl DRA7_L4PER2_MCASP7_CLKCTRL 0>, > > - <&l4per2_clkctrl DRA7_L4PER2_MCASP7_CLKCTRL 24>, > > - <&l4per2_clkctrl DRA7_L4PER2_MCASP7_CLKCTRL 28>; > > - clock-names = "fck", "ahclkx", "ahclkr"; > > + <&l4per2_clkctrl DRA7_L4PER2_MCASP7_CLKCTRL 24>; > > + clock-names = "fck", "ahclkx"; > > The equivalent change to MCASP8 is missing. Thanks for spotting it, probably should be set up the same way as MCASP4 too looking at the TRM. Tero, care to check the dra7 mcasp clocks we have defined? $ grep MCASP drivers/clk/ti/clk-7xx.c { DRA7_IPU_MCASP1_CLKCTRL, dra7_mcasp1_bit_data, CLKF_SW_SUP, "ipu-clkctrl:0000:22" }, { DRA7_L4PER2_MCASP2_CLKCTRL, dra7_mcasp2_bit_data, CLKF_SW_SUP, "l4per2-clkctrl:0154:22" }, { DRA7_L4PER2_MCASP3_CLKCTRL, dra7_mcasp3_bit_data, CLKF_SW_SUP, "l4per2-clkctrl:015c:22" }, { DRA7_L4PER2_MCASP5_CLKCTRL, dra7_mcasp5_bit_data, CLKF_SW_SUP, "l4per2-clkctrl:016c:22" }, { DRA7_L4PER2_MCASP8_CLKCTRL, dra7_mcasp8_bit_data, CLKF_SW_SUP, "l4per2-clkctrl:0184:24" }, { DRA7_L4PER2_MCASP4_CLKCTRL, dra7_mcasp4_bit_data, CLKF_SW_SUP, "l4per2-clkctrl:018c:22" }, { DRA7_L4PER2_MCASP6_CLKCTRL, dra7_mcasp6_bit_data, CLKF_SW_SUP, "l4per2-clkctrl:01f8:22" }, { DRA7_L4PER2_MCASP7_CLKCTRL, dra7_mcasp7_bit_data, CLKF_SW_SUP, "l4per2-clkctrl:01fc:22" }, Is bit 24 above correct for MCASP8 or should it too be 22 like adjacent MCASP4 in the TRM? Regards, Tony 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=-2.3 required=3.0 tests=DKIMWL_WL_HIGH,DKIM_SIGNED, DKIM_VALID,HEADER_FROM_DIFFERENT_DOMAINS,MAILING_LIST_MULTI,SPF_HELO_NONE, SPF_PASS,URIBL_BLOCKED,USER_AGENT_SANE_1 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 9222FC7618B for ; Wed, 24 Jul 2019 06:48:28 +0000 (UTC) Received: from bombadil.infradead.org (bombadil.infradead.org [198.137.202.133]) (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 667DE21743 for ; Wed, 24 Jul 2019 06:48:28 +0000 (UTC) Authentication-Results: mail.kernel.org; dkim=pass (2048-bit key) header.d=lists.infradead.org header.i=@lists.infradead.org header.b="FF6M+HzJ" DMARC-Filter: OpenDMARC Filter v1.3.2 mail.kernel.org 667DE21743 Authentication-Results: mail.kernel.org; dmarc=none (p=none dis=none) header.from=atomide.com Authentication-Results: mail.kernel.org; spf=none smtp.mailfrom=linux-arm-kernel-bounces+infradead-linux-arm-kernel=archiver.kernel.org@lists.infradead.org DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=lists.infradead.org; s=bombadil.20170209; h=Sender: Content-Transfer-Encoding:Content-Type:Cc:List-Subscribe:List-Help:List-Post: List-Archive:List-Unsubscribe:List-Id:In-Reply-To:MIME-Version:References: Message-ID:Subject:To:From:Date:Reply-To:Content-ID:Content-Description: Resent-Date:Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID: List-Owner; bh=yVJNsAmHNjtRg3Vpyq1bkIF3ICT89zOVEVuO/yi8nVk=; b=FF6M+HzJ+F94fm ev0aZYFObLieePoTTXE+BbB1qhx7wO9s+taDkZfEBp3/HmwQCPP/sMZ10ZHJsM8Hzzuhwm7PKTcr8 iHo5gIxhkLrXjW7tZpfOvkat2RPUEVftT6e5+mdATfaueKsLeLwUn30U0EvFmU93uA9RI5fxf4itQ MvRQzhuxnCIEnTT16Af2islrNZBJPhjK9tns16D3liIQm8A6GLW3J73+M/SYP3OPhVK+bbfV8enfB b//kgBlwTk6mb1w+CM2/7n85JNzGgt55FJcVN273Sf9heLj5XA0FPXuAmf+UYkY1Y6hwEFwLx4SDY kohcowVb9H+UKK7CZdQQ==; Received: from localhost ([127.0.0.1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.92 #3 (Red Hat Linux)) id 1hqB4L-0001v0-Qg; Wed, 24 Jul 2019 06:48:09 +0000 Received: from muru.com ([72.249.23.125]) by bombadil.infradead.org with esmtp (Exim 4.92 #3 (Red Hat Linux)) id 1hqB4F-0001u5-Qj for linux-arm-kernel@lists.infradead.org; Wed, 24 Jul 2019 06:48:05 +0000 Received: from atomide.com (localhost [127.0.0.1]) by muru.com (Postfix) with ESMTPS id 6EFF9816A; Wed, 24 Jul 2019 06:48:27 +0000 (UTC) Date: Tue, 23 Jul 2019 23:47:58 -0700 From: Tony Lindgren To: Suman Anna Subject: Re: [PATCH 5/8] ARM: dts: Drop bogus ahclkr clocks for dra7 mcasp 3 to 8 Message-ID: <20190724064758.GU5447@atomide.com> References: <20190723112811.44381-1-tony@atomide.com> <20190723112811.44381-6-tony@atomide.com> <2c750847-700e-c835-ee53-a656b363c36c@ti.com> MIME-Version: 1.0 Content-Disposition: inline In-Reply-To: <2c750847-700e-c835-ee53-a656b363c36c@ti.com> User-Agent: Mutt/1.11.4 (2019-03-13) X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20190723_234803_912929_E48FFF89 X-CRM114-Status: UNSURE ( 9.97 ) X-CRM114-Notice: Please train this message. X-BeenThere: linux-arm-kernel@lists.infradead.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Cc: Nishanth Menon , Tero Kristo , Dave Gerlach , Keerthy , linux-kernel@vger.kernel.org, Peter Ujfalusi , Faiz Abbas , Greg Kroah-Hartman , linux-omap@vger.kernel.org, linux-arm-kernel@lists.infradead.org, Roger Quadros Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Sender: "linux-arm-kernel" Errors-To: linux-arm-kernel-bounces+infradead-linux-arm-kernel=archiver.kernel.org@lists.infradead.org * Suman Anna [190723 21:02]: > Hi Tony, > > On 7/23/19 6:28 AM, Tony Lindgren wrote: > > The ahclkr clkctrl clock bit 28 only exists for mcasp 1 and 2 on dra7. > > Otherwise we get the following warning on beagle-x15: ... > > @@ -2962,9 +2958,8 @@ > > ; > > /* Domains (P, C): l4per_pwrdm, l4per2_clkdm */ > > clocks = <&l4per2_clkctrl DRA7_L4PER2_MCASP7_CLKCTRL 0>, > > - <&l4per2_clkctrl DRA7_L4PER2_MCASP7_CLKCTRL 24>, > > - <&l4per2_clkctrl DRA7_L4PER2_MCASP7_CLKCTRL 28>; > > - clock-names = "fck", "ahclkx", "ahclkr"; > > + <&l4per2_clkctrl DRA7_L4PER2_MCASP7_CLKCTRL 24>; > > + clock-names = "fck", "ahclkx"; > > The equivalent change to MCASP8 is missing. Thanks for spotting it, probably should be set up the same way as MCASP4 too looking at the TRM. Tero, care to check the dra7 mcasp clocks we have defined? $ grep MCASP drivers/clk/ti/clk-7xx.c { DRA7_IPU_MCASP1_CLKCTRL, dra7_mcasp1_bit_data, CLKF_SW_SUP, "ipu-clkctrl:0000:22" }, { DRA7_L4PER2_MCASP2_CLKCTRL, dra7_mcasp2_bit_data, CLKF_SW_SUP, "l4per2-clkctrl:0154:22" }, { DRA7_L4PER2_MCASP3_CLKCTRL, dra7_mcasp3_bit_data, CLKF_SW_SUP, "l4per2-clkctrl:015c:22" }, { DRA7_L4PER2_MCASP5_CLKCTRL, dra7_mcasp5_bit_data, CLKF_SW_SUP, "l4per2-clkctrl:016c:22" }, { DRA7_L4PER2_MCASP8_CLKCTRL, dra7_mcasp8_bit_data, CLKF_SW_SUP, "l4per2-clkctrl:0184:24" }, { DRA7_L4PER2_MCASP4_CLKCTRL, dra7_mcasp4_bit_data, CLKF_SW_SUP, "l4per2-clkctrl:018c:22" }, { DRA7_L4PER2_MCASP6_CLKCTRL, dra7_mcasp6_bit_data, CLKF_SW_SUP, "l4per2-clkctrl:01f8:22" }, { DRA7_L4PER2_MCASP7_CLKCTRL, dra7_mcasp7_bit_data, CLKF_SW_SUP, "l4per2-clkctrl:01fc:22" }, Is bit 24 above correct for MCASP8 or should it too be 22 like adjacent MCASP4 in the TRM? Regards, Tony _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel