[PATCH linux dev-5.3 1/2] fsi: aspeed: Remove base from trace
Eddie James
eajames at linux.vnet.ibm.com
Tue Oct 15 05:38:01 AEDT 2019
On 10/14/19 8:21 AM, Joel Stanley wrote:
> The base was showing the virtual address of the FSI master, which was
> not useful for most users of the tracepoint.
Reviewed-by: Eddie James <eajames at linux.ibm.com>
Tested-by: Eddie James <eajames at linux.ibm.com>
>
> Signed-off-by: Joel Stanley <joel at jms.id.au>
> ---
> drivers/fsi/fsi-master-aspeed.c | 5 ++---
> include/trace/events/fsi_master_aspeed.h | 20 ++++++++------------
> 2 files changed, 10 insertions(+), 15 deletions(-)
>
> diff --git a/drivers/fsi/fsi-master-aspeed.c b/drivers/fsi/fsi-master-aspeed.c
> index 2048e38e7200..d796c4012875 100644
> --- a/drivers/fsi/fsi-master-aspeed.c
> +++ b/drivers/fsi/fsi-master-aspeed.c
> @@ -194,8 +194,7 @@ static u32 opb_write(void __iomem *base, uint32_t addr, uint32_t val,
>
> status = readl(base + OPB0_STATUS);
>
> - trace_fsi_master_aspeed_opb_write(base, addr, val, size,
> - status, reg);
> + trace_fsi_master_aspeed_opb_write(addr, val, size, status, reg);
>
> /* Return error when poll timed out */
> if (ret)
> @@ -231,7 +230,7 @@ static int opb_read(void __iomem *base, uint32_t addr, size_t size, u32 *out)
>
> result = readl(base + OPB0_FSI_DATA_R);
>
> - trace_fsi_master_aspeed_opb_read(base, addr, size, result,
> + trace_fsi_master_aspeed_opb_read(addr, size, result,
> readl(base + OPB0_STATUS),
> reg);
>
> diff --git a/include/trace/events/fsi_master_aspeed.h b/include/trace/events/fsi_master_aspeed.h
> index 06ff6a14bf11..63b9ce7f0de6 100644
> --- a/include/trace/events/fsi_master_aspeed.h
> +++ b/include/trace/events/fsi_master_aspeed.h
> @@ -9,10 +9,9 @@
> #include <linux/tracepoint.h>
>
> TRACE_EVENT(fsi_master_aspeed_opb_read,
> - TP_PROTO(void __iomem *base, uint32_t addr, size_t size, uint32_t result, uint32_t status, uint32_t irq_status),
> - TP_ARGS(base, addr, size, result, status, irq_status),
> + TP_PROTO(uint32_t addr, size_t size, uint32_t result, uint32_t status, uint32_t irq_status),
> + TP_ARGS(addr, size, result, status, irq_status),
> TP_STRUCT__entry(
> - __field(void *, base)
> __field(uint32_t, addr)
> __field(size_t, size)
> __field(uint32_t, result)
> @@ -20,24 +19,22 @@ TRACE_EVENT(fsi_master_aspeed_opb_read,
> __field(uint32_t, irq_status)
> ),
> TP_fast_assign(
> - __entry->base = base;
> __entry->addr = addr;
> __entry->size = size;
> __entry->result = result;
> __entry->status = status;
> __entry->irq_status = irq_status;
> ),
> - TP_printk("fsi: opb read: base %p addr %08x size %zu: result %08x status: %08x irq_status: %08x",
> - __entry->base, __entry->addr, __entry->size, __entry->result,
> + TP_printk("fsi: opb read: addr %08x size %zu: result %08x status: %08x irq_status: %08x",
> + __entry->addr, __entry->size, __entry->result,
> __entry->status, __entry->irq_status
> )
> );
>
> TRACE_EVENT(fsi_master_aspeed_opb_write,
> - TP_PROTO(void __iomem *base, uint32_t addr, uint32_t val, size_t size, uint32_t status, uint32_t irq_status),
> - TP_ARGS(base, addr, val, size, status, irq_status),
> + TP_PROTO(uint32_t addr, uint32_t val, size_t size, uint32_t status, uint32_t irq_status),
> + TP_ARGS(addr, val, size, status, irq_status),
> TP_STRUCT__entry(
> - __field(void *, base)
> __field(uint32_t, addr)
> __field(uint32_t, val)
> __field(size_t, size)
> @@ -45,15 +42,14 @@ TRACE_EVENT(fsi_master_aspeed_opb_write,
> __field(uint32_t, irq_status)
> ),
> TP_fast_assign(
> - __entry->base = base;
> __entry->addr = addr;
> __entry->val = val;
> __entry->size = size;
> __entry->status = status;
> __entry->irq_status = irq_status;
> ),
> - TP_printk("fsi: opb write: base %p addr %08x val %08x size %zu status: %08x irq_status: %08x",
> - __entry->base, __entry->addr, __entry->val, __entry->size,
> + TP_printk("fsi: opb write: addr %08x val %08x size %zu status: %08x irq_status: %08x",
> + __entry->addr, __entry->val, __entry->size,
> __entry->status, __entry->irq_status
> )
> );
More information about the openbmc
mailing list