Messages in this thread | | | Date | Thu, 1 Jun 2023 19:50:47 +0530 | Subject | Re: [PATCH V2 2/6] clk: qcom: Update l value configuration for lucid ole and evo plls | From | Jagadeesh Kona <> |
| |
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
| |