On Tue, Apr 13, 2021 at 8:59 PM Thierry Reding <thierry.reding@xxxxxxxxx> wrote:
>
> On Mon, Apr 12, 2021 at 07:20:40PM -0300, Aline Santana Cordeiro wrote:
> > Align line break to match with the open parenthesis.
> > Issue detected by checkpatch.pl.
> > It consequently solved a few end lines with a '(',
> > issue also detected by checkpatch.pl
> >
> > Signed-off-by: Aline Santana Cordeiro <alinesantanacordeiro@xxxxxxxxx>
> > ---
> > drivers/staging/media/tegra-vde/trace.h | 111 ++++++++++++++------------------
> > 1 file changed, 50 insertions(+), 61 deletions(-)
>
> Ugh... I'd say this is one case where checkpatch.pl really shouldn't be
> complaining since this isn't a function call or declaration. It's more
> like a code snippet written with macros, so I don't think the same rules
> should apply.
>
> Adding checkpatch folks (hence quoting in full): is there anything we
> can do about this without too much effort? Or should we just accept this
> the way it is?
>
While it may be possible to add exceptions for trace headers on the
alignment rules, I don't know how many more such exceptions we will
end up adding. Such fine grained checks might be unnecessarily complex.
So I think we should just accept the way it is for now.
Joe might have a different opinion?
Thanks,
Dwaipayan.
> Thierry
>
> > diff --git a/drivers/staging/media/tegra-vde/trace.h b/drivers/staging/media/tegra-vde/trace.h
> > index e571410..1fcc573 100644
> > --- a/drivers/staging/media/tegra-vde/trace.h
> > +++ b/drivers/staging/media/tegra-vde/trace.h
> > @@ -11,79 +11,68 @@
> > #include "vde.h"
> >
> > DECLARE_EVENT_CLASS(register_access,
> > - TP_PROTO(struct tegra_vde *vde, void __iomem *base,
> > - u32 offset, u32 value),
> > - TP_ARGS(vde, base, offset, value),
> > - TP_STRUCT__entry(
> > - __string(hw_name, tegra_vde_reg_base_name(vde, base))
> > - __field(u32, offset)
> > - __field(u32, value)
> > - ),
> > - TP_fast_assign(
> > - __assign_str(hw_name, tegra_vde_reg_base_name(vde, base));
> > - __entry->offset = offset;
> > - __entry->value = value;
> > - ),
> > - TP_printk("%s:0x%03x 0x%08x", __get_str(hw_name), __entry->offset,
> > - __entry->value)
> > + TP_PROTO(struct tegra_vde *vde, void __iomem *base,
> > + u32 offset, u32 value),
> > + TP_ARGS(vde, base, offset, value),
> > + TP_STRUCT__entry(__string(hw_name, tegra_vde_reg_base_name(vde, base))
> > + __field(u32, offset)
> > + __field(u32, value)
> > + ),
> > + TP_fast_assign(__assign_str(hw_name,
> > + tegra_vde_reg_base_name(vde, base));
> > + __entry->offset = offset;
> > + __entry->value = value;
> > + ),
> > + TP_printk("%s:0x%03x 0x%08x", __get_str(hw_name), __entry->offset,
> > + __entry->value)
> > );
> >
