Re: [PATCH] mac802154_hwsim: allow users to specify the number of simulated radios dinamically instead of the previously hardcoded value of 2
Ramon Fontes <[email protected]> Sat, 7 Jun 2025 18:42:09 -0300
| Newsgroups | org.kernel.vger.linux-wpan,org.kernel.vger.netdev |
|---|---|
| Message-ID | <CAK8U23aRrvC4wC6WvcBRPA4YuyC1MPvOj8FO=cWfi43_Fdh4Zw@mail.gmail.com> |
Ok! Just a last question: Shouldn't we define a maximum number of supported radios when using unsigned? A value like -1 would wrap around and result in thousands of radios, right? That said, wouldn't it be simpler and safer to just use int instead? Em sáb., 7 de jun. de 2025 às 16:22, Andrew Lunn <[email protected]> escreveu: > > On Sat, Jun 07, 2025 at 02:54:57PM -0300, Ramon Fontes wrote: > > > handle as unsigned then this check would not be necessary? > > > > Yeah, it does make sense. However, I have a bit of an embarrassing > > question. How do I submit an updated patch in this same thread? > > You don't. You should submit a new patchset, as a new thread. > > The CI tooling works on one patchset per thread, and if you deviate > from that, the CI won't test it. And if it is not tested, it is pretty > much an automatic reject. > > https://www.kernel.org/doc/html/latest/process/maintainer-netdev.html > > It is also the merge windows right now, so anything you post will get > rejected anyway, so please post as RFC. > > Andrew