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 >>> h

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 > MAIN

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. > > > > lib/mpi/generic_mpih-mul

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 > > MAINTA

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 l-va

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. > > > > lib/mpi/gen

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 re