Mauro Carvalho Chehab <mchehab@xxxxxxxxxx> wrote: > > + dvb_class->dev_groups = dvb_class_groups, > > dvb_class->dev_uevent = dvb_uevent; > > dvb_class->devnode = dvb_devnode; > > return 0; > > The patch itself looks good, but I'm not seeing any documentation. I should probably add something to Documentation/media/dvb-drivers/udev.rst > You should likely add something to Documentation/ABI Any suggestions as to where to add stuff in there? The README there leaves a lot to be desired as to how to name elements - for instance, DVB devices can be seen through /sys/class/ and /sys/devices/. I could put it in sys-class-dvb or sys-devices-dvb - or, arguably, both. > and to the DVB uAPI (Documentation/media/uapi/dvb). Likewise, any suggestion as to where in here? As far as I can tell, the docs here don't currently mention sysfs at all. I'm guessing I'll need to create a file specifically to talk about how to use this stuff with udev. > > + port->frontends.adapter.port_num = port->nr; > > + > > Doing it for each multi-adapter device is something that bothers > me. The better would be if we could move this to the DVB Kernel, > in order to not need to check/fix every driver. I'm not sure how achievable that is: *port in this case is a private cx23885-specific structure object. > If, otherwise, this is not possible, then we need a patch fixing port_num > for all drivers that support multiple adapters. > > Also, the risk of forgetting it seems high. So, perhaps we should > add a new parameter to some function (like at dvb_register_device > or at dvb_register_frontend), in order to make the port number > a mandatory attribute. Hmmm... The cx23885 driver doesn't call either of these functions as far as I can tell - at least, not directly. Maybe by vb2_dvb_register_bus()? Note that these attribute files appear for the demux, dvr and net directories as well as for the frontend. Hmmm... further, the port number is no longer getting through and all adapters are showing port 0. The MAC address works, though. Maybe I should drop the port number. David
