On 20/05/16 14:34, Laxman Dewangan wrote: > > On Friday 20 May 2016 07:02 PM, Jon Hunter wrote: >> On 20/05/16 12:59, Laxman Dewangan wrote: >>> +/* tegra_io_pads_config_info: Tegra IO pads bit config info. >>> + * @dpd_config_bit: DPD configuration bit position. -1 if not >>> supported. >>> + * @voltage_config_bit: Voltage configuration bit position. -1 if >>> not supported. >>> + * @soc_mask: Bitwise OR of SoC masks if IO pads supported on that SoC. >>> + */ >> Comment coding style :-( > > I saw this style multiple places and so intentionally left here. > If comment is inside the code then > /* > * first-line comment > * second line > */ > > but for function, it can have in single line. > > Anyhow, I will correct in next cycle. Actually, this should be in kernel-doc format [0]. Jon [0] Documentation/kernel-doc-nano-HOWTO.txt -- To unsubscribe from this list: send the line "unsubscribe linux-tegra" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html