On 26 November 2015 at 23:37, Bin Meng <[email protected]> wrote: > Hi Simon, > > On Fri, Nov 27, 2015 at 10:51 AM, Simon Glass <[email protected]> wrote: >> The function comments use an old style and some are incorrect. Update them. >> >> Signed-off-by: Simon Glass <[email protected]> >> Reviewed-by: Bin Meng <[email protected]> >> --- >> >> Changes in v3: >> - Use 'Show' instead of 'Shows' in the the pci_header_show() function comment >> >> Changes in v2: >> - Add a new patch to update the function comments >> >> common/cmd_pci.c | 84 >> ++++++++++++++++++++++++++------------------------------ >> 1 file changed, 39 insertions(+), 45 deletions(-) >> >> diff --git a/common/cmd_pci.c b/common/cmd_pci.c >> index f8faa31..7fdd3d8 100644 >> --- a/common/cmd_pci.c >> +++ b/common/cmd_pci.c >> @@ -161,15 +161,10 @@ static struct pci_reg_info regs_cardbus[] = { >> {}, >> }; >> >> -/* >> - * Subroutine: PCI_Header_Show >> - * >> - * Description: Reads the header of the specified PCI device. >> - * >> - * Inputs: BusDevFunc Bus+Device+Function number >> - * >> - * Return: None >> +/** >> + * @pci_header_show() - Show the header of the specified PCI device. > > nits: please remove @ before pci_header_show(). > Sorry I missed this in previous review. Please fix it when applying. >
Fixed, and Applied to u-boot-dm. _______________________________________________ U-Boot mailing list [email protected] http://lists.denx.de/mailman/listinfo/u-boot
