On 8/1/23 13:23, Kalle Valo wrote:
In that case please always add "Compile tested only" to the commit log. It's important for us to know if a (non-trivial) patch is tested on a real hardware or just with a compiler.
OK.
Another problem I see that you don't always reply to review comments and that gives an impression that the comments are ignored. Please always try to reply something to the review comments, even if just a simple "ok" or "I don't agree because...".
Looking through my e-mails for the previous month, I was unable to find an unanswered review. Could you please provide an example? I'll fix it ASAP. I don't want to speculate around the workflow of others and realize that someone (especially the maintainer) may be overloaded and too busy. OTOH it's not quite clear why the trivial things like https://marc.info/?l=linux-wireless&m=169030215701718&w=2 stalls for almost a week. Should I consider this as "ignored" too? Dmitry