- User Since
- Nov 20 2015, 9:23 AM (138 w, 3 d)
Wed, Jul 11
... the numbers just happened to coincide here.
The summary says PAU06 is the combination of two already-supported chips: RT5372 radio and RT5392 MAC. But nothing in the diff mentions the latter chip.
Fri, Jun 29
Mon, Jun 25
Fri, Jun 22
Jun 9 2018
Jun 8 2018
Jun 7 2018
I think this looks good now. Now you can point someone who actually knows `rc' at it. :-)
May 30 2018
May 29 2018
May 17 2018
It looks like the GENERIC configs that you're not updating are for ARM, RISC-V, and PPC64. Why are they being excluded?
Apr 10 2018
Oh, heh, I didn't even realize that this wasn't @ian, since he and I talked about this yesterday. :-)
I started but didn't finish reading through the manpage. My comments so far are in-line.
Mar 29 2018
Out of curiosity, why specifically for chown / chgrp?
Mar 24 2018
It would be good if you explained what a hook was, somewhere before the Exported functions section.
Mar 22 2018
# Much longer than sizeof(sc->slotid_str) % kenv hint.jedec_dimm.0.slotid DIMM0LONGERSTRING
As @cem pointed out, it's even easier to use strdup().
@imp correctly observed that since the buffer is allocated to fit, then
fully initialized, there's no need to use M_ZERO.
Mar 9 2018
Mar 7 2018
Looks fine to my relatively-untrained eye.
Mar 3 2018
Modulo adding a comment, LGTM.
Mar 2 2018
Mar 1 2018
Address a few more things @jhb mentioned.
Call only bus_generic_attach(), but not bus_generic_probe(), from the attach
methods. This behaves properly for both pre-loading via loader.conf and loading
Feb 28 2018
At this point, it looks like the only outstanding issue is either adding device_probe_and_attach(), or confirming that it's not necessary. I'll try to do that tomorrow.
Address most of @avg's review comments.
Feb 27 2018
Thanks for finding time for this! I'll address many of your comments and post an updated diff later today.
Feb 23 2018
Feb 22 2018
Feb 21 2018
Remove some stray comments from the manpage.
style(9) changes: Correct formatting of function prototypes and declarations, and re-order variables.
Feb 20 2018
Address some of @cem's review comments.
Feb 19 2018
Feb 16 2018
Feb 15 2018
Feb 8 2018
Feb 6 2018
Jan 31 2018
Leading 8-space -> hard-tab. Whoops!
Use the obvious one-liner, which somehow escaped me at the time of the
original diff. Thanks, avg@!