On Thu, Jan 20, 2022 at 5:35 PM Arnd Bergmann <a...@arndb.de> wrote: > > On Thu, Jan 20, 2022 at 8:38 AM <guo...@kernel.org> wrote: > > > @@ -0,0 +1,136 @@ > > +/* SPDX-License-Identifier: GPL-2.0-only */ > > +#ifndef __ASM_COMPAT_H > > +#define __ASM_COMPAT_H > > + > > +#define compat_mode_t compat_mode_t > > +typedef u16 compat_mode_t; > > I think this one is wrong, as rv32 should get the native definition from > > include/uapi/asm-generic/posix_types.h:typedef unsigned int > __kernel_mode_t; > > I think it works if you just remove those two lines. The rest looks good to > me. Yes, you are right. compat_mode_t should be unsigned int.
> > Arnd -- Best Regards Guo Ren ML: https://lore.kernel.org/linux-csky/