Jean: Thanks. I gave this a whirl because I honestly couldn't remember exactly how these numbers were reported by the FW and it seems to work correctly.
Roland, can you pull in Jean's patch? Thanks, Tom On Thu, 2006-11-09 at 21:02 +0100, Jean Delvare wrote: > Fix the AMSO1100 firmware version computation, which was broken > due to "&&" being used where "&" should have. > > Signed-off-by: Jean Delvare <[EMAIL PROTECTED]> > --- > drivers/infiniband/hw/amso1100/c2_rnic.c | 4 ++-- > 1 file changed, 2 insertions(+), 2 deletions(-) > > --- linux-2.6.19-rc5.orig/drivers/infiniband/hw/amso1100/c2_rnic.c > 2006-11-09 10:30:33.000000000 +0100 > +++ linux-2.6.19-rc5/drivers/infiniband/hw/amso1100/c2_rnic.c 2006-11-09 > 20:50:28.000000000 +0100 > @@ -157,8 +157,8 @@ > > props->fw_ver = > ((u64)be32_to_cpu(reply->fw_ver_major) << 32) | > - ((be32_to_cpu(reply->fw_ver_minor) && 0xFFFF) << 16) | > - (be32_to_cpu(reply->fw_ver_patch) && 0xFFFF); > + ((be32_to_cpu(reply->fw_ver_minor) & 0xFFFF) << 16) | > + (be32_to_cpu(reply->fw_ver_patch) & 0xFFFF); > memcpy(&props->sys_image_guid, c2dev->netdev->dev_addr, 6); > props->max_mr_size = 0xFFFFFFFF; > props->page_size_cap = ~(C2_MIN_PAGESIZE-1); > > _______________________________________________ openib-general mailing list openib-general@openib.org http://openib.org/mailman/listinfo/openib-general To unsubscribe, please visit http://openib.org/mailman/listinfo/openib-general