From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org X-Spam-Level: X-Spam-Status: No, score=-12.7 required=3.0 tests=BAYES_00,DKIM_SIGNED, DKIM_VALID,DKIM_VALID_AU,FREEMAIL_FORGED_FROMDOMAIN,FREEMAIL_FROM, HEADER_FROM_DIFFERENT_DOMAINS,INCLUDES_CR_TRAILER,INCLUDES_PATCH, MAILING_LIST_MULTI,SPF_HELO_NONE,SPF_PASS autolearn=ham autolearn_force=no version=3.4.0 Received: from mail.kernel.org (mail.kernel.org [198.145.29.99]) by smtp.lore.kernel.org (Postfix) with ESMTP id 79725C43460 for ; Sat, 24 Apr 2021 02:10:32 +0000 (UTC) Received: from vger.kernel.org (vger.kernel.org [23.128.96.18]) by mail.kernel.org (Postfix) with ESMTP id 46C1B6137D for ; Sat, 24 Apr 2021 02:10:32 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S236612AbhDXCLH (ORCPT ); Fri, 23 Apr 2021 22:11:07 -0400 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:54676 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S236539AbhDXCLF (ORCPT ); Fri, 23 Apr 2021 22:11:05 -0400 Received: from mail-qv1-xf2e.google.com (mail-qv1-xf2e.google.com [IPv6:2607:f8b0:4864:20::f2e]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id C24FDC061574; Fri, 23 Apr 2021 19:10:15 -0700 (PDT) Received: by mail-qv1-xf2e.google.com with SMTP id x27so24783251qvd.2; Fri, 23 Apr 2021 19:10:15 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20161025; h=message-id:subject:from:to:cc:date:in-reply-to:references :user-agent:mime-version:content-transfer-encoding; bh=kcIelX1EsGDmdwC3ocBtd0zMhNA8a0bphR97MMdWuis=; b=GtqXz/xQNMqRVwYlcLPVUMocoqdYzVUPJWUnA23leuh4I/iob899wcJIyWdkjD1pLT HzYG8pOXtESdV3CWhiLV1RQBT7XKsjIYNOrNE5sIYbsN9cgiJBFGEH/x77PIH4hx2i8V kUyhiunfmiGTNSu4QeBawS4JGhK+gqsANyvi2tHNvs5lxzPjAO06HwR29IcltnnombrW UPwqtt+5fT1hrpvYgyFSXna8oKgoWT2Xec/W01S1aW6E9KFAVcfc3T9/BLdKoCz9Tkmq mZ7V3UPFXEWDfbBqgwEff9i8ljBLuLmG+ozHTxzAmyh0DXkO430fwklCdE4KmMKXETSr /P0A== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:message-id:subject:from:to:cc:date:in-reply-to :references:user-agent:mime-version:content-transfer-encoding; bh=kcIelX1EsGDmdwC3ocBtd0zMhNA8a0bphR97MMdWuis=; b=l6THvMMk+QuSoyOuhP2ACWiEaBuPqpt76OKIpRdpW4J4Q8YrrQMzj7teGsKM8tBw6X dlhJFfWmKXM9rYP08XOk9QHwzP6KC9u1/BYZcaOoDZvJRtTCxem7KNZXGzy53aCatAq4 J0Y4r95aF1ld4wE04Rt7QDggamFdFHe6uMIbWP8sMEF9JwZEKdkzSqLGvQ47CkD/VkK9 tF1AQ+g2fzBp03UYfeb9wYwoe1T6t0p2KAeMHQ+yu8aptxtq7kby2nWraHEx6qmtnRwN CcOsjm0UGMB/7P8k6JtCAyEEpFnix0QgEEtIR0KT54mdCiX7JIzXld2f/Z5r/e6MEE8v NHqA== X-Gm-Message-State: AOAM532pqeaxVvGkX1uxAz9xU0AsgrxmUn9RLFAkzj+cGySpk5/HTs/7 cwygVQz+qX34VNjfGH9x4p4= X-Google-Smtp-Source: ABdhPJxGNuD8Q0uexHAd2IERDSyUQL8artcvLH0hyQwEglMOZTYRD82ofp1LixQt+TEdr8EVqzALWQ== X-Received: by 2002:ad4:4c86:: with SMTP id bs6mr7796941qvb.39.1619230215035; Fri, 23 Apr 2021 19:10:15 -0700 (PDT) Received: from ?IPv6:2001:1284:f013:744e:ffa4:164a:a3e9:c671? ([2001:1284:f013:744e:ffa4:164a:a3e9:c671]) by smtp.gmail.com with ESMTPSA id l12sm5502413qth.72.2021.04.23.19.10.10 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Fri, 23 Apr 2021 19:10:13 -0700 (PDT) Message-ID: <3ce3294c173c954381b26ab049222c186f81ddf4.camel@gmail.com> Subject: Re: [Outreachy kernel] [PATCH v3] staging: media: atomisp: pci: Change line break to avoid an open parenthesis at the end of the line From: Aline Santana Cordeiro To: Hans Verkuil , Julia Lawall , Matthew Wilcox Cc: Mauro Carvalho Chehab , Sakari Ailus , Greg Kroah-Hartman , linux-media@vger.kernel.org, linux-staging@lists.linux.dev, linux-kernel@vger.kernel.org, outreachy-kernel@googlegroups.com Date: Fri, 23 Apr 2021 23:10:09 -0300 In-Reply-To: References: <20210421123718.GA4597@focaruja> <7aeac7041a6f6d7b3d8563f0d0bf0a4d31f379b0.camel@gmail.com> <16dd7a16a8cc69aa0f81dd6bf47f09e878c71a6b.camel@gmail.com> Content-Type: text/plain; charset="UTF-8" User-Agent: Evolution 3.40.0 (by Flathub.org) MIME-Version: 1.0 Content-Transfer-Encoding: 8bit Precedence: bulk List-ID: X-Mailing-List: linux-media@vger.kernel.org Em sex, 2021-04-23 às 11:21 +0200, Hans Verkuil escreveu: > On 21/04/2021 16:21, Aline Santana Cordeiro wrote: > > Em qua, 2021-04-21 às 15:56 +0200, Julia Lawall escreveu: > > > > > > > > > On Wed, 21 Apr 2021, Aline Santana Cordeiro wrote: > > > > > > > Em qua, 2021-04-21 às 15:08 +0200, Julia Lawall escreveu: > > > > > > > > > > > > > > > On Wed, 21 Apr 2021, Aline Santana Cordeiro wrote: > > > > > > > > > > > Change line break to avoid an open parenthesis at the end > > > > > > of > > > > > > the > > > > > > line. > > > > > > It consequently removed spaces at the start of the > > > > > > subsequent > > > > > > line. > > > > > > > > > > The message is hard to understand.  There are a lot of > > > > > singular > > > > > nouns, but > > > > > actually there are two changes.  Which change is being > > > > > described > > > > > by > > > > > the > > > > > above message?  What does "It" refer to? > > > > > > > > > > julia > > > > > > > > Checkpatch indicated two problems with this function > > > > declaration: > > > > 1) The line ending with an open parenthesis, and > > > > 2) The following line - with the function parameters - has > > > > spaces > > > > in > > > > its identation. > > > > > > > > When I changed the line break to put the function name and its > > > > parameter in the following line, both checkpath checks were > > > > eliminated. > > > > > > > > So, the main change was the line break and, also, the line > > > > break > > > > (it) > > > > removed the space in the following line. > > > > > > > > Is it better to change the message and explain only about the > > > > line > > > > break? > > > > > > The message should explain about the whole patch.  So if you > > > change > > > two > > > things, it should be clear that what you are saying covers both > > > of > > > them. > > > > Ok, I can do that. In the commit message I described just one issue > > because it is only one patch, I didn't want it to look like I was > > changing different issues in just one patch. > > > > > > > > But it seems that Matthew doesn't think that the line break is a > > > good > > > idea > > > anyway. > > > > Yes, I'm sending this email to Matthew too, because I don't know > > exactly how to proceed as Hans asked me to made some corrections > > too.  > > I've made these changes because checkpatch has indicated and with > > this > > line break, checkpatch does not indicate any check or warning > > anymore. > > But I can undo that too, I just don't know what I'm supposed to do > > with > > so many opposite opinions. > > As one of the media maintainers I can say that in this case the > preference > would be to split it up in two lines. It's one of those areas where > different maintainers have different opinions. > > Just keep in mind that this is all nitpicking and normally we > probably > wouldn't bother with this at all, but it is a good exercise to learn > about patches and contributing :-) > > Regards, > >         Hans I really appreciate all the feedbacks I've received :) Indeed we can learn a lot about all the contributing process. Thank you, Aline > > > > > > > Thank you all, > > Aline > > > > > > julia > > > > > > > > > > > Thank you, > > > > Aline > > > > > > > > > > > > > > > > Both issues detected by checkpatch.pl. > > > > > > > > > > > > Signed-off-by: Aline Santana Cordeiro < > > > > > > alinesantanacordeiro@gmail.com> > > > > > > --- > > > > > > > > > > > > Changes since v2: > > > > > >  - Insert a space between the function type and pointer > > > > > > > > > > > > Changes since v1: > > > > > >  - Keep the pointer with the function return type > > > > > >    instead of left it with the function name > > > > > > > > > > > >  drivers/staging/media/atomisp/pci/atomisp_cmd.h | 10 > > > > > > +++++---- > > > > > > - > > > > > >  1 file changed, 5 insertions(+), 5 deletions(-) > > > > > > > > > > > > diff --git > > > > > > a/drivers/staging/media/atomisp/pci/atomisp_cmd.h > > > > > > b/drivers/staging/media/atomisp/pci/atomisp_cmd.h > > > > > > index 1c0d464..639eca3 100644 > > > > > > --- a/drivers/staging/media/atomisp/pci/atomisp_cmd.h > > > > > > +++ b/drivers/staging/media/atomisp/pci/atomisp_cmd.h > > > > > > @@ -75,8 +75,8 @@ void atomisp_wdt(struct timer_list *t); > > > > > >  void atomisp_setup_flash(struct atomisp_sub_device *asd); > > > > > >  irqreturn_t atomisp_isr(int irq, void *dev); > > > > > >  irqreturn_t atomisp_isr_thread(int irq, void *isp_ptr); > > > > > > -const struct atomisp_format_bridge > > > > > > *get_atomisp_format_bridge_from_mbus( > > > > > > -    u32 mbus_code); > > > > > > +const struct atomisp_format_bridge * > > > > > > +get_atomisp_format_bridge_from_mbus(u32 mbus_code); > > > > > >  bool atomisp_is_mbuscode_raw(uint32_t code); > > > > > >  int atomisp_get_frame_pgnr(struct atomisp_device *isp, > > > > > >                            const struct ia_css_frame > > > > > > *frame, > > > > > > u32 > > > > > > *p_pgnr); > > > > > > @@ -381,9 +381,9 @@ enum mipi_port_id > > > > > > __get_mipi_port(struct > > > > > > atomisp_device *isp, > > > > > > > > > > > >  bool atomisp_is_vf_pipe(struct atomisp_video_pipe *pipe); > > > > > > > > > > > > -void atomisp_apply_css_parameters( > > > > > > -    struct atomisp_sub_device *asd, > > > > > > -    struct atomisp_css_params *css_param); > > > > > > +void atomisp_apply_css_parameters(struct > > > > > > atomisp_sub_device > > > > > > *asd, > > > > > > +                                 struct atomisp_css_params > > > > > > *css_param); > > > > > > + > > > > > >  void atomisp_free_css_parameters(struct atomisp_css_params > > > > > > *css_param); > > > > > > > > > > > >  void atomisp_handle_parameter_and_buffer(struct > > > > > > atomisp_video_pipe > > > > > > *pipe); > > > > > > -- > > > > > > 2.7.4 > > > > > > > > > > > > -- > > > > > > 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 > > > > > > outreachy-kernel+unsubscribe@googlegroups.com. > > > > > > To view this discussion on the web visit > > > > > > https://groups.google.com/d/msgid/outreachy-kernel/20210421123718.GA4597%40focaruja > > > > > > . > > > > > > > > > > > > > > > > > > -- > > > > 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 outreachy-kernel+unsubscribe@googlegroups.com. > > > > To view this discussion on the web visit    > > > > https://groups.google.com/d/msgid/outreachy-kernel/7aeac7041a6f6d7b3d8563f0d0bf0a4d31f379b0.camel%40gmail.com > > > > . > > > > >