From mboxrd@z Thu Jan 1 00:00:00 1970 From: Florian Fainelli Subject: Re: [PATCH 00/12] spi/bcm63xx: various small cleanups and fixes Date: Tue, 12 Mar 2013 00:20:33 +0100 Message-ID: <1550045.lN4y9TcD08@bender> References: <1363043627-23524-1-git-send-email-jogo@openwrt.org> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Cc: spi-devel-general-5NWGOfrQmneRv+LV9MX5uipxlwaOVQ5f@public.gmane.org, Kevin Cernekee , Mark Brown , Maxime Bizon To: Jonas Gorski Return-path: In-Reply-To: <1363043627-23524-1-git-send-email-jogo-p3rKhJxN3npAfugRpC6u6w@public.gmane.org> List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: spi-devel-general-bounces-5NWGOfrQmneRv+LV9MX5uipxlwaOVQ5f@public.gmane.org List-Id: linux-spi.vger.kernel.org On Tuesday 12 March 2013 00:13:35 Jonas Gorski wrote: > This patch series cleans up spi-bcm63xx and removes mostly redundant > checks that are already done by the spi core system itself, and also > adds some clock handling improvements. All in all nothing serious, > just some corner cases. > > The only Patch I'm not 100% sure about is patch 7/12 ("check spi > bits_per_word in spi_setup"), because I'm not sure if setup is the > right place for that. > > For all we care the spi can claim what it wants as its bit_per_word > as long as all transfers have it set to 8, so arguably we maybe > shouldn't reject it then. For the entire serie: Acked-by: Florian Fainelli thanks Jonas! -- Florian ------------------------------------------------------------------------------ Symantec Endpoint Protection 12 positioned as A LEADER in The Forrester Wave(TM): Endpoint Security, Q1 2013 and "remains a good choice" in the endpoint security space. For insight on selecting the right partner to tackle endpoint security challenges, access the full report. http://p.sf.net/sfu/symantec-dev2dev