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 mail.kernel.org (mail.kernel.org [198.145.29.99]) by smtp.lore.kernel.org (Postfix) with ESMTP id C5D01C433F5 for ; Thu, 7 Oct 2021 17:26:45 +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 92E58603E9 for ; Thu, 7 Oct 2021 17:26:45 +0000 (UTC) DMARC-Filter: OpenDMARC Filter v1.4.1 mail.kernel.org 92E58603E9 Authentication-Results: mail.kernel.org; dmarc=none (p=none dis=none) header.from=atomide.com Authentication-Results: mail.kernel.org; spf=none smtp.mailfrom=lists.infradead.org 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: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=8dJxXWQ2gXo7sVsHMY72L1N75ujJEsAlnklPGuU352k=; b=o7ZNDE7ZWRoOcQ vrpNEu8zUnBdC5E78Wy52slCyzWJ20dzK0dtZFOLRawVrCJsBKFkNLowLCjLxQ4GoYmjWMhoMAIac qSdNeaMjmus19MnhE2EnMhCCyStXcoeXffRDgJyiVaQKBROPy4I5h1a6czkr3dfyYa9LZi05Kvr6O 9RRuuU5WwFnz5Vqq2oIXqaTKNUmXxmgb24b8DI28Dr10GDOSA3QkgKNkDJaFVajrvnMq3mCTLsrRD PBpU3f/n1h2uh7ydU70JPaaIsbvtEN4V6mZ1BVyzOa6aIvXq4kdnCCtA/I80IwPtWIDxlPMXFdjdL OLCa4rJPwFVzK28S/jhA==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.94.2 #2 (Red Hat Linux)) id 1mYX8d-000N8e-Hm; Thu, 07 Oct 2021 17:24:59 +0000 Received: from muru.com ([72.249.23.125]) by bombadil.infradead.org with esmtp (Exim 4.94.2 #2 (Red Hat Linux)) id 1mYX8Z-000N7o-Vk for linux-arm-kernel@lists.infradead.org; Thu, 07 Oct 2021 17:24:57 +0000 Received: from localhost (localhost [127.0.0.1]) by muru.com (Postfix) with ESMTPS id 6E3E080C7; Thu, 7 Oct 2021 17:25:24 +0000 (UTC) Date: Thu, 7 Oct 2021 20:24:52 +0300 From: Tony Lindgren To: Geert Uytterhoeven Subject: Re: [PATCH 1/3] dt-bindings: bus: simple-pm-bus: Make clocks and power-domains optional Message-ID: References: <20211007124858.44011-1-tony@atomide.com> <20211007124858.44011-2-tony@atomide.com> MIME-Version: 1.0 Content-Disposition: inline In-Reply-To: X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20211007_102456_087696_7EA498E3 X-CRM114-Status: GOOD ( 23.73 ) 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: , Cc: "open list:OPEN FIRMWARE AND FLATTENED DEVICE TREE BINDINGS" , Geert Uytterhoeven , Rob Herring , Linux Kernel Mailing List , Rob Herring , Simon Horman , "open list:TI ETHERNET SWITCH DRIVER \(CPSW\)" , Linux ARM 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 * Geert Uytterhoeven [211007 13:27]: > Hi Tony, > > Thanks for your patch! > > On Thu, Oct 7, 2021 at 2:49 PM Tony Lindgren wrote: > > Clocks and power domains are not required by the simple-pm-bus driver. > > There are buses with read-only registers for clocks and power domains > > that are always on. > > The presence of clocks or power-domains properties is the only > distinguishing factor between simple-pm-bus and simple-bus, from a > DT point of view. So if there has to be a distinguishment, the > properties should be required Heh seems there is no need for distinguishment beyond the compatible property here though :) > If you don't have clocks and power-domains, you should use simple-bus. Except simple-bus is not the same as simple-pm-bus. We do not have simple-bus do pm_runtime_enable() as you well know having written it :) > > Even without clocks and power domains configured, simple-pm-bus is still > > different from simple-bus as simple-pm-bus enables runtime PM for the bus > > driver. > > Which you need to have working Runtime PM for child devices, right? ;-) Right. And based on what I remember we simply cannot do pm_runtime_enable() for simple-bus without breaking tons of devices. > This is not specific to DT, but to Linux. > One more reason to let Linux treat simple-pm-bus and simple-bus exactly > the same. Linux handles the clocks and power-domains (if present) > transparently anyway, through PM Domains I agree they should be treated the same way with simple-pm-bus just doing the pm_runtime_enable() being the only difference. But the clocks and power domain still should be optional. They are not required by simple-pm-bus.c driver, and may not be required by the hardware. Got any better solutions in mind? Adding yet another compatible or another driver does not seem to get us anywhere either with this :) Regards, Tony _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel