Hello Simon, Simon Glass wrote: > Hi Heiko, > > On Jan 14, 2012 11:28 PM, "Heiko Schocher" <h...@denx.de> wrote: >> Hello Simon, >> >> Simon Glass wrote: >>> Add tegra_i2c_get_dvc_bus_num() to obtain the I2C bus number of DVC bus. >>> This allows us to talk to the PMU. >>> >>> Signed-off-by: Simon Glass <s...@chromium.org> >>> --- >>> Changes in v2: >>> - Add patch to find DVC bus number >>> >>> arch/arm/include/asm/arch-tegra2/tegra2_i2c.h | 7 +++++++ >>> drivers/i2c/tegra2_i2c.c | 14 ++++++++++++++ >>> 2 files changed, 21 insertions(+), 0 deletions(-) >> just one minor comment, beside of this: >> >> Acked-by: Heiko Schocher <h...@denx.de> >> >>> diff --git a/arch/arm/include/asm/arch-tegra2/tegra2_i2c.h > b/arch/arm/include/asm/arch-tegra2/tegra2_i2c.h >>> index 86f6a01..4816cc7 100644 >>> --- a/arch/arm/include/asm/arch-tegra2/tegra2_i2c.h >>> +++ b/arch/arm/include/asm/arch-tegra2/tegra2_i2c.h >>> @@ -157,4 +157,11 @@ struct i2c_ctlr { >>> #define I2C_INT_ARBITRATION_LOST_SHIFT 2 >>> #define I2C_INT_ARBITRATION_LOST_MASK (1 << > I2C_INT_ARBITRATION_LOST_SHIFT) >>> +/** >> wrong comment style ... > > I am trying to follow the Doxygen convention. Many editors like this also > and display the parameters etc. nicely highlighted. Does yours? Hoping it > encourages more function prototype comments in U-Boot and reduce the time I > need to spend wading through code to see what a function is supposed to do.
added Wolfgang to cc: Do we want to accept this? If so, it must be documented somewhere! bye, Heiko -- DENX Software Engineering GmbH, MD: Wolfgang Denk & Detlev Zundel HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany _______________________________________________ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot