On Tue, May 02, 2023 at 06:37:41PM +0300, Jani Nikula wrote: > With CONFIG_DRM_I915_WERROR=y, we enable kernel-doc check for both > objects and headers. Now that the kernel-doc warnings have been fixed, > also enable kernel-doc -Werror to fail the build on kernel-doc warnings. > > Signed-off-by: Jani Nikula <jani.nikula@xxxxxxxxx> Reviewed-by: Rodrigo Vivi <rodrigo.vivi@xxxxxxxxx> > --- > drivers/gpu/drm/i915/Makefile | 4 ++-- > 1 file changed, 2 insertions(+), 2 deletions(-) > > diff --git a/drivers/gpu/drm/i915/Makefile b/drivers/gpu/drm/i915/Makefile > index 9af76e376ca9..f43734f13471 100644 > --- a/drivers/gpu/drm/i915/Makefile > +++ b/drivers/gpu/drm/i915/Makefile > @@ -377,7 +377,7 @@ obj-$(CONFIG_DRM_I915_GVT_KVMGT) += kvmgt.o > # > # Enable locally for CONFIG_DRM_I915_WERROR=y. See also scripts/Makefile.build > ifdef CONFIG_DRM_I915_WERROR > - cmd_checkdoc = $(srctree)/scripts/kernel-doc -none $< > + cmd_checkdoc = $(srctree)/scripts/kernel-doc -none -Werror $< > endif > > # header test > @@ -392,7 +392,7 @@ always-$(CONFIG_DRM_I915_WERROR) += \ > > quiet_cmd_hdrtest = HDRTEST $(patsubst %.hdrtest,%.h,$@) > cmd_hdrtest = $(CC) $(filter-out $(CFLAGS_GCOV), $(c_flags)) -S -o /dev/null -x c /dev/null -include $<; \ > - $(srctree)/scripts/kernel-doc -none $<; touch $@ > + $(srctree)/scripts/kernel-doc -none -Werror $<; touch $@ > > $(obj)/%.hdrtest: $(src)/%.h FORCE > $(call if_changed_dep,hdrtest) > -- > 2.39.2 >
