Re: [PATCH v6 5/8] usb: typec: cros_ec_ucsi: Add trace events

Łukasz Bartosik
2024-09-19T18:10:58Z

On Thu, Sep 19, 2024 at 2:53 PM Heikki Krogerus
<heikki.krogerus@linux.intel.com> wrote:
>
> Hi Łukasz,
>
> > > This does not look cros_ec specific. Could you check if this could be
> > > made part of the ucsi core tracepoints?
> > >
> >
> > Good point. I will look into it.
> >
> > > I can also look into this more carefully, and throw ideas if you like.
> > > Let me know.
> > >
> >
> > I would definitely like to hear your ideas on the topic :).
>
> This information is more or less the same that you would like to get
> from that netlink interface, right? If that's the case, then is there
> some reason why you want to get the same information from two
> different interfaces? Sorry if I'm missing something obvious.
>

I find both trace events and netlink useful. I elaborated on the cases
for which netlink seems more appropriate in my other reply.

Thanks,
Lukasz

> thanks,
>
> --
> heikki