On Wed, Jul 30, 2025 at 01:32:58PM +0200, Geert Uytterhoeven wrote: > On Wed, 30 Jul 2025 at 11:35, Konrad Dybcio <konradybcio@xxxxxxxxxx> wrote: ... > > + /* The protocol ID is in the teens range, simply ignore the higher bits */ > > + gchan->protocol = (u32)((u64)proto); > > A single cast "(uintptr_t)" should be sufficient. FWIW, this means (unsigned long) as Torvalds is quite against uintptr_t in the kernel. > Casing the pointer to u64 on 32-bit may trigger: > > warning: cast from pointer to integer of different size > [-Wpointer-to-int-cast] > > > return dma_get_slave_channel(&gchan->vc.chan); > > } -- With Best Regards, Andy Shevchenko
