Re: [PATCH] monitor: Fix indentation for AVRCP PASS THROUGH commands

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Hi Vikram,

On Wed, Oct 15, 2014 at 4:41 PM, Vikrampal Yadav <vikram.pal@xxxxxxxxxxx> wrote:
> Intendation for AVRCP PASS THROUGH commands' decoding fixed.
>
> btmon snippets:
>
>       Channel: 66 len 14 [PSM 23 mode 0] {chan 3}
>       AVCTP Control: Command: type 0x00 label 0 PID 0x110e
>         AV/C: Status: address 0x48 opcode 0x7c
>           Subunit: Panel
>           Opcode: Passthrough
>           Operation: 0x44 (PLAY Pressed)
>           Lenght: 0x00
> ---
>  monitor/avctp.c | 4 ++--
>  1 file changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/monitor/avctp.c b/monitor/avctp.c
> index a4e34c5..4abd18f 100644
> --- a/monitor/avctp.c
> +++ b/monitor/avctp.c
> @@ -686,13 +686,13 @@ static bool avrcp_passthrough_packet(struct avctp_frame *avctp_frame,
>         if (!l2cap_frame_get_u8(frame, &op))
>                 return false;
>
> -       print_field("%*cOperation: 0x%02x (%s %s)", (indent - 2), ' ', op,
> +       print_field("%*cOperation: 0x%02x (%s %s)", (indent - 8), ' ', op,
>                                 op2str(op), op & 0x80 ? "Released" : "Pressed");
>
>         if (!l2cap_frame_get_u8(frame, &len))
>                 return false;
>
> -       print_field("%*cLength: 0x%02x", (indent - 2), ' ', len);
> +       print_field("%*cLength: 0x%02x", (indent - 8), ' ', len);
>
>         packet_hexdump(frame->data, frame->size);
>         return true;
> --
> 1.9.1
>

Applied, please make sure the patch format is 50/72 according to the
HACKING document.

-- 
Luiz Augusto von Dentz
--
To unsubscribe from this list: send the line "unsubscribe linux-bluetooth" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html




[Index of Archives]     [Bluez Devel]     [Linux Wireless Networking]     [Linux Wireless Personal Area Networking]     [Linux ATH6KL]     [Linux USB Devel]     [Linux Media Drivers]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [Big List of Linux Books]

  Powered by Linux