Re: [PATCH] Promise 'stex' driver

[Alexey Dobriyan <adobriyan@gmail.com>]

On Sat, Jun 10, 2006 at 12:08:52PM -0400, Jeff Garzik wrote:
> --- /dev/null
> +++ b/drivers/scsi/stex.c

> +static inline void stex_gettime(u32 *time)

__le32 *time

> +{
> +	struct timeval tv;
> +	do_gettimeofday(&tv);
> +
> +	*time = cpu_to_le32(tv.tv_sec & 0xffffffff);
> +	*(time + 1) = cpu_to_le32((tv.tv_sec >> 16) >> 16);
> +}

> +static void
> +stex_send_cmd(struct st_hba *hba, struct req_msg *req, u16 tag)
> +{
> +	req->tag = cpu_to_le16(tag);
> +	req->task_attr = TASK_ATTRIBUTE_SIMPLE;
> +	req->task_manage = 0; /* not supported yet */
> +	req->payload_sz = (u8)(sizeof(struct req_msg)/sizeof(u32));

sparse warns here. Is this OK?

drivers/scsi/stex.c:442:47: warning: cast truncates bits from constant value (106 becomes 6)

-
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/