Am 1. August 2023 08:51:03 MESZ schrieb "Bjørn Mork" <bj...@mork.no>:
>Linus Walleij <linus.wall...@linaro.org> writes:
>
>>  can also create a pull request on github if it helps?
>
>Probably not.  There's a "6 approving reviews" rule on github, and not
>even close to that many active reviewers.  Sp adding new devices via
>pull requests seems impossible at the moment.
>
>
>Bjørn
>
>_______________________________________________
>openwrt-devel mailing list
>openwrt-devel@lists.openwrt.org
>https://lists.openwrt.org/mailman/listinfo/openwrt-devel

That is just an arbitrary number (probably to prevent accidental merges or so).
Most device PRs seem to be merged by Hauke.
But not all of them.
You don't need 6 reviews. But it helps when you have others that are able to 
test your code change and confirm that it works. (that won't happen much on 
adding new devices though)

But I agree that opening a PR on GitHub won't move this along. I guess, you can 
mostly revive the patch series by reminding people. Or ask them on IRC for 
review.

The overall problem is that not all patches are being looked at immediately and 
then they are being forgotten about. Not enough reviewers, yes. :/

I added some people to CC, that might be able to take a look at this. (mostly 
because I know that they are working with broadcom devices)

Also sorry, I butchered your names, my mail program (K9) doesn't support 
non-ascii characters. (so dumb)


Regards
Felix

_______________________________________________
openwrt-devel mailing list
openwrt-devel@lists.openwrt.org
https://lists.openwrt.org/mailman/listinfo/openwrt-devel

Reply via email to