Re: [PATCH] HP iLO driver

From: Michael Tokarev
Date: Fri Jun 13 2008 - 04:30:42 EST


H. Peter Anvin wrote:
Altobelli, David wrote:
Wouldn't /dev/hpilo/ccbN be a more appropriate namespace?

Yes, that would be more clear. The code is currently defined to a maximum of 1 hardware device, but this could change, and the dev path should probably reflect that.

How do you feel about /dev/hpiloX/ccbN, where X is 0, 1, ... Num_Devices-1?

I'd rather see /dev/hpilo/dXcY, the point being to try to declutter the /dev root.

For a case when there will be only one, or, in very rare theoretical
case, two devices, why bother in the first place? A subdirectory
containing only one node is more ugly.. ;)

/mjt
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/