From mboxrd@z Thu Jan 1 00:00:00 1970 From: Tomasz Figa Subject: Re: [PATCH V4] spi: s3c64xx: Enable Word transfer Date: Wed, 30 Oct 2013 18:06:30 +0100 Message-ID: <6043727.WVyVGYjJys@amdc1227> References: <1382074356-28430-1-git-send-email-rajeshwari.s@samsung.com> <20131030170029.GK2493@sirena.org.uk> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Transfer-Encoding: 7Bit Return-path: Received: from mailout3.w1.samsung.com ([210.118.77.13]:13895 "EHLO mailout3.w1.samsung.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1750882Ab3J3RGd (ORCPT ); Wed, 30 Oct 2013 13:06:33 -0400 In-reply-to: <20131030170029.GK2493@sirena.org.uk> Sender: linux-samsung-soc-owner@vger.kernel.org List-Id: linux-samsung-soc@vger.kernel.org To: Mark Brown , Rajeshwari Birje Cc: Rajeshwari S Shinde , linux-samsung-soc , linux-spi@vger.kernel.org, Simon Glass , Alim Akhtar , Doug Anderson On Wednesday 30 of October 2013 10:00:29 Mark Brown wrote: > On Wed, Oct 30, 2013 at 11:52:54AM +0530, Rajeshwari Birje wrote: > > > The following patch already sets bits_per_word_mask for > > drivers/spi/spi-s3c64xx.c in s3c64xx_spi_probe, hence I had a doubt do > > I need to set the same again. > > OK, so how did this work before then? You're just adding new code but > the driver was previously claiming to support different bits per word > (and now I look at the code there is some handling for that in code). > Is this a bug fix? That's a good question. Rajeshwari, what SPI device did you test this patch with? Does it have a driver in mainline kernel? Best regards, Tomasz