On Sat, May 06, 2023 at 06:52:09AM -0700, Guenter Roeck wrote: > There is a lot of nnecessary churn in your patches, where you reformat entire > paragrahps just to add a few "``". This makes it almost impossible to identify > the actual changes. For me it means that the review burden is so much that I'll > probably never get to actually review those changes. > > Please split the changes into subjects, following the "one logical change per > patch" rule, and please refrain from changing entire paragraphs when adding > markups. Also, a reference to the markup rules explaining if and when markups > are warranted would be nice. For example, I have no idea why "hwmon" would > warrant a markup. > > Guenter > Ack. Thanks for this. It's the sort of feedback I was hoping to receive by submitting as an RFC series. James
