* [1/4] usb: xhci: fix build warning - missing prototype
@ 2019-02-20 20:54 Jean-Philippe Menil
0 siblings, 0 replies; 9+ messages in thread
From: Jean-Philippe Menil @ 2019-02-20 20:54 UTC (permalink / raw)
To: Greg KH; +Cc: Mathias Nyman, linux-usb, Jean-Philippe Menil
On Wed, 20 Feb 2019, Greg KH wrote:
> On Wed, Feb 20, 2019 at 07:50:52PM +0200, Mathias Nyman wrote:
> > From: Jean-Philippe Menil <jpmenil@gmail.com>
> >
> > Fix build warning when building drivers/usb/host/xhci-mem.o due to missing
> > prototype for xhci_free_virt_devices_depth_first.
>
> There is no normal "build warning", this is a sparse fixup, right?
>
>
Hi Greg,
this is a build warning, at least with W=1 :
drivers/usb/host/xhci-mem.c:936:6: warning: no previous prototype for
‘xhci_free_virt_devices_depth_first’ [-Wmissing-prototypes]
void xhci_free_virt_devices_depth_first(struct xhci_hcd *xhci, int
slot_id)
Regards,
Jean-Philippe
^ permalink raw reply [flat|nested] 9+ messages in thread
* [1/4] usb: xhci: fix build warning - missing prototype
@ 2019-02-21 9:52 Jean-Philippe Menil
0 siblings, 0 replies; 9+ messages in thread
From: Jean-Philippe Menil @ 2019-02-21 9:52 UTC (permalink / raw)
To: Greg KH; +Cc: Jean-Philippe Menil, Mathias Nyman, linux-usb
On Thu, 21 Feb 2019, Greg KH wrote:
> On Thu, Feb 21, 2019 at 10:39:46AM +0100, Jean-Philippe Menil wrote:
> > On Thu, 21 Feb 2019, Greg KH wrote:
> >
> > > On Wed, Feb 20, 2019 at 09:54:12PM +0100, Jean-Philippe Menil wrote:
> > > > On Wed, 20 Feb 2019, Greg KH wrote:
> > > >
> > > > > On Wed, Feb 20, 2019 at 07:50:52PM +0200, Mathias Nyman wrote:
> > > > > > From: Jean-Philippe Menil <jpmenil@gmail.com>
> > > > > >
> > > > > > Fix build warning when building drivers/usb/host/xhci-mem.o due to missing
> > > > > > prototype for xhci_free_virt_devices_depth_first.
> > > > >
> > > > > There is no normal "build warning", this is a sparse fixup, right?
> > > > >
> > > > >
> > > >
> > > > Hi Greg,
> > > >
> > > > this is a build warning, at least with W=1 :
> > > >
> > > > drivers/usb/host/xhci-mem.c:936:6: warning: no previous prototype for
> > > > ‘xhci_free_virt_devices_depth_first’ [-Wmissing-prototypes]
> > > > void xhci_free_virt_devices_depth_first(struct xhci_hcd *xhci, int
> > > > slot_id)
> > >
> > > "W=1" is not a normal build option for anyone to use, except those that
> > > wish to find crazy issues like this :)
> > >
> > > Also, if that is the only way it shows up, you should mention it in the
> > > changelog.
> > >
> > > thanks,
> > >
> > > greg k-h
> > >
> >
> > Hmm, rereaind this mail from kernel-janitors
> > https://marc.info/?l=kernel-janitors&m=154178546220848&w=2
> > seem that you're right, this is not a normal option :)
> >
> > Mathias, do you want me to re-send something?
>
> No need, I'll go edit the changelog text...
>
> greg k-h
>
Ack,
many thanks for that, appreciated !
Regards,
Jean-Philippe
^ permalink raw reply [flat|nested] 9+ messages in thread
* [1/4] usb: xhci: fix build warning - missing prototype
@ 2019-02-21 9:49 Greg Kroah-Hartman
0 siblings, 0 replies; 9+ messages in thread
From: Greg Kroah-Hartman @ 2019-02-21 9:49 UTC (permalink / raw)
To: Jean-Philippe Menil; +Cc: Mathias Nyman, linux-usb
On Thu, Feb 21, 2019 at 10:39:46AM +0100, Jean-Philippe Menil wrote:
> On Thu, 21 Feb 2019, Greg KH wrote:
>
> > On Wed, Feb 20, 2019 at 09:54:12PM +0100, Jean-Philippe Menil wrote:
> > > On Wed, 20 Feb 2019, Greg KH wrote:
> > >
> > > > On Wed, Feb 20, 2019 at 07:50:52PM +0200, Mathias Nyman wrote:
> > > > > From: Jean-Philippe Menil <jpmenil@gmail.com>
> > > > >
> > > > > Fix build warning when building drivers/usb/host/xhci-mem.o due to missing
> > > > > prototype for xhci_free_virt_devices_depth_first.
> > > >
> > > > There is no normal "build warning", this is a sparse fixup, right?
> > > >
> > > >
> > >
> > > Hi Greg,
> > >
> > > this is a build warning, at least with W=1 :
> > >
> > > drivers/usb/host/xhci-mem.c:936:6: warning: no previous prototype for
> > > ‘xhci_free_virt_devices_depth_first’ [-Wmissing-prototypes]
> > > void xhci_free_virt_devices_depth_first(struct xhci_hcd *xhci, int
> > > slot_id)
> >
> > "W=1" is not a normal build option for anyone to use, except those that
> > wish to find crazy issues like this :)
> >
> > Also, if that is the only way it shows up, you should mention it in the
> > changelog.
> >
> > thanks,
> >
> > greg k-h
> >
>
> Hmm, rereaind this mail from kernel-janitors
> https://marc.info/?l=kernel-janitors&m=154178546220848&w=2
> seem that you're right, this is not a normal option :)
>
> Mathias, do you want me to re-send something?
No need, I'll go edit the changelog text...
greg k-h
^ permalink raw reply [flat|nested] 9+ messages in thread
* [1/4] usb: xhci: fix build warning - missing prototype
@ 2019-02-21 9:39 Jean-Philippe Menil
0 siblings, 0 replies; 9+ messages in thread
From: Jean-Philippe Menil @ 2019-02-21 9:39 UTC (permalink / raw)
To: Greg KH; +Cc: Jean-Philippe Menil, Mathias Nyman, linux-usb
On Thu, 21 Feb 2019, Greg KH wrote:
> On Wed, Feb 20, 2019 at 09:54:12PM +0100, Jean-Philippe Menil wrote:
> > On Wed, 20 Feb 2019, Greg KH wrote:
> >
> > > On Wed, Feb 20, 2019 at 07:50:52PM +0200, Mathias Nyman wrote:
> > > > From: Jean-Philippe Menil <jpmenil@gmail.com>
> > > >
> > > > Fix build warning when building drivers/usb/host/xhci-mem.o due to missing
> > > > prototype for xhci_free_virt_devices_depth_first.
> > >
> > > There is no normal "build warning", this is a sparse fixup, right?
> > >
> > >
> >
> > Hi Greg,
> >
> > this is a build warning, at least with W=1 :
> >
> > drivers/usb/host/xhci-mem.c:936:6: warning: no previous prototype for
> > ‘xhci_free_virt_devices_depth_first’ [-Wmissing-prototypes]
> > void xhci_free_virt_devices_depth_first(struct xhci_hcd *xhci, int
> > slot_id)
>
> "W=1" is not a normal build option for anyone to use, except those that
> wish to find crazy issues like this :)
>
> Also, if that is the only way it shows up, you should mention it in the
> changelog.
>
> thanks,
>
> greg k-h
>
Hmm, rereaind this mail from kernel-janitors
https://marc.info/?l=kernel-janitors&m=154178546220848&w=2
seem that you're right, this is not a normal option :)
Mathias, do you want me to re-send something?
Regards,
^ permalink raw reply [flat|nested] 9+ messages in thread
* [1/4] usb: xhci: fix build warning - missing prototype
@ 2019-02-21 8:41 Greg Kroah-Hartman
0 siblings, 0 replies; 9+ messages in thread
From: Greg Kroah-Hartman @ 2019-02-21 8:41 UTC (permalink / raw)
To: Jean-Philippe Menil; +Cc: Mathias Nyman, linux-usb
On Wed, Feb 20, 2019 at 09:54:12PM +0100, Jean-Philippe Menil wrote:
> On Wed, 20 Feb 2019, Greg KH wrote:
>
> > On Wed, Feb 20, 2019 at 07:50:52PM +0200, Mathias Nyman wrote:
> > > From: Jean-Philippe Menil <jpmenil@gmail.com>
> > >
> > > Fix build warning when building drivers/usb/host/xhci-mem.o due to missing
> > > prototype for xhci_free_virt_devices_depth_first.
> >
> > There is no normal "build warning", this is a sparse fixup, right?
> >
> >
>
> Hi Greg,
>
> this is a build warning, at least with W=1 :
>
> drivers/usb/host/xhci-mem.c:936:6: warning: no previous prototype for
> ‘xhci_free_virt_devices_depth_first’ [-Wmissing-prototypes]
> void xhci_free_virt_devices_depth_first(struct xhci_hcd *xhci, int
> slot_id)
"W=1" is not a normal build option for anyone to use, except those that
wish to find crazy issues like this :)
Also, if that is the only way it shows up, you should mention it in the
changelog.
thanks,
greg k-h
^ permalink raw reply [flat|nested] 9+ messages in thread
* [1/4] usb: xhci: fix build warning - missing prototype
@ 2019-02-20 19:17 Greg Kroah-Hartman
0 siblings, 0 replies; 9+ messages in thread
From: Greg Kroah-Hartman @ 2019-02-20 19:17 UTC (permalink / raw)
To: Mathias Nyman; +Cc: linux-usb, Jean-Philippe Menil
On Wed, Feb 20, 2019 at 07:50:52PM +0200, Mathias Nyman wrote:
> From: Jean-Philippe Menil <jpmenil@gmail.com>
>
> Fix build warning when building drivers/usb/host/xhci-mem.o due to missing
> prototype for xhci_free_virt_devices_depth_first.
There is no normal "build warning", this is a sparse fixup, right?
^ permalink raw reply [flat|nested] 9+ messages in thread
* [1/4] usb: xhci: fix build warning - missing prototype
@ 2019-02-20 18:44 Greg Kroah-Hartman
0 siblings, 0 replies; 9+ messages in thread
From: Greg Kroah-Hartman @ 2019-02-20 18:44 UTC (permalink / raw)
To: Mathias Nyman; +Cc: linux-usb, Jean-Philippe Menil
On Wed, Feb 20, 2019 at 08:01:08PM +0200, Mathias Nyman wrote:
> On 20.2.2019 19.50, Mathias Nyman wrote:
> > From: Jean-Philippe Menil <jpmenil@gmail.com>
> >
> > Fix build warning when building drivers/usb/host/xhci-mem.o due to missing
> > prototype for xhci_free_virt_devices_depth_first.
> >
> > This function is only used in xhci-mem.c so just make it static.
> >
> > Signed-off-by: Jean-Philippe Menil <jpmenil@gmail.com>
>
> My Signed-off-by was apparently lost after I did some patch juggling
>
> Signed-off-by: Mathias Nyman <mathias.nyman@linux.intel.com>
>
> Want me to resend?
Nah, I can fix it up when I apply it in the morning.
thanks,
greg k-h
^ permalink raw reply [flat|nested] 9+ messages in thread
* [1/4] usb: xhci: fix build warning - missing prototype
@ 2019-02-20 18:01 Mathias Nyman
0 siblings, 0 replies; 9+ messages in thread
From: Mathias Nyman @ 2019-02-20 18:01 UTC (permalink / raw)
To: gregkh; +Cc: linux-usb, Jean-Philippe Menil
On 20.2.2019 19.50, Mathias Nyman wrote:
> From: Jean-Philippe Menil <jpmenil@gmail.com>
>
> Fix build warning when building drivers/usb/host/xhci-mem.o due to missing
> prototype for xhci_free_virt_devices_depth_first.
>
> This function is only used in xhci-mem.c so just make it static.
>
> Signed-off-by: Jean-Philippe Menil <jpmenil@gmail.com>
My Signed-off-by was apparently lost after I did some patch juggling
Signed-off-by: Mathias Nyman <mathias.nyman@linux.intel.com>
Want me to resend?
-Mathias
^ permalink raw reply [flat|nested] 9+ messages in thread
* [1/4] usb: xhci: fix build warning - missing prototype
@ 2019-02-20 17:50 Mathias Nyman
0 siblings, 0 replies; 9+ messages in thread
From: Mathias Nyman @ 2019-02-20 17:50 UTC (permalink / raw)
To: gregkh; +Cc: linux-usb, Jean-Philippe Menil
From: Jean-Philippe Menil <jpmenil@gmail.com>
Fix build warning when building drivers/usb/host/xhci-mem.o due to missing
prototype for xhci_free_virt_devices_depth_first.
This function is only used in xhci-mem.c so just make it static.
Signed-off-by: Jean-Philippe Menil <jpmenil@gmail.com>
---
drivers/usb/host/xhci-mem.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/usb/host/xhci-mem.c b/drivers/usb/host/xhci-mem.c
index 8067f17..cf5e179 100644
--- a/drivers/usb/host/xhci-mem.c
+++ b/drivers/usb/host/xhci-mem.c
@@ -933,7 +933,7 @@ void xhci_free_virt_device(struct xhci_hcd *xhci, int slot_id)
* that tt_info, then free the child first. Recursive.
* We can't rely on udev at this point to find child-parent relationships.
*/
-void xhci_free_virt_devices_depth_first(struct xhci_hcd *xhci, int slot_id)
+static void xhci_free_virt_devices_depth_first(struct xhci_hcd *xhci, int slot_id)
{
struct xhci_virt_device *vdev;
struct list_head *tt_list_head;
^ permalink raw reply related [flat|nested] 9+ messages in thread
end of thread, other threads:[~2019-02-21 9:52 UTC | newest]
Thread overview: 9+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2019-02-20 20:54 [1/4] usb: xhci: fix build warning - missing prototype Jean-Philippe Menil
-- strict thread matches above, loose matches on Subject: below --
2019-02-21 9:52 Jean-Philippe Menil
2019-02-21 9:49 Greg Kroah-Hartman
2019-02-21 9:39 Jean-Philippe Menil
2019-02-21 8:41 Greg Kroah-Hartman
2019-02-20 19:17 Greg Kroah-Hartman
2019-02-20 18:44 Greg Kroah-Hartman
2019-02-20 18:01 Mathias Nyman
2019-02-20 17:50 Mathias Nyman
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.