[0/6] soc: qcom: llcc: Add support for QDU1000/QRU1000

Message

Komal Bajaj Aug. 10, 2023, 6:11 a.m. UTC

This patch series does the following -
 * Refactor LLCC driver to support multiple configuration
 * Add stub function for nvmem_cell_read_u8
 * Add support for multi channel DDR configuration in LLCC
 * Add LLCC support for the Qualcomm QDU1000 and QRU1000 SoCs

Changes in v7 -
 - Changed the macro name as suggested by Mukesh.
 - Added NULL check for llcc cfgs as suggested by Mukesh.
 - Updated the num_config for qdu1000 to use ARRAY_SIZE().
 - Link to v6: https://lore.kernel.org/lkml/[e-mail]/

Changes in v6 -
 - Changed variable name from num_cfgs to num_config as suggested by Mukesh.
 - Added a check for default llcc configuration as per suggestion from Mukesh.
 - Updated the commit summary for the third and fifth patch.
 - Fixed alignment in the fourth patch.
 - Used ARRAY_SIZE() to calculate the num_config as per suggested by Konrad.
 - Link to v5: https://lore.kernel.org/lkml/[e-mail]/

Changes in v5 -
 - Separated out the secure qfprom driver changes to a separate series [1].
 - Created a wrapper struct with a pointer to qcom_llcc_config and
   length of array qcom_llcc_config.
 - Added stub function for nvmem_cell_read_u8.
 - Split commit 6/6 in the previous series into two commits.
 - Link to v4: https://lore.kernel.org/lkml/[e-mail]/

Changes in v4 -
 - Created a separate driver for reading from secure fuse region as suggested.
 - Added patch for dt-bindings of secure qfprom driver accordingly.
 - Added new properties in the dt-bindings for LLCC.
 - Implemented new logic to read the nvmem cell as suggested by Bjorn.
 - Separating the DT patches from this series as per suggestion.
 - Link to v3: https://lore.kernel.org/lkml/[e-mail]/

Changes in v3 -
 - Addressed comments from Krzysztof and Mani.
 - Using qfprom to read DDR configuration from feature register.
 - Link to v2: https://lore.kernel.org/lkml/[e-mail]/

Changes in v2:
  - Addressing comments from Konrad.
  - Link to v1: https://lore.kernel.org/lkml/[e-mail]/

[1] https://lore.kernel.org/linux-arm-msm/[e-mail]/

Komal Bajaj (6):
  dt-bindings: cache: qcom,llcc: Add LLCC compatible for QDU1000/QRU1000
  soc: qcom: llcc: Refactor llcc driver to support multiple
    configuration
  nvmem: core: Add stub for nvmem_cell_read_u8
  soc: qcom: Add LLCC support for multi channel DDR
  soc: qcom: llcc: Updating the macro name
  soc: qcom: llcc: Add QDU1000 and QRU1000 LLCC support

 .../devicetree/bindings/cache/qcom,llcc.yaml  |  10 +
 drivers/soc/qcom/llcc-qcom.c                  | 363 +++++++++++++-----
 include/linux/nvmem-consumer.h                |   6 +
 include/linux/soc/qcom/llcc-qcom.h            |   2 +-
 4 files changed, 293 insertions(+), 88 deletions(-)

--
2.41.0

Comments

Mukesh Ojha Aug. 10, 2023, 2:45 p.m. UTC | #1

nit: Version v7 missed in subject..

-Mukesh

On 8/10/2023 11:41 AM, Komal Bajaj wrote:
> This patch series does the following -
>   * Refactor LLCC driver to support multiple configuration
>   * Add stub function for nvmem_cell_read_u8
>   * Add support for multi channel DDR configuration in LLCC
>   * Add LLCC support for the Qualcomm QDU1000 and QRU1000 SoCs
> 
> Changes in v7 -
>   - Changed the macro name as suggested by Mukesh.
>   - Added NULL check for llcc cfgs as suggested by Mukesh.
>   - Updated the num_config for qdu1000 to use ARRAY_SIZE().
>   - Link to v6: https://lore.kernel.org/lkml/[e-mail]/
> 
> Changes in v6 -
>   - Changed variable name from num_cfgs to num_config as suggested by Mukesh.
>   - Added a check for default llcc configuration as per suggestion from Mukesh.
>   - Updated the commit summary for the third and fifth patch.
>   - Fixed alignment in the fourth patch.
>   - Used ARRAY_SIZE() to calculate the num_config as per suggested by Konrad.
>   - Link to v5: https://lore.kernel.org/lkml/[e-mail]/
> 
> Changes in v5 -
>   - Separated out the secure qfprom driver changes to a separate series [1].
>   - Created a wrapper struct with a pointer to qcom_llcc_config and
>     length of array qcom_llcc_config.
>   - Added stub function for nvmem_cell_read_u8.
>   - Split commit 6/6 in the previous series into two commits.
>   - Link to v4: https://lore.kernel.org/lkml/[e-mail]/
> 
> Changes in v4 -
>   - Created a separate driver for reading from secure fuse region as suggested.
>   - Added patch for dt-bindings of secure qfprom driver accordingly.
>   - Added new properties in the dt-bindings for LLCC.
>   - Implemented new logic to read the nvmem cell as suggested by Bjorn.
>   - Separating the DT patches from this series as per suggestion.
>   - Link to v3: https://lore.kernel.org/lkml/[e-mail]/
> 
> Changes in v3 -
>   - Addressed comments from Krzysztof and Mani.
>   - Using qfprom to read DDR configuration from feature register.
>   - Link to v2: https://lore.kernel.org/lkml/[e-mail]/
> 
> Changes in v2:
>    - Addressing comments from Konrad.
>    - Link to v1: https://lore.kernel.org/lkml/[e-mail]/
> 
> [1] https://lore.kernel.org/linux-arm-msm/[e-mail]/
> 
> Komal Bajaj (6):
>    dt-bindings: cache: qcom,llcc: Add LLCC compatible for QDU1000/QRU1000
>    soc: qcom: llcc: Refactor llcc driver to support multiple
>      configuration
>    nvmem: core: Add stub for nvmem_cell_read_u8
>    soc: qcom: Add LLCC support for multi channel DDR
>    soc: qcom: llcc: Updating the macro name
>    soc: qcom: llcc: Add QDU1000 and QRU1000 LLCC support
> 
>   .../devicetree/bindings/cache/qcom,llcc.yaml  |  10 +
>   drivers/soc/qcom/llcc-qcom.c                  | 363 +++++++++++++-----
>   include/linux/nvmem-consumer.h                |   6 +
>   include/linux/soc/qcom/llcc-qcom.h            |   2 +-
>   4 files changed, 293 insertions(+), 88 deletions(-)
> 
> --
> 2.41.0
>