From mboxrd@z Thu Jan 1 00:00:00 1970 From: padma venkat Subject: Re: [PATCH 2/2] SPI: SAMSUNG: Bug fix for SPI with different FIFO level Date: Mon, 4 Jul 2011 15:34:19 +0530 Message-ID: References: <1309437536-9315-1-git-send-email-padma.v@samsung.com> <1309437536-9315-2-git-send-email-padma.v@samsung.com> <20110704065501.GP15152@ponder.secretlab.ca> <005501cc3a30$6c916190$45b424b0$%kim@samsung.com> Mime-Version: 1.0 Content-Type: text/plain; charset=ISO-8859-1 Content-Transfer-Encoding: QUOTED-PRINTABLE Cc: Grant Likely , Jassi Brar , Tony Nadackal , Padmavathi Venna , sbkim73@samsung.com, spi-devel-general@lists.sourceforge.net, linux-samsung-soc@vger.kernel.org, linux-arm-kernel@lists.infradead.org, tony.kn@samsung.com, naushad@samsung.com To: Kukjin Kim Return-path: In-Reply-To: <005501cc3a30$6c916190$45b424b0$%kim@samsung.com> Sender: linux-samsung-soc-owner@vger.kernel.org List-Id: linux-spi.vger.kernel.org Hi Kukjin, On Mon, Jul 4, 2011 at 3:24 PM, Kukjin Kim wrot= e: > Grant Likely wrote: >> >> On Mon, Jul 4, 2011 at 12:55 AM, Grant Likely > wrote: >> > On Fri, Jul 01, 2011 at 11:43:08AM +0530, Jassi Brar wrote: >> >> On Fri, Jul 1, 2011 at 11:29 AM, padma venkat > wrote: >> >> > Hi Jassi, >> >> > >> >> > On Fri, Jul 1, 2011 at 11:22 AM, Jassi Brar > wrote: >> >> >> On Fri, Jul 1, 2011 at 11:16 AM, padma venkat >> wrote: >> >> >>> Hi Tony, >> >> >>> >> >> >>> On Thu, Jun 30, 2011 at 4:30 PM, Tony Nadackal >> wrote: >> >> >>>> Hi Padma, >> >> >>>> With regards to your patch, even though one can check the tx= done >> status >> >> >>>> using the TX_DONE bit, the present macro itself would work > perfectly >> fine if >> >> >>>> the 'fifo_lvl_mask' is set properly. >> >> >>>> For example in 6450 channel 1, the fifo_lvl_mask should be 0= x1ff > (for >> 9bits, >> >> >>>> 15:23), while even in your patch, it is wrongly set as 0x7f(= only > 7bits). >> >> >>>> >> >> >>>> Thus, if this fifo_lvl_mask was defined correctly, the exist= ing > macro >> would >> >> >>>> itself have worked. >> >> >>> Thanks for your comment. >> >> >>> I considered changing to the fifo_lvl_mask to 1ff as you ment= ioned. >> >> >>> But I =A0think that the fifo_lvl_mask reflects the actual FIF= O > capacity >> >> >>> in the SPI driver. >> >> >>> For the failing channels the FIFO trigger level is 64 bytes a= nd so > i >> >> >>> retained that value. >> >> >>> In the driver it polls till the FIFO capacity level otherwise= it > goes >> >> >>> for DMA.So if we keep >> >> >>> the FIFO level as 1ff when the actual capacity is 7f then it = fails. >> >> >>> >> >> >>> Jassi what do you think about this? >> >> >>> >> >> >> >> >> >> 'fifo_lvl_mask' is h/w specific and can't be set for convenien= ce. >> >> >> >> >> >> I don't have access to post-s3c64xx datasheets. >> >> >> Please check and reply if TX_DONE bit is at same offset for al= l >> >> >> channels of an SoC, because >> >> >> I suspect it's otherwise. >> >> >> >> >> > Yes. The TX_DONE bit is at the same offset for all the channels= of an > SoC. >> >> > in S5P64X0,S5PV210 and S5PV310 it is at offset 25. >> >> > >> >> >> >> Then, Patches-1,2 >> >> >> >> Acked-by: Jassi Brar >> > >> > Are these bug fixes that should be in v3.0, or do I queue them up = for > v3.1? >> >> Regardless, this one touches a lot of arch/arm files, so I'd rather >> see both patches go through the samsung tree: >> >> Acked-by: Grant Likely >> > > Thanks Grant, Jassi and all, > > I will apply these 1 and 2 in my -fix tree for 3.0 with your acks. > > Thanks. > > Best regards, > Kgene. > -- > Kukjin Kim , Senior Engineer, > SW Solution Development Team, Samsung Electronics Co., Ltd. > > These patches are based on new SPI code(on which I tried to make common dev-spi for all SoCs). SO I will remake these two patches based on old code and will send the same. Thanks&Regards Padma