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

Re: [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>

> 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?

Apologies for the confusion. We weren't sure if your patch had been
picked up by the maintainer yet, so we included it in our series to
ensure the fix was applied along with our related changes.

> What is the point of this posting?

If it's already queued, we'll drop it from our next version and just
note it as a dependency.

Thanks,
Karthikeyan

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

[Krzysztof Kozlowski]

On 04/03/2026 01:22, 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?
> 
> Apologies for the confusion. We weren't sure if your patch had been
> picked up by the maintainer yet, so we included it in our series to
> ensure the fix was applied along with our related changes.
> 
>> What is the point of this posting?
> 
> If it's already queued, we'll drop it from our next version and just
> note it as a dependency.

Queued by which maintainer? Why your maintainer cannot just apply this
patch?

Best regards,
Krzysztof

Re: [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>
>> 
>>> 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?
>> 
>> Apologies for the confusion. We weren't sure if your patch had been
>> picked up by the maintainer yet, so we included it in our series to
>> ensure the fix was applied along with our related changes.
>> 
>>> What is the point of this posting?
>> 
>> If it's already queued, we'll drop it from our next version and just
>> note it as a dependency.
>
>Queued by which maintainer? Why your maintainer cannot just apply this
>patch?
Hi Krzysztof, 
 Since we were having a MAINTAINERs change for this architecture in this patch series, I though this patch can be applied to soc tree also directly, instead of a GIT PULL. I now understand a pull request is better, should I drop this patch and create a pull request for review ? because the MAINTAINER patch is also under review.

Thanks in advance,
Karthikeyan Mitran

[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

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