Netdev Archive on lore.kernel.org
help / color / mirror / Atom feed
From: Greg KH <gregkh@linuxfoundation.org>
To: Pavel Skripkin <paskripkin@gmail.com>,
	davem@davemloft.net, kuba@kernel.org, linux-usb@vger.kernel.org,
	netdev@vger.kernel.org, linux-kernel@vger.kernel.org,
	syzbot+02c9f70f3afae308464a@syzkaller.appspotmail.com
Subject: Re: [PATCH] net: pegasus: fix uninit-value in get_interrupt_interval
Date: Tue, 3 Aug 2021 07:36:57 +0200	[thread overview]
Message-ID: <YQjV+amsR+8PLVfZ@kroah.com> (raw)
In-Reply-To: <YQhQe4bdoEAef8bj@carbon>

On Mon, Aug 02, 2021 at 11:07:23PM +0300, Petko Manolov wrote:
> diff --git a/drivers/net/usb/pegasus.c b/drivers/net/usb/pegasus.c
> index 9a907182569c..eafbe8107907 100644
> --- a/drivers/net/usb/pegasus.c
> +++ b/drivers/net/usb/pegasus.c
> @@ -26,6 +26,8 @@
>   *		...
>   *		v0.9.3	simplified [get|set]_register(s), async update registers
>   *			logic revisited, receive skb_pool removed.
> + *		v1.0.1	add error checking for set_register(s)(), see if calling
> + *			get_registers() has failed and print a message accordingly.
>   */
>  
>  #include <linux/sched.h>
> @@ -45,7 +47,7 @@
>  /*
>   * Version Information
>   */
> -#define DRIVER_VERSION "v0.9.3 (2013/04/25)"
> +#define DRIVER_VERSION "v1.0.1 (2021/08/01)"

Nit, the log above, and the driver version here, mean nothing when it
comes to code in the kernel tree, both should be dropped as we have full
kernel changelog through git, and the version is bound to the kernel
release the driver came in.

thanks,

greg k-h

  parent reply	other threads:[~2021-08-03  5:37 UTC|newest]

Thread overview: 11+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2021-07-30 21:44 Pavel Skripkin
2021-08-01 12:36 ` Petko Manolov
2021-08-01 19:35   ` Pavel Skripkin
2021-08-01 20:24     ` Petko Manolov
2021-08-02 20:07     ` Petko Manolov
2021-08-02 22:18       ` Pavel Skripkin
2021-08-03  5:36       ` Greg KH [this message]
2021-08-04 10:44 ` Pavel Skripkin
2021-08-04 11:58   ` Jakub Kicinski
2021-08-04 14:30     ` [PATCH net v2] " Pavel Skripkin
2021-08-05 14:50       ` patchwork-bot+netdevbpf

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=YQjV+amsR+8PLVfZ@kroah.com \
    --to=gregkh@linuxfoundation.org \
    --cc=davem@davemloft.net \
    --cc=kuba@kernel.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=linux-usb@vger.kernel.org \
    --cc=netdev@vger.kernel.org \
    --cc=paskripkin@gmail.com \
    --cc=syzbot+02c9f70f3afae308464a@syzkaller.appspotmail.com \
    --subject='Re: [PATCH] net: pegasus: fix uninit-value in get_interrupt_interval' \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link

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).