Re: [linux-yocto][linux-yocto v5.15/standard/ti-sdk-5.10/ti-j72xx][PATCH] media: platform: vxe-vxd: Remove the redundant function umin()

[Bruce Ashfield]

merged

Bruce

In message: [linux-yocto][linux-yocto 
v5.15/standard/ti-sdk-5.10/ti-j72xx][PATCH] media: platform: vxe-vxd: Remove 
the redundant function umin()
on 18/04/2024 Xulin Sun wrote:

> commit cf716d5a0dd86f6 (minmax: add umin(a, b) and umax(a, b)) introduces
> the definition umin() in include/linux/minmax.h, remove the redundant
> function umin() in the file h264_secure_parser.c to fix below building issue:
> drivers/media/platform/vxe-vxd/decoder/h264_secure_parser.c:174:28: note: in 
> expansion of macro 'umin'
> |   174 | static inline unsigned int umin(unsigned int a, unsigned int b)
> |   |^~~~
> 
> Signed-off-by: Xulin Sun 
> ---
>  .../media/platform/vxe-vxd/decoder/h264_secure_parser.c   | 8 
>  1 file changed, 8 deletions(-)
> 
> diff --git a/drivers/media/platform/vxe-vxd/decoder/h264_secure_parser.c 
> b/drivers/media/platform/vxe-vxd/decoder/h264_secure_parser.c
> index 90f5197b17bc..9887db1067e3 100644
> --- a/drivers/media/platform/vxe-vxd/decoder/h264_secure_parser.c
> +++ b/drivers/media/platform/vxe-vxd/decoder/h264_secure_parser.c
> @@ -168,14 +168,6 @@ static const int bspp_subheightc[] = { -1, 2, 1, 1 };
>   */
>  static const int bspp_subwidthc[] = { -1, 2, 2, 1 };
>  
> -/*
> - * inline functions for Minimum and Maximum value
> - */
> -static inline unsigned int umin(unsigned int a, unsigned int b)
> -{
> - return (((a) < (b)) ? (a) : (b));
> -}
> -
>  static inline int smin(int a, int b)
>  {
>   return (((a) < (b)) ? (a) : (b));
> -- 
> 2.34.1
> 

-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.
View/Reply Online (#13847): 
https://lists.yoctoproject.org/g/linux-yocto/message/13847
Mute This Topic: https://lists.yoctoproject.org/mt/105594237/21656
Group Owner: linux-yocto+ow...@lists.yoctoproject.org
Unsubscribe: https://lists.yoctoproject.org/g/linux-yocto/unsub 
[arch...@mail-archive.com]
-=-=-=-=-=-=-=-=-=-=-=-

[linux-yocto][linux-yocto v5.15/standard/ti-sdk-5.10/ti-j72xx][PATCH] media: platform: vxe-vxd: Remove the redundant function umin()

[Xulin Sun via lists.yoctoproject.org]

commit cf716d5a0dd86f6 (minmax: add umin(a, b) and umax(a, b)) introduces
the definition umin() in include/linux/minmax.h, remove the redundant
function umin() in the file h264_secure_parser.c to fix below building issue:
drivers/media/platform/vxe-vxd/decoder/h264_secure_parser.c:174:28: note: in 
expansion of macro 'umin'
|   174 | static inline unsigned int umin(unsigned int a, unsigned int b)
|   |^~~~

Signed-off-by: Xulin Sun 
---
 .../media/platform/vxe-vxd/decoder/h264_secure_parser.c   | 8 
 1 file changed, 8 deletions(-)

diff --git a/drivers/media/platform/vxe-vxd/decoder/h264_secure_parser.c 
b/drivers/media/platform/vxe-vxd/decoder/h264_secure_parser.c
index 90f5197b17bc..9887db1067e3 100644
--- a/drivers/media/platform/vxe-vxd/decoder/h264_secure_parser.c
+++ b/drivers/media/platform/vxe-vxd/decoder/h264_secure_parser.c
@@ -168,14 +168,6 @@ static const int bspp_subheightc[] = { -1, 2, 1, 1 };
  */
 static const int bspp_subwidthc[] = { -1, 2, 2, 1 };
 
-/*
- * inline functions for Minimum and Maximum value
- */
-static inline unsigned int umin(unsigned int a, unsigned int b)
-{
-   return (((a) < (b)) ? (a) : (b));
-}
-
 static inline int smin(int a, int b)
 {
return (((a) < (b)) ? (a) : (b));
-- 
2.34.1


-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.
View/Reply Online (#13835): 
https://lists.yoctoproject.org/g/linux-yocto/message/13835
Mute This Topic: https://lists.yoctoproject.org/mt/105594237/21656
Group Owner: linux-yocto+ow...@lists.yoctoproject.org
Unsubscribe: https://lists.yoctoproject.org/g/linux-yocto/unsub 
[arch...@mail-archive.com]
-=-=-=-=-=-=-=-=-=-=-=-