* [PATCH] drm/print: cleanup coding style in drm_print.h @ 2022-09-04 20:38 ` Jingyu Wang 0 siblings, 0 replies; 5+ messages in thread From: Jingyu Wang @ 2022-09-04 20:38 UTC (permalink / raw) To: maarten.lankhorst, mripard, tzimmermann, airlied, daniel Cc: Jingyu Wang, linux-kernel, dri-devel Fix everything checkpatch.pl complained about in drm_print.h Signed-off-by: Jingyu Wang <jingyuwang_vip@163.com> --- include/drm/drm_print.h | 11 ++++++----- 1 file changed, 6 insertions(+), 5 deletions(-) diff --git a/include/drm/drm_print.h b/include/drm/drm_print.h index 22fabdeed297..4332fbeb76a8 100644 --- a/include/drm/drm_print.h +++ b/include/drm/drm_print.h @@ -1,3 +1,4 @@ +/* SPDX-License-Identifier: MIT */ /* * Copyright (C) 2016 Red Hat * @@ -103,7 +104,7 @@ __printf(2, 0) * @va: the va_list */ static inline void -drm_vprintf(struct drm_printer *p, const char *fmt, va_list *va) +drm_vprintf(struct drm_printer *p, const char *fmt, va_list * va) { struct va_format vaf = { .fmt = fmt, .va = va }; @@ -426,16 +427,16 @@ void drm_dev_dbg(const struct device *dev, enum drm_debug_category category, #define drm_info(drm, fmt, ...) \ - __drm_printk((drm), info,, fmt, ##__VA_ARGS__) + __drm_printk((drm), info, fmt, ##__VA_ARGS__) #define drm_notice(drm, fmt, ...) \ - __drm_printk((drm), notice,, fmt, ##__VA_ARGS__) + __drm_printk((drm), notice, fmt, ##__VA_ARGS__) #define drm_warn(drm, fmt, ...) \ - __drm_printk((drm), warn,, fmt, ##__VA_ARGS__) + __drm_printk((drm), warn, fmt, ##__VA_ARGS__) #define drm_err(drm, fmt, ...) \ - __drm_printk((drm), err,, "*ERROR* " fmt, ##__VA_ARGS__) + __drm_printk((drm), err, "*ERROR* " fmt, ##__VA_ARGS__) #define drm_info_once(drm, fmt, ...) \ base-commit: e47eb90a0a9ae20b82635b9b99a8d0979b757ad8 prerequisite-patch-id: f039528bc88876d6e0f64e843da089e85f6d3f58 prerequisite-patch-id: c14ed2e970be2c3f8ba746634e8161b94c148e88 -- 2.34.1 ^ permalink raw reply related [flat|nested] 5+ messages in thread
* [PATCH] drm/print: cleanup coding style in drm_print.h @ 2022-09-04 20:38 ` Jingyu Wang 0 siblings, 0 replies; 5+ messages in thread From: Jingyu Wang @ 2022-09-04 20:38 UTC (permalink / raw) To: maarten.lankhorst, mripard, tzimmermann, airlied, daniel Cc: dri-devel, linux-kernel, Jingyu Wang Fix everything checkpatch.pl complained about in drm_print.h Signed-off-by: Jingyu Wang <jingyuwang_vip@163.com> --- include/drm/drm_print.h | 11 ++++++----- 1 file changed, 6 insertions(+), 5 deletions(-) diff --git a/include/drm/drm_print.h b/include/drm/drm_print.h index 22fabdeed297..4332fbeb76a8 100644 --- a/include/drm/drm_print.h +++ b/include/drm/drm_print.h @@ -1,3 +1,4 @@ +/* SPDX-License-Identifier: MIT */ /* * Copyright (C) 2016 Red Hat * @@ -103,7 +104,7 @@ __printf(2, 0) * @va: the va_list */ static inline void -drm_vprintf(struct drm_printer *p, const char *fmt, va_list *va) +drm_vprintf(struct drm_printer *p, const char *fmt, va_list * va) { struct va_format vaf = { .fmt = fmt, .va = va }; @@ -426,16 +427,16 @@ void drm_dev_dbg(const struct device *dev, enum drm_debug_category category, #define drm_info(drm, fmt, ...) \ - __drm_printk((drm), info,, fmt, ##__VA_ARGS__) + __drm_printk((drm), info, fmt, ##__VA_ARGS__) #define drm_notice(drm, fmt, ...) \ - __drm_printk((drm), notice,, fmt, ##__VA_ARGS__) + __drm_printk((drm), notice, fmt, ##__VA_ARGS__) #define drm_warn(drm, fmt, ...) \ - __drm_printk((drm), warn,, fmt, ##__VA_ARGS__) + __drm_printk((drm), warn, fmt, ##__VA_ARGS__) #define drm_err(drm, fmt, ...) \ - __drm_printk((drm), err,, "*ERROR* " fmt, ##__VA_ARGS__) + __drm_printk((drm), err, "*ERROR* " fmt, ##__VA_ARGS__) #define drm_info_once(drm, fmt, ...) \ base-commit: e47eb90a0a9ae20b82635b9b99a8d0979b757ad8 prerequisite-patch-id: f039528bc88876d6e0f64e843da089e85f6d3f58 prerequisite-patch-id: c14ed2e970be2c3f8ba746634e8161b94c148e88 -- 2.34.1 ^ permalink raw reply related [flat|nested] 5+ messages in thread
* Re: [PATCH] drm/print: cleanup coding style in drm_print.h 2022-09-04 20:38 ` Jingyu Wang (?) @ 2022-09-05 8:45 ` Jani Nikula 2022-09-05 8:49 ` Jani Nikula -1 siblings, 1 reply; 5+ messages in thread From: Jani Nikula @ 2022-09-05 8:45 UTC (permalink / raw) To: Jingyu Wang, maarten.lankhorst, mripard, tzimmermann, airlied, daniel Cc: Jingyu Wang, linux-kernel, dri-devel On Mon, 05 Sep 2022, Jingyu Wang <jingyuwang_vip@163.com> wrote: > Fix everything checkpatch.pl complained about in drm_print.h Sorry, you just can't assume the checkpatch.pl complaints are all correct, or that it would be this trivial to fix them. This is broken. > Signed-off-by: Jingyu Wang <jingyuwang_vip@163.com> > --- > include/drm/drm_print.h | 11 ++++++----- > 1 file changed, 6 insertions(+), 5 deletions(-) > > diff --git a/include/drm/drm_print.h b/include/drm/drm_print.h > index 22fabdeed297..4332fbeb76a8 100644 > --- a/include/drm/drm_print.h > +++ b/include/drm/drm_print.h > @@ -1,3 +1,4 @@ > +/* SPDX-License-Identifier: MIT */ What's the point of having both the SPDX and full license text in the header? But unless you're the license holder, I'd be pretty careful about removing the license text. > /* > * Copyright (C) 2016 Red Hat > * > @@ -103,7 +104,7 @@ __printf(2, 0) > * @va: the va_list > */ > static inline void > -drm_vprintf(struct drm_printer *p, const char *fmt, va_list *va) > +drm_vprintf(struct drm_printer *p, const char *fmt, va_list * va) Checkpatch is just confused here. Look at all the other params, why would you add an extra space here? > { > struct va_format vaf = { .fmt = fmt, .va = va }; > > @@ -426,16 +427,16 @@ void drm_dev_dbg(const struct device *dev, enum drm_debug_category category, > > > #define drm_info(drm, fmt, ...) \ > - __drm_printk((drm), info,, fmt, ##__VA_ARGS__) > + __drm_printk((drm), info, fmt, ##__VA_ARGS__) > > #define drm_notice(drm, fmt, ...) \ > - __drm_printk((drm), notice,, fmt, ##__VA_ARGS__) > + __drm_printk((drm), notice, fmt, ##__VA_ARGS__) > > #define drm_warn(drm, fmt, ...) \ > - __drm_printk((drm), warn,, fmt, ##__VA_ARGS__) > + __drm_printk((drm), warn, fmt, ##__VA_ARGS__) > > #define drm_err(drm, fmt, ...) \ > - __drm_printk((drm), err,, "*ERROR* " fmt, ##__VA_ARGS__) > + __drm_printk((drm), err, "*ERROR* " fmt, ##__VA_ARGS__) This is macro magic, and that ,, there passes an empty arg. You remove it completely, shifting all the args to the wrong place, breaking drm error logging. BR, Jani. > > > #define drm_info_once(drm, fmt, ...) \ > > base-commit: e47eb90a0a9ae20b82635b9b99a8d0979b757ad8 > prerequisite-patch-id: f039528bc88876d6e0f64e843da089e85f6d3f58 > prerequisite-patch-id: c14ed2e970be2c3f8ba746634e8161b94c148e88 -- Jani Nikula, Intel Open Source Graphics Center ^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [PATCH] drm/print: cleanup coding style in drm_print.h 2022-09-05 8:45 ` Jani Nikula @ 2022-09-05 8:49 ` Jani Nikula 2022-09-05 10:05 ` Joe Perches 0 siblings, 1 reply; 5+ messages in thread From: Jani Nikula @ 2022-09-05 8:49 UTC (permalink / raw) To: Jingyu Wang, maarten.lankhorst, mripard, tzimmermann, airlied, daniel, Joe Perches Cc: Jingyu Wang, linux-kernel, dri-devel On Mon, 05 Sep 2022, Jani Nikula <jani.nikula@linux.intel.com> wrote: > On Mon, 05 Sep 2022, Jingyu Wang <jingyuwang_vip@163.com> wrote: >> Fix everything checkpatch.pl complained about in drm_print.h [...] >> static inline void >> -drm_vprintf(struct drm_printer *p, const char *fmt, va_list *va) >> +drm_vprintf(struct drm_printer *p, const char *fmt, va_list * va) > > Checkpatch is just confused here. Look at all the other params, why > would you add an extra space here? Joe, can you help me out here please, I can't figure out why checkpatch is complaining here: include/drm/drm_print.h:106: CHECK:SPACING: spaces preferred around that '*' (ctx:WxV) #106: FILE: include/drm/drm_print.h:106: +drm_vprintf(struct drm_printer *p, const char *fmt, va_list *va) BR, Jani. -- Jani Nikula, Intel Open Source Graphics Center ^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [PATCH] drm/print: cleanup coding style in drm_print.h 2022-09-05 8:49 ` Jani Nikula @ 2022-09-05 10:05 ` Joe Perches 0 siblings, 0 replies; 5+ messages in thread From: Joe Perches @ 2022-09-05 10:05 UTC (permalink / raw) To: Jani Nikula, Jingyu Wang, maarten.lankhorst, mripard, tzimmermann, airlied, daniel Cc: linux-kernel, dri-devel On Mon, 2022-09-05 at 11:49 +0300, Jani Nikula wrote: > On Mon, 05 Sep 2022, Jani Nikula <jani.nikula@linux.intel.com> wrote: > > On Mon, 05 Sep 2022, Jingyu Wang <jingyuwang_vip@163.com> wrote: > > > Fix everything checkpatch.pl complained about in drm_print.h > > [...] > > > > static inline void > > > -drm_vprintf(struct drm_printer *p, const char *fmt, va_list *va) > > > +drm_vprintf(struct drm_printer *p, const char *fmt, va_list * va) > > > > Checkpatch is just confused here. Look at all the other params, why > > would you add an extra space here? > > Joe, can you help me out here please, I can't figure out why checkpatch > is complaining here: > > include/drm/drm_print.h:106: CHECK:SPACING: spaces preferred around that '*' (ctx:WxV) > #106: FILE: include/drm/drm_print.h:106: > +drm_vprintf(struct drm_printer *p, const char *fmt, va_list *va) checkpatch doesn't know what a va_list is. It's similar to the FILE typedef that also causes this output. A recent link to add an exception for 'FILE' to checkpatch: https://lore.kernel.org/all/20220902111923.1488671-1-mic@digikod.net/ ^ permalink raw reply [flat|nested] 5+ messages in thread
end of thread, other threads:[~2022-09-05 10:11 UTC | newest] Thread overview: 5+ messages (download: mbox.gz follow: Atom feed -- links below jump to the message on this page -- 2022-09-04 20:38 [PATCH] drm/print: cleanup coding style in drm_print.h Jingyu Wang 2022-09-04 20:38 ` Jingyu Wang 2022-09-05 8:45 ` Jani Nikula 2022-09-05 8:49 ` Jani Nikula 2022-09-05 10:05 ` Joe Perches
This is an external index of several public inboxes, see mirroring instructions on how to clone and mirror all data and code used by this external index.