On Wed, 2018-01-24 at 13:17 +0200, Yuval Shaia wrote: > - if (pci_set_dma_mask(pdev, DMA_BIT_MASK(64)) == 0) { > + if (!pci_set_dma_mask(pdev, DMA_BIT_MASK(64))) { Sorry but I absolutely don't like this change. I think testing a return value using "== 0" is easier to read than using !. Bart.��.n��������+%������w��{.n�����{���fk��ܨ}���Ơz�j:+v�����w����ޙ��&�)ߡ�a����z�ޗ���ݢj��w�f