* [U-Boot] [PATCH 0/2 V3] exynos: Support a delay after deactivate for SPI @ 2013-05-30 5:19 Rajeshwari Shinde 2013-05-30 5:19 ` [U-Boot] [PATCH 1/2 V3] exynos: Export timer_get_us() to get microsecond timer Rajeshwari Shinde 2013-05-30 5:19 ` [U-Boot] [PATCH 2/2 V3] spi: exynos: Support a delay after deactivate Rajeshwari Shinde 0 siblings, 2 replies; 8+ messages in thread From: Rajeshwari Shinde @ 2013-05-30 5:19 UTC (permalink / raw) To: u-boot This patch set exports the function timer_get_us and adds a delay for devices that need some time to react after spi transation finishes This patch set is based on "EXYNOS: SPI: Support SPI_PREAMBLE mode" link: "http://comments.gmane.org/gmane.comp.boot-loaders.u-boot/162269" Changes in V2: - Removed #ifdefine for exported function. Changes in V3: - Rebased on "[PATCH 0/2 V5] spi: Enable SPI_PREAMBLE Mode" Rajeshwari Shinde (2): exynos: Export timer_get_us() to get microsecond timer spi: exynos: Support a delay after deactivate drivers/spi/exynos_spi.c | 19 +++++++++++++++++++ include/common.h | 6 ++++++ 2 files changed, 25 insertions(+), 0 deletions(-) -- 1.7.4.4 ^ permalink raw reply [flat|nested] 8+ messages in thread
* [U-Boot] [PATCH 1/2 V3] exynos: Export timer_get_us() to get microsecond timer 2013-05-30 5:19 [U-Boot] [PATCH 0/2 V3] exynos: Support a delay after deactivate for SPI Rajeshwari Shinde @ 2013-05-30 5:19 ` Rajeshwari Shinde 2013-05-30 5:19 ` [U-Boot] [PATCH 2/2 V3] spi: exynos: Support a delay after deactivate Rajeshwari Shinde 1 sibling, 0 replies; 8+ messages in thread From: Rajeshwari Shinde @ 2013-05-30 5:19 UTC (permalink / raw) To: u-boot This function, if implemented by the board, provides a microsecond timer. The granularity may be larger than 1us if hardware does not support this. Signed-off-by: Simon Glass <sjg@chromium.org> Signed-off-by: Rajeshwari Shinde <rajeshwari.s@samsung.com> --- Changes in V2: - Removed #ifdefine for exported function. Changes in V3: - None include/common.h | 6 ++++++ 1 files changed, 6 insertions(+), 0 deletions(-) diff --git a/include/common.h b/include/common.h index 8a1f3e4..c228d44 100644 --- a/include/common.h +++ b/include/common.h @@ -590,6 +590,12 @@ void ddr_enable_ecc(unsigned int dram_size); #endif #endif +/* + * Return the current value of a monotonically increasing microsecond timer. + * Granularity may be larger than 1us if hardware does not support this. + */ +ulong timer_get_us(void); + /* $(CPU)/cpu.c */ static inline int cpumask_next(int cpu, unsigned int mask) { -- 1.7.4.4 ^ permalink raw reply related [flat|nested] 8+ messages in thread
* [U-Boot] [PATCH 2/2 V3] spi: exynos: Support a delay after deactivate 2013-05-30 5:19 [U-Boot] [PATCH 0/2 V3] exynos: Support a delay after deactivate for SPI Rajeshwari Shinde 2013-05-30 5:19 ` [U-Boot] [PATCH 1/2 V3] exynos: Export timer_get_us() to get microsecond timer Rajeshwari Shinde @ 2013-05-30 5:19 ` Rajeshwari Shinde 2013-06-12 19:51 ` Jagan Teki 1 sibling, 1 reply; 8+ messages in thread From: Rajeshwari Shinde @ 2013-05-30 5:19 UTC (permalink / raw) To: u-boot For devices that need some time to react after a spi transaction finishes, add the ability to set a delay. Implement this as a delay on the first/next transaction to avoid any delay in the fairly common case where a SPI transaction is followed by other processing. Based on: "[U-Boot] [PATCH 0/2 V5] spi: Enable SPI_PREAMBLE Mode" Signed-off-by: Simon Glass <sjg@chromium.org> Signed-off-by: Rajeshwari Shinde <rajeshwari.s@samsung.com> --- Changes in V2: - None. Changes in V3: - Rebased on "[PATCH 0/2 V5] spi: Enable SPI_PREAMBLE Mode" drivers/spi/exynos_spi.c | 19 +++++++++++++++++++ 1 files changed, 19 insertions(+), 0 deletions(-) diff --git a/drivers/spi/exynos_spi.c b/drivers/spi/exynos_spi.c index 01378d0..03cf503 100644 --- a/drivers/spi/exynos_spi.c +++ b/drivers/spi/exynos_spi.c @@ -38,6 +38,7 @@ struct spi_bus { struct exynos_spi *regs; int inited; /* 1 if this bus is ready for use */ int node; + uint deactivate_delay_us; /* Delay to wait after deactivate */ }; /* A list of spi buses that we know about */ @@ -52,6 +53,8 @@ struct exynos_spi_slave { enum periph_id periph_id; /* Peripheral ID for this device */ unsigned int fifo_size; int skip_preamble; + struct spi_bus *bus; /* Pointer to our SPI bus info */ + ulong last_transaction_us; /* Time of last transaction end */ }; static struct spi_bus *spi_get_bus(unsigned dev_index) @@ -97,6 +100,7 @@ struct spi_slave *spi_setup_slave(unsigned int busnum, unsigned int cs, } bus = &spi_bus[busnum]; + spi_slave->bus = bus; spi_slave->regs = bus->regs; spi_slave->mode = mode; spi_slave->periph_id = bus->periph_id; @@ -107,6 +111,7 @@ struct spi_slave *spi_setup_slave(unsigned int busnum, unsigned int cs, spi_slave->fifo_size = 256; spi_slave->skip_preamble = 0; + spi_slave->last_transaction_us = timer_get_us(); spi_slave->freq = bus->frequency; if (max_hz) @@ -371,9 +376,21 @@ void spi_cs_activate(struct spi_slave *slave) { struct exynos_spi_slave *spi_slave = to_exynos_spi(slave); + /* If it's too soon to do another transaction, wait */ + if (spi_slave->bus->deactivate_delay_us && + spi_slave->last_transaction_us) { + ulong delay_us; /* The delay completed so far */ + delay_us = timer_get_us() - spi_slave->last_transaction_us; + if (delay_us < spi_slave->bus->deactivate_delay_us) + udelay(spi_slave->bus->deactivate_delay_us - delay_us); + } clrbits_le32(&spi_slave->regs->cs_reg, SPI_SLAVE_SIG_INACT); debug("Activate CS, bus %d\n", spi_slave->slave.bus); spi_slave->skip_preamble = spi_slave->mode & SPI_PREAMBLE; + + /* Remember time of this transaction so we can honour the bus delay */ + if (spi_slave->bus->deactivate_delay_us) + spi_slave->last_transaction_us = timer_get_us(); } /** @@ -423,6 +440,8 @@ static int spi_get_config(const void *blob, int node, struct spi_bus *bus) /* Use 500KHz as a suitable default */ bus->frequency = fdtdec_get_int(blob, node, "spi-max-frequency", 500000); + bus->deactivate_delay_us = fdtdec_get_int(blob, node, + "spi-deactivate-delay", 0); return 0; } -- 1.7.4.4 ^ permalink raw reply related [flat|nested] 8+ messages in thread
* [U-Boot] [PATCH 2/2 V3] spi: exynos: Support a delay after deactivate 2013-05-30 5:19 ` [U-Boot] [PATCH 2/2 V3] spi: exynos: Support a delay after deactivate Rajeshwari Shinde @ 2013-06-12 19:51 ` Jagan Teki 2013-06-12 22:06 ` Simon Glass 0 siblings, 1 reply; 8+ messages in thread From: Jagan Teki @ 2013-06-12 19:51 UTC (permalink / raw) To: u-boot Hi, On 30-05-2013 10:49, Rajeshwari Shinde wrote: > For devices that need some time to react after a spi transaction > finishes, add the ability to set a delay. > > Implement this as a delay on the first/next transaction to avoid > any delay in the fairly common case where a SPI transaction is > followed by other processing. > > Based on: > "[U-Boot] [PATCH 0/2 V5] spi: Enable SPI_PREAMBLE Mode" Please use the below format for referring previous commits ex: "<PREVIOUS_COMMIT_HEAD>" (sha1: PREVIOUS_COMMIT_ID) in this case "spi: exynos: Support SPI_PREAMBLE mode" (sha1: e4eaef8910df805d511b1cb9b2caafa7d2827fdc) > > Signed-off-by: Simon Glass <sjg@chromium.org> > Signed-off-by: Rajeshwari Shinde <rajeshwari.s@samsung.com> > --- > Changes in V2: > - None. > Changes in V3: > - Rebased on "[PATCH 0/2 V5] spi: Enable SPI_PREAMBLE Mode" > drivers/spi/exynos_spi.c | 19 +++++++++++++++++++ > 1 files changed, 19 insertions(+), 0 deletions(-) > > diff --git a/drivers/spi/exynos_spi.c b/drivers/spi/exynos_spi.c > index 01378d0..03cf503 100644 > --- a/drivers/spi/exynos_spi.c > +++ b/drivers/spi/exynos_spi.c > @@ -38,6 +38,7 @@ struct spi_bus { > struct exynos_spi *regs; > int inited; /* 1 if this bus is ready for use */ > int node; > + uint deactivate_delay_us; /* Delay to wait after deactivate */ > }; > > /* A list of spi buses that we know about */ > @@ -52,6 +53,8 @@ struct exynos_spi_slave { > enum periph_id periph_id; /* Peripheral ID for this device */ > unsigned int fifo_size; > int skip_preamble; > + struct spi_bus *bus; /* Pointer to our SPI bus info */ > + ulong last_transaction_us; /* Time of last transaction end */ > }; > > static struct spi_bus *spi_get_bus(unsigned dev_index) > @@ -97,6 +100,7 @@ struct spi_slave *spi_setup_slave(unsigned int busnum, unsigned int cs, > } > > bus = &spi_bus[busnum]; > + spi_slave->bus = bus; > spi_slave->regs = bus->regs; > spi_slave->mode = mode; > spi_slave->periph_id = bus->periph_id; > @@ -107,6 +111,7 @@ struct spi_slave *spi_setup_slave(unsigned int busnum, unsigned int cs, > spi_slave->fifo_size = 256; > > spi_slave->skip_preamble = 0; > + spi_slave->last_transaction_us = timer_get_us(); > > spi_slave->freq = bus->frequency; > if (max_hz) > @@ -371,9 +376,21 @@ void spi_cs_activate(struct spi_slave *slave) > { > struct exynos_spi_slave *spi_slave = to_exynos_spi(slave); > > + /* If it's too soon to do another transaction, wait */ > + if (spi_slave->bus->deactivate_delay_us && > + spi_slave->last_transaction_us) { > + ulong delay_us; /* The delay completed so far */ > + delay_us = timer_get_us() - spi_slave->last_transaction_us; > + if (delay_us < spi_slave->bus->deactivate_delay_us) > + udelay(spi_slave->bus->deactivate_delay_us - delay_us); > + } Add one space > clrbits_le32(&spi_slave->regs->cs_reg, SPI_SLAVE_SIG_INACT); > debug("Activate CS, bus %d\n", spi_slave->slave.bus); > spi_slave->skip_preamble = spi_slave->mode & SPI_PREAMBLE; > + > + /* Remember time of this transaction so we can honour the bus delay */ > + if (spi_slave->bus->deactivate_delay_us) > + spi_slave->last_transaction_us = timer_get_us(); > } > > /** > @@ -423,6 +440,8 @@ static int spi_get_config(const void *blob, int node, struct spi_bus *bus) > /* Use 500KHz as a suitable default */ > bus->frequency = fdtdec_get_int(blob, node, "spi-max-frequency", > 500000); > + bus->deactivate_delay_us = fdtdec_get_int(blob, node, > + "spi-deactivate-delay", 0); > > return 0; > } > Was this tested, usually what is the max deactivate delay? In this case your are passing through dts is it? Thanks, Jagan. ^ permalink raw reply [flat|nested] 8+ messages in thread
* [U-Boot] [PATCH 2/2 V3] spi: exynos: Support a delay after deactivate 2013-06-12 19:51 ` Jagan Teki @ 2013-06-12 22:06 ` Simon Glass 2013-06-13 3:17 ` Jagan Teki 0 siblings, 1 reply; 8+ messages in thread From: Simon Glass @ 2013-06-12 22:06 UTC (permalink / raw) To: u-boot Hi Jagan, On Wed, Jun 12, 2013 at 12:51 PM, Jagan Teki <jagannadh.teki@gmail.com>wrote: > Hi, > > > On 30-05-2013 10:49, Rajeshwari Shinde wrote: > >> For devices that need some time to react after a spi transaction >> finishes, add the ability to set a delay. >> >> Implement this as a delay on the first/next transaction to avoid >> any delay in the fairly common case where a SPI transaction is >> followed by other processing. >> >> Based on: >> "[U-Boot] [PATCH 0/2 V5] spi: Enable SPI_PREAMBLE Mode" >> > > Please use the below format for referring previous commits > ex: > "<PREVIOUS_COMMIT_HEAD>" > (sha1: PREVIOUS_COMMIT_ID) > > in this case > "spi: exynos: Support SPI_PREAMBLE mode" > (sha1: e4eaef8910df805d511b1cb9b2caaf**a7d2827fdc) I hope you don't mind me answering part of this... > > > >> Signed-off-by: Simon Glass <sjg@chromium.org> >> Signed-off-by: Rajeshwari Shinde <rajeshwari.s@samsung.com> >> --- >> Changes in V2: >> - None. >> Changes in V3: >> - Rebased on "[PATCH 0/2 V5] spi: Enable SPI_PREAMBLE Mode" >> drivers/spi/exynos_spi.c | 19 +++++++++++++++++++ >> 1 files changed, 19 insertions(+), 0 deletions(-) >> >> diff --git a/drivers/spi/exynos_spi.c b/drivers/spi/exynos_spi.c >> index 01378d0..03cf503 100644 >> --- a/drivers/spi/exynos_spi.c >> +++ b/drivers/spi/exynos_spi.c >> @@ -38,6 +38,7 @@ struct spi_bus { >> struct exynos_spi *regs; >> int inited; /* 1 if this bus is ready for use */ >> int node; >> + uint deactivate_delay_us; /* Delay to wait after deactivate >> */ >> }; >> >> /* A list of spi buses that we know about */ >> @@ -52,6 +53,8 @@ struct exynos_spi_slave { >> enum periph_id periph_id; /* Peripheral ID for this device >> */ >> unsigned int fifo_size; >> int skip_preamble; >> + struct spi_bus *bus; /* Pointer to our SPI bus info */ >> + ulong last_transaction_us; /* Time of last transaction end */ >> }; >> >> static struct spi_bus *spi_get_bus(unsigned dev_index) >> @@ -97,6 +100,7 @@ struct spi_slave *spi_setup_slave(unsigned int busnum, >> unsigned int cs, >> } >> >> bus = &spi_bus[busnum]; >> + spi_slave->bus = bus; >> spi_slave->regs = bus->regs; >> spi_slave->mode = mode; >> spi_slave->periph_id = bus->periph_id; >> @@ -107,6 +111,7 @@ struct spi_slave *spi_setup_slave(unsigned int >> busnum, unsigned int cs, >> spi_slave->fifo_size = 256; >> >> spi_slave->skip_preamble = 0; >> + spi_slave->last_transaction_us = timer_get_us(); >> >> spi_slave->freq = bus->frequency; >> if (max_hz) >> @@ -371,9 +376,21 @@ void spi_cs_activate(struct spi_slave *slave) >> { >> struct exynos_spi_slave *spi_slave = to_exynos_spi(slave); >> >> + /* If it's too soon to do another transaction, wait */ >> + if (spi_slave->bus->deactivate_**delay_us && >> + spi_slave->last_transaction_**us) { >> + ulong delay_us; /* The delay completed so far */ >> + delay_us = timer_get_us() - spi_slave->last_transaction_* >> *us; >> + if (delay_us < spi_slave->bus->deactivate_**delay_us) >> + udelay(spi_slave->bus->**deactivate_delay_us - >> delay_us); >> + } >> > Add one space > > > clrbits_le32(&spi_slave->regs-**>cs_reg, SPI_SLAVE_SIG_INACT); >> debug("Activate CS, bus %d\n", spi_slave->slave.bus); >> spi_slave->skip_preamble = spi_slave->mode & SPI_PREAMBLE; >> + >> + /* Remember time of this transaction so we can honour the bus >> delay */ >> + if (spi_slave->bus->deactivate_**delay_us) >> + spi_slave->last_transaction_us = timer_get_us(); >> } >> >> /** >> @@ -423,6 +440,8 @@ static int spi_get_config(const void *blob, int node, >> struct spi_bus *bus) >> /* Use 500KHz as a suitable default */ >> bus->frequency = fdtdec_get_int(blob, node, "spi-max-frequency", >> 500000); >> + bus->deactivate_delay_us = fdtdec_get_int(blob, node, >> + "spi-deactivate-delay", 0); >> >> return 0; >> } >> >> > Was this tested, usually what is the max deactivate delay? > This was tested with 50us. > In this case your are passing through dts is it? > Yes that's right. Regards, Simon > > Thanks, > Jagan. > > > ^ permalink raw reply [flat|nested] 8+ messages in thread
* [U-Boot] [PATCH 2/2 V3] spi: exynos: Support a delay after deactivate 2013-06-12 22:06 ` Simon Glass @ 2013-06-13 3:17 ` Jagan Teki 2013-08-08 13:42 ` Jagan Teki 0 siblings, 1 reply; 8+ messages in thread From: Jagan Teki @ 2013-06-13 3:17 UTC (permalink / raw) To: u-boot On Thu, Jun 13, 2013 at 3:36 AM, Simon Glass <sjg@chromium.org> wrote: > Hi Jagan, > > On Wed, Jun 12, 2013 at 12:51 PM, Jagan Teki <jagannadh.teki@gmail.com> > wrote: >> >> Hi, >> >> >> On 30-05-2013 10:49, Rajeshwari Shinde wrote: >>> >>> For devices that need some time to react after a spi transaction >>> finishes, add the ability to set a delay. >>> >>> Implement this as a delay on the first/next transaction to avoid >>> any delay in the fairly common case where a SPI transaction is >>> followed by other processing. >>> >>> Based on: >>> "[U-Boot] [PATCH 0/2 V5] spi: Enable SPI_PREAMBLE Mode" >> >> >> Please use the below format for referring previous commits >> ex: >> "<PREVIOUS_COMMIT_HEAD>" >> (sha1: PREVIOUS_COMMIT_ID) >> >> in this case >> "spi: exynos: Support SPI_PREAMBLE mode" >> (sha1: e4eaef8910df805d511b1cb9b2caafa7d2827fdc) > > > I hope you don't mind me answering part of this... > >> >> >> >>> >>> Signed-off-by: Simon Glass <sjg@chromium.org> >>> Signed-off-by: Rajeshwari Shinde <rajeshwari.s@samsung.com> >>> --- >>> Changes in V2: >>> - None. >>> Changes in V3: >>> - Rebased on "[PATCH 0/2 V5] spi: Enable SPI_PREAMBLE Mode" >>> drivers/spi/exynos_spi.c | 19 +++++++++++++++++++ >>> 1 files changed, 19 insertions(+), 0 deletions(-) >>> >>> diff --git a/drivers/spi/exynos_spi.c b/drivers/spi/exynos_spi.c >>> index 01378d0..03cf503 100644 >>> --- a/drivers/spi/exynos_spi.c >>> +++ b/drivers/spi/exynos_spi.c >>> @@ -38,6 +38,7 @@ struct spi_bus { >>> struct exynos_spi *regs; >>> int inited; /* 1 if this bus is ready for use */ >>> int node; >>> + uint deactivate_delay_us; /* Delay to wait after deactivate >>> */ >>> }; >>> >>> /* A list of spi buses that we know about */ >>> @@ -52,6 +53,8 @@ struct exynos_spi_slave { >>> enum periph_id periph_id; /* Peripheral ID for this device >>> */ >>> unsigned int fifo_size; >>> int skip_preamble; >>> + struct spi_bus *bus; /* Pointer to our SPI bus info */ >>> + ulong last_transaction_us; /* Time of last transaction end >>> */ >>> }; >>> >>> static struct spi_bus *spi_get_bus(unsigned dev_index) >>> @@ -97,6 +100,7 @@ struct spi_slave *spi_setup_slave(unsigned int busnum, >>> unsigned int cs, >>> } >>> >>> bus = &spi_bus[busnum]; >>> + spi_slave->bus = bus; >>> spi_slave->regs = bus->regs; >>> spi_slave->mode = mode; >>> spi_slave->periph_id = bus->periph_id; >>> @@ -107,6 +111,7 @@ struct spi_slave *spi_setup_slave(unsigned int >>> busnum, unsigned int cs, >>> spi_slave->fifo_size = 256; >>> >>> spi_slave->skip_preamble = 0; >>> + spi_slave->last_transaction_us = timer_get_us(); >>> >>> spi_slave->freq = bus->frequency; >>> if (max_hz) >>> @@ -371,9 +376,21 @@ void spi_cs_activate(struct spi_slave *slave) >>> { >>> struct exynos_spi_slave *spi_slave = to_exynos_spi(slave); >>> >>> + /* If it's too soon to do another transaction, wait */ >>> + if (spi_slave->bus->deactivate_delay_us && >>> + spi_slave->last_transaction_us) { >>> + ulong delay_us; /* The delay completed so far */ >>> + delay_us = timer_get_us() - >>> spi_slave->last_transaction_us; >>> + if (delay_us < spi_slave->bus->deactivate_delay_us) >>> + udelay(spi_slave->bus->deactivate_delay_us - >>> delay_us); >>> + } >> >> Add one space >> >> >>> clrbits_le32(&spi_slave->regs->cs_reg, SPI_SLAVE_SIG_INACT); >>> debug("Activate CS, bus %d\n", spi_slave->slave.bus); >>> spi_slave->skip_preamble = spi_slave->mode & SPI_PREAMBLE; >>> + >>> + /* Remember time of this transaction so we can honour the bus >>> delay */ >>> + if (spi_slave->bus->deactivate_delay_us) >>> + spi_slave->last_transaction_us = timer_get_us(); >>> } >>> >>> /** >>> @@ -423,6 +440,8 @@ static int spi_get_config(const void *blob, int node, >>> struct spi_bus *bus) >>> /* Use 500KHz as a suitable default */ >>> bus->frequency = fdtdec_get_int(blob, node, "spi-max-frequency", >>> 500000); >>> + bus->deactivate_delay_us = fdtdec_get_int(blob, node, >>> + "spi-deactivate-delay", 0); >>> >>> return 0; >>> } >>> >> >> Was this tested, usually what is the max deactivate delay? > > > This was tested with 50us. > >> >> In this case your are passing through dts is it? > > > Yes that's right. > > Regards, > Simon > >> >> >> Thanks, >> Jagan. >> >> > Can you please send the next version patches. -- Thanks, Jagan. ^ permalink raw reply [flat|nested] 8+ messages in thread
* [U-Boot] [PATCH 2/2 V3] spi: exynos: Support a delay after deactivate 2013-06-13 3:17 ` Jagan Teki @ 2013-08-08 13:42 ` Jagan Teki 2013-10-03 19:55 ` Jagan Teki 0 siblings, 1 reply; 8+ messages in thread From: Jagan Teki @ 2013-08-08 13:42 UTC (permalink / raw) To: u-boot HI Rajeshwari, Can you please send the next version patch w.r.t to current tree. and also fix the comments on last version. Planning to apply on tree, feel free to ask if you have any concerns. -- Thanks, Jagan. On Thu, Jun 13, 2013 at 8:47 AM, Jagan Teki <jagannadh.teki@gmail.com> wrote: > On Thu, Jun 13, 2013 at 3:36 AM, Simon Glass <sjg@chromium.org> wrote: >> Hi Jagan, >> >> On Wed, Jun 12, 2013 at 12:51 PM, Jagan Teki <jagannadh.teki@gmail.com> >> wrote: >>> >>> Hi, >>> >>> >>> On 30-05-2013 10:49, Rajeshwari Shinde wrote: >>>> >>>> For devices that need some time to react after a spi transaction >>>> finishes, add the ability to set a delay. >>>> >>>> Implement this as a delay on the first/next transaction to avoid >>>> any delay in the fairly common case where a SPI transaction is >>>> followed by other processing. >>>> >>>> Based on: >>>> "[U-Boot] [PATCH 0/2 V5] spi: Enable SPI_PREAMBLE Mode" >>> >>> >>> Please use the below format for referring previous commits >>> ex: >>> "<PREVIOUS_COMMIT_HEAD>" >>> (sha1: PREVIOUS_COMMIT_ID) >>> >>> in this case >>> "spi: exynos: Support SPI_PREAMBLE mode" >>> (sha1: e4eaef8910df805d511b1cb9b2caafa7d2827fdc) >> >> >> I hope you don't mind me answering part of this... >> >>> >>> >>> >>>> >>>> Signed-off-by: Simon Glass <sjg@chromium.org> >>>> Signed-off-by: Rajeshwari Shinde <rajeshwari.s@samsung.com> >>>> --- >>>> Changes in V2: >>>> - None. >>>> Changes in V3: >>>> - Rebased on "[PATCH 0/2 V5] spi: Enable SPI_PREAMBLE Mode" >>>> drivers/spi/exynos_spi.c | 19 +++++++++++++++++++ >>>> 1 files changed, 19 insertions(+), 0 deletions(-) >>>> >>>> diff --git a/drivers/spi/exynos_spi.c b/drivers/spi/exynos_spi.c >>>> index 01378d0..03cf503 100644 >>>> --- a/drivers/spi/exynos_spi.c >>>> +++ b/drivers/spi/exynos_spi.c >>>> @@ -38,6 +38,7 @@ struct spi_bus { >>>> struct exynos_spi *regs; >>>> int inited; /* 1 if this bus is ready for use */ >>>> int node; >>>> + uint deactivate_delay_us; /* Delay to wait after deactivate >>>> */ >>>> }; >>>> >>>> /* A list of spi buses that we know about */ >>>> @@ -52,6 +53,8 @@ struct exynos_spi_slave { >>>> enum periph_id periph_id; /* Peripheral ID for this device >>>> */ >>>> unsigned int fifo_size; >>>> int skip_preamble; >>>> + struct spi_bus *bus; /* Pointer to our SPI bus info */ >>>> + ulong last_transaction_us; /* Time of last transaction end >>>> */ >>>> }; >>>> >>>> static struct spi_bus *spi_get_bus(unsigned dev_index) >>>> @@ -97,6 +100,7 @@ struct spi_slave *spi_setup_slave(unsigned int busnum, >>>> unsigned int cs, >>>> } >>>> >>>> bus = &spi_bus[busnum]; >>>> + spi_slave->bus = bus; >>>> spi_slave->regs = bus->regs; >>>> spi_slave->mode = mode; >>>> spi_slave->periph_id = bus->periph_id; >>>> @@ -107,6 +111,7 @@ struct spi_slave *spi_setup_slave(unsigned int >>>> busnum, unsigned int cs, >>>> spi_slave->fifo_size = 256; >>>> >>>> spi_slave->skip_preamble = 0; >>>> + spi_slave->last_transaction_us = timer_get_us(); >>>> >>>> spi_slave->freq = bus->frequency; >>>> if (max_hz) >>>> @@ -371,9 +376,21 @@ void spi_cs_activate(struct spi_slave *slave) >>>> { >>>> struct exynos_spi_slave *spi_slave = to_exynos_spi(slave); >>>> >>>> + /* If it's too soon to do another transaction, wait */ >>>> + if (spi_slave->bus->deactivate_delay_us && >>>> + spi_slave->last_transaction_us) { >>>> + ulong delay_us; /* The delay completed so far */ >>>> + delay_us = timer_get_us() - >>>> spi_slave->last_transaction_us; >>>> + if (delay_us < spi_slave->bus->deactivate_delay_us) >>>> + udelay(spi_slave->bus->deactivate_delay_us - >>>> delay_us); >>>> + } >>> >>> Add one space >>> >>> >>>> clrbits_le32(&spi_slave->regs->cs_reg, SPI_SLAVE_SIG_INACT); >>>> debug("Activate CS, bus %d\n", spi_slave->slave.bus); >>>> spi_slave->skip_preamble = spi_slave->mode & SPI_PREAMBLE; >>>> + >>>> + /* Remember time of this transaction so we can honour the bus >>>> delay */ >>>> + if (spi_slave->bus->deactivate_delay_us) >>>> + spi_slave->last_transaction_us = timer_get_us(); >>>> } >>>> >>>> /** >>>> @@ -423,6 +440,8 @@ static int spi_get_config(const void *blob, int node, >>>> struct spi_bus *bus) >>>> /* Use 500KHz as a suitable default */ >>>> bus->frequency = fdtdec_get_int(blob, node, "spi-max-frequency", >>>> 500000); >>>> + bus->deactivate_delay_us = fdtdec_get_int(blob, node, >>>> + "spi-deactivate-delay", 0); >>>> >>>> return 0; >>>> } >>>> >>> >>> Was this tested, usually what is the max deactivate delay? >> >> >> This was tested with 50us. >> >>> >>> In this case your are passing through dts is it? >> >> >> Yes that's right. >> >> Regards, >> Simon >> >>> >>> >>> Thanks, >>> Jagan. >>> >>> >> > > Can you please send the next version patches. > > -- > Thanks, > Jagan. ^ permalink raw reply [flat|nested] 8+ messages in thread
* [U-Boot] [PATCH 2/2 V3] spi: exynos: Support a delay after deactivate 2013-08-08 13:42 ` Jagan Teki @ 2013-10-03 19:55 ` Jagan Teki 0 siblings, 0 replies; 8+ messages in thread From: Jagan Teki @ 2013-10-03 19:55 UTC (permalink / raw) To: u-boot Any update on this! ^ permalink raw reply [flat|nested] 8+ messages in thread
end of thread, other threads:[~2013-10-03 19:55 UTC | newest] Thread overview: 8+ messages (download: mbox.gz follow: Atom feed -- links below jump to the message on this page -- 2013-05-30 5:19 [U-Boot] [PATCH 0/2 V3] exynos: Support a delay after deactivate for SPI Rajeshwari Shinde 2013-05-30 5:19 ` [U-Boot] [PATCH 1/2 V3] exynos: Export timer_get_us() to get microsecond timer Rajeshwari Shinde 2013-05-30 5:19 ` [U-Boot] [PATCH 2/2 V3] spi: exynos: Support a delay after deactivate Rajeshwari Shinde 2013-06-12 19:51 ` Jagan Teki 2013-06-12 22:06 ` Simon Glass 2013-06-13 3:17 ` Jagan Teki 2013-08-08 13:42 ` Jagan Teki 2013-10-03 19:55 ` Jagan Teki
This is a public inbox, see mirroring instructions for how to clone and mirror all data and code used for this inbox