Message ID | 1399044179-17912-1-git-send-email-mq@suse.cz |
---|---|
State | Superseded |
Headers | show |
On Fri, May 02, 2014 at 05:22:59PM +0200, Jan Moskyto Matejka wrote: > Fixing this warning: > drivers/pwm/pwm-lpss.c: In function ‘pwm_lpss_probe_pci’: > drivers/pwm/pwm-lpss.c:192:2: warning: passing argument 3 of ‘pwm_lpss_probe’ discards ‘const’ qualifier from pointer target type [enabled by default] > lpwm = pwm_lpss_probe(&pdev->dev, &pdev->resource[0], info); > ^ > drivers/pwm/pwm-lpss.c:130:30: note: expected ‘struct pwm_lpss_boardinfo *’ but argument is of type ‘const struct pwm_lpss_boardinfo *’ > static struct pwm_lpss_chip *pwm_lpss_probe(struct device *dev, > > That warning was introduced in commit 093e00bb3f82f3c67e2d1682e316fc012bcd0d92 > ("pwm: lpss: Add support for PCI devices"). > > Signed-off-by: Jan Moskyto Matejka <mq@suse.cz> > --- > drivers/pwm/pwm-lpss.c | 1 + > 1 file changed, 1 insertion(+) This was reported over a week ago by the kbuild test robot and I've had fixes for this (and an additional sparse warning) in my tree for a while now. Interestingly everybody at Intel seems to have forgotten about this driver after their patches got merged since nobody cared to respond. I've now pushed out the patch that I had locally. Thanks, Thierry
On Wed, 2014-05-07 at 10:41 +0200, Thierry Reding wrote: > On Fri, May 02, 2014 at 05:22:59PM +0200, Jan Moskyto Matejka wrote: > > Fixing this warning: > > drivers/pwm/pwm-lpss.c: In function ‘pwm_lpss_probe_pci’: > > drivers/pwm/pwm-lpss.c:192:2: warning: passing argument 3 of ‘pwm_lpss_probe’ discards ‘const’ qualifier from pointer target type [enabled by default] > > lpwm = pwm_lpss_probe(&pdev->dev, &pdev->resource[0], info); > > ^ > > drivers/pwm/pwm-lpss.c:130:30: note: expected ‘struct pwm_lpss_boardinfo *’ but argument is of type ‘const struct pwm_lpss_boardinfo *’ > > static struct pwm_lpss_chip *pwm_lpss_probe(struct device *dev, > > > > That warning was introduced in commit 093e00bb3f82f3c67e2d1682e316fc012bcd0d92 > > ("pwm: lpss: Add support for PCI devices"). > > > > Signed-off-by: Jan Moskyto Matejka <mq@suse.cz> > > --- > > drivers/pwm/pwm-lpss.c | 1 + > > 1 file changed, 1 insertion(+) > > This was reported over a week ago by the kbuild test robot and I've had > fixes for this (and an additional sparse warning) in my tree for a while > now. Interestingly everybody at Intel seems to have forgotten about this > driver after their patches got merged since nobody cared to respond. No - like everyone else we're all juggling multiple things. Also remember there have just been a bunch of public holidays on large parts of the planet. I did see Jan's email but I didn't realise it needed a response, it was clearly and obviously correct. Alan -- To unsubscribe from this list: send the line "unsubscribe linux-pwm" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
diff --git a/drivers/pwm/pwm-lpss.c b/drivers/pwm/pwm-lpss.c index c718ad1..c4620c1 100644 --- a/drivers/pwm/pwm-lpss.c +++ b/drivers/pwm/pwm-lpss.c @@ -129,6 +129,7 @@ static const struct pwm_ops pwm_lpss_ops = { static struct pwm_lpss_chip *pwm_lpss_probe(struct device *dev, struct resource *r, + const struct pwm_lpss_boardinfo *info) { struct pwm_lpss_chip *lpwm;
Fixing this warning: drivers/pwm/pwm-lpss.c: In function ‘pwm_lpss_probe_pci’: drivers/pwm/pwm-lpss.c:192:2: warning: passing argument 3 of ‘pwm_lpss_probe’ discards ‘const’ qualifier from pointer target type [enabled by default] lpwm = pwm_lpss_probe(&pdev->dev, &pdev->resource[0], info); ^ drivers/pwm/pwm-lpss.c:130:30: note: expected ‘struct pwm_lpss_boardinfo *’ but argument is of type ‘const struct pwm_lpss_boardinfo *’ static struct pwm_lpss_chip *pwm_lpss_probe(struct device *dev, That warning was introduced in commit 093e00bb3f82f3c67e2d1682e316fc012bcd0d92 ("pwm: lpss: Add support for PCI devices"). Signed-off-by: Jan Moskyto Matejka <mq@suse.cz> --- drivers/pwm/pwm-lpss.c | 1 + 1 file changed, 1 insertion(+)