[lkml]   [2017]   [Sep]   [8]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
Messages in this thread
SubjectRe: [Outreachy kernel] [PATCH] staging: typec: tcpm: Comparison to NULL changed as "!port->partner_altmode[pmdata->altmodes]"

On Fri, 8 Sep 2017, Harsha Sharma wrote:

> Fixes coding style issue

Good job on getting the From issue sorted out.

The subject line change description is a bit verbose. One doesn't want
to see the whole code, but rather the concept of what has changed. If you
were going to tell someone about your patch, maybe someone ho knows about
C but not about the Linux kernel, what is the first thing you would say?

Afterwards, there is perhaps not so much to say about such a change, but
try to come up with something less generic for the log message, preferably
something that doesn't use "fix". What is the purpose of doing the
change. Just because a tool said to do it is not the best answer. How
does it make the code better?


> Signed-off-by: Harsha Sharma <>
> ---
> drivers/staging/typec/tcpm.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
> diff --git a/drivers/staging/typec/tcpm.c b/drivers/staging/typec/tcpm.c
> index 630fe75..9f4a5a5 100644
> --- a/drivers/staging/typec/tcpm.c
> +++ b/drivers/staging/typec/tcpm.c
> @@ -982,7 +982,7 @@ static void svdm_consume_modes(struct tcpm_port *port, const __le32 *payload,
> }
> port->partner_altmode[pmdata->altmodes] =
> typec_partner_register_altmode(port->partner, paltmode);
> - if (port->partner_altmode[pmdata->altmodes] == NULL) {
> + if (!port->partner_altmode[pmdata->altmodes]) {
> tcpm_log(port,
> "Failed to register alternate modes for SVID 0x%04x",
> paltmode->svid);
> --
> 1.9.1
> --
> You received this message because you are subscribed to the Google Groups "outreachy-kernel" group.
> To unsubscribe from this group and stop receiving emails from it, send an email to
> To post to this group, send email to
> To view this discussion on the web visit
> For more options, visit

 \ /
  Last update: 2017-09-08 16:44    [W:0.028 / U:10.640 seconds]
©2003-2018 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site