On 05/05/16 05:52, David Kershner wrote:
> From: Erik Arfvidson <erik.arfvidson@xxxxxxxxxx>
>
> The following struct members were never used:
> putfile_active_buffer::pnext
> putfile_request::file_request_number
> putfile_request::data_sequence_number
>
> Signed-off-by: Erik Arfvidson <erik.arfvidson@xxxxxxxxxx>
> Signed-off-by: David Kershner <david.kershner@xxxxxxxxxx>
> ---
> drivers/staging/unisys/visorbus/visorchipset.c | 6 ------
> 1 file changed, 6 deletions(-)
>
> diff --git a/drivers/staging/unisys/visorbus/visorchipset.c b/drivers/staging/unisys/visorbus/visorchipset.c
> index 6d9bbae..bb0ca31 100644
> --- a/drivers/staging/unisys/visorbus/visorchipset.c
> +++ b/drivers/staging/unisys/visorbus/visorchipset.c
> @@ -150,8 +150,6 @@ static LIST_HEAD(putfile_request_list);
> struct putfile_active_buffer {
> /* a payload from a controlvm message, containing a file data buffer */
> struct parser_context *parser_ctx;
> - /* points within data area of parser_ctx to next byte of data */
> - u8 *pnext;
> /* # bytes left from <pnext> to the end of this data buffer */
Small thing but, now this comment mentions a pnext that isn't there.
Thanks,
Luis
> size_t bytes_remaining;
> };
> @@ -166,14 +164,10 @@ struct putfile_request {
>
> /* header from original TransmitFile request */
> struct controlvm_message_header controlvm_header;
> - u64 file_request_number; /* from original TransmitFile request */
>
> /* link to next struct putfile_request */
> struct list_head next_putfile_request;
>
> - /* most-recent sequence number supplied via a controlvm message */
> - u64 data_sequence_number;
> -
> /* head of putfile_buffer_entry list, which describes the data to be
> * supplied as putfile data;
> * - this list is added to when controlvm messages come in that supply
>
_______________________________________________
devel mailing list
devel@xxxxxxxxxxxxxxxxxxxxxx
http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel
