<div dir="ltr">Okay sorry I missed out on the coding standards docs.<div>yes code blocks edited without telling I will change its settings</div><div><br></div><div><br></div></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Thu, Feb 27, 2020 at 9:18 PM Gedare Bloom <<a href="mailto:gedare@rtems.org">gedare@rtems.org</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">Hi Suyash,<br>
<br>
I have a  few comments for you.<br>
<br>
First, the commit message should follow the guidance at<br>
<a href="https://docs.rtems.org/branches/master/eng/vc-users.html#creating-a-patch" rel="noreferrer" target="_blank">https://docs.rtems.org/branches/master/eng/vc-users.html#creating-a-patch</a><br>
and follow through the link about Commit Messages. We like to have a<br>
short "tag" at the start of the commit to indicate the area of RTEMS<br>
the commit applies to, in this case, maybe "bsps/shared: ignore return<br>
value in display driver"  -- We don't currently have a set of standard<br>
tags.<br>
<br>
Second, avoid making unnecessary changes in whitespace elsewhere in<br>
source code. My guess is that your text editor did this for you<br>
automatically. You may want to investigate how to disable it from<br>
rewriting the entire file for formatting on open.<br>
<br>
Third, if a return value is unused, there are two options to consider:<br>
(a) it should be used, or (b) it should be ignored. Either way, we<br>
would prefer to be explicit about the programmer intent. In this case,<br>
does it make sense to check the return status of<br>
rtems_semaphore_release? Can it fail? Would it matter if it does?<br>
<br>
To explicitly ignore a return value, you can do this instead:<br>
<br>
rc = rtems_semaphore_release(...)<br>
(void) rc;<br>
<br>
As directed by our coding standards: "Use ‘(void) unused;’ to mark<br>
unused parameters and set-but-unused variables immediately after being<br>
set."<br>
<br>
<br>
On Thu, Feb 27, 2020 at 3:24 AM suyash singh <<a href="mailto:suyashsingh234@gmail.com" target="_blank">suyashsingh234@gmail.com</a>> wrote:<br>
><br>
> ---<br>
>  bsps/shared/dev/display/disp_hcms29xx.c | 8 ++++----<br>
>  1 file changed, 4 insertions(+), 4 deletions(-)<br>
><br>
> diff --git a/bsps/shared/dev/display/disp_hcms29xx.c b/bsps/shared/dev/display/disp_hcms29xx.c<br>
> index 5730b36ea9..9d3e7220cf 100644<br>
> --- a/bsps/shared/dev/display/disp_hcms29xx.c<br>
> +++ b/bsps/shared/dev/display/disp_hcms29xx.c<br>
> @@ -530,7 +530,7 @@ static rtems_task disp_hcms29xx_update_task<br>
>  +---------------------------------------------------------------------------+<br>
>  | Input Parameters:                                                         |<br>
>  \*-------------------------------------------------------------------------*/<br>
> -   rtems_task_argument argument<br>
> +   rtems_task_argument argument<br>
>  )<br>
>  /*-------------------------------------------------------------------------*\<br>
>  | Return Value:                                                             |<br>
> @@ -597,7 +597,7 @@ static rtems_task disp_hcms29xx_update_task<br>
>           (int) strlen(softc_ptr->disp_param.disp_buffer);<br>
>        }<br>
>        if (rc == RTEMS_SUCCESSFUL) {<br>
> -       rc = rtems_semaphore_release(softc_ptr->disp_param.trns_sema_id);<br>
> +        rtems_semaphore_release(softc_ptr->disp_param.trns_sema_id);<br>
>        }<br>
>        /*<br>
>         * set initial offset to negative value<br>
> @@ -911,7 +911,7 @@ static rtems_driver_address_table disp_hcms29xx_ops = {<br>
><br>
>  static disp_hcms29xx_drv_t disp_hcms29xx_drv_tbl = {<br>
>    {/* public fields */<br>
> -    .ops =         &disp_hcms29xx_ops,<br>
> +    .ops =         &disp_hcms29xx_ops,<br>
>      .size =        sizeof (disp_hcms29xx_drv_t),<br>
>    },<br>
>    { /* our private fields */<br>
> @@ -927,6 +927,6 @@ static disp_hcms29xx_drv_t disp_hcms29xx_drv_tbl = {<br>
>    }<br>
>  };<br>
><br>
> -rtems_libi2c_drv_t *disp_hcms29xx_driver_descriptor =<br>
> +rtems_libi2c_drv_t *disp_hcms29xx_driver_descriptor =<br>
>    &disp_hcms29xx_drv_tbl.libi2c_drv_entry;<br>
><br>
> --<br>
> 2.17.1<br>
><br>
> _______________________________________________<br>
> devel mailing list<br>
> <a href="mailto:devel@rtems.org" target="_blank">devel@rtems.org</a><br>
> <a href="http://lists.rtems.org/mailman/listinfo/devel" rel="noreferrer" target="_blank">http://lists.rtems.org/mailman/listinfo/devel</a><br>
</blockquote></div>