Re: [PATCH RFC 00/10] media file tree reorg - part 1

[Mauro Carvalho Chehab <mchehab@xxxxxxxxxx>]

Em 15-06-2012 04:39, Hans Verkuil escreveu:
> On 14/06/12 22:35, Mauro Carvalho Chehab wrote:
>> As discussed a while ago, breaking media drivers by V4L or DVB
>> is confusing, as:
>>     - hybrid devices are at V4L drivers;
>>     - DVB-only devices for chips that support analog are at
>>       V4L drivers;
>>     - Analog support addition on a DVB driver would require it
>>       to move to V4L drivers.
>>
>> Instead, move all drivers into a per-bus directory, and common drivers
>> used by more than one driver into /common.
>>
>> This is the part 1 of this idea: it moves the core drivers to
>> /drivers/media/foo-core, and re-arranges the DVB files.
>>
>> After this patch series, the directory structure will be:
>>
>> drivers/media/
>> |-- common
>> |   `--<common drivers>
>> |-- dvb-core
>> |-- dvb-frontends
>> |-- firewire
>> |-- mmc
>> |   `--<mmc/sdio drivers>
>> |-- pci
>> |   `--<pci/pcie drivers>
>> |-- radio
>> |   `--<radio drivers>
>> |-- rc
>> |   `-- keymaps
>> |-- tuners
>> |-- usb
>> |   `--<usb drivers>
>> |-- v4l2-core
>> `-- video
>>
>> PS.: The "video" directory is currently unchanged. It currently
>>       contains subdevs, common V4L drivers, and V4L bridges.
>>
>> On this series, I avoided mixing the file tree reorganization with
>> menu improvements. Those will happen together with the second part,
>> when the devices under video will be moved to /common, /usb, /pci...
>> dirs.
> 
> Looks good to me. I like that saa7146 gets its own directory :-)

Yes, this is good.

> One request: before you commit this, can you go through the pending patches for 3.6 and apply all the non-controversial ones? Otherwise everyone will have to rebase their work.

Yeah, that's my plan.

> Regards,
> 
>      Hans
Regards,
Mauro

--
To unsubscribe from this list: send the line "unsubscribe linux-media" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html