> > - s64 offset = (void *)v - (void *)location; > > + uintptr_t offset = (void *)v - (void *)location; > > s64 is signed, uintptr is not, so this might change behavior and needs > an explanation. >
Yes, it should be intptr_t. Thanks!
- [PATCH 0/5] Building for 32-bit RISC-V kernel Zong Li
- [PATCH] RISC-V: Add conditional marco for boot_sec_... Zong Li
- [PATCH 1/5] RISC-V: Add conditional macro for zone ... Zong Li
- Re: [PATCH 1/5] RISC-V: Add conditional macro f... Christoph Hellwig
- [PATCH 4/5] RISC-V: Change variable type for 32-bit... Zong Li
- Re: [PATCH 4/5] RISC-V: Change variable type fo... Christoph Hellwig
- Re: [PATCH 4/5] RISC-V: Change variable typ... Zong Li
- [PATCH 3/5] RISC-V: Add definiion of extract symbol... Zong Li
- [PATCH 5/5] RISC-V: Use fixed width integer types f... Zong Li
- Re: [PATCH 5/5] RISC-V: Use fixed width integer... Christoph Hellwig
- Re: [PATCH 5/5] RISC-V: Use fixed width int... Zong Li
- Re: [PATCH 5/5] RISC-V: Use fixed width int... Andreas Schwab
- Re: [PATCH 5/5] RISC-V: Use fixed width... Christoph Hellwig
- [PATCH 2/5] RISC-V: Select GENERIC_UCMPDI2 on RV32I Zong Li
