and one more hint, if is expression in ruby, so you can write it this way:
```
dcb_default = if card["driver"] != "bnx2x" && dcb_capable == "yes"
     "yes"
   else
     "no"
   end
```

but when I read your code I think you can achieve it with simple
```
# exception for Broadcom cards: DCB_REQUIRED should be set to "no" (bnc #728658) 
dcb_default = "no" if card["driver"] == "bnx2x"
```