zephyr: modify log type from info to debug #50
Conversation
GaofengZhangNXP
requested review from
jukkar,
krish2718 and
rlubos
as code owners
|
Please elaborate in the commit message why this "fix" is needed, I have hard time understanding why we need to change this. |
GaofengZhangNXP
changed the title
GaofengZhangNXP
force-pushed
the
gaofeng_hostap_local1
branch
from
e1a74d9 to
a70fe5e
Compare
yes, you are right, I only modify log type. |
|
hi sirs, |
|
Can you please raise a manifest PR for this? And then once CI is green we can merge this. |
|
| @@ -269,7 +269,7 @@ static void hostapd_cli_recv_pending(struct wpa_ctrl *ctrl, struct hostapd_data | |||
| wpa_printf(MSG_DEBUG, "Received len: %d, msg_len:%d - %s->END\n", | |||
There was a problem hiding this comment.
@GaofengZhangNXP Please change the commit subject to zephyr: modify log type from info to debug
now there the hostap mentioned twice which looks weird. As this is zephyr specific change, lets use the zephyr: ... prefix here.
There was a problem hiding this comment.
Hi @GaofengZhangNXP please modify as jukkar's comment.
zephyr: modify log type from info to debug
|
@jukkar @krish2718 , if you haven't any other comments, please help merge it, thanks |
print too long msg when eap enterprise. we should not print the log, so set print type to debug. Signed-off-by: Gaofeng Zhang <[email protected]>
GaofengZhangNXP
force-pushed
the
gaofeng_hostap_local1
branch
from
a70fe5e to
6fd30ff
Compare
GaofengZhangNXP
changed the title
|
Hi @jukkar , please help review the patch, if you have not any concern, please help merge it, thank you very much. |
|
Hi @jukkar , please help review the patch, if you have not any other suggestion, please help merge it, thank you very much. |
|
Hi @krish2718 , @jukkar could you pls review and merge this change? thanks |
print too long msg when eap enterprise.
we should not print the log, so set print type to debug.