[U-Boot] [PATCH v2 19/38] x86: fsp: Tidy up comment style a little
Bin Meng
bmeng.cn at gmail.com
Wed Oct 2 14:06:44 UTC 2019
Hi Simon,
On Wed, Sep 25, 2019 at 10:12 PM Simon Glass <sjg at chromium.org> wrote:
>
> The comments in the FSP code use a different style from the rest of the
> x86 code. I am not sure it this is intentional.
I guess it's because when it was written I was not familiar with the
documenting style :)
>
> Signed-off-by: Simon Glass <sjg at chromium.org>
> ---
>
> Changes in v2: None
>
> arch/x86/include/asm/fsp/fsp_support.h | 42 ++++++++++++-------------
> arch/x86/include/asm/fsp1/fsp_support.h | 30 ++++++++++++------
> arch/x86/include/asm/hob.h | 18 +++++------
> arch/x86/lib/hob.c | 8 ++---
> 4 files changed, 54 insertions(+), 44 deletions(-)
>
[snip]
> diff --git a/arch/x86/include/asm/hob.h b/arch/x86/include/asm/hob.h
> index b4239821aaa..72151ea045e 100644
> --- a/arch/x86/include/asm/hob.h
> +++ b/arch/x86/include/asm/hob.h
> @@ -135,7 +135,7 @@ struct hob_guid {
> *
> * @hdr: A pointer to a HOB.
> *
> - * @return: A pointer to the next HOB in the HOB list.
> + * @return A pointer to the next HOB in the HOB list.
> */
> static inline const struct hob_header *get_next_hob(const struct hob_header
> *hdr)
> @@ -152,8 +152,8 @@ static inline const struct hob_header *get_next_hob(const struct hob_header
> *
> * @hdr: A pointer to a HOB.
> *
> - * @retval true: The HOB specified by hdr is the last HOB in the HOB list.
> - * @retval false: The HOB specified by hdr is not the last HOB in the HOB list.
> + * @return true: The HOB specified by hdr is the last HOB in the HOB list.
> + * @return false: The HOB specified by hdr is not the last HOB in the HOB list.
> */
> static inline bool end_of_hob(const struct hob_header *hdr)
> {
> @@ -169,7 +169,7 @@ static inline bool end_of_hob(const struct hob_header *hdr)
> *
> * @hdr: A pointer to a HOB.
> *
> - * @return: A pointer to the data buffer in a HOB.
> + * @return A pointer to the data buffer in a HOB.
> */
> static inline void *get_guid_hob_data(const struct hob_header *hdr)
> {
> @@ -185,7 +185,7 @@ static inline void *get_guid_hob_data(const struct hob_header *hdr)
> *
> * @hdr: A pointer to a HOB.
> *
> - * @return: The size of the data buffer.
> + * @return The size of the data buffer.
> */
> static inline u16 get_guid_hob_data_size(const struct hob_header *hdr)
> {
> @@ -198,7 +198,7 @@ static inline u16 get_guid_hob_data_size(const struct hob_header *hdr)
> * @type: HOB type to search
> * @hob_list: A pointer to the HOB list
> *
> - * @retval: A HOB object with matching type; Otherwise NULL.
> + *@return A HOB object with matching type; Otherwise NULL.
should have a space between * and @
> */
> const struct hob_header *hob_get_next_hob(uint type, const void *hob_list);
>
> @@ -208,7 +208,7 @@ const struct hob_header *hob_get_next_hob(uint type, const void *hob_list);
> * @guid: GUID to search
> * @hob_list: A pointer to the HOB list
> *
> - * @retval: A HOB object with matching GUID; Otherwise NULL.
> + *@return A HOB object with matching GUID; Otherwise NULL.
should have a space between * and @
> */
> const struct hob_header *hob_get_next_guid_hob(const efi_guid_t *guid,
> const void *hob_list);
> @@ -221,8 +221,8 @@ const struct hob_header *hob_get_next_guid_hob(const efi_guid_t *guid,
> * If the GUID HOB is located, the length will be updated.
> * @guid A pointer to HOB GUID.
> *
> - * @retval NULL: Failed to find the GUID HOB.
> - * @retval others: GUID HOB data buffer pointer.
> + * @return NULL: Failed to find the GUID HOB.
> + * @return others: GUID HOB data buffer pointer.
> */
> void *hob_get_guid_hob_data(const void *hob_list, u32 *len,
> const efi_guid_t *guid);
> diff --git a/arch/x86/lib/hob.c b/arch/x86/lib/hob.c
> index dcee29b04cf..f2c47240ee8 100644
> --- a/arch/x86/lib/hob.c
> +++ b/arch/x86/lib/hob.c
> @@ -13,7 +13,7 @@
> * @type: HOB type to search
> * @hob_list: A pointer to the HOB list
> *
> - * @retval: A HOB object with matching type; Otherwise NULL.
> + * @return A HOB object with matching type; Otherwise NULL.
> */
> const struct hob_header *hob_get_next_hob(uint type, const void *hob_list)
> {
> @@ -38,7 +38,7 @@ const struct hob_header *hob_get_next_hob(uint type, const void *hob_list)
> * @guid: GUID to search
> * @hob_list: A pointer to the HOB list
> *
> - * @retval: A HOB object with matching GUID; Otherwise NULL.
> + * @return A HOB object with matching GUID; Otherwise NULL.
> */
> const struct hob_header *hob_get_next_guid_hob(const efi_guid_t *guid,
> const void *hob_list)
> @@ -65,8 +65,8 @@ const struct hob_header *hob_get_next_guid_hob(const efi_guid_t *guid,
> * If the GUID HOB is located, the length will be updated.
> * @guid A pointer to HOB GUID.
> *
> - * @retval NULL: Failed to find the GUID HOB.
> - * @retval others: GUID HOB data buffer pointer.
> + * @return NULL: Failed to find the GUID HOB.
> + * @return others: GUID HOB data buffer pointer.
> */
> void *hob_get_guid_hob_data(const void *hob_list, u32 *len,
> const efi_guid_t *guid)
> --
Other than above,
Reviewed-by: Bin Meng <bmeng.cn at gmail.com>
Regards,
Bin
More information about the U-Boot
mailing list