On ١١/٣/٢٠٢٣ ١٦:٣٨, Julia Lawall wrote:
On Sat, 11 Mar 2023, Menna Mahmoud wrote:
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?
The existing code indeed looks better to me. So you can skip this issue.
julia
Okay, thanks Julia.
-Menna
> Signed-off-by: Menna Mahmoud eng.mennamahmoud.mm@gmail.com > --- > 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 > > >