On Mon, 2025-04-07 at 13:15 -0700, Kees Cook wrote:
> lots of quotes

It'd be really good if you could trim the quotes a bit ...

> > +           wkc->mac_id_n_color =
> > +                   cpu_to_le32(FW_CMD_ID_AND_COLOR(mvmvif->id,
> > +                                                   mvmvif->color));
> > +           wkc->num_keys = 1;
> 
> Looks like struct iwl_mvm_wep_key_cmd::num_keys is the counted_by for
> struct iwl_mvm_wep_key_cmd::wep_key?

No no no. I'm still burned by you adding this to various places
elsewhere in the wireless stack, let's not.

Btw, Gustavo, I think it was probably the other iwlwifi patch, but
please also make sure kernel-doc doesn't start complaining, we are
trying keep it that way.

johannes

Reply via email to