<div dir="auto"><div><br>Hi again, Petr.</div><div dir="auto"><br><div class="gmail_quote" dir="auto"><div dir="ltr" class="gmail_attr">A domingo, 1/03/2020, 13:47, Petr Štetiar <<a href="mailto:ynezz@true.cz">ynezz@true.cz</a>> escreveu:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Rui Salvaterra <<a href="mailto:rsalvaterra@gmail.com" target="_blank" rel="noreferrer">rsalvaterra@gmail.com</a>> [2020-03-01 13:39:37]:<br>
<br>
> Ah, silly me, you're absolutely right. This is just half the patch, and<br>
> it's perfectly fine the way it is, so I'm not enabling anything new, just<br>
> exposing the existing kconfig. Please ignore my previous email.<br>
<br>
Well, I've re-read it again.<br>
<br>
Your commit description forget to mention, that you're actually removing ADC<br>
based RNG feeding for ar5k, which is missing in that upstream rng.c.<br></blockquote></div></div><div dir="auto"><br></div><div dir="auto">Yes, I don't want to regress that, of course.</div><div dir="auto"><br></div><div dir="auto"><div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
So commit subject and commit description needs some work. You should consider<br>
as well Felix's remark and if I were your, I would simply handle this via<br>
upstream, once upstreamed, simply backported it to OpenWrt.<br></blockquote></div></div><div dir="auto"><br></div><div dir="auto"><div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"></blockquote></div></div><div dir="auto">That's my preferred course of action too, upstream first. As for Felix's concerns, I'm very thoroughly testing on all the hardware I can get my hands on. I'm worried the most about AR5008, which is the earliest one, but now I have a card to test it.</div><div dir="auto"><br></div><div dir="auto">Thanks,</div><div dir="auto">Rui</div></div>