Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1752363Ab3FQSto (ORCPT ); Mon, 17 Jun 2013 14:49:44 -0400 Received: from caramon.arm.linux.org.uk ([78.32.30.218]:43152 "EHLO caramon.arm.linux.org.uk" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751090Ab3FQStn (ORCPT ); Mon, 17 Jun 2013 14:49:43 -0400 Date: Mon, 17 Jun 2013 19:48:56 +0100 From: Russell King - ARM Linux To: Tomasz Figa Cc: linux-samsung-soc@vger.kernel.org, linux-arm-kernel@lists.infradead.org, linux-kernel@vger.kernel.org, linux-spi@vger.kernel.org, alsa-devel@alsa-project.org, Kukjin Kim , Vinod Koul , Dan Williams , Linus Walleij , Alessandro Rubini , Giancarlo Asnaghi , Mark Brown , Grant Likely , Sangbeom Kim , Liam Girdwood , Jaroslav Kysela , Takashi Iwai , Padmavathi Venna , Thomas Abraham , Arnd Bergmann , Olof Johansson , Heiko =?iso-8859-1?Q?St=FCbner?= , Sylwester Nawrocki , Alban Bedel Subject: Re: [RFC PATCH 01/11] dma: amba-pl08x: Use bitmap to pass variant specific quirks Message-ID: <20130617184856.GN2718@n2100.arm.linux.org.uk> References: <1371416058-22047-1-git-send-email-tomasz.figa@gmail.com> <1371416058-22047-2-git-send-email-tomasz.figa@gmail.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <1371416058-22047-2-git-send-email-tomasz.figa@gmail.com> User-Agent: Mutt/1.5.19 (2009-01-05) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 592 Lines: 12 On Sun, Jun 16, 2013 at 10:54:08PM +0200, Tomasz Figa wrote: > Instead of defining new bool field in vendor_data struct for each quirk, > it is more reasonable to use a single flags field and make each quirk > use single bits. Please explain why this is better over the existing system, and why it is not just churn for code modification's sake. -- 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/