On Fri, Oct 28, 2011 at 09:33:11AM +0200, walter harms wrote: > > > Am 28.10.2011 08:45, schrieb Dan Carpenter: > > On Thu, Oct 20, 2011 at 09:39:23AM +0200, walter harms wrote: > >> I am not sure how SNDRV_MINOR_HWDEPS is used further > > > > Either device = SNDRV_MINOR_HWDEPS or device = SNDRV_MINOR_HWDEPS + 1 > > are fine (equivalent) but it's a fair point. Your version is nicer. > > > > I'll redo the patch. > > > > An other question is: > Should the user be informed that the device is modified ? > The Problem i see is that a user parameter is silently modified > and buggy code will come through. That would be an API change. This is cut and pasted code, and we've had this exact discussion before for sound/core/rawmidi.c a year ago. You were involved in the discussion too. :P http://marc.info/?l=kernel-janitors&m=128401699203676&w=2 regards, dan carpenter -- To unsubscribe from this list: send the line "unsubscribe kernel-janitors" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html