From mboxrd@z Thu Jan 1 00:00:00 1970 From: Kumar Gala Subject: Re: [PATCH] [SPI][POWERPC] spi_mpc83xx: in "QE mode" spiclk is sysclk/2 Date: Thu, 2 Aug 2007 10:19:17 -0500 Message-ID: <59836CD3-C46E-4B51-8FFE-5F40107CAA85@kernel.crashing.org> References: <20070802142641.GA6309@localhost.localdomain> Mime-Version: 1.0 (Apple Message framework v752.2) Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Cc: spi-devel-general-5NWGOfrQmneRv+LV9MX5uipxlwaOVQ5f@public.gmane.org, linuxppc-dev-mnsaURCQ41sdnm+yROfE0A@public.gmane.org To: Anton Vorontsov Return-path: In-Reply-To: <20070802142641.GA6309-bi+AKbBUZKY6gyzm1THtWbp2dZbC/Bob@public.gmane.org> List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Sender: spi-devel-general-bounces-5NWGOfrQmneRv+LV9MX5uipxlwaOVQ5f@public.gmane.org Errors-To: spi-devel-general-bounces-5NWGOfrQmneRv+LV9MX5uipxlwaOVQ5f@public.gmane.org List-Id: linux-spi.vger.kernel.org On Aug 2, 2007, at 9:26 AM, Anton Vorontsov wrote: > For MPC8349E input to SPIBRG is SYSCLK, but it's SYSCLK/2 for > MPC8323E (running in so-called "QE mode"). QE mode or the SPI in QE running in CPU mode? - k > > This fixes clocking issues I've noticed recently. > > Probably someday mpc83xx_spi->sysclk should be renamed to > mpc83xx_spi->spiclk to be less confusing. > > Signed-off-by: Anton Vorontsov > --- > drivers/spi/spi_mpc83xx.c | 6 +++++- > 1 files changed, 5 insertions(+), 1 deletions(-) > > diff --git a/drivers/spi/spi_mpc83xx.c b/drivers/spi/spi_mpc83xx.c > index 0c16a2b..55230f7 100644 > --- a/drivers/spi/spi_mpc83xx.c > +++ b/drivers/spi/spi_mpc83xx.c > @@ -429,13 +429,17 @@ static int __init mpc83xx_spi_probe(struct > platform_device *dev) > mpc83xx_spi->bitbang.chipselect = mpc83xx_spi_chipselect; > mpc83xx_spi->bitbang.setup_transfer = mpc83xx_spi_setup_transfer; > mpc83xx_spi->bitbang.txrx_bufs = mpc83xx_spi_bufs; > - mpc83xx_spi->sysclk = pdata->sysclk; > mpc83xx_spi->activate_cs = pdata->activate_cs; > mpc83xx_spi->deactivate_cs = pdata->deactivate_cs; > mpc83xx_spi->qe_mode = pdata->qe_mode; > mpc83xx_spi->get_rx = mpc83xx_spi_rx_buf_u8; > mpc83xx_spi->get_tx = mpc83xx_spi_tx_buf_u8; > > + if (mpc83xx_spi->qe_mode) > + mpc83xx_spi->sysclk = pdata->sysclk / 2; > + else > + mpc83xx_spi->sysclk = pdata->sysclk; > + > mpc83xx_spi->rx_shift = 0; > mpc83xx_spi->tx_shift = 0; > if (mpc83xx_spi->qe_mode) { > -- > 1.5.0.6 > _______________________________________________ > Linuxppc-dev mailing list > Linuxppc-dev-mnsaURCQ41sdnm+yROfE0A@public.gmane.org > https://ozlabs.org/mailman/listinfo/linuxppc-dev ------------------------------------------------------------------------- This SF.net email is sponsored by: Splunk Inc. Still grepping through log files to find problems? Stop. Now Search log events and configuration files using AJAX and a browser. Download your FREE copy of Splunk now >> http://get.splunk.com/