Re: [patch] qla4xxx: remove some bogus casts

[Bart Van Assche <Bart.VanAssche@sandisk.com>]

On Sat, 2015-08-22 at 18:41 -0700, Bart Van Assche wrote:
> On 08/18/15 01:47, Dan Carpenter wrote:
> > On Mon, Aug 17, 2015 at 09:45:45PM -0700, Bart Van Assche wrote:
> > > On 08/17/2015 07:36 AM, Dan Carpenter wrote:
> > > > These casts are wrong and unnecessary.  They annoy static checkers
> > > > because they imply we are planning to write sizeof(long) bytes to a
> > > > sizeof(u32) buffer which would corrupt memory.
> > > 
> > > Hello Dan,
> > > 
> > > Can you verify whether that patch is still needed after having
> > > applied the patch series I posted in July (and that was acknowledged
> > > by QLogic): http://thread.gmane.org/gmane.linux.scsi/103135 ?
> > > 
> > 
> > I think it is needed still?
> 
> Hello Dan,
> 
> You are right, this patch is indeed needed. Hence:
> 
> Reviewed-by: Bart Van Assche <bart.vanassche@sandisk.com>

Hello Martin,

Do you need more reviews to queue this patch?

Thanks,

Bart.