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 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 smtp.lore.kernel.org (Postfix) with ESMTPS id 9C405C433F5 for ; Thu, 31 Mar 2022 15:02:05 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=lists.infradead.org; s=bombadil.20210309; h=Sender: Content-Transfer-Encoding:Content-Type:List-Subscribe:List-Help:List-Post: List-Archive:List-Unsubscribe:List-Id:In-Reply-To:MIME-Version:References: Message-ID:Subject:Cc: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=a3/enOlE/HfXC+8aLU5QSZqviIZ9A7giM7UiglqngnE=; b=kJZVazYsUa/+vK MJbexasZi1YLa2TYPyxWwKoZt7OOhFxTKVL+OTvjEBZo/WFwax4m59ImZUI3j0m5BLYMfgcseZMCf B6/L3T1GIn7fm2Cxv/YXCDQak5EXnAuLFCIrF73Ai6PxAT7tUxystN26VqjU1nWp+bPGhxYE4yvyU MJ4yAUTcYw7zdo+08az2X8/M/AVWB+QwJAopxBtr9UVN+IZz9By+42v7dgMpxQKcI0dIL0P+9+yIJ 9daSAL2iBEvLmUSsPtGySXssEEfzvz/C/o5zR5R6/xFNHZZbDfTWZSNPwb2keGENzwG/WTpbQNV4h V/ckCknC6flOij4YpEoQ==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.94.2 #2 (Red Hat Linux)) id 1nZwI3-002fEH-St; Thu, 31 Mar 2022 15:00:48 +0000 Received: from muru.com ([72.249.23.125]) by bombadil.infradead.org with esmtp (Exim 4.94.2 #2 (Red Hat Linux)) id 1nZwI0-002fDP-Gs; Thu, 31 Mar 2022 15:00:45 +0000 Received: from localhost (localhost [127.0.0.1]) by muru.com (Postfix) with ESMTPS id AA72380DB; Thu, 31 Mar 2022 14:58:34 +0000 (UTC) Date: Thu, 31 Mar 2022 18:00:42 +0300 From: Tony Lindgren To: Maxime Ripard Cc: Marek Szyprowski , Mike Turquette , Stephen Boyd , linux-clk@vger.kernel.org, Dmitry Osipenko , 'Linux Samsung SOC' , linux-amlogic@lists.infradead.org, linux-omap@vger.kernel.org, linux-arm-kernel@lists.infradead.org Subject: Re: [PATCH v2 3/3] clk: Drop the rate range on clk_put Message-ID: References: <20220325161144.1901695-1-maxime@cerno.tech> <20220325161144.1901695-4-maxime@cerno.tech> <366a0232-bb4a-c357-6aa8-636e398e05eb@samsung.com> <20220330084710.3r6b5pjspz5hdmy6@houat> <20220331095456.dyyxsiu2b3yw2vvs@houat> MIME-Version: 1.0 Content-Disposition: inline In-Reply-To: <20220331095456.dyyxsiu2b3yw2vvs@houat> X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20220331_080044_664433_245185C9 X-CRM114-Status: GOOD ( 12.41 ) X-BeenThere: linux-arm-kernel@lists.infradead.org X-Mailman-Version: 2.1.34 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Sender: "linux-arm-kernel" Errors-To: linux-arm-kernel-bounces+linux-arm-kernel=archiver.kernel.org@lists.infradead.org * Maxime Ripard [220331 09:52]: > On Thu, Mar 31, 2022 at 12:42:10PM +0300, Tony Lindgren wrote: > > It seems the dts assigned-clock-parents no longer works now? > > That would make some kind of sense, __set_clk_parents calls clk_put on > both the assigned clock and its parent. > > Could you see what parent (and why?) it tries to enforce then? It picks the other option available for the mux clock that only has two options. No idea why, but if you have some debug patch in mind I can give it a try. > It looks like the gpt1_fck driver might favor another parent for that > rate, which, if it's an invalid configuration, shouldn't really happen? Hmm there's a gate clock and a mux clock, there's not really a rate selection available here for the sources. Regards, Tony _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel