Date: Thu, 9 Dec 2010 14:58:58 +0100
From: Samuel Ortiz <sameo@linux.intel.com>
To: Stephen Warren <swarren@nvidia.com>
Cc: linux-kernel@vger.kernel.org, achew@nvidia.com, olofj@chromium.org
Subject: Re: [PATCH] mfd: tps6586x: Remove device ID check
Message-ID: <20101209135857.GK8884@sortiz-mobl>
References: <1291314802-18447-1-git-send-email-swarren@nvidia.com>
MIME-Version: 1.0
Content-Type: text/plain; charset=us-ascii
Content-Disposition: inline
In-Reply-To: <1291314802-18447-1-git-send-email-swarren@nvidia.com>
User-Agent: Mutt/1.5.20 (2009-06-14)
Sender: linux-kernel-owner@vger.kernel.org
Content-Length: 989
Lines: 25

Hi Stephen,

On Thu, Dec 02, 2010 at 11:33:22AM -0700, Stephen Warren wrote:
> ... and convert it to a dev_info print at probe time.
> 
> There are many variants of this chip with different values of VERSIONCRC.
> The set of values is large, and not useful to enumerate. All are SW
> compatible. The difference lies in default settings of the various power
> rails, and other similar differences. The driver, or clients of the
> driver, shouldn't be affected by this, since all rails should be
> programmed into the desired state in all cases for correct operation.
Fair enough, but then you can remove the TPS658621A_VERSIONCRC and
TPS658621C_VERSIONCRC defines.

Cheers,
Samuel.

-- 
Intel Open Source Technology Centre
http://oss.intel.com/
--
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/