Messages in this thread | | | From | "Jason A. Donenfeld" <> | Date | Tue, 22 Feb 2022 11:13:45 +0100 | Subject | Re: [PATCH v3] ath9k: use hw_random API instead of directly dumping into random.c |
| |
On 2/22/22, Ard Biesheuvel <ardb@kernel.org> wrote: > On Mon, 21 Feb 2022 at 11:57, Kalle Valo <kvalo@kernel.org> wrote: >> >> "Jason A. Donenfeld" <Jason@zx2c4.com> wrote: >> >> > Hardware random number generators are supposed to use the hw_random >> > framework. This commit turns ath9k's kthread-based design into a proper >> > hw_random driver. >> > >> > Cc: Toke Høiland-Jørgensen <toke@redhat.com> >> > Cc: Kalle Valo <kvalo@kernel.org> >> > Cc: Rui Salvaterra <rsalvaterra@gmail.com> >> > Cc: Dominik Brodowski <linux@dominikbrodowski.net> >> > Cc: Herbert Xu <herbert@gondor.apana.org.au> >> > Signed-off-by: Jason A. Donenfeld <Jason@zx2c4.com> >> > Tested-by: Rui Salvaterra <rsalvaterra@gmail.com> >> > Acked-by: Toke Høiland-Jørgensen <toke@toke.dk> >> > Signed-off-by: Kalle Valo <quic_kvalo@quicinc.com> >> >> Patch applied to ath-next branch of ath.git, thanks. >> >> fcd09c90c3c5 ath9k: use hw_random API instead of directly dumping into >> random.c >> > > With this patch, it seems we end up registering the hw_rng every time > the link goes up, and unregister it again when the link goes down, > right? > > Wouldn't it be better to split off this driver from the 802.11 link > state handling? >
I really have no idea how this thing works, and I tried hard to change as little as possible in converting it to the API. You may want to send some follow-up patches if you have hardware to experiment with. One consideration does leap out, which is that in my experience wifi cards use a lot less power when they're set "down", as though a decent amount of hardware is being switched off. I think this ath9k rng call might be using the ADC to gather samples of ether from somewhere. I imagine this gets shutdown too as part of that dame circuitry.
| |