On Tue, Jul 19, 2022 at 08:22:15PM +0200, Alexander Gordeev wrote: > On Fri, Jul 15, 2022 at 12:36:10PM +0800, Jason Wang wrote: > > The double `that' is duplicated in line 19, remove one. > > > > Signed-off-by: Jason Wang <wangborong@xxxxxxxxxx> > > --- > > arch/s390/lib/delay.c | 2 +- > > 1 file changed, 1 insertion(+), 1 deletion(-) > > > > diff --git a/arch/s390/lib/delay.c b/arch/s390/lib/delay.c > > index f7f5adea8940..c32bc8f7c1fd 100644 > > --- a/arch/s390/lib/delay.c > > +++ b/arch/s390/lib/delay.c > > @@ -16,7 +16,7 @@ void __delay(unsigned long loops) > > /* > > * To end the bloody studid and useless discussion about the > > * BogoMips number I took the liberty to define the __delay > > - * function in a way that that resulting BogoMips number will > > + * function in a way that resulting BogoMips number will > > * yield the megahertz number of the cpu. The important function > > * is udelay and that is done using the tod clock. -- martin. > > */ > > Applied, thanks! Ahem :) That should have been actually "fixed" to "that the", instead of removing the double "that". And by the way, just two lines above is another obvious typo... But seriously, the whole comment is entirely outdated and simply wrong. We don't use __delay() to calculate the BogoMips number, nor does the function return a value that correlates with the megahertz number of the cpu nowadays. It just loops a random time (where the cpu is free to detect the pointless loop and optimize it away anyway). So the right thing to do would be to remove the complete comment, and maybe add a new comment - but please don't apply this patch.
