Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1752554Ab1FWEhe (ORCPT ); Thu, 23 Jun 2011 00:37:34 -0400 Received: from mail-pw0-f46.google.com ([209.85.160.46]:34919 "EHLO mail-pw0-f46.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751593Ab1FWEhd (ORCPT ); Thu, 23 Jun 2011 00:37:33 -0400 DomainKey-Signature: a=rsa-sha1; c=nofws; d=gmail.com; s=gamma; h=message-id:date:from:user-agent:mime-version:to:cc:subject :references:in-reply-to:content-type:content-transfer-encoding; b=A+t+scoRWak70YYpUSnyc66hzG2VIP5Smpvx+u/h0wXT8VlHaF7pO5wQ7X/9jZE2/Q MLDeNUgG347f4DFcYtG2baMt5BV6fIxJ2F5IaPrfRn0OofMYOy9qZ3DLzyZd+LIUa6SE S8ecJD2DbtH45mRtRMODXcdS6PLW3rpTsNkww= Message-ID: <4E02C30A.5040106@gmail.com> Date: Wed, 22 Jun 2011 21:37:30 -0700 From: Dirk Brandewie User-Agent: Mozilla/5.0 (X11; U; Linux x86_64; en-US; rv:1.9.2.17) Gecko/20110428 Fedora/3.1.10-1.fc13 Thunderbird/3.1.10 MIME-Version: 1.0 To: "glikely@secretlab.ca" CC: linux-kernel@vger.kernel.org, spi-devel-general@lists.sourceforge.net Subject: Re: [PATCH 01/11] spi-dw: expose platform data stucture. References: <1308794413-11069-1-git-send-email-dirk.brandewie@gmail.com> <1308794413-11069-2-git-send-email-dirk.brandewie@gmail.com> <4E02BA68.2010108@gmail.com> <127d80e7-0bb6-4961-891e-2ab791ac6368@email.android.com> In-Reply-To: <127d80e7-0bb6-4961-891e-2ab791ac6368@email.android.com> Content-Type: text/plain; charset=UTF-8; format=flowed Content-Transfer-Encoding: 7bit Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 1920 Lines: 52 On 06/22/2011 09:03 PM, glikely@secretlab.ca wrote: > > > Dirk Brandewie wrote: > >> On 06/22/2011 08:47 PM, Grant Likely wrote: >>> On Wed, Jun 22, 2011 at 8:00 PM, wrote: >>>> From: Dirk Brandewie >>>> >>>> Expose the platform data structure for use by client drivers. ATM >>>> there are not any in-tree drivers using the driver (that I can >>>> find). This patch exposes the platform data needed for client >> drivers. >>> >>> ? Why would client drivers want to muck with this configuration? I >>> can understand the dw_spi driver being able to have per-spi_device >>> configuration, but spi_drivers absolutely should not have visibility >>> into bus-specific details. Am I misunderstanding something. >>> >> >> Most of these config options don't need to be client configurable IMHO >> but they >> are being used ATM by drivers that aren't upstream and the current >> controller >> driver uses them. This patch is to give a smooth transition >> (bisectable) to my >> change that reworks the core message and transfer handling code. >> >> This allows me to provide patches to the developers of the out of tree >> drivers >> that should be coming in RSN and exposes the interface they are using >> now. > > My question still stands. Are you expecting spi_driver code to manipulate this data? > > The current drivers behaviour is driven by this data provided by the client. This makes the current client drivers work since some have not picked picked up your change moving dw_spi.h out of include/linux/spi (right answer IMHO) and provides the interface they are using now. --Dirk -- 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/