On 04/20/2015 07:04 PM, Hal Rosenstock wrote: > On 4/20/2015 4:33 AM, Michael Wang wrote: >> >> Use raw management helpers to reform IB-core cm. >> >> Cc: Hal Rosenstock <h...@dev.mellanox.co.il> >> Cc: Steve Wise <sw...@opengridcomputing.com> >> Cc: Tom Talpey <t...@talpey.com> >> Cc: Jason Gunthorpe <jguntho...@obsidianresearch.com> >> Cc: Doug Ledford <dledf...@redhat.com> >> Cc: Ira Weiny <ira.we...@intel.com> >> Cc: Sean Hefty <sean.he...@intel.com> >> Signed-off-by: Michael Wang <yun.w...@profitbricks.com> >> --- >> drivers/infiniband/core/cm.c | 20 +++++++++++++++++--- >> 1 file changed, 17 insertions(+), 3 deletions(-) >> >> diff --git a/drivers/infiniband/core/cm.c b/drivers/infiniband/core/cm.c >> index e28a494..3c10b75 100644 >> --- a/drivers/infiniband/core/cm.c >> +++ b/drivers/infiniband/core/cm.c >> @@ -3761,9 +3761,7 @@ static void cm_add_one(struct ib_device *ib_device) >> unsigned long flags; >> int ret; >> u8 i; >> - >> - if (rdma_node_get_transport(ib_device->node_type) != RDMA_TRANSPORT_IB) >> - return; >> + int count = 0; > > Nit: Should the int count line be moved above u8 i declaration so > declarations are naturally aligned ?
Make sense, will be in next version :-) Regards, Michael Wang > > -- Hal > > <snip...> > -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
