r349062 bdrewery June 15, 2019, 5:08 p.m.
This is somewhat of a follow-up to r335746.

MFC after:	2 weeks
Sponsored by:	DellEMC
r349061 bdrewery June 15, 2019, 5:08 p.m.
This is in the case of not having any .depend.foo.o yet.  Don't force add *.h
as a dependency for those. They are built in beforebuild already when in

This change allows custom rules, like in bin/sh/Makefile for mksyntax, to not
have cyclic dependency problems when connected to the .depend.* handling.

This is purposely not copied to sys/conf/kern.post.mk as it handles
generating headers slightly differently.

MFC after:	2 weeks
Sponsored by:	DellEMC
r349060 ian June 15, 2019, 4:59 p.m.
r349059 ian June 15, 2019, 4:56 p.m.
driver is compiled into the kernel but pwmbus will be loaded as a module
when needed (and because of that, pwmbus_attach_bus() is going away in
the near future).  Instead, just directly do what that function did:
register the fdt xfef handle, and attach the pwmbus.
r349058 ian June 15, 2019, 4:36 p.m.
resources if they got allocated (because detach() gets called from attach()
to handle various failures), and delete the pwmbus child if it got created.
r349057 ian June 15, 2019, 4:16 p.m.
still includes it all.
r349056 0mp June 15, 2019, 12:09 p.m.
Approved by:	src (emaste)
Event:		Berlin Devsummit 2019
r349055 marius June 15, 2019, 11:07 a.m.
structs with placeholders (in the latter case, IFLIB_MAX_TX_BYTES
  etc. are also only ever used for these write-only members if at all,
  so both these macros and members can just go). Using these spares
  may render it possible to merge certain iflib(9) fixes to stable/12.
  Otherwise, changes extending struct if_irq or struct if_shared_ctx
  in any way would break KBI as instances of these are allocated by
  the driver front-ends (by contrast, struct if_pkt_info as well as
  struct if_softc_ctx instances are provided by iflib(9) and, thus,
  may grow at least at the end without breaking KBI).
- Make the pvi_name in struct pci_vendor_info const char * as device
  identifiers in hardware lookup tables aren't to be expected to ever
  change at runtime.
- Similarly, make the pci_vendor_info_t of struct if_shared_ctx which
  is used to point to the struct pci_vendor_info arrays provided by
  the driver front-ends const.
- Remove the ETH_ADDR_LEN macro from iflib.h; this was duplicating
  ETHER_ADDR_LEN of <net/ethernet.h> with iflib(9) actually only
  consuming the latter macro.
- Make the name argument of iflib_io_tqg_attach(9) const, matching
  the taskqgroup_attach_cpu(9) this function wraps as well as e. g.
- Remove the orphaned iflib_qset_lock_get() prototype.
- Remove some extraneous empty lines.
r349054 marius June 15, 2019, 9:45 a.m.
in r336335.
r349049 emaste June 15, 2019, 9:15 a.m.
During review for PR 205707.

Event:		Berlin Devsummit 2019
r349048 delphij June 15, 2019, 7:23 a.m.

MFC after:	2 weeks
r349047 delphij June 15, 2019, 6:51 a.m.
MFC after:	2 weeks
r349046 dougm June 15, 2019, 4:30 a.m.
the lost information in new comments.

Reported by: alc
Reviewed by: alc
Approved by: kib (mentor)
Differential Revision: https://reviews.freebsd.org/D20632
r349044 julian June 15, 2019, 12:47 a.m.
I just happenned to have 3rd party code using 'var' as the output variable
which drew my attention to this. variables defined inside macros should be
prefixed to avoid getting shadowed varable wanrings from clang.
r349043 grog June 14, 2019, 11:53 p.m.