Skip site navigation (1)Skip section navigation (2)
Date:      Wed, 18 May 2016 23:41:59 +0000 (UTC)
From:      Oleksandr Tymoshenko <gonzo@FreeBSD.org>
To:        src-committers@freebsd.org, svn-src-all@freebsd.org, svn-src-head@freebsd.org
Subject:   svn commit: r300175 - head/sys/arm/amlogic/aml8726
Message-ID:  <201605182341.u4INfxl1078451@repo.freebsd.org>

next in thread | raw e-mail | index | archive | help
Author: gonzo
Date: Wed May 18 23:41:58 2016
New Revision: 300175
URL: https://svnweb.freebsd.org/changeset/base/300175

Log:
  Use OF_prop_free instead of direct call to free(9)
  
  Reviewed by:	ganbold

Modified:
  head/sys/arm/amlogic/aml8726/aml8726_ccm.c
  head/sys/arm/amlogic/aml8726/aml8726_mmc.c
  head/sys/arm/amlogic/aml8726/aml8726_pinctrl.c
  head/sys/arm/amlogic/aml8726/aml8726_sdxc-m8.c
  head/sys/arm/amlogic/aml8726/aml8726_usb_phy-m3.c
  head/sys/arm/amlogic/aml8726/aml8726_usb_phy-m6.c

Modified: head/sys/arm/amlogic/aml8726/aml8726_ccm.c
==============================================================================
--- head/sys/arm/amlogic/aml8726/aml8726_ccm.c	Wed May 18 23:41:55 2016	(r300174)
+++ head/sys/arm/amlogic/aml8726/aml8726_ccm.c	Wed May 18 23:41:58 2016	(r300175)
@@ -141,7 +141,7 @@ aml8726_ccm_configure_gates(struct aml87
 		AML_CCM_UNLOCK(sc);
 	}
 
-	free(functions, M_OFWPROP);
+	OF_prop_free(functions);
 
 	return (0);
 }

Modified: head/sys/arm/amlogic/aml8726/aml8726_mmc.c
==============================================================================
--- head/sys/arm/amlogic/aml8726/aml8726_mmc.c	Wed May 18 23:41:55 2016	(r300174)
+++ head/sys/arm/amlogic/aml8726/aml8726_mmc.c	Wed May 18 23:41:58 2016	(r300175)
@@ -605,11 +605,11 @@ aml8726_mmc_attach(device_t dev)
 	else {
 		device_printf(dev, "unknown function attribute %.*s in FDT\n",
 		    len, function_name);
-		free(function_name, M_OFWPROP);
+		OF_prop_free(function_name);
 		return (ENXIO);
 	}
 
-	free(function_name, M_OFWPROP);
+	OF_prop_free(function_name);
 
 	sc->pwr_en.dev = NULL;
 
@@ -661,7 +661,7 @@ aml8726_mmc_attach(device_t dev)
 			device_printf(dev,
 			    "unknown voltage attribute %.*s in FDT\n",
 			    len, voltage);
-			free(voltages, M_OFWPROP);
+			OF_prop_free(voltages);
 			return (ENXIO);
 		}
 
@@ -678,7 +678,7 @@ aml8726_mmc_attach(device_t dev)
 		}
 	}
 
-	free(voltages, M_OFWPROP);
+	OF_prop_free(voltages);
 
 	sc->vselect.dev = NULL;
 

Modified: head/sys/arm/amlogic/aml8726/aml8726_pinctrl.c
==============================================================================
--- head/sys/arm/amlogic/aml8726/aml8726_pinctrl.c	Wed May 18 23:41:55 2016	(r300174)
+++ head/sys/arm/amlogic/aml8726/aml8726_pinctrl.c	Wed May 18 23:41:58 2016	(r300175)
@@ -210,11 +210,11 @@ aml8726_pinctrl_configure_pins(device_t 
 	if (f->name == NULL) {
 		device_printf(dev, "unknown function attribute %.*s in FDT\n",
 		    len, function_name);
-		free(function_name, M_OFWPROP);
+		OF_prop_free(function_name);
 		return (ENXIO);
 	}
 
-	free(function_name, M_OFWPROP);
+	OF_prop_free(function_name);
 
 	len = OF_getprop_alloc(node, "amlogic,pull",
 	    sizeof(char), (void **)&pull);
@@ -234,12 +234,12 @@ aml8726_pinctrl_configure_pins(device_t 
 			device_printf(dev,
 			    "unknown pull attribute %.*s in FDT\n",
 			    len, pull);
-			free(pull, M_OFWPROP);
+			OF_prop_free(pull);
 			return (ENXIO);
 		}
 	}
 
-	free(pull, M_OFWPROP);
+	OF_prop_free(pull);
 
 	/*
 	 * Setting the pull direction isn't supported on all SoC.
@@ -403,7 +403,7 @@ aml8726_pinctrl_configure_pins(device_t 
 		AML_PINCTRL_UNLOCK(sc);
 	}
 
-	free(pins, M_OFWPROP);
+	OF_prop_free(pins);
 
 	return (0);
 }

Modified: head/sys/arm/amlogic/aml8726/aml8726_sdxc-m8.c
==============================================================================
--- head/sys/arm/amlogic/aml8726/aml8726_sdxc-m8.c	Wed May 18 23:41:55 2016	(r300174)
+++ head/sys/arm/amlogic/aml8726/aml8726_sdxc-m8.c	Wed May 18 23:41:58 2016	(r300175)
@@ -821,7 +821,7 @@ aml8726_sdxc_attach(device_t dev)
 			device_printf(dev,
 			    "unknown voltage attribute %.*s in FDT\n",
 			    len, voltage);
-			free(voltages, M_OFWPROP);
+			OF_prop_free(voltages);
 			return (ENXIO);
 		}
 
@@ -838,7 +838,7 @@ aml8726_sdxc_attach(device_t dev)
 		}
 	}
 
-	free(voltages, M_OFWPROP);
+	OF_prop_free(voltages);
 
 	sc->vselect.dev = NULL;
 

Modified: head/sys/arm/amlogic/aml8726/aml8726_usb_phy-m3.c
==============================================================================
--- head/sys/arm/amlogic/aml8726/aml8726_usb_phy-m3.c	Wed May 18 23:41:55 2016	(r300174)
+++ head/sys/arm/amlogic/aml8726/aml8726_usb_phy-m3.c	Wed May 18 23:41:58 2016	(r300175)
@@ -139,7 +139,7 @@ aml8726_usb_phy_mode(const char *dwcotg_
 		    AML_USB_PHY_MISC_ID_OVERIDE_DEVICE;
 	}
 
-	free(usb_mode, M_OFWPROP);
+	OF_prop_free(usb_mode);
 
 	return (0);
 }
@@ -214,7 +214,7 @@ aml8726_usb_phy_attach(device_t dev)
 		}
 	}
 
-	free(prop, M_OFWPROP);
+	OF_prop_free(prop);
 
 	if (err) {
 		device_printf(dev, "unable to parse gpio\n");

Modified: head/sys/arm/amlogic/aml8726/aml8726_usb_phy-m6.c
==============================================================================
--- head/sys/arm/amlogic/aml8726/aml8726_usb_phy-m6.c	Wed May 18 23:41:55 2016	(r300174)
+++ head/sys/arm/amlogic/aml8726/aml8726_usb_phy-m6.c	Wed May 18 23:41:58 2016	(r300175)
@@ -164,7 +164,7 @@ aml8726_usb_phy_attach(device_t dev)
 			sc->force_aca = TRUE;
 	}
 
-	free(force_aca, M_OFWPROP);
+	OF_prop_free(force_aca);
 
 	err = 0;
 
@@ -187,7 +187,7 @@ aml8726_usb_phy_attach(device_t dev)
 		}
 	}
 
-	free(prop, M_OFWPROP);
+	OF_prop_free(prop);
 
 	len = OF_getencprop_alloc(node, "usb-hub-rst",
 	    3 * sizeof(pcell_t), (void **)&prop);
@@ -200,7 +200,7 @@ aml8726_usb_phy_attach(device_t dev)
 			err = 1;
 	}
 
-	free(prop, M_OFWPROP);
+	OF_prop_free(prop);
 
 	if (err) {
 		device_printf(dev, "unable to parse gpio\n");



Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?201605182341.u4INfxl1078451>