Hi Paul,
Just a couple of nits:
On Thu, 27 Jul 2023 20:37:00 -0700 "Paul E. McKenney" <[email protected]> wrote:
>
> [ sfr: Apply kernel test robot feedback. ]
This was a fix for my own build testing (I am not a bot (yet) :-)).
> diff --git a/include/linux/init.h b/include/linux/init.h
> index 266c3e1640d4..29e75bbe7984 100644
> --- a/include/linux/init.h
> +++ b/include/linux/init.h
> @@ -112,6 +112,7 @@
> #define __REFCONST .section ".ref.rodata", "a"
>
> #ifndef __ASSEMBLY__
> +
Please remove this added blank line.
--
Cheers,
Stephen Rothwell
On Fri, Jul 28, 2023 at 02:06:35PM +1000, Stephen Rothwell wrote:
> Hi Paul,
>
> Just a couple of nits:
>
> On Thu, 27 Jul 2023 20:37:00 -0700 "Paul E. McKenney" <[email protected]> wrote:
> >
> > [ sfr: Apply kernel test robot feedback. ]
>
> This was a fix for my own build testing (I am not a bot (yet) :-)).
Ah!
This says "Stephen Rothwell made a change in response to kernel test robot
feedback". I saw your change and saw the kernel test robot complaint,
and gave the robot the benefit of the doubt.
But if you saw this build bug yourself and object to crediting kernel
test robot for also spotting it, please let me know and I will remove
this line.
> > diff --git a/include/linux/init.h b/include/linux/init.h
> > index 266c3e1640d4..29e75bbe7984 100644
> > --- a/include/linux/init.h
> > +++ b/include/linux/init.h
> > @@ -112,6 +112,7 @@
> > #define __REFCONST .section ".ref.rodata", "a"
> >
> > #ifndef __ASSEMBLY__
> > +
>
> Please remove this added blank line.
Good eyes, done!
I should also split out the modifications to include/linux/init.h,
which are now whitespace-only, shouldn't I?
Thanx, Paul