[v3,0/4] clk: qcom : add sm8250 LPASS GFM drivers

Message

Srinivas Kandagatla Oct. 16, 2020, 2:12 p.m. UTC

This patchset adds support for GFM Muxes found in LPASS
(Low Power Audio SubSystem) IP in Audio Clock Controller
and Always ON clock controller.

Clocks derived from these muxes are consumed by LPASS Digital Codec.
Currently the driver for Audio and Always ON clock controller only
supports GFM Muxes, however it should be easy to add more clock
support when required

Changes since v2:
- updated Kconfig text and help as suggested by Stephen
- removed unnecessary header file include
- use fw_name instead of name for parents
- use devm_platform_ioremap_resource

verified dt_binding_check to pass on linux master,

Srinivas Kandagatla (4):
  dt-bindings: clock: Add support for LPASS Audio Clock Controller
  dt-bindings: clock: Add support for LPASS Always ON Controller
  clk: qcom: Add support to LPASS AUDIO_CC Glitch Free Mux clocks
  clk: qcom: Add support to LPASS AON_CC Glitch Free Mux clocks

 .../bindings/clock/qcom,aoncc-sm8250.yaml     |  58 ++++
 .../bindings/clock/qcom,audiocc-sm8250.yaml   |  58 ++++
 drivers/clk/qcom/Kconfig                      |   6 +
 drivers/clk/qcom/Makefile                     |   1 +
 drivers/clk/qcom/lpass-gfm-sm8250.c           | 320 ++++++++++++++++++
 .../clock/qcom,sm8250-lpass-aoncc.h           |  11 +
 .../clock/qcom,sm8250-lpass-audiocc.h         |  13 +
 7 files changed, 467 insertions(+)
 create mode 100644 Documentation/devicetree/bindings/clock/qcom,aoncc-sm8250.yaml
 create mode 100644 Documentation/devicetree/bindings/clock/qcom,audiocc-sm8250.yaml
 create mode 100644 drivers/clk/qcom/lpass-gfm-sm8250.c
 create mode 100644 include/dt-bindings/clock/qcom,sm8250-lpass-aoncc.h
 create mode 100644 include/dt-bindings/clock/qcom,sm8250-lpass-audiocc.h

Comments

Stephen Boyd Oct. 19, 2020, 10:42 p.m. UTC | #1

Quoting Srinivas Kandagatla (2020-10-16 07:12:37)
> This patchset adds support for GFM Muxes found in LPASS
> (Low Power Audio SubSystem) IP in Audio Clock Controller
> and Always ON clock controller.
> 
> Clocks derived from these muxes are consumed by LPASS Digital Codec.
> Currently the driver for Audio and Always ON clock controller only
> supports GFM Muxes, however it should be easy to add more clock
> support when required
> 
> Changes since v2:
> - updated Kconfig text and help as suggested by Stephen
> - removed unnecessary header file include
> - use fw_name instead of name for parents
> - use devm_platform_ioremap_resource
> 
> verified dt_binding_check to pass on linux master,

Which linux master? Can you try on linux-next?

Srinivas Kandagatla Oct. 20, 2020, 9:01 a.m. UTC | #2

On 19/10/2020 23:42, Stephen Boyd wrote:
> Quoting Srinivas Kandagatla (2020-10-16 07:12:37)
>> This patchset adds support for GFM Muxes found in LPASS
>> (Low Power Audio SubSystem) IP in Audio Clock Controller
>> and Always ON clock controller.
>>
>> Clocks derived from these muxes are consumed by LPASS Digital Codec.
>> Currently the driver for Audio and Always ON clock controller only
>> supports GFM Muxes, however it should be easy to add more clock
>> support when required
>>
>> Changes since v2:
>> - updated Kconfig text and help as suggested by Stephen
>> - removed unnecessary header file include
>> - use fw_name instead of name for parents
>> - use devm_platform_ioremap_resource
>>
>> verified dt_binding_check to pass on linux master,
> Which linux master? Can you try on linux-next?

I meant linus master branch. However I learned that Bot runs only based 
on rc1 tag (5.9-rc1 in this case) which is why its still reporting this 
errors even though the dependent patches are already in linus master branch!

I will resend this version patchset once rc1 is out!

thanks,
srini