[Skiboot] [PATCH V2 2/5] hw/bt: Convert retry_count to send_count

[Alistair Popple]

Looks the same as the last version, which was ack'd:

Acked-by: Alistair Popple <alistair at popple.id.au>

On Thu, 12 Nov 2015 17:22:56 Cyril Bur wrote:
> It is useful to know if a message has been sent at all, the counter used to
> retry sending can be used for this.
> 
> Signed-off-by: Cyril Bur <cyril.bur at au1.ibm.com>
> ---
>  hw/bt.c | 14 ++++++++------
>  1 file changed, 8 insertions(+), 6 deletions(-)
> 
> diff --git a/hw/bt.c b/hw/bt.c
> index d4ba4c6..253ca2a 100644
> --- a/hw/bt.c
> +++ b/hw/bt.c
> @@ -67,7 +67,7 @@
>  /*
>   * Maximum number of times to attempt sending a message before giving up.
>   */
> -#define BT_MAX_RETRY_COUNT 1
> +#define BT_MAX_SEND_COUNT 2
>  
>  #define BT_QUEUE_DEBUG 0
>  
> @@ -110,7 +110,7 @@ struct bt_msg {
>  	struct list_node link;
>  	unsigned long tb;
>  	uint8_t seq;
> -	uint8_t retry_count;
> +	uint8_t send_count;
>  	struct ipmi_msg ipmi_msg;
>  };
>  
> @@ -217,6 +217,7 @@ static void bt_send_msg(struct bt_msg *bt_msg)
>  		bt_outb(ipmi_msg->data[i], BT_HOST2BMC);
>  
>  	BT_Q_DBG(bt_msg, "Message sent to host");
> +	bt_msg->send_count++;
>  
>  	bt_outb(BT_CTRL_H2B_ATN, BT_CTRL);
>  	bt_set_state(BT_STATE_RESP_WAIT);
> @@ -321,13 +322,14 @@ static void bt_expire_old_msg(uint64_t tb)
>  	bt_msg = list_top(&bt.msgq, struct bt_msg, link);
>  
>  	if (bt_msg && bt_msg->tb > 0 && (bt_msg->tb + BT_MSG_TIMEOUT) < tb) {
> -		if (bt_msg->retry_count < BT_MAX_RETRY_COUNT) {
> +		if (bt_msg->send_count < BT_MAX_SEND_COUNT) {
>  			/* A message timeout is usually due to the BMC
>  			clearing the H2B_ATN flag without actually
>  			doing anything. The data will still be in the
>  			FIFO so just reset the flag.*/
>  			BT_Q_ERR(bt_msg, "Retry sending message");
> -			bt_msg->retry_count++;
> +			bt_msg->send_count++;
> +
>  			bt_msg->tb = tb;
>  			bt_outb(BT_CTRL_H2B_ATN, BT_CTRL);
>  		} else {
> @@ -353,7 +355,7 @@ static void print_debug_queue_info(void)
>  		printed = false;
>  		BT_DBG("-------- BT Msg Queue --------\n");
>  		list_for_each(&bt.msgq, msg, link) {
> -			BT_Q_DBG(msg, "[]");
> +			BT_Q_DBG(msg, "[ sent %d ]", msg->send_count);
>  		}
>  		prlog(PR_DEBUG, "-----------------------------\n");
>  	} else if (!printed) {
> @@ -435,7 +437,7 @@ static void bt_add_msg(struct bt_msg *bt_msg)
>  {
>  	bt_msg->tb = 0;
>  	bt_msg->seq = ipmi_seq++;
> -	bt_msg->retry_count = 0;
> +	bt_msg->send_count = 0;
>  	bt.queue_len++;
>  	if (bt.queue_len > BT_MAX_QUEUE_LEN) {
>  		/* Maximum queue length exceeded - remove the oldest message
>