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 F2916C54E69 for ; Fri, 15 Mar 2024 10:25:25 +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:MIME-Version:Message-ID:In-reply-to: Date:Subject:Cc:To:From:References:Reply-To:Content-ID:Content-Description: Resent-Date:Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID: List-Owner; bh=dLyOdeQNbEopoqqI4orsBAV/xBY9vLs5w6skAO9tYmY=; b=w3fy6i0lcq7AgB eMogTSNmrybVq2Tsz61puZHuTKB5hpUTPJARSQcDxtW9C7d/C1CNbifQG4KTinSE/UMQ1G8q4YZe3 1DZdZaM56dcuPYpC4xfBcHWgjHeQRAZY8xhTZm3mnOtaVvbvsPz71Trw+Q8upzz5R2r+KYKmO4ge5 5tocPL6pwYIprzWjkC3pFOnqxRuu85KnsYrlqnFSRrkn3sBvzubaIXOtBmvHnKUcQVyOMwy1WMTcd WbLQ480rcDJQXC1isyMr4rs9TWBsm1gQar2EZy0Ddvs61e+P10F84ebFBhT81iXm/tUAraCiMQH1l 8mn2rKIFReAzU8EKYKuw==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.97.1 #2 (Red Hat Linux)) id 1rl4kS-0000000HSV9-237d; Fri, 15 Mar 2024 10:25:12 +0000 Received: from mail-wr1-x429.google.com ([2a00:1450:4864:20::429]) by bombadil.infradead.org with esmtps (Exim 4.97.1 #2 (Red Hat Linux)) id 1rl4kP-0000000HSTv-3Ic1 for linux-arm-kernel@lists.infradead.org; Fri, 15 Mar 2024 10:25:11 +0000 Received: by mail-wr1-x429.google.com with SMTP id ffacd0b85a97d-33ed4cf02ebso170469f8f.0 for ; Fri, 15 Mar 2024 03:25:08 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=baylibre-com.20230601.gappssmtp.com; s=20230601; t=1710498307; x=1711103107; darn=lists.infradead.org; h=mime-version:message-id:in-reply-to:date:subject:cc:to:from :user-agent:references:from:to:cc:subject:date:message-id:reply-to; bh=I3EB/6PldhsbhWY9+bYWiu2w8/wr6W5dSP5c/9sIMoI=; b=ayZqBI399we34JjarUgplIfxu4WUxUPY+RhqdhQkMg9kcgmt0lJpUmWtqFCpUh11VT diB5xIeJB8KoSwZU0f8oaAMSLdfxA/HGFJrdlR/IC+hOA+omsnJOxSqyqe65r4vVBlJD EGBqIio/RPpIBJS13Oegr9BLGUw3D1UFbV/0nXAjTYf84TZm0vJpcuKdy0CLZosB1rPi cfIkInYQJmvC3NnY25wXUEodlsyyku1kCj68DUaffaS4LPSVhNpc5haEZw+VnUERowN2 pgcGEvJXxiC1RboFQvf/8oBJIzex0DfLb6KydfDmuWZGPfc349GmfYQZJ9o2MDl8/4tQ Ip3Q== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20230601; t=1710498307; x=1711103107; h=mime-version:message-id:in-reply-to:date:subject:cc:to:from :user-agent:references:x-gm-message-state:from:to:cc:subject:date :message-id:reply-to; bh=I3EB/6PldhsbhWY9+bYWiu2w8/wr6W5dSP5c/9sIMoI=; b=WA8jOSOtyXJeQqmz24ryVbHT6sAz+Ed1AJD9SBB7DGjO0bZ0091iwiX5F+rUIQcskC LtyPxkdkWqYq2WnDVQf4Tu8vVgo+ZuwbBZT9mk6qBjTH851qPu0D0VUcGgpyf02wIhCh 2f51ESEawleEMBtVB7iEF/oPYZcelU8Q8vm9+sixeskk6FMmIqOsEw1Pvyja8RMOULAz yCQ95bC57WnDMpdcLoYgo9s+2nc3/yxixQFe4U0qDi/UkdDNxylkE51oUGilOl3LrBV+ TMJxRyf3T0p4ktMqwVKFcMoIllbrHIOBLvnPFb598FsJZ3iRtP3FDPWCO4dRrh0sOj7x E0vw== X-Forwarded-Encrypted: i=1; AJvYcCUzCyZNvgIQP5j50YmUHl0GPSQRwX8zlemPdOBReNoBU/SO+bcD5YUxZyaWHXJjp4Hd9VtD3uG8kb/Q1lgRkmVOAcRXOI9ayIWwCOj8Gx8EHUsIWvE= X-Gm-Message-State: AOJu0Yz3nK6Z9DqkFDBwNGOztNN8JLe96GpOoyO8j0/XgRFJNzPCeYWD yqgQLTXpmtNf4EC0q8A4xX4/TQqIydCnfbOpHGNgl3gVdUiY9arMYwEwg7iJcH8= X-Google-Smtp-Source: AGHT+IGWDipfwojUVR3R0h/HAg8+ErMHQU1L9DU1QDqUHJY9+2A1t5wePivY0yDxiYGWnr60Q0NDnQ== X-Received: by 2002:a5d:4cc1:0:b0:33e:7f5c:732c with SMTP id c1-20020a5d4cc1000000b0033e7f5c732cmr3060666wrt.62.1710498307090; Fri, 15 Mar 2024 03:25:07 -0700 (PDT) Received: from localhost ([2a01:e0a:3c5:5fb1:8151:4d0a:14d8:1124]) by smtp.gmail.com with ESMTPSA id o14-20020adfe80e000000b0033e8c50fc3fsm2818997wrm.90.2024.03.15.03.25.06 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Fri, 15 Mar 2024 03:25:06 -0700 (PDT) References: <20240314232201.2102178-1-jan.dakinevich@salutedevices.com> <20240314232201.2102178-14-jan.dakinevich@salutedevices.com> User-agent: mu4e 1.10.8; emacs 29.2 From: Jerome Brunet To: Krzysztof Kozlowski Cc: Jan Dakinevich , Neil Armstrong , Jerome Brunet , Michael Turquette , Stephen Boyd , Rob Herring , Krzysztof Kozlowski , Conor Dooley , Philipp Zabel , Kevin Hilman , Martin Blumenstingl , Liam Girdwood , Mark Brown , Linus Walleij , Jaroslav Kysela , Takashi Iwai , linux-amlogic@lists.infradead.org, linux-clk@vger.kernel.org, devicetree@vger.kernel.org, linux-kernel@vger.kernel.org, linux-arm-kernel@lists.infradead.org, alsa-devel@alsa-project.org, linux-sound@vger.kernel.org, linux-gpio@vger.kernel.org, kernel@salutedevices.com Subject: Re: [PATCH 13/25] ASoC: dt-bindings: meson: axg-pdm: document 'sysrate' property Date: Fri, 15 Mar 2024 11:22:59 +0100 In-reply-to: Message-ID: <1jil1nhjwd.fsf@starbuckisacylon.baylibre.com> MIME-Version: 1.0 X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20240315_032509_856329_AE5C8D8F X-CRM114-Status: GOOD ( 17.50 ) 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 On Fri 15 Mar 2024 at 11:00, Krzysztof Kozlowski wrote: > On 15/03/2024 00:21, Jan Dakinevich wrote: >> This option allow to redefine the rate of DSP system clock. > > And why is it suitable for bindings? Describe the hardware, not what you > want to do in the driver. > >> >> Signed-off-by: Jan Dakinevich >> --- >> Documentation/devicetree/bindings/sound/amlogic,axg-pdm.yaml | 4 ++++ >> 1 file changed, 4 insertions(+) >> >> diff --git a/Documentation/devicetree/bindings/sound/amlogic,axg-pdm.yaml b/Documentation/devicetree/bindings/sound/amlogic,axg-pdm.yaml >> index df21dd72fc65..d2f23a59a6b6 100644 >> --- a/Documentation/devicetree/bindings/sound/amlogic,axg-pdm.yaml >> +++ b/Documentation/devicetree/bindings/sound/amlogic,axg-pdm.yaml >> @@ -40,6 +40,10 @@ properties: >> resets: >> maxItems: 1 >> >> + sysrate: >> + $ref: /schemas/types.yaml#/definitions/uint32 >> + description: redefine rate of DSP system clock > > No vendor prefix, so is it a generic property? Also, missing unit > suffix, but more importantly I don't understand why this is a property > of hardware. +1. The appropriate way to set rate of the clock before the driver take over is 'assigned-rate', if you need to customize this for different platform. Then you don't have to deal with it in the device driver. > > Best regards, > Krzysztof -- Jerome _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel