> -----Original Message----- > From: Bart Van Assche [mailto:Bart.VanAssche@xxxxxxx] > Sent: Thursday, December 21, 2017 10:43 AM > To: jejb@xxxxxxxxxxxxxxxxxx; Raghava Aditya Renukunta > <RaghavaAditya.Renukunta@xxxxxxxxxxxxx>; linux-scsi@xxxxxxxxxxxxxxx; > martin.petersen@xxxxxxxxxx > Cc: dl-esc-Aacraid Linux Driver <aacraid@xxxxxxxxxxxxx>; > gpiccoli@xxxxxxxxxxxxxxxxxx; Tom White <tom.white@xxxxxxxxxxxxx>; > Scott Benesh <scott.benesh@xxxxxxxxxxxxx> > Subject: Re: [PATCH 21/29] scsi: aacraid: Refactor resolve luns code and scsi > functions > > EXTERNAL EMAIL > > > On Thu, 2017-12-21 at 09:34 -0800, Raghava Aditya Renukunta wrote: > > + if (bus == CONTAINER_CHANNEL) > > + bus = CONTAINER_CHANNEL; > > Sorry but I don't see how the above code can be useful? I agree, I did not want to change legacy code a bit too much. I will refactor this in the next iteration. > Thanks, > > Bart.