mbox series

[v5,0/2] arm64: dts: qcom: pwm: Drop PWM reg dependency + update MAINTAINERS

Message ID 20220927235108.227566-1-bryan.odonoghue@linaro.org
Headers show
Series arm64: dts: qcom: pwm: Drop PWM reg dependency + update MAINTAINERS | expand

Message

Bryan O'Donoghue Sept. 27, 2022, 11:51 p.m. UTC 
V5:
- Resend the yaml fix
- Update MAINTAINERS to mark Lee as Maintainer not Supporter of the YAML
  files for MFD.
  I rely on scripts/get_mainter.pl to tell me who needs to be mailed and
  who doesn't, given the YAML change has to go through Lee, the MAINTAINERS
  file should make that show up in get_maintainers.pl

V4:
- Churn patch#1 prefix to "dt-bindings: mfd: qcom,spmi-pmic:" - Krzysztof
- Add's Krzysztof's RB as indicated

V3:
- Splits dtsi and yaml
- Uses Krzysztof's suggested commit log in the yaml

V2:
The accompanying patch removes reg = <> and pwm@reg from the yaml and dtsi.
This follows on from discussions between Bupesh, Dmitry, Bjorn, Krzysztof and myself.

https://lore.kernel.org/all/20220719205058.1004942-1-bhupesh.sharma@linaro.org/
https://lore.kernel.org/all/20220721195502.1525214-1-bhupesh.sharma@linaro.org/
https://lore.kernel.org/all/20220822120300.2633790-1-bryan.odonoghue@linaro.org/

The previous discussion tended towards either removing pwm@reg and reg = <> or
extending out the yaml to support multiple reg declarations for PWM compatible.

This patch does the former. I've left node: label in place, dropped both pwm@reg
and reg = <> I kept "label: nodename" though because it looked more like what we
already have for rpm regulators.

Per our previous discussion I've modified the yaml and dtsi in one go.

Bryan O'Donoghue (2):
  dt-bindings: mfd: qcom,spmi-pmic: Drop PWM reg dependency
  MAINTAINERS: Mark Lee as Maintainer not Supporter for MFD yaml

 Documentation/devicetree/bindings/mfd/qcom,spmi-pmic.yaml | 8 ++++----
 MAINTAINERS                                               | 2 +-
 2 files changed, 5 insertions(+), 5 deletions(-)

Comments

Bryan O'Donoghue Sept. 28, 2022, 12:01 a.m. UTC | #1 
On 28/09/2022 00:51, Bryan O'Donoghue wrote:
> Running scripts/get_maintainer.pl on
> Documentation/devicetree/bindings/mfd/qcom,spmi-pmic.yaml gives Andy Gross,
> Bjorn Andersson, Rob Herring and Krzysztof Kozlowski but I'm told my yaml
> changes needs to go through Lee.
> 
> Change up the MAINTAINERS file so that get_maintainers.pl includes Lee as
> Maintainer in the output list as per
> Documentation/process/submitting-patches.rst.
> 
> Signed-off-by: Bryan O'Donoghue <bryan.odonoghue@linaro.org>
> ---
>   MAINTAINERS | 2 +-
>   1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/MAINTAINERS b/MAINTAINERS
> index 936490dcc97b6..ee3bafe9a08d9 100644
> --- a/MAINTAINERS
> +++ b/MAINTAINERS
> @@ -13905,7 +13905,7 @@ F:	drivers/media/i2c/mt9v111.c
>   
>   MULTIFUNCTION DEVICES (MFD)
>   M:	Lee Jones <lee@kernel.org>
> -S:	Supported
> +S:	Maintained
>   T:	git git://git.kernel.org/pub/scm/linux/kernel/git/lee/mfd.git
>   F:	Documentation/devicetree/bindings/mfd/
>   F:	drivers/mfd/

Hmm.

Actually maybe Documentation/process/submitting-patches.rst should be 
updated to state that anybody appearing as "supported" in 
get_maintainers.pl should be mailed, instead.

I'll V6 the yaml standalone and propose a change to 
Documentation/process/submitting-patches.rst separately.

---
bod