From mboxrd@z Thu Jan  1 00:00:00 1970
Return-Path: <cyrus@holtmann.org>
Date: Fri, 2 Mar 2012 15:29:08 +0200
From: Andrei Emeltchenko <andrei.emeltchenko.news@gmail.com>
To: johan.hedberg@gmail.com
Cc: linux-bluetooth@vger.kernel.org
Subject: Re: [PATCH 4/5] Bluetooth: mgmt: Add defines for command sizes
Message-ID: <20120302132906.GA6674@aemeltch-MOBL1>
References: <1330634012-4502-1-git-send-email-johan.hedberg@gmail.com>
 <1330634012-4502-4-git-send-email-johan.hedberg@gmail.com>
MIME-Version: 1.0
Content-Type: text/plain; charset=us-ascii
In-Reply-To: <1330634012-4502-4-git-send-email-johan.hedberg@gmail.com>
Sender: linux-bluetooth-owner@vger.kernel.org
List-ID: <linux-bluetooth.vger.kernel.org>

Hi Johan,

On Thu, Mar 01, 2012 at 12:33:31PM -0800, johan.hedberg@gmail.com wrote:
> From: Johan Hedberg <johan.hedberg@intel.com>
> 
> These defines are shorter than "sizeof(struct mgmt_cp_foo_bar...)" and

Maybe matter of taste but I would leave sizeof.

Best regards 
Andrei Emeltchenko