<div dir="auto"><div><br><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Mon, Nov 29, 2021, 9:05 AM Ferruh Yigit <<a href="mailto:ferruh.yigit@intel.com">ferruh.yigit@intel.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">On 11/29/2021 1:59 PM, Bruce Richardson wrote:<br>
> On Mon, Nov 29, 2021 at 12:49:56PM +0000, Ferruh Yigit wrote:<br>
>> Hi Josh, Thanks for the patch.<br>
>><br>
>> The typo fixes for comments and logs are more straightforward, but I have<br>
>> mixed feeling about the variable / macro typo fixes, let's see what others<br>
>> think.<br>
>><br>
>> And I think we should get this patch either one of the first patch or last<br>
>> patch to prevent conflict with other patches.<br>
> <br>
> +1 to being a first patch to start things off clean.<br>
> <br>
> For the macros, since there is quick a bit of content in the patch, could<br>
> we have the patch split into a separate patch for macro changes to review<br>
> separately, allowing the simple doc/printf spelling changes to go in<br>
> immediately?<br></blockquote></div></div><div dir="auto"><br></div><div dir="auto">Yeah, this is doable. I'll see if I can do it now. It's a lot of files, plus I'll need to figure out the command sequence to reply to this thread. If someone is available on Slack for some handholding, that might help me get it done faster.</div><div dir="auto"><br></div><div dir="auto"><div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Also as far as I can see there are multiple build errors related to the<br>
variable / macro renames, which was the concern.<br></blockquote></div></div><div dir="auto"><br></div><div dir="auto">I'm used to having access to GitHub based CI that let me test before making a PR. I've locally dropped the peer change, but didn't resubmit since I figured I'd wait for replies.</div><div dir="auto"><br></div><div dir="auto">(Last night I installed meson and ninja, so I might be able to do a local build.)</div><div dir="auto"><div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
</blockquote></div></div></div>