Re: [PATCH 3/5] lib/mpi: Fix for building for MIPS32 with Clang

2019-08-12 Thread Nathan Chancellor
On Mon, Aug 12, 2019 at 10:40:49PM +0300, Jussi Kivilinna wrote: > That's quite complicated approach. Fast and easier process would be if you > just own the patch yourself. Libgcrypt (and target file in libgcrypt) > is LGPL v2.1+, so the license is compatible with kernel and you are good > to go

Re: [PATCH 3/5] lib/mpi: Fix for building for MIPS32 with Clang

2019-08-12 Thread Jussi Kivilinna
Hello, On 12.8.2019 20.14, Nathan Chancellor wrote: > On Mon, Aug 12, 2019 at 10:35:53AM +0300, Jussi Kivilinna wrote: >> Hello, >> >> On 12.8.2019 6.31, Nathan Chancellor wrote: >>> From: Vladimir Serbinenko >>> >>> clang doesn't recognise =l / =h assembly operand specifiers but apparently >>>

Re: [PATCH 3/5] lib/mpi: Fix for building for MIPS32 with Clang

2019-08-12 Thread Paul Burton
Hi Nathan, On Sun, Aug 11, 2019 at 10:23:55PM -0700, Nathan Chancellor wrote: > I noticed you didn't pick up this patch with the other ones you > applied. I just wanted to make sure it wasn't because it was sent to > the wrong person. This set of files doesn't appear to have an owner in >

Re: [PATCH 3/5] lib/mpi: Fix for building for MIPS32 with Clang

2019-08-12 Thread Nathan Chancellor
On Mon, Aug 12, 2019 at 10:35:53AM +0300, Jussi Kivilinna wrote: > Hello, > > On 12.8.2019 6.31, Nathan Chancellor wrote: > > From: Vladimir Serbinenko > > > > clang doesn't recognise =l / =h assembly operand specifiers but apparently > > handles C version well. > > > >

Re: [PATCH 3/5] lib/mpi: Fix for building for MIPS32 with Clang

2019-08-12 Thread Herbert Xu
On Sun, Aug 11, 2019 at 10:26:53PM -0700, Nathan Chancellor wrote: > > > I noticed you didn't pick up this patch with the other ones you > > applied. I just wanted to make sure it wasn't because it was sent to > > the wrong person. This set of files doesn't appear to have an owner in > >

Re: [PATCH 3/5] lib/mpi: Fix for building for MIPS32 with Clang

2019-08-12 Thread Jussi Kivilinna
Hello, On 12.8.2019 6.31, Nathan Chancellor wrote: > From: Vladimir Serbinenko > > clang doesn't recognise =l / =h assembly operand specifiers but apparently > handles C version well. > > lib/mpi/generic_mpih-mul1.c:37:24: error: invalid use of a cast in a > inline asm context requiring an

Re: [PATCH 3/5] lib/mpi: Fix for building for MIPS32 with Clang

2019-08-11 Thread Nathan Chancellor
On Sun, Aug 11, 2019 at 10:23:55PM -0700, Nathan Chancellor wrote: > On Sun, Aug 11, 2019 at 08:31:18PM -0700, Nathan Chancellor wrote: > > From: Vladimir Serbinenko > > > > clang doesn't recognise =l / =h assembly operand specifiers but apparently > > handles C version well. > > > >

Re: [PATCH 3/5] lib/mpi: Fix for building for MIPS32 with Clang

2019-08-11 Thread Nathan Chancellor
On Sun, Aug 11, 2019 at 08:31:18PM -0700, Nathan Chancellor wrote: > From: Vladimir Serbinenko > > clang doesn't recognise =l / =h assembly operand specifiers but apparently > handles C version well. > > lib/mpi/generic_mpih-mul1.c:37:24: error: invalid use of a cast in a > inline asm context

[PATCH 3/5] lib/mpi: Fix for building for MIPS32 with Clang

2019-08-11 Thread Nathan Chancellor
From: Vladimir Serbinenko clang doesn't recognise =l / =h assembly operand specifiers but apparently handles C version well. lib/mpi/generic_mpih-mul1.c:37:24: error: invalid use of a cast in a inline asm context requiring an l-value: remove the cast or build with -fheinous-gnu-extensions