Messages in this thread | | | From | KY Srinivasan <> | Subject | RE: [PATCH 20/20] Staging: hv: netvsc: Rename deviceInfo as device_info | Date | Tue, 13 Sep 2011 18:15:28 +0000 |
| |
> -----Original Message----- > From: Joe Perches [mailto:joe@perches.com] > Sent: Tuesday, September 13, 2011 2:11 PM > To: KY Srinivasan > Cc: gregkh@suse.de; linux-kernel@vger.kernel.org; > devel@linuxdriverproject.org; virtualization@lists.osdl.org; Haiyang Zhang > Subject: Re: [PATCH 20/20] Staging: hv: netvsc: Rename deviceInfo as > device_info > > On Tue, 2011-09-13 at 10:59 -0700, K. Y. Srinivasan wrote: > > Rename deviceInfo as device_info. > [] > > diff --git a/drivers/staging/hv/rndis_filter.c b/drivers/staging/hv/rndis_filter.c > [] > > @@ -732,15 +732,15 @@ int rndis_filter_device_add(struct hv_device *dev, > [] > > dev_info(&dev->device, "Device MAC %pM link state %s", > > rndis_device->hw_mac_adr, > > - ((deviceInfo->link_state) ? ("down\n") : ("up\n"))); > > + ((device_info->link_state) ? ("down\n") : ("up\n"))); > > Maybe get rid of the superfluous parentheses too? > > dev_info(&dev->device, "Device MAC %pM link state %s\n", > rndis_device->hw_mac_adr, > device_info->link_state ? "down" : "up"); > > Also, struct rndis_device uses > u32 link_stat; > that is copied into struct netvsc_device_info > bool link_state; > could you please rationalize the names and types > of those 2 variables?
Thanks Joe; I will get a patch out soon.
Regards,
K. Y
| |