* [PATCH 1/1] iscsi: kill redundant casts
@ 2014-06-09 19:57 Nick Black
[not found] ` <1402343822-27701-1-git-send-email-nlb-hpIqsD4AKlfQT0dZR+AlfA@public.gmane.org>
0 siblings, 1 reply; 2+ messages in thread
From: Nick Black @ 2014-06-09 19:57 UTC (permalink / raw)
To: Mike Christie, open-iscsi; +Cc: linux-kernel, Nick Black
Remove two redundant casts from char * to char *.
Signed-off-by: Nick Black <nlb@google.com>
---
drivers/scsi/scsi_transport_iscsi.c | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/drivers/scsi/scsi_transport_iscsi.c b/drivers/scsi/scsi_transport_iscsi.c
index 0102a2d..38a1de3 100644
--- a/drivers/scsi/scsi_transport_iscsi.c
+++ b/drivers/scsi/scsi_transport_iscsi.c
@@ -3059,7 +3059,7 @@ iscsi_get_chap(struct iscsi_transport *transport, struct nlmsghdr *nlh)
evchap->u.get_chap.host_no = ev->u.get_chap.host_no;
evchap->u.get_chap.chap_tbl_idx = ev->u.get_chap.chap_tbl_idx;
evchap->u.get_chap.num_entries = ev->u.get_chap.num_entries;
- buf = (char *) ((char *)evchap + sizeof(*evchap));
+ buf = (char *)evchap + sizeof(*evchap);
memset(buf, 0, chap_buf_size);
err = transport->get_chap(shost, ev->u.get_chap.chap_tbl_idx,
@@ -3463,7 +3463,7 @@ iscsi_get_host_stats(struct iscsi_transport *transport, struct nlmsghdr *nlh)
evhost_stats->type = nlh->nlmsg_type;
evhost_stats->u.get_host_stats.host_no =
ev->u.get_host_stats.host_no;
- buf = (char *)((char *)evhost_stats + sizeof(*evhost_stats));
+ buf = (char *)evhost_stats + sizeof(*evhost_stats);
memset(buf, 0, host_stats_size);
err = transport->get_host_stats(shost, buf, host_stats_size);
--
2.0.0.526.g5318336
^ permalink raw reply related [flat|nested] 2+ messages in thread
* Re: [PATCH 1/1] iscsi: kill redundant casts
[not found] ` <1402343822-27701-1-git-send-email-nlb-hpIqsD4AKlfQT0dZR+AlfA@public.gmane.org>
@ 2014-06-10 6:48 ` Mike Christie
0 siblings, 0 replies; 2+ messages in thread
From: Mike Christie @ 2014-06-10 6:48 UTC (permalink / raw)
To: open-iscsi-/JYPxA39Uh5TLH3MbocFFw; +Cc: linux-scsi, Nick Black
Switching linux-kernel for linux-scsi mailing list.
On 6/9/14, 2:57 PM, 'Nick Black' via open-iscsi wrote:
> Remove two redundant casts from char * to char *.
>
> Signed-off-by: Nick Black <nlb-hpIqsD4AKlfQT0dZR+AlfA@public.gmane.org>
> ---
> drivers/scsi/scsi_transport_iscsi.c | 4 ++--
> 1 file changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/drivers/scsi/scsi_transport_iscsi.c b/drivers/scsi/scsi_transport_iscsi.c
> index 0102a2d..38a1de3 100644
> --- a/drivers/scsi/scsi_transport_iscsi.c
> +++ b/drivers/scsi/scsi_transport_iscsi.c
> @@ -3059,7 +3059,7 @@ iscsi_get_chap(struct iscsi_transport *transport, struct nlmsghdr *nlh)
> evchap->u.get_chap.host_no = ev->u.get_chap.host_no;
> evchap->u.get_chap.chap_tbl_idx = ev->u.get_chap.chap_tbl_idx;
> evchap->u.get_chap.num_entries = ev->u.get_chap.num_entries;
> - buf = (char *) ((char *)evchap + sizeof(*evchap));
> + buf = (char *)evchap + sizeof(*evchap);
> memset(buf, 0, chap_buf_size);
>
> err = transport->get_chap(shost, ev->u.get_chap.chap_tbl_idx,
> @@ -3463,7 +3463,7 @@ iscsi_get_host_stats(struct iscsi_transport *transport, struct nlmsghdr *nlh)
> evhost_stats->type = nlh->nlmsg_type;
> evhost_stats->u.get_host_stats.host_no =
> ev->u.get_host_stats.host_no;
> - buf = (char *)((char *)evhost_stats + sizeof(*evhost_stats));
> + buf = (char *)evhost_stats + sizeof(*evhost_stats);
> memset(buf, 0, host_stats_size);
>
> err = transport->get_host_stats(shost, buf, host_stats_size);
>
Looks ok to me.
Acked-by: Mike Christie <michaelc-hcNo3dDEHLuVc3sceRu5cw@public.gmane.org>
--
You received this message because you are subscribed to the Google Groups "open-iscsi" group.
To unsubscribe from this group and stop receiving emails from it, send an email to open-iscsi+unsubscribe-/JYPxA39Uh5TLH3MbocFF+G/Ez6ZCGd0@public.gmane.org
To post to this group, send email to open-iscsi-/JYPxA39Uh5TLH3MbocFF+G/Ez6ZCGd0@public.gmane.org
Visit this group at http://groups.google.com/group/open-iscsi.
For more options, visit https://groups.google.com/d/optout.
^ permalink raw reply [flat|nested] 2+ messages in thread
end of thread, other threads:[~2014-06-10 6:48 UTC | newest]
Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2014-06-09 19:57 [PATCH 1/1] iscsi: kill redundant casts Nick Black
[not found] ` <1402343822-27701-1-git-send-email-nlb-hpIqsD4AKlfQT0dZR+AlfA@public.gmane.org>
2014-06-10 6:48 ` Mike Christie
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.