From 5f066b29ce993c0fd67fc8b6ab156a0a0fa02c6c Mon Sep 17 00:00:00 2001 From: Martin Roth Date: Sun, 4 Jan 2015 16:47:39 -0700 Subject: doxygen fixes: change @var to @param var These files were trying to document the parameters, but didn't have the syntax quite right. Change the comments from @varname to @param varname as required by doxygen. Change-Id: I63662094d3f1686e3e35b61925b580eb06e72e28 Signed-off-by: Martin Roth Reviewed-on: http://review.coreboot.org/8100 Reviewed-by: Edward O'Callaghan Tested-by: build bot (Jenkins) Reviewed-by: Alexandru Gagniuc --- src/drivers/intel/gma/intel_dp.c | 6 +++--- src/drivers/ti/tps65090/tps65090.c | 5 +++-- src/drivers/ti/tps65090/tps65090.h | 20 ++++++++++---------- src/include/spi_flash.h | 6 +++--- src/lib/ramtest.c | 8 ++++---- src/soc/intel/common/hda_verb.c | 8 ++++---- src/soc/qualcomm/ipq806x/timer.c | 2 +- src/soc/samsung/exynos5250/fb.c | 2 +- src/southbridge/intel/lynxpoint/hda_verb.c | 8 ++++---- 9 files changed, 33 insertions(+), 32 deletions(-) diff --git a/src/drivers/intel/gma/intel_dp.c b/src/drivers/intel/gma/intel_dp.c index dc2a957cdd..55839e149a 100644 --- a/src/drivers/intel/gma/intel_dp.c +++ b/src/drivers/intel/gma/intel_dp.c @@ -38,7 +38,7 @@ /** * is_edp - is the given port attached to an eDP panel (either CPU or PCH) - * @intel_dp: DP struct + * @param intel_dp: DP struct * * If a CPU or PCH DP output is attached to an eDP panel, this function * will return 1, and 0 otherwise. @@ -50,7 +50,7 @@ static int is_edp(struct intel_dp *intel_dp) /** * is_pch_edp - is the port on the PCH and attached to an eDP panel? - * @intel_dp: DP struct + * @param intel_dp: DP struct * * Returns 1 if the given DP struct corresponds to a PCH DP port attached * to an eDP panel, 0 otherwise. Helpful for determining whether we @@ -63,7 +63,7 @@ static int is_pch_edp(struct intel_dp *intel_dp) /** * is_cpu_edp - is the port on the CPU and attached to an eDP panel? - * @intel_dp: DP struct + * @param intel_dp: DP struct * * Returns 1 if the given DP struct corresponds to a CPU eDP port. */ diff --git a/src/drivers/ti/tps65090/tps65090.c b/src/drivers/ti/tps65090/tps65090.c index 35a050f36f..58bb3ce95d 100644 --- a/src/drivers/ti/tps65090/tps65090.c +++ b/src/drivers/ti/tps65090/tps65090.c @@ -87,8 +87,9 @@ static int tps65090_i2c_read(unsigned int bus, /** * Set the power state for a FET * - * @fet_id Fet number to set (1..MAX_FET_NUM) - * @set 1 to power on FET, 0 to power off + * @param bus + * @param fet_id Fet number to set (1..MAX_FET_NUM) + * @param set 1 to power on FET, 0 to power off * @return FET_ERR_COMMS if we got a comms error, FET_ERR_NOT_READY if the * FET failed to change state. If all is ok, returns 0. */ diff --git a/src/drivers/ti/tps65090/tps65090.h b/src/drivers/ti/tps65090/tps65090.h index b38db59892..a79158b697 100644 --- a/src/drivers/ti/tps65090/tps65090.h +++ b/src/drivers/ti/tps65090/tps65090.h @@ -67,8 +67,8 @@ enum { /** * Enable FET * - * @bus I2C bus number the TPS65090 is on - * @fet_id FET ID, value between 1 and 7 + * @param bus I2C bus number the TPS65090 is on + * @param fet_id FET ID, value between 1 and 7 * return 0 on success, non-0 on failure */ int tps65090_fet_enable(unsigned int bus, enum fet_id fet_id); @@ -76,8 +76,8 @@ int tps65090_fet_enable(unsigned int bus, enum fet_id fet_id); /** * Disable FET * - * @bus I2C bus number the TPS65090 is on - * @fet_id FET ID, value between 1 and 7 + * @param bus I2C bus number the TPS65090 is on + * @param fet_id FET ID, value between 1 and 7 * @return 0 on success, non-0 on failure */ int tps65090_fet_disable(unsigned int bus, enum fet_id fet_id); @@ -85,8 +85,8 @@ int tps65090_fet_disable(unsigned int bus, enum fet_id fet_id); /** * Is FET enabled? * - * @bus I2C bus number the TPS65090 is on - * @fet_id FET ID, value between 1 and 7 + * @param bus I2C bus number the TPS65090 is on + * @param fet_id FET ID, value between 1 and 7 * @return 1 enabled, 0 disabled, negative value on failure */ int tps65090_fet_is_enabled(unsigned int bus, enum fet_id fet_id); @@ -94,15 +94,15 @@ int tps65090_fet_is_enabled(unsigned int bus, enum fet_id fet_id); /** * Enable / disable the battery charger * - * @bus I2C bus number the TPS65090 is on - * @enable 0 to disable charging, non-zero to enable + * @param bus I2C bus number the TPS65090 is on + * @param enable 0 to disable charging, non-zero to enable */ int tps65090_set_charge_enable(unsigned int bus, int enable); /** * Check whether we have enabled battery charging * - * @bus I2C bus number the TPS65090 is on + * @param bus I2C bus number the TPS65090 is on * @return 1 if enabled, 0 if disabled */ int tps65090_is_charging(unsigned int bus); @@ -110,7 +110,7 @@ int tps65090_is_charging(unsigned int bus); /** * Return the value of the status register * - * @bus I2C bus number the TPS65090 is on + * @param bus I2C bus number the TPS65090 is on * @return status register value, or -1 on error */ int tps65090_get_status(unsigned int bus); diff --git a/src/include/spi_flash.h b/src/include/spi_flash.h index 1a78b5a904..8ac7912f91 100644 --- a/src/include/spi_flash.h +++ b/src/include/spi_flash.h @@ -30,9 +30,9 @@ /** * container_of - cast a member of a structure out to the containing structure - * @ptr: the pointer to the member. - * @type: the type of the container struct this is embedded in. - * @member: the name of the member within the struct. + * @param ptr: the pointer to the member. + * @param type: the type of the container struct this is embedded in. + * @param member: the name of the member within the struct. * */ #define container_of(ptr, type, member) ({ \ diff --git a/src/lib/ramtest.c b/src/lib/ramtest.c index e9173fa7e6..03c4aa7d78 100644 --- a/src/lib/ramtest.c +++ b/src/lib/ramtest.c @@ -52,9 +52,9 @@ static void phys_memory_barrier(void) * memory bus. To test most address lines, addresses are scattered * using 256B, 4kB and 64kB increments. * - * @idx Index to test pattern (0=