Hello Kurt. If you're still interested in tmscim.
Your garl...@suse.de address bounces.... btw: Your MAINTAINERS entry is out of date. MAINTAINERS-DC390/AM53C974 SCSI driver MAINTAINERS:M: Kurt Garloff <garl...@suse.de> MAINTAINERS:W: http://www.garloff.de/kurt/linux/dc390/ MAINTAINERS-M: Guennadi Liakhovetski <g.liakhovet...@gmx.de> MAINTAINERS-S: Maintained MAINTAINERS-F: drivers/scsi/tmscsim.* -------- Forwarded Message -------- From: Joe Perches <j...@perches.com> To: Fabian Frederick <f...@skynet.be> Cc: linux-kernel <linux-kernel@vger.kernel.org>, Kurt Garloff <garl...@suse.de>, akpm <a...@linux-foundation.org> Subject: Re: [PATCH 1/1] drivers/scsi/tmscsim.c: replace shift loop by ilog2 On Tue, 2014-05-20 at 18:48 +0200, Fabian Frederick wrote: > Cc: Kurt Garloff <garl...@suse.de> > Cc: Andrew Morton <a...@linux-foundation.org> > Signed-off-by: Fabian Frederick <f...@skynet.be> > --- > drivers/scsi/tmscsim.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/drivers/scsi/tmscsim.c b/drivers/scsi/tmscsim.c [] > @@ -1611,7 +1611,7 @@ dc390_Reselect( struct dc390_acb* pACB ) > printk (KERN_ERR "DC390: Reselection must select host adapter: > %02x!\n", lun); > else > lun ^= 1 << pACB->pScsiHost->this_id; /* Mask AdapterID */ > - id = 0; while (lun >>= 1) id++; > + id = ilog2(lun); > /* Get LUN */ > lun = DC390_read8 (ScsiFifo); > if (!(lun & IDENTIFY_BASE)) printk (KERN_ERR "DC390: Resel: Expect > identify message!\n"); Hey Fabian. You've submitted several of these now. Have you gone through all of these to make sure that the ilog2(<foo>) uses are always using a non-zero <foo>? If so, that should be mentioned in the changelog. These should probably add #include <linux/log2.h> It would have been nicer to use a patch series and a cover letter so I could make this reply to to cover letter. Please take the time to setup and use git send-email instead of sylpheed. -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
