On Thu, Oct 05, 2017 at 11:01:45AM +0100, Colin King wrote: > From: Colin Ian King <colin.king@xxxxxxxxxxxxx> > > The structure event_regs is local to the source and does not need to > be in global scope, so make it static. > > Cleans up sparse warning: > symbol 'event_regs' was not declared. Should it be static > > Signed-off-by: Colin Ian King <colin.king@xxxxxxxxxxxxx> Applied with Logan's ack to pci/switchtec for v4.15, thanks! > --- > drivers/pci/switch/switchtec.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/drivers/pci/switch/switchtec.c b/drivers/pci/switch/switchtec.c > index af81b2dec42e..da45dbea20ce 100644 > --- a/drivers/pci/switch/switchtec.c > +++ b/drivers/pci/switch/switchtec.c > @@ -943,7 +943,7 @@ static u32 __iomem *pff_ev_reg(struct switchtec_dev *stdev, > #define EV_PAR(i, r)[i] = {offsetof(struct part_cfg_regs, r), part_ev_reg} > #define EV_PFF(i, r)[i] = {offsetof(struct pff_csr_regs, r), pff_ev_reg} > > -const struct event_reg { > +static const struct event_reg { > size_t offset; > u32 __iomem *(*map_reg)(struct switchtec_dev *stdev, > size_t offset, int index); > -- > 2.14.1 > -- To unsubscribe from this list: send the line "unsubscribe kernel-janitors" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html