On 14/06/2013 15:18, Jagan Teki wrote:
This does not scale. What if a (trivial) patch touches 10 drivers?
Yes, you are correct if the changes is common to across the all drivers or If the changes which are more in 10 spi drivers out of 20 or > 10 for your example - use "spi: " and I think it's good to use above syntax as you changesd two drivers.
I am just showing the possibilities of using the coding guidelines, nothing more.
There is another issue: patman recognizes each <subpart>: and checks for an alias to find the e-mail to send patches, and stops to go on if it is not found. In this example, you can use patman if there is an entry for spi: (and this is you as SPI custodian) and also for mxc_spi:, and usually there is no maintainer for each driver. I agree that using a long chain does not scale, and I think the hidden rule to use subsystem: <subject> works well in most cases, Axel did in this patchset.
Regards, Stefano