Re: [PATCH V2 2/6] clk: qcom: Update l value configuration for lucid ole and evo plls

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Hi Bryan,

Thanks for your review!

On 5/26/2023 6:52 PM, Bryan O'Donoghue wrote:
On 25/05/2023 18:21, Jagadeesh Kona wrote:
-    .config_ctl_hi1_val = 0x32AA299C,
+    .config_ctl_hi1_val = 0x32aa299c,

You should separate out this change from this patch.

There's nothing wrong with having a preliminary patch to change all upper-case HEX to lowercase hex.

You should do that since the log here "update l value" != change UCASE to lcase hex.

Sure, will remove this in the next series.
---
0xb0d

Thanks & Regards,
Jagadeesh



[Index of Archives]     [Linux ARM Kernel]     [Linux ARM]     [Linux Omap]     [Fedora ARM]     [Linux for Sparc]     [IETF Annouce]     [Security]     [Bugtraq]     [Linux MIPS]     [ECOS]     [Asterisk Internet PBX]     [Linux API]

  Powered by Linux