On Wed, May 08, 2024 at 06:19:38PM +0300, Andy Shevchenko wrote: > *-objs suffix is reserved rather for (user-space) host programs while > usually *-y suffix is used for kernel drivers (although *-objs works > for that purpose for now). > > Let's correct the old usages of *-objs in Makefiles. > > Signed-off-by: Andy Shevchenko <andriy.shevchenko@xxxxxxxxxxxxxxx> > --- > > Note, the original approach is weirdest from the existing. > Only a few drivers use this (-objs-y) one most likely by mistake. > > fs/lockd/Makefile | 9 ++++----- > 1 file changed, 4 insertions(+), 5 deletions(-) > > diff --git a/fs/lockd/Makefile b/fs/lockd/Makefile > index ac9f9d84510e..fe3e23dd29c3 100644 > --- a/fs/lockd/Makefile > +++ b/fs/lockd/Makefile > @@ -7,8 +7,7 @@ ccflags-y += -I$(src) # needed for trace events > > obj-$(CONFIG_LOCKD) += lockd.o > > -lockd-objs-y += clntlock.o clntproc.o clntxdr.o host.o svc.o svclock.o \ > - svcshare.o svcproc.o svcsubs.o mon.o trace.o xdr.o > -lockd-objs-$(CONFIG_LOCKD_V4) += clnt4xdr.o xdr4.o svc4proc.o > -lockd-objs-$(CONFIG_PROC_FS) += procfs.o > -lockd-objs := $(lockd-objs-y) > +lockd-y := clntlock.o clntproc.o clntxdr.o host.o svc.o svclock.o \ > + svcshare.o svcproc.o svcsubs.o mon.o trace.o xdr.o > +lockd-$(CONFIG_LOCKD_V4) += clnt4xdr.o xdr4.o svc4proc.o > +lockd-$(CONFIG_PROC_FS) += procfs.o > -- > 2.43.0.rc1.1336.g36b5255a03ac > Acked-by: Chuck Lever <chuck.lever@xxxxxxxxxx> Unless, of course, you'd like me to take this through the nfsd tree. -- Chuck Lever
