On 2021/4/8 5:56, Bart Van Assche wrote:
On 4/5/21 11:39 PM, luojiaxing wrote:
On 2021/4/3 0:01, Bart Van Assche wrote:
On 4/2/21 2:08 AM, Luo Jiaxing wrote:
#define AAP1_MEMMAP(r, c) \
- (*(u32 *)((u8*)pm8001_ha->memoryMap.region[AAP1].virt_ptr +
(r) * 32 \
+ (*(u32 *)((u8 *)pm8001_ha->memoryMap.region[AAP1].virt_ptr +
(r) * 32 \
+ (c)))
Since this macro is being modified, please convert it into an inline
function such that the type of the arguments can be verified by the
compiler.
Sure, but still keep the function name as AAP1_MEMMAP?
The coding style requires lower case names for functions so the
function name probably should be converted to lower case.
Sure, and sorry to post v2 before you reply, you can ignore it and
directly check v3.
Thanks
Jiaxing
Thanks,
Bart.
.
