Re: [PATCH 1/2] staging: greybus: Fix Alignment with parenthesis

From: Menna Mahmoud
Date: Sat Mar 11 2023 - 09:07:05 EST



On ١١‏/٣‏/٢٠٢٣ ١٤:٥٧, Menna Mahmoud wrote:

On ١١‏/٣‏/٢٠٢٣ ١٤:٥٥, Julia Lawall wrote:

On Sat, 11 Mar 2023, Menna Mahmoud wrote:

On ١١/٣/٢٠٢٣ ١٠:٥٩, Julia Lawall wrote:
On Fri, 10 Mar 2023, Menna Mahmoud wrote:

Fix " CHECK: Alignment should match open parenthesis "
Reported by checkpath
See the message in the other mail about the log message.

Also, you should not have two patches with the same subject.  Here, the
changes are on the same file and are essentially the same, even involving
the same function call.  So they can be together in one patch.

julia
okay, I will. appreciate your feedback. thanks.
Please put some blank lines around your response, so it is easier to find.

thanks,
julia


Okay, I will.

thanks,

Menna



Hi Julia,

according to Alex feedback

" I think this type of alignment is not a major problem,
and alignment isn't done this way in general in this
driver, it's probably OK to keep it that way. - Alex "


,I won't resubmit these patches, right?


 -Menna




Signed-off-by: Menna Mahmoud <eng.mennamahmoud.mm@xxxxxxxxx>
---
   drivers/staging/greybus/fw-core.c | 2 +-
   1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/staging/greybus/fw-core.c
b/drivers/staging/greybus/fw-core.c
index 57bebf24636b..f562cb12d5ad 100644
--- a/drivers/staging/greybus/fw-core.c
+++ b/drivers/staging/greybus/fw-core.c
@@ -89,7 +89,7 @@ static int gb_fw_core_probe(struct gb_bundle *bundle,
               }

               connection = gb_connection_create(bundle, cport_id,
-                        gb_fw_mgmt_request_handler);
+
gb_fw_mgmt_request_handler);
               if (IS_ERR(connection)) {
                   ret = PTR_ERR(connection);
                   dev_err(&bundle->dev,
--
2.34.1



>