Re: [PATCH 13/42] scsi: add get_{status,host}_byte() accessor function

[Bart Van Assche <bvanassche@xxxxxxx>]

On 4/21/21 10:47 AM, Hannes Reinecke wrote:
> Add accessor functions for the host and status byte.
>
> Signed-off-by: Hannes Reinecke <hare@xxxxxxx>
> ---
>   include/scsi/scsi_cmnd.h | 10 ++++++++++
>   1 file changed, 10 insertions(+)
>
> diff --git a/include/scsi/scsi_cmnd.h b/include/scsi/scsi_cmnd.h
> index a1eb7732aa1b..0ac18a7d8ac6 100644
> --- a/include/scsi/scsi_cmnd.h
> +++ b/include/scsi/scsi_cmnd.h
> @@ -316,6 +316,11 @@ static inline void set_status_byte(struct scsi_cmnd *cmd, char status)
>   	cmd->result = (cmd->result & 0xffffff00) | status;
>   }
>   
> +static inline unsigned char get_status_byte(struct scsi_cmnd *cmd)
> +{
> +	return cmd->result & 0xff;
> +}

So in addition to the status_byte() macro, get_status_byte() is 
introduced? That seems like a potential source of confusion to me.

Thanks,

Bart.