<div dir="ltr"><div class="gmail_extra"><div class="gmail_quote">On Sat, Sep 27, 2014 at 10:23 PM, Peer, Ilan <span dir="ltr">&lt;<a href="mailto:ilan.peer@intel.com" target="_blank">ilan.peer@intel.com</a>&gt;</span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div class="HOEnZb"><div class="h5"><br>
<br>
&gt; -----Original Message-----<br>
&gt; From: <a href="mailto:hostap-bounces@lists.shmoo.com">hostap-bounces@lists.shmoo.com</a> [mailto:<a href="mailto:hostap-">hostap-</a><br>
&gt; <a href="mailto:bounces@lists.shmoo.com">bounces@lists.shmoo.com</a>] On Behalf Of Dmitry Shmidt<br>
&gt; Sent: Wednesday, September 24, 2014 20:39<br>
&gt; To: <a href="mailto:hostap@lists.shmoo.com">hostap@lists.shmoo.com</a><br>
&gt; Subject: [PATCH] Android: Skip explicit conf_p2p_dev loading for main<br>
&gt; interface<br>
&gt;<br>
&gt; It will be loaded later for Non-netdev p2p interface.<br>
&gt;<br>
&gt; Change-Id: Ib53daf22c84f2581b499462dc09a74ccfbac226a<br>
&gt; Signed-off-by: Dmitry Shmidt &lt;<a href="mailto:dimitrysh@google.com">dimitrysh@google.com</a>&gt;<br>
&gt; ---<br>
&gt;  wpa_supplicant/wpa_supplicant.c | 2 ++<br>
&gt;  1 file changed, 2 insertions(+)<br>
&gt;<br>
&gt; diff --git a/wpa_supplicant/wpa_supplicant.c<br>
&gt; b/wpa_supplicant/wpa_supplicant.c index f20bc62..9d21fe0 100644<br>
&gt; --- a/wpa_supplicant/wpa_supplicant.c<br>
&gt; +++ b/wpa_supplicant/wpa_supplicant.c<br>
&gt; @@ -3509,7 +3509,9 @@ static int wpa_supplicant_init_iface(struct<br>
&gt; wpa_supplicant *wpa_s,<br>
&gt;<br>
&gt;  #ifdef CONFIG_P2P<br>
&gt;               wpa_s-&gt;conf_p2p_dev = os_rel2abs_path(iface-<br>
&gt; &gt;conf_p2p_dev);<br>
&gt; +#ifndef ANDROID_P2P<br>
&gt;               wpa_config_read(wpa_s-&gt;conf_p2p_dev, wpa_s-&gt;conf);<br>
&gt; +#endif<br>
<br>
</div></div>I think it would be cleaner to do the reading only if iface-&gt;p2p_mgmt is set (happens only in case a dedicated P2P DEVICE interface is added).<br></blockquote><div><br></div><div>1) iface-&gt;p2p_mgmt will be set later.</div><div>2) In case p2p has separate Netdev - it will have its own iterface, in case for Non-netdev it will still load this file</div><div>taking it from parent. Can you please describe the situation when we need to load this file for main interface?</div><div><br></div><div>Thanks,</div><div>Dmitry</div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<br>
Regards,<br>
<br>
Ilan.<br>
</blockquote></div><br></div></div>