netdev - Re: [PATCH v6 0/4] Add pinctrl support for SDX75

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: <c0c3db1d-e83c-3610-ed61-db84cd88b569@quicinc.com>
Date: Wed, 10 May 2023 17:45:54 +0530
From: Rohit Agarwal <quic_rohiagar@...cinc.com>
To: <andy.shevchenko@...il.com>, <agross@...nel.org>, <andersson@...nel.org>,
        <konrad.dybcio@...aro.org>, <linus.walleij@...aro.org>,
        <robh+dt@...nel.org>, <krzysztof.kozlowski+dt@...aro.org>,
        <richardcochran@...il.com>, <manivannan.sadhasivam@...aro.org>,
        Mukesh Ojha <quic_mojha@...cinc.com>
CC: <linux-arm-msm@...r.kernel.org>, <linux-gpio@...r.kernel.org>,
        <devicetree@...r.kernel.org>, <linux-kernel@...r.kernel.org>,
        <netdev@...r.kernel.org>
Subject: Re: [PATCH v6 0/4] Add pinctrl support for SDX75


On 5/10/2023 5:08 PM, Rohit Agarwal wrote:
> Hi,
>
> Changes in v6:
>   - Refactoring as per suggestions from Andy to remove msm_function and
>     reusing the pinfunction and pingroup struct with macros as well.

Patch 2/4 didnt go through in the mailing list linux-arm-msm because of 
char length.
BOUNCE linux-arm-msm@...r.kernel.org: Message too long (>100000 chars)

Here is the link for it. 
https://lore.kernel.org/all/1683718725-14869-3-git-send-email-quic_rohiagar@quicinc.com/
Please suggest if this patch needs to be broken down.

Thanks,
Rohit.
> Changes in v5:
>   - Refactor the pinctrl target files based on the new macro and
>     structure defined as suggested by Andy.
>
> Changes in v4:
>   - Fixed the bindings check and rebased on linux-next.
>
> Changes in v3:
>   - Rebased the bindings on linux-next as suggested by Krzysztof.
>
> Changes in v2:
>   - Updated the bindings to clear the bindings check.
>
> This patch series adds pinctrl bindings and tlmm support for SDX75.
>
> Thanks,
> Rohit.
>
> Rohit Agarwal (4):
>    dt-bindings: pinctrl: qcom: Add SDX75 pinctrl devicetree compatible
>    pinctrl: qcom: Remove the msm_function struct
>    pinctrl: qcom: Refactor generic qcom pinctrl driver
>    pinctrl: qcom: Add SDX75 pincontrol driver
>
>   .../bindings/pinctrl/qcom,sdx75-tlmm.yaml          |  169 +++
>   drivers/pinctrl/qcom/Kconfig                       |   30 +-
>   drivers/pinctrl/qcom/Makefile                      |    3 +-
>   drivers/pinctrl/qcom/pinctrl-apq8064.c             |  104 +-
>   drivers/pinctrl/qcom/pinctrl-apq8084.c             |  264 ++--
>   drivers/pinctrl/qcom/pinctrl-ipq4019.c             |  104 +-
>   drivers/pinctrl/qcom/pinctrl-ipq5332.c             |  206 ++-
>   drivers/pinctrl/qcom/pinctrl-ipq6018.c             |  260 ++--
>   drivers/pinctrl/qcom/pinctrl-ipq8064.c             |  114 +-
>   drivers/pinctrl/qcom/pinctrl-ipq8074.c             |  240 ++-
>   drivers/pinctrl/qcom/pinctrl-mdm9607.c             |  276 ++--
>   drivers/pinctrl/qcom/pinctrl-mdm9615.c             |   90 +-
>   drivers/pinctrl/qcom/pinctrl-msm.c                 |   13 +-
>   drivers/pinctrl/qcom/pinctrl-msm.h                 |   42 +-
>   drivers/pinctrl/qcom/pinctrl-msm8226.c             |  156 +-
>   drivers/pinctrl/qcom/pinctrl-msm8660.c             |  252 ++-
>   drivers/pinctrl/qcom/pinctrl-msm8909.c             |  268 ++--
>   drivers/pinctrl/qcom/pinctrl-msm8916.c             |  556 ++++---
>   drivers/pinctrl/qcom/pinctrl-msm8953.c             |  424 +++---
>   drivers/pinctrl/qcom/pinctrl-msm8960.c             |  464 +++---
>   drivers/pinctrl/qcom/pinctrl-msm8976.c             |  212 ++-
>   drivers/pinctrl/qcom/pinctrl-msm8994.c             |  564 ++++---
>   drivers/pinctrl/qcom/pinctrl-msm8996.c             |  508 +++----
>   drivers/pinctrl/qcom/pinctrl-msm8998.c             |  380 +++--
>   drivers/pinctrl/qcom/pinctrl-msm8x74.c             |  474 +++---
>   drivers/pinctrl/qcom/pinctrl-qcm2290.c             |  230 ++-
>   drivers/pinctrl/qcom/pinctrl-qcs404.c              |  388 +++--
>   drivers/pinctrl/qcom/pinctrl-qdf2xxx.c             |    6 +-
>   drivers/pinctrl/qcom/pinctrl-qdu1000.c             |  249 ++-
>   drivers/pinctrl/qcom/pinctrl-sa8775p.c             |  308 ++--
>   drivers/pinctrl/qcom/pinctrl-sc7180.c              |  254 ++--
>   drivers/pinctrl/qcom/pinctrl-sc7280.c              |  322 ++--
>   drivers/pinctrl/qcom/pinctrl-sc8180x.c             |  286 ++--
>   drivers/pinctrl/qcom/pinctrl-sc8280xp.c            |  358 +++--
>   drivers/pinctrl/qcom/pinctrl-sdm660.c              |  387 +++--
>   drivers/pinctrl/qcom/pinctrl-sdm670.c              |  284 ++--
>   drivers/pinctrl/qcom/pinctrl-sdm845.c              |  286 ++--
>   drivers/pinctrl/qcom/pinctrl-sdx55.c               |  190 ++-
>   drivers/pinctrl/qcom/pinctrl-sdx65.c               |  194 ++-
>   drivers/pinctrl/qcom/pinctrl-sdx75.c               | 1601 ++++++++++++++++++++
>   drivers/pinctrl/qcom/pinctrl-sm6115.c              |  162 +-
>   drivers/pinctrl/qcom/pinctrl-sm6125.c              |  282 ++--
>   drivers/pinctrl/qcom/pinctrl-sm6350.c              |  296 ++--
>   drivers/pinctrl/qcom/pinctrl-sm6375.c              |  358 +++--
>   drivers/pinctrl/qcom/pinctrl-sm8150.c              |  286 ++--
>   drivers/pinctrl/qcom/pinctrl-sm8250.c              |  258 ++--
>   drivers/pinctrl/qcom/pinctrl-sm8350.c              |  298 ++--
>   drivers/pinctrl/qcom/pinctrl-sm8450.c              |  300 ++--
>   drivers/pinctrl/qcom/pinctrl-sm8550.c              |  320 ++--
>   49 files changed, 7763 insertions(+), 6313 deletions(-)
>   create mode 100644 Documentation/devicetree/bindings/pinctrl/qcom,sdx75-tlmm.yaml
>   create mode 100644 drivers/pinctrl/qcom/pinctrl-sdx75.c
>