Re: [RFC bluetooth-next] mac802154: add trace functionality for driver ops

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Hi,

I did not test yet, but found some review things.

On Wed, May 27, 2015 at 01:54:50PM +0530, Varka Bhadram wrote:
...
> +		__field(bool, mode)
> +	),
> +	TP_fast_assign(
> +		LOCAL_ASSIGN;
> +		__entry->mode = mode;
> +	),
> +	TP_printk(LOCAL_PR_FMT ", lbt mode: %s", LOCAL_PR_ARG,
> +		  BOOL_TO_STR(__entry->mode))
> +);
> +
> +TRACE_EVENT(802154_drv_set_short_addr,
> +	TP_PROTO(struct ieee802154_local *local, __le16 short_addr),
> +	TP_ARGS(local, short_addr),
> +	TP_STRUCT__entry(
> +		LOCAL_ENTRY
> +		__field(__le16, short_addr)
> +	),
> +	TP_fast_assign(
> +		LOCAL_ASSIGN;
> +		__entry->short_addr = short_addr;
> +	),
> +	TP_printk(LOCAL_PR_FMT ", sa: 0x%04x", LOCAL_PR_ARG,
> +		  __entry->short_addr)

care about byteorder here. you need cpu_to_le16 in the format string,
please.


> +);
> +
> +TRACE_EVENT(802154_drv_set_pan_id,
> +	TP_PROTO(struct ieee802154_local *local, __le16 pan_id),
> +	TP_ARGS(local, pan_id),
> +	TP_STRUCT__entry(
> +		LOCAL_ENTRY
> +		__field(__le16, pan_id)
> +	),
> +	TP_fast_assign(
> +		LOCAL_ASSIGN;
> +		__entry->pan_id = pan_id;
> +	),
> +	TP_printk(LOCAL_PR_FMT ", pan id: 0x%04x", LOCAL_PR_ARG,
> +		  __entry->pan_id)

same here.

> +);
> +
> +TRACE_EVENT(802154_drv_set_extended_addr,
> +	TP_PROTO(struct ieee802154_local *local, __le64 extended_addr),
> +	TP_ARGS(local, extended_addr),
> +	TP_STRUCT__entry(
> +		LOCAL_ENTRY
> +		__field(__le64, extended_addr)
> +	),
> +	TP_fast_assign(
> +		LOCAL_ASSIGN;
> +		__entry->extended_addr = extended_addr;
> +	),
> +	TP_printk(LOCAL_PR_FMT ", extended addr %llx", LOCAL_PR_ARG,
> +		  __entry->extended_addr)
> +);
> +

same here.

> +TRACE_EVENT(802154_drv_set_pan_coord,
> +	TP_PROTO(struct ieee802154_local *local, bool is_coord),
> +	TP_ARGS(local, is_coord),
> +	TP_STRUCT__entry(
> +		LOCAL_ENTRY
> +		__field(bool, is_coord)
> +	),
> +	TP_fast_assign(
> +		LOCAL_ASSIGN;
> +		__entry->is_coord = is_coord;
> +	),
> +	TP_printk(LOCAL_PR_FMT ", is_coord: %s", LOCAL_PR_ARG,
> +		  BOOL_TO_STR(__entry->is_coord))
> +);
> +

- Alex
--
To unsubscribe from this list: send the line "unsubscribe linux-wpan" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html




[Index of Archives]     [Linux NFS]     [Linux NILFS]     [Linux USB Devel]     [Linux Audio Users]     [Photo]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]

  Powered by Linux