Skip to contentSkip to conversation

thermal: Remove Amit Kucheria from MAINTAINERS

PATCH Superseded Rafael J. Wysocki 6 participants 8 messages Oct 6, 2023 – Oct 7, 2023

Subsystems (2)

v1 of 3 Superseded Latest is v3 →

lore
Share & tools
lore
#0 lore raw

PATCH 1/1 thermal: Remove Amit Kucheria from MAINTAINERS

From: Rafael J. Wysocki <[email protected]>

Amit Kucheria has not been participating in kernel development in any way or form for quite some time, so it is not useful to list him as a designated reviewer for the thermal subsystem or as the maintainer of the thermal zone device bindings.

Remove him from those two places accordingly.

---
 Documentation/devicetree/bindings/thermal/thermal-zones.yaml |    3 ---
 MAINTAINERS                                                  |    1 -
 2 files changed, 4 deletions(-)
Documentation/devicetree/bindings/thermal/thermal-zones.yaml +0 -3
Index: linux-pm/Documentation/devicetree/bindings/thermal/thermal-zones.yaml
===================================================================
@@ -7,9 +7,6 @@ $schema: http://devicetree.org/meta-sche
 
 title: Thermal zone
 
-maintainers:
-  - Amit Kucheria <[email protected]>
-
 description: |
   Thermal management is achieved in devicetree by describing the sensor hardware
   and the software abstraction of cooling devices and thermal zones required to
MAINTAINERS +0 -1
Index: linux-pm/MAINTAINERS
===================================================================
@@ -21363,7 +21363,6 @@ F:	drivers/media/radio/radio-raremono.c
 THERMAL
 M:	Rafael J. Wysocki <[email protected]>
 M:	Daniel Lezcano <[email protected]>
-R:	Amit Kucheria <[email protected]>
 R:	Zhang Rui <[email protected]>
 L:	[email protected]
 S:	Supported
maintainer
#1 lore raw
Quoted 7 lines · Rafael J. Wysocki

From: Rafael J. Wysocki <[email protected]>

Amit Kucheria has not been participating in kernel development in any way or form for quite some time, so it is not useful to list him as a designated reviewer for the thermal subsystem or as the maintainer of the thermal zone device bindings.

Quoted 8 lines

Remove him from those two places accordingly.

---
 Documentation/devicetree/bindings/thermal/thermal-zones.yaml |    3 ---
 MAINTAINERS                                                  |    1 -
 2 files changed, 4 deletions(-)

My bot found errors running 'make DT_CHECKER_FLAGS=-m dt_binding_check' on your patch (DT_CHECKER_FLAGS is new in v5.13):

yamllint warnings/errors:

dtschema/dtc warnings/errors:
/builds/robherring/dt-review-ci/linux/Documentation/devicetree/bindings/thermal/thermal-zones.yaml: 'maintainers' is a required property
hint: Metaschema for devicetree binding documentation
from schema $id: http://devicetree.org/meta-schemas/base.yaml#

doc reference errors (make refcheckdocs):

See https://patchwork.ozlabs.org/project/devicetree-bindings/patch/5716404.DvuYhMxLoT@kreacher

The base for the series is generally the latest rc1. A different dependency should be noted in *this* patch.

If you already ran 'make dt_binding_check' and didn't see the above error(s), then make sure 'yamllint' is installed and dt-schema is up to date:

pip3 install dtschema --upgrade

Please check and re-submit after running the above command yourself. Note that DT_SCHEMA_FILES can be set to your schema file to speed up checking your schema. However, it must be unset to test all examples with your schema.

Acked-by on v1 1/1
#2 lore raw
In reply to Rafael J. Wysocki
Quoted 7 lines · Rafael J. Wysocki

From: Rafael J. Wysocki <[email protected]>

Amit Kucheria has not been participating in kernel development in any way or form for quite some time, so it is not useful to list him as a designated reviewer for the thermal subsystem or as the maintainer of the thermal zone device bindings.

Quoted 5 lines

Remove him from those two places accordingly.

---
 Documentation/devicetree/bindings/thermal/thermal-zones.yaml |    3 ---

Best regards,
Krzysztof

#3 lore raw
Quoted 8 lines · Krzysztof Kozlowski
Quoted 7 lines · Rafael J. Wysocki

From: Rafael J. Wysocki <[email protected]>

Amit Kucheria has not been participating in kernel development in any way or form for quite some time, so it is not useful to list him as a designated reviewer for the thermal subsystem or as the maintainer of the thermal zone device bindings.

Quoted 7 lines
Quoted 5 lines

Remove him from those two places accordingly.

---
 Documentation/devicetree/bindings/thermal/thermal-zones.yaml |    3 ---

and unAcked. We need a maintainer for the bindings. Someone else from thermal?

Best regards,
Krzysztof

reviewer
#4 lore raw
Quoted 11 lines · Krzysztof Kozlowski
Quoted 10 lines · Krzysztof Kozlowski
Quoted 9 lines · Rafael J. Wysocki

From: Rafael J. Wysocki <[email protected]>

Amit Kucheria has not been participating in kernel development in any way or form for quite some time, so it is not useful to list him as a designated reviewer for the thermal subsystem or as the maintainer of the thermal zone device bindings.

Remove him from those two places accordingly.

Quoted 9 lines
Quoted 5 lines
Quoted 3 lines
---
  Documentation/devicetree/bindings/thermal/thermal-zones.yaml |    3 ---

and unAcked. We need a maintainer for the bindings. Someone else from thermal?

I'm going to handle the review in thermal subsystem. Although, I forgot about this 'binding' thing...

Daniel, what do you think?

#5 lore raw
Quoted 20 lines · Lukasz Luba
Quoted 17 lines · Krzysztof Kozlowski
Quoted 15 lines · Krzysztof Kozlowski
Quoted 7 lines · Rafael J. Wysocki

From: Rafael J. Wysocki <[email protected]>

Amit Kucheria has not been participating in kernel development in any way or form for quite some time, so it is not useful to list him as a designated reviewer for the thermal subsystem or as the maintainer of the thermal zone device bindings.

Quoted 5 lines

Remove him from those two places accordingly.

---
  Documentation/devicetree/bindings/thermal/thermal-zones.yaml | 3 ---

Quoted 8 lines
Quoted 3 lines

and unAcked. We need a maintainer for the bindings. Someone else from thermal?

I'm going to handle the review in thermal subsystem. Although, I forgot about this 'binding' thing...

Daniel, what do you think?

I can handle the bindings, I rewrote the thermal-of code and worked with Amit on the txt to yaml conversion.

Signature · 5 lines
<http://www.linaro.org/> Linaro.org │ Open source software for ARM SoCs

Follow Linaro:  <http://www.facebook.com/pages/Linaro> Facebook |
<http://twitter.com/#!/linaroorg> Twitter |
<http://www.linaro.org/linaro-blog/> Blog
#6 lore raw
Quoted 27 lines · Daniel Lezcano
Quoted 25 lines · Lukasz Luba
Quoted 21 lines · Krzysztof Kozlowski
Quoted 8 lines · Krzysztof Kozlowski
Quoted 7 lines · Rafael J. Wysocki

From: Rafael J. Wysocki <[email protected]>

Amit Kucheria has not been participating in kernel development in any way or form for quite some time, so it is not useful to list him as a designated reviewer for the thermal subsystem or as the maintainer of the thermal zone device bindings.

Quoted 8 lines
Quoted 6 lines

Remove him from those two places accordingly.

---
Documentation/devicetree/bindings/thermal/thermal-zones.yaml |
3 ---

and unAcked. We need a maintainer for the bindings. Someone else from thermal?

Quoted 7 lines
Quoted 4 lines

I'm going to handle the review in thermal subsystem. Although, I forgot about this 'binding' thing...

Daniel, what do you think?

I can handle the bindings, I rewrote the thermal-of code and worked with Amit on the txt to yaml conversion.

Sounds good!

I'll send a v3 of the patch then with this change included, please ACK it.

#7 lore raw
In reply to Krzysztof Kozlowski
Quoted 12 lines · Krzysztof Kozlowski
Quoted 10 lines · Krzysztof Kozlowski
Quoted 9 lines · Rafael J. Wysocki

From: Rafael J. Wysocki <[email protected]>

Amit Kucheria has not been participating in kernel development in any way or form for quite some time, so it is not useful to list him as a designated reviewer for the thermal subsystem or as the maintainer of the thermal zone device bindings.

Remove him from those two places accordingly.

Quoted 7 lines
Quoted 5 lines
Quoted 3 lines
---
 Documentation/devicetree/bindings/thermal/thermal-zones.yaml |    3 ---

and unAcked. We need a maintainer for the bindings.

Well, yes, we do, but how useful is it to hold on to the stale record?

Surely, it doesn't help anyone.

Quoted 1 line

Someone else from thermal?