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/

Reply via email to