[PATCH v3 0/2] Update Axiado reviewers and ax3000.dtsi fix

[PATCH v3 0/2] Update Axiado reviewers and ax3000.dtsi fix

[Karthikeyan Mitran]

----
This series updates Axiado SoC maintainership and fixes devicetree formatting.

A maintainer has moved from the project, so we are adding three new maintainers
(Prasad, Tzu-Hao, and Karthikeyan) along with a reviewer group for broader
coverage.

Patch 1: Removed one and added new maintainers and reviewer group for Axiado SoC
Patch 2: Convert ax3000.dtsi hex values to lowercase

Changes since v2:
- Updated commit message to clarify maintainer additions
- v2: https://lore.kernel.org/linux-devicetree/20260220044550.4152148-1-kmitran@axiado.com/

Signed-off-by: Karthikeyan Mitran <kmitran@axiado.com>
---
---
Krzysztof Kozlowski (1):
      arm64: dts: axiado: Use lowercase hex

Prasad Bolisetty (1):
      MAINTAINERS: Update Axiado reviewer

 MAINTAINERS                            | 5 ++++-
 arch/arm64/boot/dts/axiado/ax3000.dtsi | 2 +-
 2 files changed, 5 insertions(+), 2 deletions(-)
---
base-commit: 8f0b4cce4481fb22653697cced8d0d04027cb1e8
change-id: 20260226-maintainers-addition-and-axiado-ax3000_dtsi-update-2f37e16b9053

Best regards,
-- 
Karthikeyan Mitran <kmitran@axiado.com>

[PATCH v3 1/2] arm64: dts: axiado: Use lowercase hex

[Karthikeyan Mitran]

From: Krzysztof Kozlowski <krzysztof.kozlowski@oss.qualcomm.com>

The DTS code coding style expects lowercase hex for values and unit
addresses.

Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@oss.qualcomm.com>
Reviewed-by: Tzu-Hao Wei <twei@axiado.com>
Signed-off-by: Tzu-Hao Wei <twei@axiado.com>
Signed-off-by: Karthikeyan Mitran <kmitran@axiado.com>
---
 arch/arm64/boot/dts/axiado/ax3000.dtsi | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/arch/arm64/boot/dts/axiado/ax3000.dtsi b/arch/arm64/boot/dts/axiado/ax3000.dtsi
index 792f52e0c7dd..d515f3141529 100644
--- a/arch/arm64/boot/dts/axiado/ax3000.dtsi
+++ b/arch/arm64/boot/dts/axiado/ax3000.dtsi
@@ -480,7 +480,7 @@ uart0: serial@80520000 {
 
 		uart1: serial@805a0000 {
 			compatible = "axiado,ax3000-uart", "cdns,uart-r1p12";
-			reg = <0x00 0x805A0000 0x00 0x100>;
+			reg = <0x00 0x805a0000 0x00 0x100>;
 			interrupt-parent = <&gic500>;
 			interrupts = <GIC_SPI 113 IRQ_TYPE_LEVEL_HIGH>;
 			clock-names = "uart_clk", "pclk";

-- 
2.34.1

[PATCH v3 2/2] MAINTAINERS: Update Axiado reviewer

[Karthikeyan Mitran]

From: Prasad Bolisetty <pbolisetty@axiado.com>

Adding 3 new maintainers Prasad,Tzu-Hao, and Karthikeyan
also adding a group reviewer entry for review coverage.

Signed-off-by: Prasad Bolisetty <pbolisetty@axiado.com>
Acked-by: Harshit Shah <hshah@axiado.com>
Signed-off-by: Tzu-Hao Wei <twei@axiado.com>
Signed-off-by: Karthikeyan Mitran <kmitran@axiado.com>
---
 MAINTAINERS | 5 ++++-
 1 file changed, 4 insertions(+), 1 deletion(-)

diff --git a/MAINTAINERS b/MAINTAINERS
index 5b11839cba9d..52f1228bf137 100644
--- a/MAINTAINERS
+++ b/MAINTAINERS
@@ -2554,7 +2554,10 @@ F:	arch/arm/mach-aspeed/
 N:	aspeed
 
 ARM/AXIADO ARCHITECTURE
-M:	Harshit Shah <hshah@axiado.com>
+M:	Prasad Bolisetty <pbolisetty@axiado.com>
+M:	Tzu-Hao Wei <twei@axiado.com>
+M:	Karthikeyan Mitran <kmitran@axiado.com>
+R:	Axiado Reviewers <linux-maintainer@axiado.com>
 L:	linux-arm-kernel@lists.infradead.org (moderated for non-subscribers)
 S:	Maintained
 F:	Documentation/devicetree/bindings/arm/axiado.yaml

-- 
2.34.1

Re: [PATCH v3 2/2] MAINTAINERS: Update Axiado reviewer

[Krzysztof Kozlowski]

On 27/02/2026 01:31, Karthikeyan Mitran wrote:
> From: Prasad Bolisetty <pbolisetty@axiado.com>
> 
> Adding 3 new maintainers Prasad,Tzu-Hao, and Karthikeyan
> also adding a group reviewer entry for review coverage.

Same comments as before. You are doing much more in the patch which is
not explained at all.

You received comments, so respond or implement them, not just send the same.


Best regards,
Krzysztof

Re: [PATCH v3 1/2] arm64: dts: axiado: Use lowercase hex

[Krzysztof Kozlowski]

On 27/02/2026 01:31, Karthikeyan Mitran wrote:
> From: Krzysztof Kozlowski <krzysztof.kozlowski@oss.qualcomm.com>
> 
> The DTS code coding style expects lowercase hex for values and unit
> addresses.
> 
> Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@oss.qualcomm.com>
> Reviewed-by: Tzu-Hao Wei <twei@axiado.com>
> Signed-off-by: Tzu-Hao Wei <twei@axiado.com>
> Signed-off-by: Karthikeyan Mitran <kmitran@axiado.com>

It is not a big deal, but I just don't understand what are you doing
here with my patch. I sent it, it got reviewed, so why are you posting
it again for review?

What is the point of this posting?

Best regards,
Krzysztof

Re: [PATCH v3 2/2] MAINTAINERS: Update Axiado reviewer

[Karthikeyan Mitran]

On 27/02/2026 01:31, [Reviewer Name] wrote:
> Same comments as before. You are doing much more in the patch which is
> not explained at all.

I did explain this in the cover letter:
"A maintainer has moved from the project, so we are adding three new
maintainers (Prasad, Tzu-Hao, and Karthikeyan) along with a reviewer
group for broader coverage."

The outgoing maintainer also provided their Acked-by in the commit.

You're right that this should be in the commit message, not just the
cover letter. I understand the heightened scrutiny of maintainer changes
post-XZ, which is why proper documentation is important.

> You received comments, so respond or implement them, not just send the same.

I've sent v4 with the full explanation moved into the commit message so
it's part of the permanent git history.

Thanks,
Karthikeyan Mitran