21 Feb
2019
21 Feb
'19
4:04 p.m.
On Thu, Feb 21, 2019 at 02:29:57PM +0200, Andy Shevchenko wrote:
On Wed, Feb 20, 2019 at 02:18:01PM -0600, Pierre-Louis Bossart wrote:
as discussed in the last review, we didn't use min() since it adds a number of obscure warnings with sparse. I haven't found any time to look into this so for now left the code as is. Sparse is a useful tool for the address checks (topology/DMAs with _le32, __iomem, etc), the fewer warnings we get the better.
Shouldn't be sparse itself get fixed?
Ideally. We had this discussion last time :/