[U-Boot] [EXT] Re: [PATCH 7/7] scsi: dts: a3700: add scsi node

[Ken Ma]

Hi Stefan

I think it's a good way, but I wonder why the codes calls ffs() but not fls()?
If the linkmap is 0xffff, it seems that ffs() returns 1 while fls() returns 16, I think max_id should be 16 then.

Yours,
Ken

-----Original Message-----
From: Stefan Roese [mailto:sr at denx.de] 
Sent: 2017年3月24日 21:24
To: Ken Ma; u-boot at lists.denx.de
Cc: Simon Glass; Michal Simek; Kostya Porotchkin; Hua Jing; Wilson Ding
Subject: Re: [EXT] Re: [PATCH 7/7] scsi: dts: a3700: add scsi node

Hi Ken,

btw, regarding the max-lun and max-id you could perhaps take a look at this recent patch, also dealing with these parameters:

https://patchwork.ozlabs.org/patch/743160/

I've not looked too close into this, just wanted to point it out to you.

Thanks,
Stefan