mbox series

[v5,0/2] pinctrl: qcom: Add SM8350 pinctrl support

Message ID 20210121171747.3161543-1-vkoul@kernel.org
Headers show
Series pinctrl: qcom: Add SM8350 pinctrl support | expand

Message

Vinod Koul Jan. 21, 2021, 5:17 p.m. UTC 
This adds binding and driver for TLMM block found in SM8350 SoC

Changes in v5:
 - rebase and revise binding based on Bjorn's qcom common TLMM binding

Changes in v4:
 - rename to qcom,sm8350-tlmm along with binding and driver structs
 - fix some nits in binding pointer by Rob

Vinod Koul (2):
  dt-bindings: pinctrl: qcom: Add SM8350 pinctrl bindings
  pinctrl: qcom: Add SM8350 pinctrl driver

 .../bindings/pinctrl/qcom,sm8350-pinctrl.yaml |  146 ++
 drivers/pinctrl/qcom/Kconfig                  |    9 +
 drivers/pinctrl/qcom/Makefile                 |    1 +
 drivers/pinctrl/qcom/pinctrl-sm8350.c         | 1649 +++++++++++++++++
 4 files changed, 1805 insertions(+)
 create mode 100644 Documentation/devicetree/bindings/pinctrl/qcom,sm8350-pinctrl.yaml
 create mode 100644 drivers/pinctrl/qcom/pinctrl-sm8350.c

Comments

Bjorn Andersson Jan. 21, 2021, 5:27 p.m. UTC | #1 
On Thu 21 Jan 11:17 CST 2021, Vinod Koul wrote:

> This adds pincontrol driver for tlmm block found in SM8350 SoC
> 
> This patch is based on initial code downstream by Raghavendra.
> 

> Signed-off-by: Raghavendra Rao Ananta <rananta@codeaurora.org>

With Raghavendra's s-o-b here he should be From:, but based on the
changes you've done I don't think he has certified the origin of this
patch anymore.

So the line crediting his work above and your alone S-o-b seems more
reasonable.


For the content of the patch:

Reviewed-by: Bjorn Andersson <bjorn.andersson@linaro.org>

Regards,
Bjorn

> Signed-off-by: Jeevan Shriram <jshriram@codeaurora.org>
> Signed-off-by: Vinod Koul <vkoul@kernel.org>