> -----Original Message----- > From: Greg KH [mailto:[email protected]] > Sent: Monday, August 29, 2011 2:10 PM > To: KY Srinivasan > Cc: [email protected]; [email protected]; > [email protected]; [email protected]; Haiyang Zhang > Subject: Re: [PATCH 44/46] Staging: hv: vmbus: Fix checkpatch warnings in > connection.c > > On Sat, Aug 27, 2011 at 11:31:43AM -0700, K. Y. Srinivasan wrote: > > Fix checkpatch warnings in connection.c. > > > > Signed-off-by: K. Y. Srinivasan <[email protected]> > > Signed-off-by: Haiyang Zhang <[email protected]> > > --- > > drivers/staging/hv/connection.c | 13 +++++++------ > > 1 files changed, 7 insertions(+), 6 deletions(-) > > > > diff --git a/drivers/staging/hv/connection.c > > b/drivers/staging/hv/connection.c > > index ca92ca3..9e99c04 100644 > > --- a/drivers/staging/hv/connection.c > > +++ b/drivers/staging/hv/connection.c > > @@ -220,11 +220,11 @@ static void process_chn_event(u32 relid) > > channel = relid2channel(relid); > > > > spin_lock_irqsave(&channel->inbound_lock, flags); > > - if (channel && (channel->onchannel_callback != NULL)) { > > + if (channel && (channel->onchannel_callback != NULL)) > > channel->onchannel_callback(channel- > >channel_callback_context); > > - } else { > > I agree with Joe here, if channel really was NULL, you just oopsed. > > I'll apply this one, but please send me a follow-on one fixing this bug.
Thanks Greg. I will fix this. I got these patches out just before Hurricane Irene hit the east coast. While we were lucky that it was not as bad as was predicted, we lost power and we still don't have power. I have come to a public library in a nearby town to check my email. So, my responses will be sporadic over the next couple of days (until we get power). I will try to address the issues you have raised as quickly as possible. Regards, K. Y _______________________________________________ devel mailing list [email protected] http://driverdev.linuxdriverproject.org/mailman/listinfo/devel
