You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
For some projects, code coverage is the name of the game and by forcing people to add a DMA driver dependency they might not use, we might not do them any favor.
It might be possible to provide 2 drivers, or specify the dependency in the name, like:
drivers/spi/spi-xyz.c
drivers/spi/spi-xyz+dma.c
This would be a strange case where, to avoid dead code, we might have duplicated code (that will be harder to maintain, of course).
reacted with thumbs up emoji reacted with thumbs down emoji reacted with laugh emoji reacted with hooray emoji reacted with confused emoji reacted with heart emoji reacted with rocket emoji reacted with eyes emoji
-
For some projects, code coverage is the name of the game and by forcing people to add a DMA driver dependency they might not use, we might not do them any favor.
It might be possible to provide 2 drivers, or specify the dependency in the name, like:
This would be a strange case where, to avoid dead code, we might have duplicated code (that will be harder to maintain, of course).
Beta Was this translation helpful? Give feedback.
All reactions