On Thu, Jun 20, 2024 at 12:21:10PM +0200, Wolfram Sang wrote: > Hi Yilun, > > > Sorry for late reply. > > No worries, thank you for the review. > > > First of all, please limit your subject/shortlog/summary within 70 > > chars. > > I'll try but it will be hard when it contains such a long function name. You don't have to include the funtion name. The code change itself explains. Thanks, Yilun > > > > There is a confusing pattern in the kernel to use a variable named 'timeout' to > > > > Please make the changelog line wrapped at less than 75 chars. > > Good point. My scripts reformat generated paragraphs now accordingly. > > > > + int ret = 0; > > > + long time_left; > > > > Please use reverse xmas tree when it is easy to follow. > > Will fix. > > All the best, > > Wolfram >
