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 CFBA0C433F5 for ; Mon, 28 Feb 2022 15:28:27 +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-Type: Content-Transfer-Encoding:List-Subscribe:List-Help:List-Post:List-Archive: List-Unsubscribe:List-Id:In-Reply-To:From:References:Cc:To:Subject: MIME-Version:Date:Message-ID:Reply-To:Content-ID:Content-Description: Resent-Date:Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID: List-Owner; bh=5iSSr6V1iYM4PvfefHpypZ3KlOyJoDZmCmGVZ36bhjY=; b=VXr5DZQTd7obgZ 5h25baIV6IZPNPYDdu3ySZGPqq0XQHrU4JLn802wdYgUfufB1WEXqUXmbXE3PhfiAK30DiAD9side Rb40UwLo/bpDs1YMAfXzLeLP52r5EfoIliocJL9h2jYE9tCilN4RTcL6E1flE7csPK2YYuIPJ3Ucm +ZI682nKg3dPD8LrW9htd1jcTNS09o2DssqX7lgzj0mbyRf+kJJKqkJyXeB98Eop2OUK0jaJd/vrp APnOZaBJ8aK4wEK5Cm8JYiyu+nnHNGHXdCdA3w3orwvXoyxR7tbEdnRr9YbZl9tXDrnl+aa6GOCiz +Icbbsw9duGB1TRo4rbQ==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.94.2 #2 (Red Hat Linux)) id 1nOhvj-00D9R5-Fm; Mon, 28 Feb 2022 15:27:19 +0000 Received: from foss.arm.com ([217.140.110.172]) by bombadil.infradead.org with esmtp (Exim 4.94.2 #2 (Red Hat Linux)) id 1nOhvg-00D9Pk-Gu for linux-arm-kernel@lists.infradead.org; Mon, 28 Feb 2022 15:27:18 +0000 Received: from usa-sjc-imap-foss1.foss.arm.com (unknown [10.121.207.14]) by usa-sjc-mx-foss1.foss.arm.com (Postfix) with ESMTP id 6F133ED1; Mon, 28 Feb 2022 07:27:15 -0800 (PST) Received: from [10.57.39.47] (unknown [10.57.39.47]) by usa-sjc-imap-foss1.foss.arm.com (Postfix) with ESMTPSA id 00FD03F73D; Mon, 28 Feb 2022 07:27:13 -0800 (PST) Message-ID: <1763f668-2582-c05e-7ac8-d635bc84ef82@arm.com> Date: Mon, 28 Feb 2022 15:27:08 +0000 MIME-Version: 1.0 User-Agent: Mozilla/5.0 (Windows NT 10.0; rv:91.0) Gecko/20100101 Thunderbird/91.6.1 Subject: Re: [PATCH 1/3] dt-bindings: spi: Update clocks property for ARM pl022 Content-Language: en-GB To: Mark Brown Cc: Kuldeep Singh , linux-arm-kernel@lists.infradead.org, linux-spi@vger.kernel.org, devicetree@vger.kernel.org, linux-kernel@vger.kernel.org, Rob Herring , Linus Walleij References: <20220228124345.99474-1-singh.kuldeep87k@gmail.com> <20220228124345.99474-2-singh.kuldeep87k@gmail.com> <6e14bdd4-5924-dd58-41ef-5bdb5561913b@arm.com> From: Robin Murphy In-Reply-To: X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20220228_072716_648422_4CCB5B1E X-CRM114-Status: GOOD ( 14.02 ) 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-Transfer-Encoding: 7bit Content-Type: text/plain; charset="us-ascii"; Format="flowed" Sender: "linux-arm-kernel" Errors-To: linux-arm-kernel-bounces+linux-arm-kernel=archiver.kernel.org@lists.infradead.org On 2022-02-28 15:11, Mark Brown wrote: > On Mon, Feb 28, 2022 at 02:26:12PM +0000, Robin Murphy wrote: > >> Who says that minItems is missing? Looking at the PL022 TRM[1] it seems >> clear that SSPCLK is pretty fundamental to useful operation. If that DT ever >> worked, it must be that the same clock is wired to both inputs, and the fact >> that that's how the neighbouring PL011 is described is strongly suggestive. > > Well, it could also be that the clock is wired to some other clock which > is always on (which I guess is why the driver allows this in the first > place, there's a lot of sloppy code around stuff like that in the tree). I wouldn't say the driver "allows" it, so much as it just blindly grabs the first clock assuming it's SSPCLK per the binding, and thus it will happen to work out if the underlying physical clock is the same as, or equivalent to, the APB PCLK. Otherwise, it's already into some degree of not working properly, by virtue of reading the wrong clock rate. Robin. _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel