----- Original Message -----
> From: "Johannes Thumshirn" <[email protected]>
> To: "Martin K . Petersen" <[email protected]>, "James Bottomley" 
> <[email protected]>, "Jiri Kosina"
> <[email protected]>
> Cc: "Linux SCSI Mailinglist" <[email protected]>, 
> [email protected], "Johannes Thumshirn"
> <[email protected]>
> Sent: Thursday, July 7, 2016 9:41:58 AM
> Subject: [PATCH trivial] fcoe: use defines from ethtool for 20Gbit and 40Gbit 
> speeds
> 
> Use defines from ethtool for 20Gbit and 40Gbit speeds instead of magic
> numbers.
> 
> Signed-off-by: Johannes Thumshirn <[email protected]>
> ---
>  drivers/scsi/fcoe/fcoe_transport.c | 4 ++--
>  1 file changed, 2 insertions(+), 2 deletions(-)
> 
> diff --git a/drivers/scsi/fcoe/fcoe_transport.c
> b/drivers/scsi/fcoe/fcoe_transport.c
> index 641c60e..7028dd3 100644
> --- a/drivers/scsi/fcoe/fcoe_transport.c
> +++ b/drivers/scsi/fcoe/fcoe_transport.c
> @@ -133,10 +133,10 @@ int fcoe_link_speed_update(struct fc_lport *lport)
>               case SPEED_10000:
>                       lport->link_speed = FC_PORTSPEED_10GBIT;
>                       break;
> -             case 20000:
> +             case SPEED_20000:
>                       lport->link_speed = FC_PORTSPEED_20GBIT;
>                       break;
> -             case 40000:
> +             case SPEED_40000:
>                       lport->link_speed = FC_PORTSPEED_40GBIT;
>                       break;
>               default:
> --
> 1.8.5.6
> 
> --
> To unsubscribe from this list: send the line "unsubscribe linux-scsi" in
> the body of a message to [email protected]
> More majordomo info at  http://vger.kernel.org/majordomo-info.html
> 

Looks fine, simple change
Reviewed-by: Laurence Oberman <[email protected]>

--
To unsubscribe from this list: send the line "unsubscribe linux-scsi" in
the body of a message to [email protected]
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Reply via email to