On 2021/6/7 17:06, Andy Shevchenko wrote: > On Mon, Jun 7, 2021 at 11:56 AM Leizhen (ThunderTown) > <thunder.leizhen@xxxxxxxxxx> wrote: >> On 2021/6/7 16:52, Leizhen (ThunderTown) wrote: >>> On 2021/6/7 16:39, Andy Shevchenko wrote: >>>> On Mon, Jun 7, 2021 at 6:21 AM Zhen Lei <thunder.leizhen@xxxxxxxxxx> wrote: >>>> >>>>> Fix some spelling mistakes in comments: >>>>> thats ==> that's >>>>> unitialized ==> uninitialized >>>>> panicing ==> panicking >>>>> sucess ==> success >>>>> possitive ==> positive >>>>> intepreted ==> interpreted >>>> >>>> Thanks for the fix! Is it done with the help of the codespell tool? If >>>> not, can you run it and check if it suggests more fixes? >>> >>> Yes, it's detected by codespell tool. But to avoid too many changes in one patch, I tried >>> breaking it down into smaller patches(If it can be classified) to make it easier to review. >>> In fact, the other patch I just posted included the rest. >> >> https://lkml.org/lkml/2021/6/7/151 >> >> All the remaining spelling mistakes are fixed by the patch above. I can combine the two of >> them into one patch if you think it's necessary. > > No, it's good to keep them split. What I meant is to use the tool > against the same subset of the files you have done your patch for. But > please mention in the commit message that you have used that tool, so > reviewers will not waste time on the comments like mine. Okay, thanks for the advice. > >
