On Tue, Jan 10, 2017 at 10:44:23PM +0530, Bart Van Assche wrote:
> On Sat, 2017-01-07 at 22:09 +0530, Varun Prakash wrote:
> > -int
> > -iscsit_check_dataout_hdr(struct iscsi_conn *conn, unsigned char *buf,
> > - struct iscsi_cmd **out_cmd)
> > +int __iscsit_check_dataout_hdr(struct iscsi_conn *conn, unsigned char *buf,
> > + struct iscsi_cmd *cmd, bool *success)
> > {
>
> Is there any reason that 'buf' is declared as 'unsigned char *' instead of
> 'void *' in the iscsit_check_dataout_hdr() and __iscsit_check_dataout_hdr()
> declarations? If the data type would be changed into 'void *' several casts
> could be left out.
In this patch I maintained existing data type of 'buf', I will change it
to 'void * buf' in v2.
--
To unsubscribe from this list: send the line "unsubscribe target-devel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
