mbox series

[v2,0/3] dt-bindings: soc: qcom: aoss: Support sc8180x and convert to YAML

Message ID 20210709174142.1274554-1-bjorn.andersson@linaro.org
Headers show
Series dt-bindings: soc: qcom: aoss: Support sc8180x and convert to YAML | expand

Message

Bjorn Andersson July 9, 2021, 5:41 p.m. UTC 
Introduce generic and sc8180x specific compatible for the aoss QMP binding and
add the generic to the implementation. Then convert the binding to YAML.

Bjorn Andersson (3):
  dt-bindings: soc: qcom: aoss: Add SC8180X and generic compatible
  dt-bindings: soc: qcom: aoss: Convert to YAML
  soc: qcom: aoss: Add generic compatible

 .../bindings/soc/qcom/qcom,aoss-qmp.txt       |  87 -------------
 .../bindings/soc/qcom/qcom,aoss-qmp.yaml      | 114 ++++++++++++++++++
 drivers/soc/qcom/qcom_aoss.c                  |   1 +
 3 files changed, 115 insertions(+), 87 deletions(-)
 delete mode 100644 Documentation/devicetree/bindings/soc/qcom/qcom,aoss-qmp.txt
 create mode 100644 Documentation/devicetree/bindings/soc/qcom/qcom,aoss-qmp.yaml

Comments

Sibi Sankar July 16, 2021, 5:01 p.m. UTC | #1 
On 7/9/21 11:11 PM, Bjorn Andersson wrote:
> It seems we don't need platform specific implementation for the AOSS
> QMP, so let's introduce a generic compatible to avoid having to update
> the driver for each platform.
> 
> Signed-off-by: Bjorn Andersson <bjorn.andersson@linaro.org>

Reviewed-by: Sibi Sankar <sibis@codeaurora.org>
Tested-by: Sibi Sankar <sibis@codeaurora.org>

> ---
> 
> Changes since v1:
> - None
> 
>   drivers/soc/qcom/qcom_aoss.c | 1 +
>   1 file changed, 1 insertion(+)
> 
> diff --git a/drivers/soc/qcom/qcom_aoss.c b/drivers/soc/qcom/qcom_aoss.c
> index 934fcc4d2b05..92a1af70a649 100644
> --- a/drivers/soc/qcom/qcom_aoss.c
> +++ b/drivers/soc/qcom/qcom_aoss.c
> @@ -602,6 +602,7 @@ static const struct of_device_id qmp_dt_match[] = {
>   	{ .compatible = "qcom,sm8150-aoss-qmp", },
>   	{ .compatible = "qcom,sm8250-aoss-qmp", },
>   	{ .compatible = "qcom,sm8350-aoss-qmp", },
> +	{ .compatible = "qcom,aoss-qmp", },
>   	{}
>   };
>   MODULE_DEVICE_TABLE(of, qmp_dt_match);
>