Re: [PATCH 2/8] netdev: add netdev_get_station

[James Prestwood <prestwoj@gmail.com>]

[-- Attachment #1: Type: text/plain, Size: 1096 bytes --]

Hi Denis,

> >   struct wiphy *netdev_get_wiphy(struct netdev *netdev);
> >   const uint8_t *netdev_get_address(struct netdev *netdev);
> >   uint32_t netdev_get_ifindex(struct netdev *netdev);
> > @@ -174,6 +182,10 @@ int netdev_neighbor_report_req(struct netdev
> > *netdev,
> >   int netdev_set_rssi_report_levels(struct netdev *netdev, const
> > int8_t *levels,
> >   					size_t levels_num);
> >   
> > +int netdev_get_station(struct netdev *netdev, const uint8_t *mac,
> 
> Do you want to add a convenience method to grab the current
> station?  The mac 
> parameter is for cases where you might have multiple ones, like AP.

I could, although its not too bad just passing in handshake->aa... But
if you want a current station API and a separate one for AP I don't
mind either way.

> 
> > +			netdev_get_station_cb_t cb, void *user_data,
> > +			netdev_destroy_func_t destroy);
> > +
> >   void netdev_handshake_failed(struct handshake_state *hs, uint16_t
> > reason_code);
> >   
> >   struct netdev *netdev_find(int ifindex);
> > 
> 
> Regards,
> -Denis