From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from buildserver.ru.mvista.com (unknown [213.79.90.228]) by ozlabs.org (Postfix) with ESMTP id 55760B6EF0 for ; Fri, 27 Nov 2009 05:42:00 +1100 (EST) Date: Thu, 26 Nov 2009 21:41:57 +0300 From: Anton Vorontsov To: Grant Likely Subject: Re: spi_mpc8xxx.c: chip select polarity problem Message-ID: <20091126184157.GA16607@oksana.dev.rtsoft.ru> References: <200911161742.46663.to-fleischer@t-online.de> <200911211708.47253.to-fleischer@t-online.de> <200911252141.59549.to-fleischer@t-online.de> <20091126121204.GA3558@oksana.dev.rtsoft.ru> MIME-Version: 1.0 Content-Type: text/plain; charset=utf-8 In-Reply-To: Cc: Torsten Fleischer , spi-devel-general@lists.sourceforge.net, linuxppc-dev@lists.ozlabs.org Reply-To: avorontsov@ru.mvista.com List-Id: Linux on PowerPC Developers Mail List List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , On Thu, Nov 26, 2009 at 11:16:34AM -0700, Grant Likely wrote: [...] > The spi-cs-high property is defined in > Documentation/powerpc/dts-bindings/spi-bus.txt, but it definitely was > a mistake Yup. > Currently the spi-cs-high property is parsed in the > of_register_spi_devices() function, but the CS polarity needs to be > known before registering devices. It needs to be factored out into > another utility function callable by spi bus drivers so that it can > get polarity data at probe time. Untill we have this, Torsten's patch is a real improvement, and works for non-broken hw/fw. So I think it should be applied. Thanks, -- Anton Vorontsov email: cbouatmailru@gmail.com irc://irc.freenode.net/bd2