Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]

Groups > linux.kernel > #1264754 > unrolled thread

[PATCH 9/9] Staging: lustre: llite_internal: Remove function prototype

Back to article view | Back to linux.kernel

This discussion starts older than the indexed window; earlier articles aren't shown. The article labeled Started by below is the oldest one visible, not the original post.

Contents

  [PATCH 9/9] Staging: lustre: llite_internal: Remove function prototype Shivani Bhardwaj <shivanib134@gmail.com> - 2015-11-07 08:50 +0100
    Re: [PATCH 9/9] Staging: lustre: llite_internal: Remove function  prototype Dan Carpenter <dan.carpenter@oracle.com> - 2015-11-07 11:50 +0100
      Re: [PATCH 9/9] Staging: lustre: llite_internal: Remove function prototype Shivani Bhardwaj <shivanib134@gmail.com> - 2015-11-07 16:30 +0100
        Re: [PATCH 9/9] Staging: lustre: llite_internal: Remove function  prototype Dan Carpenter <dan.carpenter@oracle.com> - 2015-11-07 21:50 +0100
          Re: [PATCH 9/9] Staging: lustre: llite_internal: Remove function prototype Shivani Bhardwaj <shivanib134@gmail.com> - 2015-11-08 09:40 +0100

#1264754 — [PATCH 9/9] Staging: lustre: llite_internal: Remove function prototype

Remove the prototype of the function ll_finish_md_op_data() as it is no
longer needed.

Signed-off-by: Shivani Bhardwaj <shivanib134@gmail.com>
---
 drivers/staging/lustre/lustre/llite/llite_internal.h | 1 -
 1 file changed, 1 deletion(-)

diff --git a/drivers/staging/lustre/lustre/llite/llite_internal.h b/drivers/staging/lustre/lustre/llite/llite_internal.h
index 157c3284..d12d483 100644
--- a/drivers/staging/lustre/lustre/llite/llite_internal.h
+++ b/drivers/staging/lustre/lustre/llite/llite_internal.h
@@ -792,7 +792,6 @@ struct md_op_data *ll_prep_md_op_data(struct md_op_data *op_data,
 				      struct inode *i1, struct inode *i2,
 				      const char *name, int namelen,
 				      int mode, __u32 opc, void *data);
-void ll_finish_md_op_data(struct md_op_data *op_data);
 int ll_get_obd_name(struct inode *inode, unsigned int cmd, unsigned long arg);
 char *ll_get_fsname(struct super_block *sb, char *buf, int buflen);
 void ll_open_cleanup(struct super_block *sb, struct ptlrpc_request *open_req);
-- 
2.1.0

--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

[toc] | [next] | [standalone]

#1264806 — Re: [PATCH 9/9] Staging: lustre: llite_internal: Remove function prototype

This one should have been folded in with the last one.  Really the whole
series could have been sent as one patch, but especially the last two
should have been folded together.

regards,
dan carpenter

--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

[toc] | [prev] | [next] | [standalone]

#1264862

On Sat, Nov 7, 2015 at 4:15 PM, Dan Carpenter <dan.carpenter@oracle.com> wrote:
> This one should have been folded in with the last one.  Really the whole
> series could have been sent as one patch, but especially the last two
> should have been folded together.
>
> regards,
> dan carpenter
>

Thanks Dan! Could you please suggest whether I should be resending the
complete series?
I'll take care for future patches.

Thank you
Shivani
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

[toc] | [prev] | [next] | [standalone]

#1264931 — Re: [PATCH 9/9] Staging: lustre: llite_internal: Remove function prototype

Don't resend, it's fine.  I don't like re-reviewing patches and you
don't like sending them.  It's not like it introduces a bug or generate
a warning with our current tools.

But it's still important to understand how the one thing per patch rule
works (don't do half a thing per patch) so I wanted you to be aware of
it for future patches as you said.

regards,
dan carpenter

--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

[toc] | [prev] | [next] | [standalone]

#1265039

On Sun, Nov 8, 2015 at 2:17 AM, Dan Carpenter <dan.carpenter@oracle.com> wrote:
> Don't resend, it's fine.  I don't like re-reviewing patches and you
> don't like sending them.  It's not like it introduces a bug or generate
> a warning with our current tools.
>
> But it's still important to understand how the one thing per patch rule
> works (don't do half a thing per patch) so I wanted you to be aware of
> it for future patches as you said.
>

I'll take care next time.

Thank you
Shivani

> regards,
> dan carpenter
>
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

[toc] | [prev] | [standalone]

Back to top | Article view | linux.kernel

csiph-web