- User Since
- Nov 20 2015, 9:23 AM (151 w, 6 d)
Thu, Oct 4
Aug 28 2018
I went ahead and moved the news.xml change as you suggested; I'm including it there for posterity.
Re-order change to news.xml to be in reverse-chronological order.
Aug 27 2018
Actually include the new key in the diff...
Aug 25 2018
LGTM from a purely technical standpoint. (I'm actively avoiding the larger policy discussion around drm(4) and drm2(4).)
Aug 1 2018
Jul 31 2018
Fix a typo in UPDATING: "jdedc_ts" -> "jedec_ts".
Jul 24 2018
Jul 21 2018
Looks good to me.
Jul 11 2018
... 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.
Jun 29 2018
Jun 25 2018
Jun 22 2018
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.