Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S965087AbbFJORX (ORCPT ); Wed, 10 Jun 2015 10:17:23 -0400 Received: from mail-la0-f49.google.com ([209.85.215.49]:33978 "EHLO mail-la0-f49.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751308AbbFJONu (ORCPT ); Wed, 10 Jun 2015 10:13:50 -0400 MIME-Version: 1.0 In-Reply-To: References: Date: Wed, 10 Jun 2015 16:13:49 +0200 Message-ID: Subject: Re: [PATCH v2 00/10] kdbus: macros fixes From: David Herrmann To: Sergei Zviagintsev Cc: Greg Kroah-Hartman , Daniel Mack , David Herrmann , Djalal Harouni , linux-kernel Content-Type: text/plain; charset=UTF-8 Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 1787 Lines: 45 Hi On Tue, Jun 9, 2015 at 10:59 PM, Sergei Zviagintsev wrote: > Fix type conversion and style issues in item macros. > > First submission was in 2 separate emails: > https://lkml.kernel.org/g/1433414370-17557-1-git-send-email-sergei@s15v.net > https://lkml.kernel.org/g/1433414370-17557-2-git-send-email-sergei@s15v.net > > Changes in v2: > > - fixed similar type cast issue in KDBUS_ITEMS_END macro > - all arguments in iteration macros are enclosed in parentheses, > instead of using loop cursor without them > - summary phrases and commit messages updated > - documentation updates are excluded to their own patches > - added fixes for samples/kdbus > - added fixes for selftests/kdbus > > > Sergei Zviagintsev (10): > kdbus: fix operator precedence issues in item macros > kdbus: use parentheses uniformly in KDBUS_ITEMS_FOREACH macro > Documentation/kdbus: fix operator precedence issue in KDBUS_ITEM_NEXT > macro > Documentation/kdbus: use parentheses uniformly in KDBUS_ITEM_FOREACH > macro > selftests/kdbus: fix trivial style issues > selftests/kdbus: fix precedence issues in macros > selftests/kdbus: use parentheses in iteration macros uniformly > samples/kdbus: add whitespace > samples/kdbus: fix operator precedence issue in KDBUS_ITEM_NEXT macro > samples/kdbus: use parentheses uniformly in KDBUS_FOREACH macro Next time, please merge most of these changes to reduce the number of patches and review-overhead. But looks all good to me, much appreciated. Thanks David -- 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/