Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S966843Ab3HHWgZ (ORCPT ); Thu, 8 Aug 2013 18:36:25 -0400 Received: from ns.pmeerw.net ([87.118.82.44]:43302 "EHLO pmeerw.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S966767Ab3HHWgY (ORCPT ); Thu, 8 Aug 2013 18:36:24 -0400 Date: Fri, 9 Aug 2013 00:36:22 +0200 (CEST) From: Peter Meerwald X-X-Sender: pmeerw@pmeerw.net To: Bryan Wu cc: Ricardo Ribalda Delgado , Richard Purdie , Linux LED Subsystem , LKML Subject: Re: [PATCH v2 1/3] leds-pca9633: Add support for PCA9634 In-Reply-To: Message-ID: References: <1375980335-19943-1-git-send-email-ricardo.ribalda@gmail.com> <1375980335-19943-2-git-send-email-ricardo.ribalda@gmail.com> User-Agent: Alpine 2.01 (DEB 1266 2009-07-14) MIME-Version: 1.0 Content-Type: TEXT/PLAIN; charset=US-ASCII Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 1047 Lines: 33 Hello, > > Add support for PCA9634 chip, which belongs to the same family as the > > 9633 but with support for 8 outputs instead of 4. > Basically I like this method to add a new chip supporting. Please find > my comments below. me too :) > What about just rename the whole file to leds-pca963x.c. And rename > some pca9633 to pca963x in the driver. there are other, similar I2C LED driver chips which might be handled with the current pca9633 driver, e.g. the pca9685 (which is supported under pwm/ by the way) people have argued that the numbering scheme of chips is hard to predict; hence, the driver name should be determined by the first device supported to avoid subsequent renaming -- but I have no strong feelings about this regards, p. -- Peter Meerwald +43-664-2444418 (mobile) -- 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/