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=-11.5 required=3.0 tests=BAYES_00,DKIMWL_WL_HIGH, DKIM_SIGNED,DKIM_VALID,HEADER_FROM_DIFFERENT_DOMAINS,INCLUDES_PATCH, MAILING_LIST_MULTI,NICE_REPLY_A,SIGNED_OFF_BY,SPF_HELO_NONE,SPF_PASS, URIBL_BLOCKED,USER_AGENT_SANE_1 autolearn=unavailable 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 C9320C433E0 for ; Mon, 20 Jul 2020 09:40:15 +0000 (UTC) Received: from merlin.infradead.org (merlin.infradead.org [205.233.59.134]) (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 97300208E4 for ; Mon, 20 Jul 2020 09:40:15 +0000 (UTC) Authentication-Results: mail.kernel.org; dkim=pass (2048-bit key) header.d=lists.infradead.org header.i=@lists.infradead.org header.b="JJ6n/hb9"; dkim=fail reason="signature verification failed" (2048-bit key) header.d=st.com header.i=@st.com header.b="B2lHbtZg" DMARC-Filter: OpenDMARC Filter v1.3.2 mail.kernel.org 97300208E4 Authentication-Results: mail.kernel.org; dmarc=fail (p=none dis=none) header.from=st.com Authentication-Results: mail.kernel.org; spf=none smtp.mailfrom=linux-arm-kernel-bounces+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=merlin.20170209; h=Sender:Content-Transfer-Encoding: Content-Type:Cc:List-Subscribe:List-Help:List-Post:List-Archive: List-Unsubscribe:List-Id:MIME-Version:Content-ID:In-Reply-To:References: Message-ID:Date:Subject:To:From:Reply-To:Content-Description:Resent-Date: Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID:List-Owner; bh=TqKY/Qt2jjFE68AHJ5gj4R/xZPyWHs6hyuvBv5+41UE=; b=JJ6n/hb9Sb4+/uv4beksFXDd7 V78UL+DaPqptwPqP0sTk9uJEgRsQY/H15NPzKGpi3RRQ0el5Nd2w4Kq8pNz1Fwg7ax2tw9F5rYEQF E593/iXNoHn3H/AXkEQzxyiF2qMcd+C3YolbRQZoT9cIAIYvzlVl9CCpySPrxugrn66QI1vrp0Ga5 kseg68bR+a2RBXw1EBY+hGYoOD0koh5dJc267Vu4B0ximI5rNX1t9enxysPEQTSzUm/O+mRvW3bXt JD+tg5l+6Ec80m/8xYA2fewUGnEzMXa3CgpTfp5prI86jixKCp0Ho7fW26fPsq35N83hUy2WJJjQO v/Z8BC5Wg==; Received: from localhost ([::1] helo=merlin.infradead.org) by merlin.infradead.org with esmtp (Exim 4.92.3 #3 (Red Hat Linux)) id 1jxSFt-0002QS-FL; Mon, 20 Jul 2020 09:38:41 +0000 Received: from mx08-00178001.pphosted.com ([91.207.212.93] helo=mx07-00178001.pphosted.com) by merlin.infradead.org with esmtps (Exim 4.92.3 #3 (Red Hat Linux)) id 1jxSFq-0002PA-42 for linux-arm-kernel@lists.infradead.org; Mon, 20 Jul 2020 09:38:39 +0000 Received: from pps.filterd (m0046661.ppops.net [127.0.0.1]) by mx07-00178001.pphosted.com (8.16.0.42/8.16.0.42) with SMTP id 06K9MgFI017123; Mon, 20 Jul 2020 11:38:31 +0200 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=st.com; h=from : to : cc : subject : date : message-id : references : in-reply-to : content-type : content-id : content-transfer-encoding : mime-version; s=STMicroelectronics; bh=lC/LymgalsLxxn7CqvsDbLQJCr894y+aoN1Rm4+7KFI=; b=B2lHbtZgdj+u9WxQ68HEI/gS7/Z6eRdJaSi0mfGaOMbatZ1z9rMWQWn4GzR0qeO50ycY TiUjpfpwqmTMnc9JOFHouSKPbThhLA50opdyH507nGH0RLC1FLJtACNSznORxknYSY2i fLj3ACREU2o1kKxTYNft5iLv0mGJEWWtKB2UO4wlTPvO6U3UE/RKWDQqWUMfnmXIhz5u 0npGd1vsXUjyFiWFfyWM8eEo5LhQuK7i7yoMEm4Z1WMenrye9nY1Mxxf4VrN82557cUs eEtCazhSqDwD3civFpNN0lVi49nZ28VbghP657uXePejNwTBJSsGrppIeJvCSbj5U40e eA== Received: from beta.dmz-eu.st.com (beta.dmz-eu.st.com [164.129.1.35]) by mx07-00178001.pphosted.com with ESMTP id 32bsagr125-1 (version=TLSv1.2 cipher=ECDHE-RSA-AES256-GCM-SHA384 bits=256 verify=NOT); Mon, 20 Jul 2020 11:38:31 +0200 Received: from euls16034.sgp.st.com (euls16034.sgp.st.com [10.75.44.20]) by beta.dmz-eu.st.com (STMicroelectronics) with ESMTP id C6925100034; Mon, 20 Jul 2020 11:38:30 +0200 (CEST) Received: from Webmail-eu.st.com (sfhdag3node1.st.com [10.75.127.7]) by euls16034.sgp.st.com (STMicroelectronics) with ESMTP id A7FE82B3E71; Mon, 20 Jul 2020 11:38:30 +0200 (CEST) Received: from SFHDAG3NODE3.st.com (10.75.127.9) by SFHDAG3NODE1.st.com (10.75.127.7) with Microsoft SMTP Server (TLS) id 15.0.1473.3; Mon, 20 Jul 2020 11:38:30 +0200 Received: from SFHDAG3NODE3.st.com ([fe80::3507:b372:7648:476]) by SFHDAG3NODE3.st.com ([fe80::3507:b372:7648:476%20]) with mapi id 15.00.1347.000; Mon, 20 Jul 2020 11:38:30 +0200 From: Benjamin GAIGNARD To: Rob Herring Subject: Re: [PATCH v7 1/3] dt-bindings: media: stm32-dcmi: Add DCMI min frequency property Thread-Topic: [PATCH v7 1/3] dt-bindings: media: stm32-dcmi: Add DCMI min frequency property Thread-Index: AQHWT6e/b2wyFUUnLkaf//MXpHPgJqj/oKMAgAC3YwCAANDPAIAPCWCA Date: Mon, 20 Jul 2020 09:38:30 +0000 Message-ID: References: <20200701130129.30961-1-benjamin.gaignard@st.com> <20200701130129.30961-2-benjamin.gaignard@st.com> <20200709203718.GA837160@bogus> <20e4907a-f218-3e43-1111-7d4b9ee6d945@st.com> In-Reply-To: Accept-Language: en-US Content-Language: en-US X-MS-Has-Attach: X-MS-TNEF-Correlator: user-agent: Mozilla/5.0 (X11; Linux x86_64; rv:68.0) Gecko/20100101 Thunderbird/68.10.0 x-ms-exchange-messagesentrepresentingtype: 1 x-ms-exchange-transport-fromentityheader: Hosted x-originating-ip: [10.75.127.51] Content-ID: MIME-Version: 1.0 X-Proofpoint-Virus-Version: vendor=fsecure engine=2.50.10434:6.0.235, 18.0.687 definitions=2020-07-20_05:2020-07-17, 2020-07-20 signatures=0 X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20200720_053838_421009_1A0815D9 X-CRM114-Status: GOOD ( 25.15 ) 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: "devicetree@vger.kernel.org" , "vincent.guittot@linaro.org" , Alexandre TORGUE , "rjw@rjwysocki.net" , "linux-kernel@vger.kernel.org" , "valentin.schneider@arm.com" , "mcoquelin.stm32@gmail.com" , Hugues FRUCHET , "mchehab@kernel.org" , "linux-stm32@st-md-mailman.stormreply.com" , "linux-arm-kernel@lists.infradead.org" , "linux-media@vger.kernel.org" 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 7/10/20 10:01 PM, Rob Herring wrote: > On Fri, Jul 10, 2020 at 1:33 AM Benjamin GAIGNARD > wrote: >> >> >> On 7/9/20 10:37 PM, Rob Herring wrote: >>> On Wed, Jul 01, 2020 at 03:01:27PM +0200, Benjamin Gaignard wrote: >>>> Document st,stm32-dcmi-min-frequency property which is used to >>>> request CPUs minimum frequency when streaming frames. >>>> >>>> Signed-off-by: Benjamin Gaignard >>>> --- >>>> Documentation/devicetree/bindings/media/st,stm32-dcmi.yaml | 8 ++++++++ >>>> 1 file changed, 8 insertions(+) >>>> >>>> diff --git a/Documentation/devicetree/bindings/media/st,stm32-dcmi.yaml b/Documentation/devicetree/bindings/media/st,stm32-dcmi.yaml >>>> index 3fe778cb5cc3..05ca85a2411a 100644 >>>> --- a/Documentation/devicetree/bindings/media/st,stm32-dcmi.yaml >>>> +++ b/Documentation/devicetree/bindings/media/st,stm32-dcmi.yaml >>>> @@ -44,6 +44,13 @@ properties: >>>> bindings defined in >>>> Documentation/devicetree/bindings/media/video-interfaces.txt. >>>> >>>> + st,stm32-dcmi-min-frequency: >>>> + description: DCMI minimum CPUs frequency requirement (in KHz). >>>> + allOf: >>>> + - $ref: /schemas/types.yaml#/definitions/uint32 >>>> + - minimum: 0 >>>> + - default: 0 >>> I think this is questionable to be in DT and if it is, it's something >>> that's hardly specific to ST or this block. IIRC, we already have a way >>> to specify minimum OPPs. >> This binding is only needed on some STM32 SoC when DVFS is activated >> with low frequency setting in opp. The value also depends of the targeted >> video format and framerate. > As those 2 things are not in the DT, then neither should this value be. > >> It is not an opp because it doesn't define a voltage-current-frequency >> combination >> but only set a minimum target for the CPUs frequency to guaranty a good >> reaction >> time when handling IRQs for the sensor. > OPPs can be frequency only. This is pretty clearly defining the CPU > must Operate at a certain minimum Performance Point. Hi Rob, My goal here wasn't to define an OPP for the CPU since it is the camera interface node but to avoid let the CPU go to down. I haven't found how to use OPP bindings, can you provide me hints or examples of how I should do it? Thanks, Benjamin > > Rob _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel