Messages in this thread | | | From | Mathieu Poirier <> | Date | Wed, 4 Mar 2020 10:13:45 -0700 | Subject | Re: [PATCH] coresight: cti: Remove unnecessary NULL check in cti_sig_type_name |
| |
On Mon, 2 Mar 2020 at 14:34, Nathan Chancellor <natechancellor@gmail.com> wrote: > > Clang warns: > > drivers/hwtracing/coresight/coresight-cti-sysfs.c:948:11: warning: > address of array 'grp->sig_types' will always evaluate to 'true' > [-Wpointer-bool-conversion] > if (grp->sig_types) { > ~~ ~~~~~^~~~~~~~~ > 1 warning generated. > > sig_types is at the end of a struct so it cannot be NULL. > > Fixes: 85b6684eab65 ("coresight: cti: Add connection information to sysfs") > Link: https://github.com/ClangBuiltLinux/linux/issues/914 > Signed-off-by: Nathan Chancellor <natechancellor@gmail.com> > --- > drivers/hwtracing/coresight/coresight-cti-sysfs.c | 6 ++---- > 1 file changed, 2 insertions(+), 4 deletions(-) > > diff --git a/drivers/hwtracing/coresight/coresight-cti-sysfs.c b/drivers/hwtracing/coresight/coresight-cti-sysfs.c > index abb7f492c2cb..214d6552b494 100644 > --- a/drivers/hwtracing/coresight/coresight-cti-sysfs.c > +++ b/drivers/hwtracing/coresight/coresight-cti-sysfs.c > @@ -945,10 +945,8 @@ cti_sig_type_name(struct cti_trig_con *con, int used_count, bool in) > int idx = 0; > struct cti_trig_grp *grp = in ? con->con_in : con->con_out; > > - if (grp->sig_types) { > - if (used_count < grp->nr_sigs) > - idx = grp->sig_types[used_count]; > - } > + if (used_count < grp->nr_sigs) > + idx = grp->sig_types[used_count]; > return sig_type_names[idx]; > }
Applied - thanks, Mathieu
> > -- > 2.25.1 >
| |