Message ID | 20200403081544.37061-1-yanaijie@huawei.com |
---|---|
State | Accepted |
Headers | show |
Series | mtd: physmap_of_gemini: remove defined but not used symbol 'syscon_match' | expand |
Hi Jason, + Linus W. Jason Yan <yanaijie@huawei.com> wrote on Fri, 3 Apr 2020 16:15:44 +0800: > It's not used by anyone now, remove it. Fix the following gcc warning: > > drivers/mtd/maps/physmap-gemini.c:49:34: warning: ‘syscon_match’ defined > but not used [-Wunused-const-variable=] > static const struct of_device_id syscon_match[] = { > ^~~~~~~~~~~~ > > Reported-by: Hulk Robot <hulkci@huawei.com> > Signed-off-by: Jason Yan <yanaijie@huawei.com> > --- I am not sure this is the right solution. This compatible has been added in this driver by Linus in 9d3b5086f6d4 ("mtd: physmap_of_gemini: Handle pin control"). I believe he had a good reason to do so but you are right that it is not used. Linus, any input? > drivers/mtd/maps/physmap-gemini.c | 5 ----- > 1 file changed, 5 deletions(-) > > diff --git a/drivers/mtd/maps/physmap-gemini.c b/drivers/mtd/maps/physmap-gemini.c > index a289c8b5cabf..d4a46e159d38 100644 > --- a/drivers/mtd/maps/physmap-gemini.c > +++ b/drivers/mtd/maps/physmap-gemini.c > @@ -46,11 +46,6 @@ > > #define FLASH_PARALLEL_HIGH_PIN_CNT (1 << 20) /* else low pin cnt */ > > -static const struct of_device_id syscon_match[] = { > - { .compatible = "cortina,gemini-syscon" }, > - { }, > -}; > - > struct gemini_flash { > struct device *dev; > struct pinctrl *p; Thanks, Miquèl
On Mon, Apr 27, 2020 at 3:53 PM Miquel Raynal <miquel.raynal@bootlin.com> wrote: > Jason Yan <yanaijie@huawei.com> wrote on Fri, 3 Apr 2020 16:15:44 +0800: > > > It's not used by anyone now, remove it. Fix the following gcc warning: > > > > drivers/mtd/maps/physmap-gemini.c:49:34: warning: ‘syscon_match’ defined > > but not used [-Wunused-const-variable=] > > static const struct of_device_id syscon_match[] = { > > ^~~~~~~~~~~~ > > > > Reported-by: Hulk Robot <hulkci@huawei.com> > > Signed-off-by: Jason Yan <yanaijie@huawei.com> > > --- > > I am not sure this is the right solution. This compatible has been > added in this driver by Linus in 9d3b5086f6d4 ("mtd: physmap_of_gemini: > Handle pin control"). I believe he had a good reason to do so but you > are right that it is not used. Linus, any input? Reviewed-by: Linus Walleij <linus.walleij@linaro.org> It's probably just a remnant of earlier development, sorry for that. We now locate the syscon by phandle: rmap = syscon_regmap_lookup_by_phandle(np, "syscon"); Yours, Linus Walleij
Hi Linus, Linus Walleij <linus.walleij@linaro.org> wrote on Tue, 28 Apr 2020 16:16:55 +0200: > On Mon, Apr 27, 2020 at 3:53 PM Miquel Raynal <miquel.raynal@bootlin.com> wrote: > > Jason Yan <yanaijie@huawei.com> wrote on Fri, 3 Apr 2020 16:15:44 +0800: > > > > > It's not used by anyone now, remove it. Fix the following gcc warning: > > > > > > drivers/mtd/maps/physmap-gemini.c:49:34: warning: ‘syscon_match’ defined > > > but not used [-Wunused-const-variable=] > > > static const struct of_device_id syscon_match[] = { > > > ^~~~~~~~~~~~ > > > > > > Reported-by: Hulk Robot <hulkci@huawei.com> > > > Signed-off-by: Jason Yan <yanaijie@huawei.com> > > > --- > > > > I am not sure this is the right solution. This compatible has been > > added in this driver by Linus in 9d3b5086f6d4 ("mtd: physmap_of_gemini: > > Handle pin control"). I believe he had a good reason to do so but you > > are right that it is not used. Linus, any input? > > Reviewed-by: Linus Walleij <linus.walleij@linaro.org> > > It's probably just a remnant of earlier development, sorry for that. > We now locate the syscon by phandle: > rmap = syscon_regmap_lookup_by_phandle(np, "syscon"); No problem, thanks for the confirmation. Thanks, Miquèl
diff --git a/drivers/mtd/maps/physmap-gemini.c b/drivers/mtd/maps/physmap-gemini.c index a289c8b5cabf..d4a46e159d38 100644 --- a/drivers/mtd/maps/physmap-gemini.c +++ b/drivers/mtd/maps/physmap-gemini.c @@ -46,11 +46,6 @@ #define FLASH_PARALLEL_HIGH_PIN_CNT (1 << 20) /* else low pin cnt */ -static const struct of_device_id syscon_match[] = { - { .compatible = "cortina,gemini-syscon" }, - { }, -}; - struct gemini_flash { struct device *dev; struct pinctrl *p;
It's not used by anyone now, remove it. Fix the following gcc warning: drivers/mtd/maps/physmap-gemini.c:49:34: warning: ‘syscon_match’ defined but not used [-Wunused-const-variable=] static const struct of_device_id syscon_match[] = { ^~~~~~~~~~~~ Reported-by: Hulk Robot <hulkci@huawei.com> Signed-off-by: Jason Yan <yanaijie@huawei.com> --- drivers/mtd/maps/physmap-gemini.c | 5 ----- 1 file changed, 5 deletions(-)