Re: [PATCH] vicodec: change codec license to LGPL

2018-09-02 Thread Mauro Carvalho Chehab
Em Sun, 2 Sep 2018 12:37:04 +0200
Hans Verkuil  escreveu:

> The FWHT codec can also be used by userspace utilities and libraries, but
> since the current license is GPL and not LGPL it is not possible to include
> it in e.g. gstreamer, since LGPL is required for that.
> 
> Change the license of these four files to LGPL.
> 
> Signed-off-by: Hans Verkuil 
> ---
> Tom, if you agree to this, can you give your 'Signed-off-by' line? I cannot
> make this change for the codec-fwht.c/h files without it. I think this change
> makes sense.
> 
> Regards,
> 
>   Hans
> ---
> diff --git a/drivers/media/platform/vicodec/codec-fwht.c 
> b/drivers/media/platform/vicodec/codec-fwht.c
> index 47939160560e..36656031b295 100644
> --- a/drivers/media/platform/vicodec/codec-fwht.c
> +++ b/drivers/media/platform/vicodec/codec-fwht.c
> @@ -1,4 +1,4 @@
> -// SPDX-License-Identifier: GPL-2.0+
> +// SPDX-License-Identifier: LGPL-2.1+

There aren't much C files under LGPL at the Kernel. Yeah, I know it
is compatible with GPL-2.0+, but I would prefer it the tag would
be, instead:

// SPDX-License-Identifier: GPL-2.0+ OR LGPL-2.1+

as this makes easier if one uses some software to parse the Kernel
tree.

(same applies to the other files).

Regards,
Mauro

>  /*
>   * Copyright 2016 Tom aan de Wiel
>   * Copyright 2018 Cisco Systems, Inc. and/or its affiliates. All rights 
> reserved.
> diff --git a/drivers/media/platform/vicodec/codec-fwht.h 
> b/drivers/media/platform/vicodec/codec-fwht.h
> index 1f9e47331197..3e9391fec5fe 100644
> --- a/drivers/media/platform/vicodec/codec-fwht.h
> +++ b/drivers/media/platform/vicodec/codec-fwht.h
> @@ -1,4 +1,4 @@
> -/* SPDX-License-Identifier: GPL-2.0+ */
> +/* SPDX-License-Identifier: LGPL-2.1+ */
>  /*
>   * Copyright 2016 Tom aan de Wiel
>   * Copyright 2018 Cisco Systems, Inc. and/or its affiliates. All rights 
> reserved.
> diff --git a/drivers/media/platform/vicodec/codec-v4l2-fwht.c 
> b/drivers/media/platform/vicodec/codec-v4l2-fwht.c
> index cfcf84b8574d..6b06aa382cbb 100644
> --- a/drivers/media/platform/vicodec/codec-v4l2-fwht.c
> +++ b/drivers/media/platform/vicodec/codec-v4l2-fwht.c
> @@ -1,4 +1,4 @@
> -// SPDX-License-Identifier: GPL-2.0
> +// SPDX-License-Identifier: LGPL-2.1
>  /*
>   * A V4L2 frontend for the FWHT codec
>   *
> diff --git a/drivers/media/platform/vicodec/codec-v4l2-fwht.h 
> b/drivers/media/platform/vicodec/codec-v4l2-fwht.h
> index 7794c186d905..95d1756556db 100644
> --- a/drivers/media/platform/vicodec/codec-v4l2-fwht.h
> +++ b/drivers/media/platform/vicodec/codec-v4l2-fwht.h
> @@ -1,4 +1,4 @@
> -/* SPDX-License-Identifier: GPL-2.0 */
> +/* SPDX-License-Identifier: LGPL-2.1 */
>  /*
>   * Copyright 2018 Cisco Systems, Inc. and/or its affiliates. All rights 
> reserved.
>   */



Thanks,
Mauro


Re: cron job: media_tree daily build: ERRORS

2018-09-02 Thread Jasmin J.
Hi!

This was really hard to fix and I am still not ready.
But it should now compile for 4.19, 4.18 and 4.17.

Compiling is one thing, but functional is another. I added some inline
functions to compat.h (ida_alloc_min, ...) in 7283154cd56d to support them for
older Kernels. If someone is using an older Kernel, I would be happy to get a
feedback if my implementation works.

BR,
   Jasmin

*

On 09/01/2018 06:55 AM, Hans Verkuil wrote:
> This message is generated daily by a cron job that builds media_tree for
> the kernels and architectures in the list below.
> 
> Results of the daily build of media_tree:
> 
> date: Sat Sep  1 05:00:11 CEST 2018
> media-tree git hash:  d842a7cf938b6e0f8a1aa9f1aec0476c9a599310
> media_build git hash: 27f99df152c7e5ae155c6e7888e12234eaa0fc25
> v4l-utils git hash:   f44f00e8b4ac6e9aa05bac8953e3fcc89e1fe198
> edid-decode git hash: b2da1516df3cc2756bfe8d1fa06d7bf2562ba1f4
> gcc version:  i686-linux-gcc (GCC) 8.1.0
> sparse version:   0.5.2
> smatch version:   0.5.1
> host hardware:x86_64
> host os:  4.18.5-marune
> 
> linux-git-arm-at91: OK
> linux-git-arm-davinci: OK
> linux-git-arm-multi: OK
> linux-git-arm-pxa: OK
> linux-git-arm-stm32: OK
> linux-git-arm64: OK
> linux-git-i686: OK
> linux-git-mips: OK
> linux-git-powerpc64: OK
> linux-git-sh: WARNINGS
> linux-git-x86_64: OK
> Check COMPILE_TEST: OK
> linux-2.6.36.4-i686: ERRORS
> linux-2.6.36.4-x86_64: ERRORS
> linux-2.6.37.6-i686: ERRORS
> linux-2.6.37.6-x86_64: ERRORS
> linux-2.6.38.8-i686: ERRORS
> linux-2.6.38.8-x86_64: ERRORS
> linux-2.6.39.4-i686: ERRORS
> linux-2.6.39.4-x86_64: ERRORS
> linux-3.0.101-i686: ERRORS
> linux-3.0.101-x86_64: ERRORS
> linux-3.1.10-i686: ERRORS
> linux-3.1.10-x86_64: ERRORS
> linux-3.2.102-i686: ERRORS
> linux-3.2.102-x86_64: ERRORS
> linux-3.3.8-i686: ERRORS
> linux-3.3.8-x86_64: ERRORS
> linux-3.4.113-i686: ERRORS
> linux-3.4.113-x86_64: ERRORS
> linux-3.5.7-i686: ERRORS
> linux-3.5.7-x86_64: ERRORS
> linux-3.6.11-i686: ERRORS
> linux-3.6.11-x86_64: ERRORS
> linux-3.7.10-i686: ERRORS
> linux-3.7.10-x86_64: ERRORS
> linux-3.8.13-i686: ERRORS
> linux-3.8.13-x86_64: ERRORS
> linux-3.9.11-i686: ERRORS
> linux-3.9.11-x86_64: ERRORS
> linux-3.10.108-i686: ERRORS
> linux-3.10.108-x86_64: ERRORS
> linux-3.11.10-i686: ERRORS
> linux-3.11.10/arch/x86/include/asm/msr.h:131,
> linux-3.11.10-x86_64: ERRORS
> linux-3.12.74-i686: ERRORS
> linux-3.12.74-x86_64: ERRORS
> linux-3.13.11-i686: ERRORS
> linux-3.13.11-x86_64: ERRORS
> linux-3.14.79-i686: ERRORS
> linux-3.14.79-x86_64: ERRORS
> linux-3.15.10-i686: ERRORS
> linux-3.15.10-x86_64: ERRORS
> linux-3.16.57-i686: ERRORS
> linux-3.16.57-x86_64: ERRORS
> linux-3.17.8-i686: ERRORS
> linux-3.17.8-x86_64: ERRORS
> linux-3.18.119-i686: ERRORS
> linux-3.18.119-x86_64: ERRORS
> linux-3.19.8-i686: ERRORS
> linux-3.19.8-x86_64: ERRORS
> linux-4.0.9-i686: ERRORS
> linux-4.0.9-x86_64: ERRORS
> linux-4.1.52-i686: ERRORS
> linux-4.1.52-x86_64: ERRORS
> linux-4.2.8-i686: ERRORS
> linux-4.2.8-x86_64: ERRORS
> linux-4.3.6-i686: ERRORS
> linux-4.3.6-x86_64: ERRORS
> linux-4.4.152-i686: ERRORS
> linux-4.4.152-x86_64: ERRORS
> linux-4.5.7-i686: ERRORS
> linux-4.5.7-x86_64: ERRORS
> linux-4.6.7-i686: ERRORS
> linux-4.6.7-x86_64: ERRORS
> linux-4.7.10-i686: ERRORS
> linux-4.7.10-x86_64: ERRORS
> linux-4.8.17-i686: ERRORS
> linux-4.8.17-x86_64: ERRORS
> linux-4.9.124-i686: ERRORS
> linux-4.9.124-x86_64: ERRORS
> linux-4.10.17-i686: ERRORS
> linux-4.10.17-x86_64: ERRORS
> linux-4.11.12-i686: ERRORS
> linux-4.11.12-x86_64: ERRORS
> linux-4.12.14-i686: ERRORS
> linux-4.12.14-x86_64: ERRORS
> linux-4.13.16-i686: ERRORS
> linux-4.13.16-x86_64: ERRORS
> linux-4.14.67-i686: ERRORS
> linux-4.14.67-x86_64: ERRORS
> linux-4.15.18-i686: ERRORS
> linux-4.15.18-x86_64: ERRORS
> linux-4.16.18-i686: ERRORS
> linux-4.16.18-x86_64: ERRORS
> linux-4.17.19-i686: ERRORS
> linux-4.17.19-x86_64: ERRORS
> linux-4.18.5-i686: ERRORS
> linux-4.18.5-x86_64: ERRORS
> linux-4.19-rc1-i686: OK
> linux-4.19-rc1-x86_64: OK
> apps: OK
> specified for parameter 'printk_address'
> specified for parameter 'printk_address'
> specified for parameter 'socket_seq_show'
> specified for parameter 'probe_kernel_write'
> specified for parameter '__bitmap_intersects'
> specified for parameter '__bitmap_intersects'
> specified for parameter 'numa_set_node'
> specified for parameter 'do_splice_direct'
> specified for parameter 'queued_spin_lock_slowpath'
> specifiers or '...' before 'atomic_long_t'
> specifiers or '...' before 'atomic_long_t'
> specifiers or '...' before 'atomic_long_t'
> specified for parameter 'next_online_pgdat'
> specified for parameter 'next_online_pgdat'
> specified for parameter 'next_online_pgdat'
> specifiers or '...' before 'seqcount_t'
> specified for parameter '__page_ref_mod'
> specified for parameter 'spi_busnum_to_master'
> specified for 

[PATCH] vicodec: change codec license to LGPL

2018-09-02 Thread Hans Verkuil
The FWHT codec can also be used by userspace utilities and libraries, but
since the current license is GPL and not LGPL it is not possible to include
it in e.g. gstreamer, since LGPL is required for that.

Change the license of these four files to LGPL.

Signed-off-by: Hans Verkuil 
---
Tom, if you agree to this, can you give your 'Signed-off-by' line? I cannot
make this change for the codec-fwht.c/h files without it. I think this change
makes sense.

Regards,

Hans
---
diff --git a/drivers/media/platform/vicodec/codec-fwht.c 
b/drivers/media/platform/vicodec/codec-fwht.c
index 47939160560e..36656031b295 100644
--- a/drivers/media/platform/vicodec/codec-fwht.c
+++ b/drivers/media/platform/vicodec/codec-fwht.c
@@ -1,4 +1,4 @@
-// SPDX-License-Identifier: GPL-2.0+
+// SPDX-License-Identifier: LGPL-2.1+
 /*
  * Copyright 2016 Tom aan de Wiel
  * Copyright 2018 Cisco Systems, Inc. and/or its affiliates. All rights 
reserved.
diff --git a/drivers/media/platform/vicodec/codec-fwht.h 
b/drivers/media/platform/vicodec/codec-fwht.h
index 1f9e47331197..3e9391fec5fe 100644
--- a/drivers/media/platform/vicodec/codec-fwht.h
+++ b/drivers/media/platform/vicodec/codec-fwht.h
@@ -1,4 +1,4 @@
-/* SPDX-License-Identifier: GPL-2.0+ */
+/* SPDX-License-Identifier: LGPL-2.1+ */
 /*
  * Copyright 2016 Tom aan de Wiel
  * Copyright 2018 Cisco Systems, Inc. and/or its affiliates. All rights 
reserved.
diff --git a/drivers/media/platform/vicodec/codec-v4l2-fwht.c 
b/drivers/media/platform/vicodec/codec-v4l2-fwht.c
index cfcf84b8574d..6b06aa382cbb 100644
--- a/drivers/media/platform/vicodec/codec-v4l2-fwht.c
+++ b/drivers/media/platform/vicodec/codec-v4l2-fwht.c
@@ -1,4 +1,4 @@
-// SPDX-License-Identifier: GPL-2.0
+// SPDX-License-Identifier: LGPL-2.1
 /*
  * A V4L2 frontend for the FWHT codec
  *
diff --git a/drivers/media/platform/vicodec/codec-v4l2-fwht.h 
b/drivers/media/platform/vicodec/codec-v4l2-fwht.h
index 7794c186d905..95d1756556db 100644
--- a/drivers/media/platform/vicodec/codec-v4l2-fwht.h
+++ b/drivers/media/platform/vicodec/codec-v4l2-fwht.h
@@ -1,4 +1,4 @@
-/* SPDX-License-Identifier: GPL-2.0 */
+/* SPDX-License-Identifier: LGPL-2.1 */
 /*
  * Copyright 2018 Cisco Systems, Inc. and/or its affiliates. All rights 
reserved.
  */