[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [PATCH 1/8] xen/arm: bootfdt.c: Remove unused-but-set variable



Hi Michal,

On 27/04/2022 10:49, Michal Orzel wrote:
Function device_tree_node_compatible defines and sets a variable
mlen but does not make use of it. Remove this variable.

Signed-off-by: Michal Orzel <michal.orzel@xxxxxxx>

Reviewed-by: Julien Grall <jgrall@xxxxxxxxxx>

---
  xen/arch/arm/bootfdt.c | 3 ---
  1 file changed, 3 deletions(-)

diff --git a/xen/arch/arm/bootfdt.c b/xen/arch/arm/bootfdt.c
index e318ef9603..29671c8df0 100644
--- a/xen/arch/arm/bootfdt.c
+++ b/xen/arch/arm/bootfdt.c
@@ -36,11 +36,8 @@ static bool __init device_tree_node_compatible(const void 
*fdt, int node,
                                                 const char *match)
  {
      int len, l;
-    int mlen;
      const void *prop;
- mlen = strlen(match);
-
      prop = fdt_getprop(fdt, node, "compatible", &len);
      if ( prop == NULL )
          return false;

Cheers,

--
Julien Grall



 


Rackspace

Lists.xenproject.org is hosted with RackSpace, monitoring our
servers 24x7x365 and backed by RackSpace's Fanatical Support®.