[PATCH 1/9] wifi: mac80211: allow enabling chanctx until hw registration
Johannes Berg
johannes at sipsolutions.net
Fri Aug 19 10:16:58 PDT 2022
Hi Sean,
> > If we really, really need to allow changing the ops, then we should
> > probably make a much larger change to not even pass the ops until
> > register, though I'm not really sure it's worth it just to have mt7921
> > avoid loading the firmware from disk before allocation?
> Thanks for your input. I thought I'd try to write a patch to follow up
> on the idea you mentioned here.
>
I think you will introduce a bug into mt7921 when you do this, and I'm
curious if you will find it ;-)
Seriously though, this approach also seems fragile, and I don't even
know if other bugs would be introduced. And splitting into three
functions (alloc -> set_ops -> register) also feels a bit awkward.
Is there really no chance you could add bits to the firmware _file_
format so you can query the capabilities before you actually _run_ the
firmware? I guess you could even validate it at runtime again (and just
fail is somebody messed up the file), but it would make things a lot
simpler, I'm sure.
johannes
More information about the Linux-mediatek
mailing list