Add __printflike() to bus_describe_intr() to enable -Wformat checks.
Fix a few places that were passing a raw string as the format to use a "%s" format string instead. MFC after: 2 months
This commit is contained in:
parent
1c2702c041
commit
c62700a16c
@ -4364,7 +4364,7 @@ t4_alloc_irq(struct adapter *sc, struct irq *irq, int rid,
|
||||
"failed to setup interrupt for rid %d, name %s: %d\n",
|
||||
rid, name, rc);
|
||||
} else if (name)
|
||||
bus_describe_intr(sc->dev, irq->res, irq->tag, name);
|
||||
bus_describe_intr(sc->dev, irq->res, irq->tag, "%s", name);
|
||||
|
||||
return (rc);
|
||||
}
|
||||
|
@ -1210,7 +1210,7 @@ _iflib_irq_alloc(if_ctx_t ctx, if_irq_t irq, int rid,
|
||||
rid, name ? name : "unknown", rc);
|
||||
return (rc);
|
||||
} else if (name)
|
||||
bus_describe_intr(dev, res, tag, name);
|
||||
bus_describe_intr(dev, res, tag, "%s", name);
|
||||
|
||||
irq->ii_tag = tag;
|
||||
return (0);
|
||||
|
@ -534,7 +534,7 @@ int bus_setup_intr(device_t dev, struct resource *r, int flags,
|
||||
int bus_teardown_intr(device_t dev, struct resource *r, void *cookie);
|
||||
int bus_bind_intr(device_t dev, struct resource *r, int cpu);
|
||||
int bus_describe_intr(device_t dev, struct resource *irq, void *cookie,
|
||||
const char *fmt, ...);
|
||||
const char *fmt, ...) __printflike(4, 5);
|
||||
int bus_set_resource(device_t dev, int type, int rid,
|
||||
rman_res_t start, rman_res_t count);
|
||||
int bus_get_resource(device_t dev, int type, int rid,
|
||||
|
@ -306,7 +306,7 @@ dmar_alloc_irq(device_t dev, struct dmar_unit *unit, int idx)
|
||||
dmd->name, error);
|
||||
goto err4;
|
||||
}
|
||||
bus_describe_intr(dev, dmd->irq_res, dmd->intr_handle, dmd->name);
|
||||
bus_describe_intr(dev, dmd->irq_res, dmd->intr_handle, "%s", dmd->name);
|
||||
error = PCIB_MAP_MSI(pcib, dev, dmd->irq, &msi_addr, &msi_data);
|
||||
if (error != 0) {
|
||||
device_printf(dev, "cannot map %s interrupt, %d\n",
|
||||
|
Loading…
Reference in New Issue
Block a user