On Sun, May 06, 2018 at 03:03:04PM -0700, Nathan Chancellor wrote: > Fixes checkpatch.pl warnings. > > Signed-off-by: Nathan Chancellor <natechancellor@xxxxxxxxx> > --- > drivers/staging/ks7010/ks_hostif.c | 4 ++-- > drivers/staging/ks7010/ks_wlan.h | 4 ++-- > drivers/staging/ks7010/ks_wlan_net.c | 8 ++++---- > 3 files changed, 8 insertions(+), 8 deletions(-) > > diff --git a/drivers/staging/ks7010/ks_hostif.c b/drivers/staging/ks7010/ks_hostif.c > index 628171091786..43090922daff 100644 > --- a/drivers/staging/ks7010/ks_hostif.c > +++ b/drivers/staging/ks7010/ks_hostif.c > @@ -475,7 +475,7 @@ void hostif_mib_get_confirm(struct ks_wlan_private *priv) > case DOT11_MAC_ADDRESS: > hostif_sme_enqueue(priv, SME_GET_MAC_ADDRESS); > ether_addr_copy(priv->eth_addr, priv->rxp); > - priv->mac_address_valid = true; > + priv->mac_address_valid = 1; Wait, why? This should be bool, not an int. Why would checkpatch say this is incorrect? confused, greg k-h _______________________________________________ devel mailing list devel@xxxxxxxxxxxxxxxxxxxxxx http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel
