From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S965550AbeEIQmT (ORCPT ); Wed, 9 May 2018 12:42:19 -0400 Received: from smtp.codeaurora.org ([198.145.29.96]:60494 "EHLO smtp.codeaurora.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S935232AbeEIQmR (ORCPT ); Wed, 9 May 2018 12:42:17 -0400 DMARC-Filter: OpenDMARC Filter v1.3.2 smtp.codeaurora.org CECBB60712 Authentication-Results: pdx-caf-mail.web.codeaurora.org; dmarc=none (p=none dis=none) header.from=codeaurora.org Authentication-Results: pdx-caf-mail.web.codeaurora.org; spf=none smtp.mailfrom=ilina@codeaurora.org Date: Wed, 9 May 2018 10:42:15 -0600 From: Lina Iyer To: andy.gross@linaro.org, david.brown@linaro.org, linux-arm-msm@vger.kernel.org, linux-soc@vger.kernel.org Cc: rnayak@codeaurora.org, bjorn.andersson@linaro.org, linux-kernel@vger.kernel.org, sboyd@kernel.org, evgreen@chromium.org, dianders@chromium.org Subject: Re: [PATCH v8 0/2] drivers/qcom: add Command DB support Message-ID: <20180509164215.GF4698@codeaurora.org> References: <20180410175724.7165-1-ilina@codeaurora.org> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii; format=flowed Content-Disposition: inline In-Reply-To: <20180410175724.7165-1-ilina@codeaurora.org> User-Agent: Mutt/1.9.5 (2018-04-13) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Hi Andy, David, These patches have been reviewed by our peers and have recieved no further comments in the past few weeks. Would you consider merging this in your trees? Thanks, Lina On Tue, Apr 10 2018 at 11:57 -0600, Lina Iyer wrote: >Changes since v7: > - Add reviewed-by tags > - Fix example > >Changes since v6: > - Fix 'reg' property > - Add reviewed-by tags > >Changes since v5: > - Remove indirection of command db start address > - Rebase on top of 4.16 > >Changes since v4: > - Address comments from Stephen and Bjorn > >These patches add support for reading a shared memory database in the newer >QCOM SoCs called Command DB. With the new architecture on SDM845, shared >resources like clocks, regulators etc., have dynamic properties. These >properties may change based on external components, board configurations or >available feature set. A remote processor detects these parameters and fills up >the database with the resource and available state information. Platform >drivers that need these shared resources will need to query this database to >get the address and properties and vote for the state. > >The information in the database is static. The database is read-only memory >location that is available for Linux. A pre-defined string is used as a key into >an entry in the database. Generally, platform drivers query the database only >at init to get the information they need. > >[v1]: https://www.spinics.net/lists/linux-arm-msm/msg32462.html >[v2]: https://lkml.org/lkml/2018/2/8/588 >[v3]: https://lkml.org/lkml/2018/2/16/842 >[v4]: https://patchwork.kernel.org/patch/10242935/ >[v5]: https://lkml.org/lkml/2018/3/14/787 >[v6]: https://lkml.org/lkml/2018/4/5/451 >[v7]: https://lkml.org/lkml/2018/4/6/756 > >Lina Iyer (2): > drivers: qcom: add command DB driver > dt-bindings: introduce Command DB for QCOM SoCs > > .../bindings/reserved-memory/qcom,cmd-db.txt | 37 +++ > drivers/of/platform.c | 1 + > drivers/soc/qcom/Kconfig | 9 + > drivers/soc/qcom/Makefile | 1 + > drivers/soc/qcom/cmd-db.c | 310 +++++++++++++++++++++ > include/soc/qcom/cmd-db.h | 45 +++ > 6 files changed, 403 insertions(+) > create mode 100644 Documentation/devicetree/bindings/reserved-memory/qcom,cmd-db.txt > create mode 100644 drivers/soc/qcom/cmd-db.c > create mode 100644 include/soc/qcom/cmd-db.h > >-- >The Qualcomm Innovation Center, Inc. is a member of the Code Aurora Forum, >a Linux Foundation Collaborative Project >