On Thu, Feb 11, 2021 at 12:40:41AM +0530, karthik alapati wrote: > @@ -867,10 +845,8 @@ static void PHY_HandleSwChnlAndSetBW8723B( > if (bSetBandWidth) > pHalData->bSetChnlBW = true; > > - if (!pHalData->bSetChnlBW && !pHalData->bSwChnl) { > - /* DBG_871X("<= PHY_HandleSwChnlAndSetBW8812: bSwChnl %d, bSetChnlBW %d\n", pHalData->bSwChnl, pHalData->bSetChnlBW); */ > + if (!pHalData->bSetChnlBW && !pHalData->bSwChnl) > return; > - } In this case, the + line is correct. Good job. > > > if (pHalData->bSwChnl) { > @@ -929,9 +905,7 @@ void PHY_SetSwChnlBWMode8723B( > u8 Offset80 > ) > { > - /* DBG_871X("%s() ===>\n", __func__); */ > > PHY_HandleSwChnlAndSetBW8723B(Adapter, true, true, channel, Bandwidth, Offset40, Offset80, channel); > > - /* DBG_871X("<==%s()\n", __func__); */ Please delete the blank lines as well. regards, dan carpenter _______________________________________________ devel mailing list devel@xxxxxxxxxxxxxxxxxxxxxx http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel