Received-SPF: pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) client-ip=209.132.180.67;
MIME-Version: 1.0
In-Reply-To: <20180725111459.10800-1-yuehaibing@huawei.com>
References: <20180725111459.10800-1-yuehaibing@huawei.com>
From:   Michael Chan <michael.chan@broadcom.com>
Date:   Fri, 27 Jul 2018 08:02:52 -0700
Message-ID: <CACKFLim+u+KPfJjj2fZ2BBDdzNDiKLV-hdeF8jJ0R-pW7G9nfw@mail.gmail.com>
Subject: Re: [PATCH net-next] bnxt_en: combine 'else if' and 'else' into
 single branche
To:     YueHaibing <yuehaibing@huawei.com>
Cc:     David Miller <davem@davemloft.net>,
        open list <linux-kernel@vger.kernel.org>,
        Netdev <netdev@vger.kernel.org>,
        Vasundhara Volam <vasundhara-v.volam@broadcom.com>
Content-Type: text/plain; charset="UTF-8"
Sender: linux-kernel-owner@vger.kernel.org
Precedence: bulk

On Wed, Jul 25, 2018 at 4:14 AM, YueHaibing <yuehaibing@huawei.com> wrote:
> The else-if branch and else branch set mac_ok to true similarly,
> so combine the two into single else branch.
>
> Aslo add comments to explain the two conditions, which
> from Michael Chan and Vasundhara Volam.
>
> Signed-off-by: YueHaibing <yuehaibing@huawei.com>

Acked-by: Michael Chan <michael.chan@broadcom.com>

Thanks.