* [PATCH 1/3] video: fbdev: vt8623fb: suppress build warning @ 2014-11-26 22:07 Lad, Prabhakar 2014-11-26 22:07 ` [PATCH 2/3] video: fbdev: s3fb: " Lad, Prabhakar ` (2 more replies) 0 siblings, 3 replies; 10+ messages in thread From: Lad, Prabhakar @ 2014-11-26 22:07 UTC (permalink / raw) To: Tomi Valkeinen, Jean-Christophe Plagniol-Villard Cc: linux-fbdev, linux-kernel, Lad, Prabhakar this patch fixes following build warning: drivers/video/fbdev/vt8623fb.c: In function ‘vt8623_pci_probe’: drivers/video/fbdev/vt8623fb.c:734:23: warning: cast to pointer from integer of different size [-Wint-to-pointer-cast] par->state.vgabase = (void __iomem *) vga_res.start; ^ Signed-off-by: Lad, Prabhakar <prabhakar.csengg@gmail.com> --- drivers/video/fbdev/vt8623fb.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/video/fbdev/vt8623fb.c b/drivers/video/fbdev/vt8623fb.c index 5c7cbc6..ea7f056 100644 --- a/drivers/video/fbdev/vt8623fb.c +++ b/drivers/video/fbdev/vt8623fb.c @@ -731,7 +731,7 @@ static int vt8623_pci_probe(struct pci_dev *dev, const struct pci_device_id *id) pcibios_bus_to_resource(dev->bus, &vga_res, &bus_reg); - par->state.vgabase = (void __iomem *) vga_res.start; + par->state.vgabase = (void __iomem *) (unsigned long) vga_res.start; /* Find how many physical memory there is on card */ memsize1 = (vga_rseq(par->state.vgabase, 0x34) + 1) >> 1; -- 1.9.1 ^ permalink raw reply related [flat|nested] 10+ messages in thread
* [PATCH 2/3] video: fbdev: s3fb: suppress build warning 2014-11-26 22:07 [PATCH 1/3] video: fbdev: vt8623fb: suppress build warning Lad, Prabhakar @ 2014-11-26 22:07 ` Lad, Prabhakar 2014-11-26 22:07 ` [PATCH 3/3] video: fbdev: arkfb: " Lad, Prabhakar 2014-12-03 11:49 ` [PATCH 1/3] video: fbdev: vt8623fb: " Tomi Valkeinen 2 siblings, 0 replies; 10+ messages in thread From: Lad, Prabhakar @ 2014-11-26 22:07 UTC (permalink / raw) To: Tomi Valkeinen, Jean-Christophe Plagniol-Villard Cc: linux-fbdev, linux-kernel, Lad, Prabhakar this patch fixes following build warning: drivers/video/fbdev/s3fb.c: In function ‘s3_pci_probe’: drivers/video/fbdev/s3fb.c:1185:23: warning: cast to pointer from integer of different size [-Wint-to-pointer-cast] par->state.vgabase = (void __iomem *) vga_res.start; ^ Signed-off-by: Lad, Prabhakar <prabhakar.csengg@gmail.com> --- drivers/video/fbdev/s3fb.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/video/fbdev/s3fb.c b/drivers/video/fbdev/s3fb.c index c43b969..f0ae61a 100644 --- a/drivers/video/fbdev/s3fb.c +++ b/drivers/video/fbdev/s3fb.c @@ -1182,7 +1182,7 @@ static int s3_pci_probe(struct pci_dev *dev, const struct pci_device_id *id) pcibios_bus_to_resource(dev->bus, &vga_res, &bus_reg); - par->state.vgabase = (void __iomem *) vga_res.start; + par->state.vgabase = (void __iomem *) (unsigned long) vga_res.start; /* Unlock regs */ cr38 = vga_rcrt(par->state.vgabase, 0x38); -- 1.9.1 ^ permalink raw reply related [flat|nested] 10+ messages in thread
* [PATCH 3/3] video: fbdev: arkfb: suppress build warning 2014-11-26 22:07 [PATCH 1/3] video: fbdev: vt8623fb: suppress build warning Lad, Prabhakar 2014-11-26 22:07 ` [PATCH 2/3] video: fbdev: s3fb: " Lad, Prabhakar @ 2014-11-26 22:07 ` Lad, Prabhakar 2014-12-03 11:49 ` [PATCH 1/3] video: fbdev: vt8623fb: " Tomi Valkeinen 2 siblings, 0 replies; 10+ messages in thread From: Lad, Prabhakar @ 2014-11-26 22:07 UTC (permalink / raw) To: Tomi Valkeinen, Jean-Christophe Plagniol-Villard Cc: linux-fbdev, linux-kernel, Lad, Prabhakar this patch fixes following build warning: drivers/video/fbdev/arkfb.c: In function ‘ark_pci_probe’: drivers/video/fbdev/arkfb.c:1019:23: warning: cast to pointer from integer of different size [-Wint-to-pointer-cast] par->state.vgabase = (void __iomem *) vga_res.start; ^ Signed-off-by: Lad, Prabhakar <prabhakar.csengg@gmail.com> --- drivers/video/fbdev/arkfb.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/video/fbdev/arkfb.c b/drivers/video/fbdev/arkfb.c index adc4ea2..b305a1e 100644 --- a/drivers/video/fbdev/arkfb.c +++ b/drivers/video/fbdev/arkfb.c @@ -1016,7 +1016,7 @@ static int ark_pci_probe(struct pci_dev *dev, const struct pci_device_id *id) pcibios_bus_to_resource(dev->bus, &vga_res, &bus_reg); - par->state.vgabase = (void __iomem *) vga_res.start; + par->state.vgabase = (void __iomem *) (unsigned long) vga_res.start; /* FIXME get memsize */ regval = vga_rseq(par->state.vgabase, 0x10); -- 1.9.1 ^ permalink raw reply related [flat|nested] 10+ messages in thread
* Re: [PATCH 1/3] video: fbdev: vt8623fb: suppress build warning 2014-11-26 22:07 [PATCH 1/3] video: fbdev: vt8623fb: suppress build warning Lad, Prabhakar 2014-11-26 22:07 ` [PATCH 2/3] video: fbdev: s3fb: " Lad, Prabhakar 2014-11-26 22:07 ` [PATCH 3/3] video: fbdev: arkfb: " Lad, Prabhakar @ 2014-12-03 11:49 ` Tomi Valkeinen 2014-12-03 18:29 ` Prabhakar Lad 2 siblings, 1 reply; 10+ messages in thread From: Tomi Valkeinen @ 2014-12-03 11:49 UTC (permalink / raw) To: Lad, Prabhakar, Jean-Christophe Plagniol-Villard Cc: linux-fbdev, linux-kernel [-- Attachment #1: Type: text/plain, Size: 1190 bytes --] On 27/11/14 00:07, Lad, Prabhakar wrote: > this patch fixes following build warning: > drivers/video/fbdev/vt8623fb.c: In function ‘vt8623_pci_probe’: > drivers/video/fbdev/vt8623fb.c:734:23: warning: cast to pointer from integer of different size [-Wint-to-pointer-cast] > par->state.vgabase = (void __iomem *) vga_res.start; > ^ > Signed-off-by: Lad, Prabhakar <prabhakar.csengg@gmail.com> > --- > drivers/video/fbdev/vt8623fb.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/drivers/video/fbdev/vt8623fb.c b/drivers/video/fbdev/vt8623fb.c > index 5c7cbc6..ea7f056 100644 > --- a/drivers/video/fbdev/vt8623fb.c > +++ b/drivers/video/fbdev/vt8623fb.c > @@ -731,7 +731,7 @@ static int vt8623_pci_probe(struct pci_dev *dev, const struct pci_device_id *id) > > pcibios_bus_to_resource(dev->bus, &vga_res, &bus_reg); > > - par->state.vgabase = (void __iomem *) vga_res.start; > + par->state.vgabase = (void __iomem *) (unsigned long) vga_res.start; This does look quite ugly... Where does the warning come from in the first place. Isn't vga_res.start (resource_size_t) the size of a pointer? Tomi [-- Attachment #2: OpenPGP digital signature --] [-- Type: application/pgp-signature, Size: 819 bytes --] ^ permalink raw reply [flat|nested] 10+ messages in thread
* Re: [PATCH 1/3] video: fbdev: vt8623fb: suppress build warning 2014-12-03 11:49 ` [PATCH 1/3] video: fbdev: vt8623fb: " Tomi Valkeinen @ 2014-12-03 18:29 ` Prabhakar Lad 2014-12-04 7:05 ` Tomi Valkeinen 0 siblings, 1 reply; 10+ messages in thread From: Prabhakar Lad @ 2014-12-03 18:29 UTC (permalink / raw) To: Tomi Valkeinen; +Cc: Jean-Christophe Plagniol-Villard, LFBDEV, LKML On Wed, Dec 3, 2014 at 11:49 AM, Tomi Valkeinen <tomi.valkeinen@ti.com> wrote: > On 27/11/14 00:07, Lad, Prabhakar wrote: >> this patch fixes following build warning: >> drivers/video/fbdev/vt8623fb.c: In function ‘vt8623_pci_probe’: >> drivers/video/fbdev/vt8623fb.c:734:23: warning: cast to pointer from integer of different size [-Wint-to-pointer-cast] >> par->state.vgabase = (void __iomem *) vga_res.start; >> ^ >> Signed-off-by: Lad, Prabhakar <prabhakar.csengg@gmail.com> >> --- >> drivers/video/fbdev/vt8623fb.c | 2 +- >> 1 file changed, 1 insertion(+), 1 deletion(-) >> >> diff --git a/drivers/video/fbdev/vt8623fb.c b/drivers/video/fbdev/vt8623fb.c >> index 5c7cbc6..ea7f056 100644 >> --- a/drivers/video/fbdev/vt8623fb.c >> +++ b/drivers/video/fbdev/vt8623fb.c >> @@ -731,7 +731,7 @@ static int vt8623_pci_probe(struct pci_dev *dev, const struct pci_device_id *id) >> >> pcibios_bus_to_resource(dev->bus, &vga_res, &bus_reg); >> >> - par->state.vgabase = (void __iomem *) vga_res.start; >> + par->state.vgabase = (void __iomem *) (unsigned long) vga_res.start; > > This does look quite ugly... Where does the warning come from in the > first place. Isn't vga_res.start (resource_size_t) the size of a pointer? > Yes looks ugly, I am not sure what you meant from 'where does this warning come from' its in the commit message. Thanks, --Prabhakar Lad ^ permalink raw reply [flat|nested] 10+ messages in thread
* Re: [PATCH 1/3] video: fbdev: vt8623fb: suppress build warning 2014-12-03 18:29 ` Prabhakar Lad @ 2014-12-04 7:05 ` Tomi Valkeinen 2014-12-04 7:46 ` Sudip Mukherjee 0 siblings, 1 reply; 10+ messages in thread From: Tomi Valkeinen @ 2014-12-04 7:05 UTC (permalink / raw) To: Prabhakar Lad; +Cc: Jean-Christophe Plagniol-Villard, LFBDEV, LKML [-- Attachment #1: Type: text/plain, Size: 1712 bytes --] On 03/12/14 20:29, Prabhakar Lad wrote: > On Wed, Dec 3, 2014 at 11:49 AM, Tomi Valkeinen <tomi.valkeinen@ti.com> wrote: >> On 27/11/14 00:07, Lad, Prabhakar wrote: >>> this patch fixes following build warning: >>> drivers/video/fbdev/vt8623fb.c: In function ‘vt8623_pci_probe’: >>> drivers/video/fbdev/vt8623fb.c:734:23: warning: cast to pointer from integer of different size [-Wint-to-pointer-cast] >>> par->state.vgabase = (void __iomem *) vga_res.start; >>> ^ >>> Signed-off-by: Lad, Prabhakar <prabhakar.csengg@gmail.com> >>> --- >>> drivers/video/fbdev/vt8623fb.c | 2 +- >>> 1 file changed, 1 insertion(+), 1 deletion(-) >>> >>> diff --git a/drivers/video/fbdev/vt8623fb.c b/drivers/video/fbdev/vt8623fb.c >>> index 5c7cbc6..ea7f056 100644 >>> --- a/drivers/video/fbdev/vt8623fb.c >>> +++ b/drivers/video/fbdev/vt8623fb.c >>> @@ -731,7 +731,7 @@ static int vt8623_pci_probe(struct pci_dev *dev, const struct pci_device_id *id) >>> >>> pcibios_bus_to_resource(dev->bus, &vga_res, &bus_reg); >>> >>> - par->state.vgabase = (void __iomem *) vga_res.start; >>> + par->state.vgabase = (void __iomem *) (unsigned long) vga_res.start; >> >> This does look quite ugly... Where does the warning come from in the >> first place. Isn't vga_res.start (resource_size_t) the size of a pointer? >> > Yes looks ugly, I am not sure what you meant from 'where does this warning > come from' its in the commit message. I meant why is there a warning at all. With a quick glance, vga_res.start is the size of a pointer. So the sizes of the integer and the pointer should be the same. But the warning still says "of different size". Tomi [-- Attachment #2: OpenPGP digital signature --] [-- Type: application/pgp-signature, Size: 819 bytes --] ^ permalink raw reply [flat|nested] 10+ messages in thread
* Re: [PATCH 1/3] video: fbdev: vt8623fb: suppress build warning 2014-12-04 7:05 ` Tomi Valkeinen @ 2014-12-04 7:46 ` Sudip Mukherjee 2014-12-04 7:56 ` Tomi Valkeinen 0 siblings, 1 reply; 10+ messages in thread From: Sudip Mukherjee @ 2014-12-04 7:46 UTC (permalink / raw) To: Tomi Valkeinen Cc: Prabhakar Lad, Jean-Christophe Plagniol-Villard, LFBDEV, LKML On Thu, Dec 04, 2014 at 09:05:46AM +0200, Tomi Valkeinen wrote: > On 03/12/14 20:29, Prabhakar Lad wrote: > > On Wed, Dec 3, 2014 at 11:49 AM, Tomi Valkeinen <tomi.valkeinen@ti.com> wrote: > >> On 27/11/14 00:07, Lad, Prabhakar wrote: > >>> this patch fixes following build warning: > >>> drivers/video/fbdev/vt8623fb.c: In function ‘vt8623_pci_probe’: > >>> drivers/video/fbdev/vt8623fb.c:734:23: warning: cast to pointer from integer of different size [-Wint-to-pointer-cast] > >>> par->state.vgabase = (void __iomem *) vga_res.start; > >>> ^ > >>> Signed-off-by: Lad, Prabhakar <prabhakar.csengg@gmail.com> > >>> --- > >>> drivers/video/fbdev/vt8623fb.c | 2 +- > >>> 1 file changed, 1 insertion(+), 1 deletion(-) > >>> > >>> diff --git a/drivers/video/fbdev/vt8623fb.c b/drivers/video/fbdev/vt8623fb.c > >>> index 5c7cbc6..ea7f056 100644 > >>> --- a/drivers/video/fbdev/vt8623fb.c > >>> +++ b/drivers/video/fbdev/vt8623fb.c > >>> @@ -731,7 +731,7 @@ static int vt8623_pci_probe(struct pci_dev *dev, const struct pci_device_id *id) > >>> > >>> pcibios_bus_to_resource(dev->bus, &vga_res, &bus_reg); > >>> > >>> - par->state.vgabase = (void __iomem *) vga_res.start; > >>> + par->state.vgabase = (void __iomem *) (unsigned long) vga_res.start; > >> > >> This does look quite ugly... Where does the warning come from in the > >> first place. Isn't vga_res.start (resource_size_t) the size of a pointer? > >> > > Yes looks ugly, I am not sure what you meant from 'where does this warning > > come from' its in the commit message. > > I meant why is there a warning at all. With a quick glance, > vga_res.start is the size of a pointer. So the sizes of the integer and > the pointer should be the same. But the warning still says "of different > size". poking my nose into your discussion. I tried to see the warning, and I re-compiled like make W=1 M=drivers/video/fbdev/ (before that make clean M=drivers/video/fbdev was done) I can see warning with many other files, but drivers/video/fbdev/vt8623fb.o was quiet and there was no warning. I tested with next=20141203. did i miss something in checking the warning ? thanks sudip > > Tomi > > ^ permalink raw reply [flat|nested] 10+ messages in thread
* Re: [PATCH 1/3] video: fbdev: vt8623fb: suppress build warning 2014-12-04 7:46 ` Sudip Mukherjee @ 2014-12-04 7:56 ` Tomi Valkeinen 2014-12-04 13:29 ` Geert Uytterhoeven 0 siblings, 1 reply; 10+ messages in thread From: Tomi Valkeinen @ 2014-12-04 7:56 UTC (permalink / raw) To: Prabhakar Lad Cc: Sudip Mukherjee, Jean-Christophe Plagniol-Villard, LFBDEV, LKML [-- Attachment #1: Type: text/plain, Size: 2392 bytes --] On 04/12/14 09:46, Sudip Mukherjee wrote: > On Thu, Dec 04, 2014 at 09:05:46AM +0200, Tomi Valkeinen wrote: >> On 03/12/14 20:29, Prabhakar Lad wrote: >>> On Wed, Dec 3, 2014 at 11:49 AM, Tomi Valkeinen <tomi.valkeinen@ti.com> wrote: >>>> On 27/11/14 00:07, Lad, Prabhakar wrote: >>>>> this patch fixes following build warning: >>>>> drivers/video/fbdev/vt8623fb.c: In function ‘vt8623_pci_probe’: >>>>> drivers/video/fbdev/vt8623fb.c:734:23: warning: cast to pointer from integer of different size [-Wint-to-pointer-cast] >>>>> par->state.vgabase = (void __iomem *) vga_res.start; >>>>> ^ >>>>> Signed-off-by: Lad, Prabhakar <prabhakar.csengg@gmail.com> >>>>> --- >>>>> drivers/video/fbdev/vt8623fb.c | 2 +- >>>>> 1 file changed, 1 insertion(+), 1 deletion(-) >>>>> >>>>> diff --git a/drivers/video/fbdev/vt8623fb.c b/drivers/video/fbdev/vt8623fb.c >>>>> index 5c7cbc6..ea7f056 100644 >>>>> --- a/drivers/video/fbdev/vt8623fb.c >>>>> +++ b/drivers/video/fbdev/vt8623fb.c >>>>> @@ -731,7 +731,7 @@ static int vt8623_pci_probe(struct pci_dev *dev, const struct pci_device_id *id) >>>>> >>>>> pcibios_bus_to_resource(dev->bus, &vga_res, &bus_reg); >>>>> >>>>> - par->state.vgabase = (void __iomem *) vga_res.start; >>>>> + par->state.vgabase = (void __iomem *) (unsigned long) vga_res.start; >>>> >>>> This does look quite ugly... Where does the warning come from in the >>>> first place. Isn't vga_res.start (resource_size_t) the size of a pointer? >>>> >>> Yes looks ugly, I am not sure what you meant from 'where does this warning >>> come from' its in the commit message. >> >> I meant why is there a warning at all. With a quick glance, >> vga_res.start is the size of a pointer. So the sizes of the integer and >> the pointer should be the same. But the warning still says "of different >> size". > > poking my nose into your discussion. > I tried to see the warning, and I re-compiled like make W=1 M=drivers/video/fbdev/ (before that make clean M=drivers/video/fbdev was done) > I can see warning with many other files, but drivers/video/fbdev/vt8623fb.o was quiet and there was no warning. > I tested with next=20141203. > did i miss something in checking the warning ? I don't see the warning either when compiling for arm or x86_64. On what architecture do you see the warning? Tomi [-- Attachment #2: OpenPGP digital signature --] [-- Type: application/pgp-signature, Size: 819 bytes --] ^ permalink raw reply [flat|nested] 10+ messages in thread
* Re: [PATCH 1/3] video: fbdev: vt8623fb: suppress build warning 2014-12-04 7:56 ` Tomi Valkeinen @ 2014-12-04 13:29 ` Geert Uytterhoeven 2014-12-04 13:40 ` Tomi Valkeinen 0 siblings, 1 reply; 10+ messages in thread From: Geert Uytterhoeven @ 2014-12-04 13:29 UTC (permalink / raw) To: Tomi Valkeinen Cc: Prabhakar Lad, Sudip Mukherjee, Jean-Christophe Plagniol-Villard, LFBDEV, LKML On Thu, Dec 4, 2014 at 8:56 AM, Tomi Valkeinen <tomi.valkeinen@ti.com> wrote: > On 04/12/14 09:46, Sudip Mukherjee wrote: >> On Thu, Dec 04, 2014 at 09:05:46AM +0200, Tomi Valkeinen wrote: >>> On 03/12/14 20:29, Prabhakar Lad wrote: >>>> On Wed, Dec 3, 2014 at 11:49 AM, Tomi Valkeinen <tomi.valkeinen@ti.com> wrote: >>>>> On 27/11/14 00:07, Lad, Prabhakar wrote: >>>>>> this patch fixes following build warning: >>>>>> drivers/video/fbdev/vt8623fb.c: In function ‘vt8623_pci_probe’: >>>>>> drivers/video/fbdev/vt8623fb.c:734:23: warning: cast to pointer from integer of different size [-Wint-to-pointer-cast] >>>>>> par->state.vgabase = (void __iomem *) vga_res.start; >>>>>> ^ >>>>>> Signed-off-by: Lad, Prabhakar <prabhakar.csengg@gmail.com> >>>>>> --- >>>>>> drivers/video/fbdev/vt8623fb.c | 2 +- >>>>>> 1 file changed, 1 insertion(+), 1 deletion(-) >>>>>> >>>>>> diff --git a/drivers/video/fbdev/vt8623fb.c b/drivers/video/fbdev/vt8623fb.c >>>>>> index 5c7cbc6..ea7f056 100644 >>>>>> --- a/drivers/video/fbdev/vt8623fb.c >>>>>> +++ b/drivers/video/fbdev/vt8623fb.c >>>>>> @@ -731,7 +731,7 @@ static int vt8623_pci_probe(struct pci_dev *dev, const struct pci_device_id *id) >>>>>> >>>>>> pcibios_bus_to_resource(dev->bus, &vga_res, &bus_reg); >>>>>> >>>>>> - par->state.vgabase = (void __iomem *) vga_res.start; >>>>>> + par->state.vgabase = (void __iomem *) (unsigned long) vga_res.start; >>>>> >>>>> This does look quite ugly... Where does the warning come from in the >>>>> first place. Isn't vga_res.start (resource_size_t) the size of a pointer? >>>>> >>>> Yes looks ugly, I am not sure what you meant from 'where does this warning >>>> come from' its in the commit message. >>> >>> I meant why is there a warning at all. With a quick glance, >>> vga_res.start is the size of a pointer. So the sizes of the integer and >>> the pointer should be the same. But the warning still says "of different >>> size". >> >> poking my nose into your discussion. >> I tried to see the warning, and I re-compiled like make W=1 M=drivers/video/fbdev/ (before that make clean M=drivers/video/fbdev was done) >> I can see warning with many other files, but drivers/video/fbdev/vt8623fb.o was quiet and there was no warning. >> I tested with next=20141203. >> did i miss something in checking the warning ? > > I don't see the warning either when compiling for arm or x86_64. On what > architecture do you see the warning? On 32-bit systems with PHYS_ADDR_T_64BIT=y, resource_size_t is u64, while pointers are still 32-bit. Gr{oetje,eeting}s, Geert -- Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert@linux-m68k.org In personal conversations with technical people, I call myself a hacker. But when I'm talking to journalists I just say "programmer" or something like that. -- Linus Torvalds ^ permalink raw reply [flat|nested] 10+ messages in thread
* Re: [PATCH 1/3] video: fbdev: vt8623fb: suppress build warning 2014-12-04 13:29 ` Geert Uytterhoeven @ 2014-12-04 13:40 ` Tomi Valkeinen 0 siblings, 0 replies; 10+ messages in thread From: Tomi Valkeinen @ 2014-12-04 13:40 UTC (permalink / raw) To: Geert Uytterhoeven, Prabhakar Lad Cc: Sudip Mukherjee, Jean-Christophe Plagniol-Villard, LFBDEV, LKML [-- Attachment #1: Type: text/plain, Size: 2864 bytes --] On 04/12/14 15:29, Geert Uytterhoeven wrote: > On Thu, Dec 4, 2014 at 8:56 AM, Tomi Valkeinen <tomi.valkeinen@ti.com> wrote: >> On 04/12/14 09:46, Sudip Mukherjee wrote: >>> On Thu, Dec 04, 2014 at 09:05:46AM +0200, Tomi Valkeinen wrote: >>>> On 03/12/14 20:29, Prabhakar Lad wrote: >>>>> On Wed, Dec 3, 2014 at 11:49 AM, Tomi Valkeinen <tomi.valkeinen@ti.com> wrote: >>>>>> On 27/11/14 00:07, Lad, Prabhakar wrote: >>>>>>> this patch fixes following build warning: >>>>>>> drivers/video/fbdev/vt8623fb.c: In function ‘vt8623_pci_probe’: >>>>>>> drivers/video/fbdev/vt8623fb.c:734:23: warning: cast to pointer from integer of different size [-Wint-to-pointer-cast] >>>>>>> par->state.vgabase = (void __iomem *) vga_res.start; >>>>>>> ^ >>>>>>> Signed-off-by: Lad, Prabhakar <prabhakar.csengg@gmail.com> >>>>>>> --- >>>>>>> drivers/video/fbdev/vt8623fb.c | 2 +- >>>>>>> 1 file changed, 1 insertion(+), 1 deletion(-) >>>>>>> >>>>>>> diff --git a/drivers/video/fbdev/vt8623fb.c b/drivers/video/fbdev/vt8623fb.c >>>>>>> index 5c7cbc6..ea7f056 100644 >>>>>>> --- a/drivers/video/fbdev/vt8623fb.c >>>>>>> +++ b/drivers/video/fbdev/vt8623fb.c >>>>>>> @@ -731,7 +731,7 @@ static int vt8623_pci_probe(struct pci_dev *dev, const struct pci_device_id *id) >>>>>>> >>>>>>> pcibios_bus_to_resource(dev->bus, &vga_res, &bus_reg); >>>>>>> >>>>>>> - par->state.vgabase = (void __iomem *) vga_res.start; >>>>>>> + par->state.vgabase = (void __iomem *) (unsigned long) vga_res.start; >>>>>> >>>>>> This does look quite ugly... Where does the warning come from in the >>>>>> first place. Isn't vga_res.start (resource_size_t) the size of a pointer? >>>>>> >>>>> Yes looks ugly, I am not sure what you meant from 'where does this warning >>>>> come from' its in the commit message. >>>> >>>> I meant why is there a warning at all. With a quick glance, >>>> vga_res.start is the size of a pointer. So the sizes of the integer and >>>> the pointer should be the same. But the warning still says "of different >>>> size". >>> >>> poking my nose into your discussion. >>> I tried to see the warning, and I re-compiled like make W=1 M=drivers/video/fbdev/ (before that make clean M=drivers/video/fbdev was done) >>> I can see warning with many other files, but drivers/video/fbdev/vt8623fb.o was quiet and there was no warning. >>> I tested with next=20141203. >>> did i miss something in checking the warning ? >> >> I don't see the warning either when compiling for arm or x86_64. On what >> architecture do you see the warning? > > On 32-bit systems with PHYS_ADDR_T_64BIT=y, resource_size_t is u64, > while pointers are still 32-bit. Ah, I see. Yes, I can reproduce the warning with that config. So, still rather ugly, but looks correct to me, so I'll apply the series. Tomi [-- Attachment #2: OpenPGP digital signature --] [-- Type: application/pgp-signature, Size: 819 bytes --] ^ permalink raw reply [flat|nested] 10+ messages in thread
end of thread, other threads:[~2014-12-04 13:41 UTC | newest] Thread overview: 10+ messages (download: mbox.gz / follow: Atom feed) -- links below jump to the message on this page -- 2014-11-26 22:07 [PATCH 1/3] video: fbdev: vt8623fb: suppress build warning Lad, Prabhakar 2014-11-26 22:07 ` [PATCH 2/3] video: fbdev: s3fb: " Lad, Prabhakar 2014-11-26 22:07 ` [PATCH 3/3] video: fbdev: arkfb: " Lad, Prabhakar 2014-12-03 11:49 ` [PATCH 1/3] video: fbdev: vt8623fb: " Tomi Valkeinen 2014-12-03 18:29 ` Prabhakar Lad 2014-12-04 7:05 ` Tomi Valkeinen 2014-12-04 7:46 ` Sudip Mukherjee 2014-12-04 7:56 ` Tomi Valkeinen 2014-12-04 13:29 ` Geert Uytterhoeven 2014-12-04 13:40 ` Tomi Valkeinen
This is a public inbox, see mirroring instructions for how to clone and mirror all data and code used for this inbox; as well as URLs for NNTP newsgroup(s).