Re: [PATCH 2/2] [media] firedtv: Port it to use rc_core

[Mauro Carvalho Chehab <mchehab@redhat.com>]

Em 28-05-2012 11:14, Stefan Richter escreveu:
> On May 28 Stefan Richter wrote:
>>> +	idev->phys = "/ir0";		/* FIXME */  
>>
>> Something similar to drivers/media/dvb/dvb-usb/dvb-usb-remote.c::
>>
>> 	usb_make_path(d->udev, d->rc_phys, sizeof(d->rc_phys));
>> 	strlcat(d->rc_phys, "/ir0", sizeof(d->rc_phys));
>>
>> should be implemented for this, right?
> 
> PS:
> The current input device looks like this:
> 
> /sys/devices/pci0000:00/0000:00:02.0/0000:02:00.0/0000:03:01.0/0000:04:00.0/fw7/fw7.0/input/input8/device -> ../../../fw7.0
> 
> "fw7.0" is dev_name(dev) in fdtv_register_rc() or dev_name(fdtv->device)
> in general in firedtv.
> 
> The last numeric name before fw7, i.e. 0000:04:00.0, is the name of the PCI
> device of the FireWire controller.  fw7 is the name of the FireDTV node;
> fw7.0 is the name of the (only) unit within the FireDTV node which
> implements the DVB receiver and IR receiver.  What would be needed from
> this?
> 
> FWIW, usb_make_path() results in "usb-%s-%s" % (usb_device.bus.bus_name,
> usb_device.devpath).

IMO, the physical patch is "fw7.0". We add a /ir0 for the first IR device (as a
driver might have more than one IR).

So, the phys should be "fw7.0/ir0".

Regards,
Mauro