Hi Kalle, I will change the debug level and send the updated patches again. Thanks, Prameela On Mon, Aug 14, 2017 at 10:24 AM, Kalle Valo <kvalo@xxxxxxxxxxxxxx> wrote: > Amitkumar Karwar <amitkarwar@xxxxxxxxx> writes: > >> From: Prameela Rani Garnepudi <prameela.j04cs@xxxxxxxxx> >> >> Station structures are maintained in driver with required >> fields. When mac80211 callback sta_add is called, driver >> iterates through list of connected stations to check available >> index and assigns station id which is important for further >> communication to that station. Then peer notify frame is send >> to firmware to inform the firmware about new station connection. >> >> Signed-off-by: Prameela Rani Garnepudi <prameela.j04cs@xxxxxxxxx> >> Signed-off-by: Amitkumar Karwar <amit.karwar@xxxxxxxxxxxxxxxxxx> > > [...] > >> @@ -1157,18 +1159,81 @@ static int rsi_mac80211_sta_add(struct ieee80211_hw *hw, >> { >> struct rsi_hw *adapter = hw->priv; >> struct rsi_common *common = adapter->priv; >> + bool sta_exist = false; >> + struct rsi_sta *rsta; >> + >> + print_hex_dump(KERN_INFO, "Station add: ", DUMP_PREFIX_OFFSET, >> + 16, 1, sta->addr, ETH_ALEN, 0); > > The driver should not spam the logs every time a client connects, so > this should be a debug message. > > -- > Kalle Valo -- G. Prameela Rani (M.Tech,CSE)