linux-kernel - Re: [PATCH v8 0/2] drivers/qcom: add Command DB support

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <20180509164215.GF4698@codeaurora.org>
Date:   Wed, 9 May 2018 10:42:15 -0600
From:   Lina Iyer <ilina@...eaurora.org>
To:     andy.gross@...aro.org, david.brown@...aro.org,
        linux-arm-msm@...r.kernel.org, linux-soc@...r.kernel.org
Cc:     rnayak@...eaurora.org, bjorn.andersson@...aro.org,
        linux-kernel@...r.kernel.org, sboyd@...nel.org,
        evgreen@...omium.org, dianders@...omium.org
Subject: Re: [PATCH v8 0/2]  drivers/qcom: add Command DB support

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
>