qemu-devel.nongnu.org archive mirror
 help / color / mirror / Atom feed
* [PATCH-for-7.0] hw/net/rocker: Remove unused definitions
@ 2021-11-16 19:39 Philippe Mathieu-Daudé
  2021-12-15 10:37 ` Philippe Mathieu-Daudé
  2021-12-18  0:25 ` Richard Henderson
  0 siblings, 2 replies; 3+ messages in thread
From: Philippe Mathieu-Daudé @ 2021-11-16 19:39 UTC (permalink / raw)
  To: qemu-devel
  Cc: qemu-trivial, Philippe Mathieu-Daudé, Jason Wang, Jiri Pirko

Signed-off-by: Philippe Mathieu-Daudé <philmd@redhat.com>
---
 hw/net/rocker/rocker.h | 5 -----
 1 file changed, 5 deletions(-)

diff --git a/hw/net/rocker/rocker.h b/hw/net/rocker/rocker.h
index 412fa44d017..d22bbd2bf80 100644
--- a/hw/net/rocker/rocker.h
+++ b/hw/net/rocker/rocker.h
@@ -36,13 +36,8 @@ static inline GCC_FMT_ATTR(1, 2) int DPRINTF(const char *fmt, ...)
 }
 #endif
 
-#define __le16 uint16_t
-#define __le32 uint32_t
-#define __le64 uint64_t
-
 #define __be16 uint16_t
 #define __be32 uint32_t
-#define __be64 uint64_t
 
 static inline bool ipv4_addr_is_multicast(__be32 addr)
 {
-- 
2.31.1



^ permalink raw reply related	[flat|nested] 3+ messages in thread

* Re: [PATCH-for-7.0] hw/net/rocker: Remove unused definitions
  2021-11-16 19:39 [PATCH-for-7.0] hw/net/rocker: Remove unused definitions Philippe Mathieu-Daudé
@ 2021-12-15 10:37 ` Philippe Mathieu-Daudé
  2021-12-18  0:25 ` Richard Henderson
  1 sibling, 0 replies; 3+ messages in thread
From: Philippe Mathieu-Daudé @ 2021-12-15 10:37 UTC (permalink / raw)
  To: qemu-devel; +Cc: qemu-trivial, Jason Wang, Jiri Pirko

ping for review?

On 11/16/21 20:39, Philippe Mathieu-Daudé wrote:
> Signed-off-by: Philippe Mathieu-Daudé <philmd@redhat.com>
> ---
>  hw/net/rocker/rocker.h | 5 -----
>  1 file changed, 5 deletions(-)
> 
> diff --git a/hw/net/rocker/rocker.h b/hw/net/rocker/rocker.h
> index 412fa44d017..d22bbd2bf80 100644
> --- a/hw/net/rocker/rocker.h
> +++ b/hw/net/rocker/rocker.h
> @@ -36,13 +36,8 @@ static inline GCC_FMT_ATTR(1, 2) int DPRINTF(const char *fmt, ...)
>  }
>  #endif
>  
> -#define __le16 uint16_t
> -#define __le32 uint32_t
> -#define __le64 uint64_t
> -
>  #define __be16 uint16_t
>  #define __be32 uint32_t
> -#define __be64 uint64_t
>  
>  static inline bool ipv4_addr_is_multicast(__be32 addr)
>  {
> 



^ permalink raw reply	[flat|nested] 3+ messages in thread

* Re: [PATCH-for-7.0] hw/net/rocker: Remove unused definitions
  2021-11-16 19:39 [PATCH-for-7.0] hw/net/rocker: Remove unused definitions Philippe Mathieu-Daudé
  2021-12-15 10:37 ` Philippe Mathieu-Daudé
@ 2021-12-18  0:25 ` Richard Henderson
  1 sibling, 0 replies; 3+ messages in thread
From: Richard Henderson @ 2021-12-18  0:25 UTC (permalink / raw)
  To: Philippe Mathieu-Daudé, qemu-devel
  Cc: qemu-trivial, Jason Wang, Jiri Pirko

On 11/16/21 11:39 AM, Philippe Mathieu-Daudé wrote:
> Signed-off-by: Philippe Mathieu-Daudé <philmd@redhat.com>
> ---
>   hw/net/rocker/rocker.h | 5 -----
>   1 file changed, 5 deletions(-)
> 
> diff --git a/hw/net/rocker/rocker.h b/hw/net/rocker/rocker.h
> index 412fa44d017..d22bbd2bf80 100644
> --- a/hw/net/rocker/rocker.h
> +++ b/hw/net/rocker/rocker.h
> @@ -36,13 +36,8 @@ static inline GCC_FMT_ATTR(1, 2) int DPRINTF(const char *fmt, ...)
>   }
>   #endif
>   
> -#define __le16 uint16_t
> -#define __le32 uint32_t
> -#define __le64 uint64_t
> -
>   #define __be16 uint16_t
>   #define __be32 uint32_t
> -#define __be64 uint64_t

Surely the __beN defines should go away as well.
If they're still used, that seems like a bug itself.


r~


^ permalink raw reply	[flat|nested] 3+ messages in thread

end of thread, other threads:[~2021-12-18  0:26 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2021-11-16 19:39 [PATCH-for-7.0] hw/net/rocker: Remove unused definitions Philippe Mathieu-Daudé
2021-12-15 10:37 ` Philippe Mathieu-Daudé
2021-12-18  0:25 ` Richard Henderson

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).