Re: [PATCH v2 38/51] IB/qib: Add qib_sysfs.c

Ralph Campbell
2010-04-08T22:26:41Z

On Thu, 2010-04-08 at 15:08 -0700, Jason Gunthorpe wrote:
> On Thu, Apr 08, 2010 at 02:29:53PM -0700, Ralph Campbell wrote:
>
> > One other place that has multiple values is dumping the QSFP
> > data from the IB cable.
>
> I was going to comment that building your own I2C subsystem is kinda
> strange, can't the in-kernel stuff be used?
>
> Jason

It is a bit strange but we have had this discussion before with
the ib_ipath driver. Basically, the devices connected to the bus
weren't really I2C compliant. Since we had to support the
non-standard parts, it was easier to have only one set of code
that could handle both.