2021-03-29 05:57:23

by Stephen Rothwell

[permalink] [raw]
Subject: linux-next: build failure after merge of the staging tree

Hi all,

After merging the staging tree, today's linux-next build (x86_64
allmodconfig) failed like this:

drivers/iio/adc/ti-ads131e08.c: In function 'ads131e08_read_reg':
drivers/iio/adc/ti-ads131e08.c:180:5: error: 'struct spi_transfer' has no member named 'delay_usecs'
180 | .delay_usecs = st->sdecode_delay_us,
| ^~~~~~~~~~~
drivers/iio/adc/ti-ads131e08.c: In function 'ads131e08_write_reg':
drivers/iio/adc/ti-ads131e08.c:206:5: error: 'struct spi_transfer' has no member named 'delay_usecs'
206 | .delay_usecs = st->sdecode_delay_us,
| ^~~~~~~~~~~

Caused by commit

d935eddd2799 ("iio: adc: Add driver for Texas Instruments ADS131E0x ADC family")

interacting with commit

3ab1cce55337 ("spi: core: remove 'delay_usecs' field from spi_transfer")

from the spi tree.

I have applied the following merge fix patch.

From: Stephen Rothwell <[email protected]>
Date: Mon, 29 Mar 2021 16:51:22 +1100
Subject: [PATCH] iio: adc: merge fix for "spi: core: remove 'delay_usecs'
field from spi_transfer"

Signed-off-by: Stephen Rothwell <[email protected]>
---
drivers/iio/adc/ti-ads131e08.c | 10 ++++++++--
1 file changed, 8 insertions(+), 2 deletions(-)

diff --git a/drivers/iio/adc/ti-ads131e08.c b/drivers/iio/adc/ti-ads131e08.c
index 0060d5f0abb0..764dab087b41 100644
--- a/drivers/iio/adc/ti-ads131e08.c
+++ b/drivers/iio/adc/ti-ads131e08.c
@@ -177,7 +177,10 @@ static int ads131e08_read_reg(struct ads131e08_state *st, u8 reg)
{
.tx_buf = &st->tx_buf,
.len = 2,
- .delay_usecs = st->sdecode_delay_us,
+ .delay = {
+ .value = st->sdecode_delay_us,
+ .unit = SPI_DELAY_UNIT_USECS,
+ },
}, {
.rx_buf = &st->rx_buf,
.len = 1,
@@ -203,7 +206,10 @@ static int ads131e08_write_reg(struct ads131e08_state *st, u8 reg, u8 value)
{
.tx_buf = &st->tx_buf,
.len = 3,
- .delay_usecs = st->sdecode_delay_us,
+ .delay = {
+ .value = st->sdecode_delay_us,
+ .unit = SPI_DELAY_UNIT_USECS,
+ },
}
};

--
2.30.0

--
Cheers,
Stephen Rothwell


Attachments:
(No filename) (499.00 B)
OpenPGP digital signature

2021-03-29 06:17:45

by Greg KH

[permalink] [raw]
Subject: Re: linux-next: build failure after merge of the staging tree

On Mon, Mar 29, 2021 at 04:55:25PM +1100, Stephen Rothwell wrote:
> Hi all,
>
> After merging the staging tree, today's linux-next build (x86_64
> allmodconfig) failed like this:
>
> drivers/iio/adc/ti-ads131e08.c: In function 'ads131e08_read_reg':
> drivers/iio/adc/ti-ads131e08.c:180:5: error: 'struct spi_transfer' has no member named 'delay_usecs'
> 180 | .delay_usecs = st->sdecode_delay_us,
> | ^~~~~~~~~~~
> drivers/iio/adc/ti-ads131e08.c: In function 'ads131e08_write_reg':
> drivers/iio/adc/ti-ads131e08.c:206:5: error: 'struct spi_transfer' has no member named 'delay_usecs'
> 206 | .delay_usecs = st->sdecode_delay_us,
> | ^~~~~~~~~~~
>
> Caused by commit
>
> d935eddd2799 ("iio: adc: Add driver for Texas Instruments ADS131E0x ADC family")
>
> interacting with commit
>
> 3ab1cce55337 ("spi: core: remove 'delay_usecs' field from spi_transfer")
>
> from the spi tree.
>
> I have applied the following merge fix patch.
>
> From: Stephen Rothwell <[email protected]>
> Date: Mon, 29 Mar 2021 16:51:22 +1100
> Subject: [PATCH] iio: adc: merge fix for "spi: core: remove 'delay_usecs'
> field from spi_transfer"
>
> Signed-off-by: Stephen Rothwell <[email protected]>
> ---
> drivers/iio/adc/ti-ads131e08.c | 10 ++++++++--
> 1 file changed, 8 insertions(+), 2 deletions(-)
>
> diff --git a/drivers/iio/adc/ti-ads131e08.c b/drivers/iio/adc/ti-ads131e08.c
> index 0060d5f0abb0..764dab087b41 100644
> --- a/drivers/iio/adc/ti-ads131e08.c
> +++ b/drivers/iio/adc/ti-ads131e08.c
> @@ -177,7 +177,10 @@ static int ads131e08_read_reg(struct ads131e08_state *st, u8 reg)
> {
> .tx_buf = &st->tx_buf,
> .len = 2,
> - .delay_usecs = st->sdecode_delay_us,
> + .delay = {
> + .value = st->sdecode_delay_us,
> + .unit = SPI_DELAY_UNIT_USECS,
> + },
> }, {
> .rx_buf = &st->rx_buf,
> .len = 1,
> @@ -203,7 +206,10 @@ static int ads131e08_write_reg(struct ads131e08_state *st, u8 reg, u8 value)
> {
> .tx_buf = &st->tx_buf,
> .len = 3,
> - .delay_usecs = st->sdecode_delay_us,
> + .delay = {
> + .value = st->sdecode_delay_us,
> + .unit = SPI_DELAY_UNIT_USECS,
> + },
> }
> };
>
> --
> 2.30.0
>

Thanks for the fix, looks correct to me.

greg k-h

2021-03-29 07:26:59

by Alexandru Ardelean

[permalink] [raw]
Subject: Re: linux-next: build failure after merge of the staging tree

On Mon, 29 Mar 2021 at 09:15, Greg KH <[email protected]> wrote:
>
> On Mon, Mar 29, 2021 at 04:55:25PM +1100, Stephen Rothwell wrote:
> > Hi all,
> >
> > After merging the staging tree, today's linux-next build (x86_64
> > allmodconfig) failed like this:
> >
> > drivers/iio/adc/ti-ads131e08.c: In function 'ads131e08_read_reg':
> > drivers/iio/adc/ti-ads131e08.c:180:5: error: 'struct spi_transfer' has no member named 'delay_usecs'
> > 180 | .delay_usecs = st->sdecode_delay_us,
> > | ^~~~~~~~~~~
> > drivers/iio/adc/ti-ads131e08.c: In function 'ads131e08_write_reg':
> > drivers/iio/adc/ti-ads131e08.c:206:5: error: 'struct spi_transfer' has no member named 'delay_usecs'
> > 206 | .delay_usecs = st->sdecode_delay_us,
> > | ^~~~~~~~~~~
> >
> > Caused by commit
> >
> > d935eddd2799 ("iio: adc: Add driver for Texas Instruments ADS131E0x ADC family")
> >
> > interacting with commit
> >
> > 3ab1cce55337 ("spi: core: remove 'delay_usecs' field from spi_transfer")
> >
> > from the spi tree.
> >
> > I have applied the following merge fix patch.
> >
> > From: Stephen Rothwell <[email protected]>
> > Date: Mon, 29 Mar 2021 16:51:22 +1100
> > Subject: [PATCH] iio: adc: merge fix for "spi: core: remove 'delay_usecs'
> > field from spi_transfer"
> >

Reviewed-by: Alexandru Ardelean <[email protected]>

> > Signed-off-by: Stephen Rothwell <[email protected]>
> > ---
> > drivers/iio/adc/ti-ads131e08.c | 10 ++++++++--
> > 1 file changed, 8 insertions(+), 2 deletions(-)
> >
> > diff --git a/drivers/iio/adc/ti-ads131e08.c b/drivers/iio/adc/ti-ads131e08.c
> > index 0060d5f0abb0..764dab087b41 100644
> > --- a/drivers/iio/adc/ti-ads131e08.c
> > +++ b/drivers/iio/adc/ti-ads131e08.c
> > @@ -177,7 +177,10 @@ static int ads131e08_read_reg(struct ads131e08_state *st, u8 reg)
> > {
> > .tx_buf = &st->tx_buf,
> > .len = 2,
> > - .delay_usecs = st->sdecode_delay_us,
> > + .delay = {
> > + .value = st->sdecode_delay_us,
> > + .unit = SPI_DELAY_UNIT_USECS,
> > + },
> > }, {
> > .rx_buf = &st->rx_buf,
> > .len = 1,
> > @@ -203,7 +206,10 @@ static int ads131e08_write_reg(struct ads131e08_state *st, u8 reg, u8 value)
> > {
> > .tx_buf = &st->tx_buf,
> > .len = 3,
> > - .delay_usecs = st->sdecode_delay_us,
> > + .delay = {
> > + .value = st->sdecode_delay_us,
> > + .unit = SPI_DELAY_UNIT_USECS,
> > + },
> > }
> > };
> >
> > --
> > 2.30.0
> >
>
> Thanks for the fix, looks correct to me.
>
> greg k-h

2021-04-26 01:31:21

by Stephen Rothwell

[permalink] [raw]
Subject: Re: linux-next: build failure after merge of the staging tree

Hi all,

On Mon, 29 Mar 2021 16:55:25 +1100 Stephen Rothwell <[email protected]> wrote:
>
> After merging the staging tree, today's linux-next build (x86_64
> allmodconfig) failed like this:
>
> drivers/iio/adc/ti-ads131e08.c: In function 'ads131e08_read_reg':
> drivers/iio/adc/ti-ads131e08.c:180:5: error: 'struct spi_transfer' has no member named 'delay_usecs'
> 180 | .delay_usecs = st->sdecode_delay_us,
> | ^~~~~~~~~~~
> drivers/iio/adc/ti-ads131e08.c: In function 'ads131e08_write_reg':
> drivers/iio/adc/ti-ads131e08.c:206:5: error: 'struct spi_transfer' has no member named 'delay_usecs'
> 206 | .delay_usecs = st->sdecode_delay_us,
> | ^~~~~~~~~~~
>
> Caused by commit
>
> d935eddd2799 ("iio: adc: Add driver for Texas Instruments ADS131E0x ADC family")
>
> interacting with commit
>
> 3ab1cce55337 ("spi: core: remove 'delay_usecs' field from spi_transfer")
>
> from the spi tree.
>
> I have applied the following merge fix patch.
>
> From: Stephen Rothwell <[email protected]>
> Date: Mon, 29 Mar 2021 16:51:22 +1100
> Subject: [PATCH] iio: adc: merge fix for "spi: core: remove 'delay_usecs'
> field from spi_transfer"
>
> Signed-off-by: Stephen Rothwell <[email protected]>
> ---
> drivers/iio/adc/ti-ads131e08.c | 10 ++++++++--
> 1 file changed, 8 insertions(+), 2 deletions(-)
>
> diff --git a/drivers/iio/adc/ti-ads131e08.c b/drivers/iio/adc/ti-ads131e08.c
> index 0060d5f0abb0..764dab087b41 100644
> --- a/drivers/iio/adc/ti-ads131e08.c
> +++ b/drivers/iio/adc/ti-ads131e08.c
> @@ -177,7 +177,10 @@ static int ads131e08_read_reg(struct ads131e08_state *st, u8 reg)
> {
> .tx_buf = &st->tx_buf,
> .len = 2,
> - .delay_usecs = st->sdecode_delay_us,
> + .delay = {
> + .value = st->sdecode_delay_us,
> + .unit = SPI_DELAY_UNIT_USECS,
> + },
> }, {
> .rx_buf = &st->rx_buf,
> .len = 1,
> @@ -203,7 +206,10 @@ static int ads131e08_write_reg(struct ads131e08_state *st, u8 reg, u8 value)
> {
> .tx_buf = &st->tx_buf,
> .len = 3,
> - .delay_usecs = st->sdecode_delay_us,
> + .delay = {
> + .value = st->sdecode_delay_us,
> + .unit = SPI_DELAY_UNIT_USECS,
> + },
> }
> };
>
> --
> 2.30.0

With the merge window opening, this is a reminder that I am still
applying the above fix to the merge of the staging tree.

--
Cheers,
Stephen Rothwell


Attachments:
(No filename) (499.00 B)
OpenPGP digital signature

2021-04-26 14:44:01

by Mark Brown

[permalink] [raw]
Subject: Re: linux-next: build failure after merge of the staging tree

On Mon, Mar 29, 2021 at 08:14:50AM +0200, Greg KH wrote:

> Thanks for the fix, looks correct to me.

Here's the SPI pull request if you want to pull it in:

The following changes since commit a38fd8748464831584a19438cbb3082b5a2dab15:

Linux 5.12-rc2 (2021-03-05 17:33:41 -0800)

are available in the Git repository at:

https://git.kernel.org/pub/scm/linux/kernel/git/broonie/spi.git tags/spi-v5.13

for you to fetch changes up to 86527bcbc88922ea40df05d28189ee15489d2cf1:

spi: Rename enable1 to activate in spi_set_cs() (2021-04-23 15:36:18 +0100)

----------------------------------------------------------------
spi: Updates for v5.13

The only core work for SPI this time around is the completion of the
conversion to the new style method for specifying transfer delays,
meaning we can cope with what most controllers support more directly
using conversions in the core rather than open coding in drivers.
Otherwise it's a good stack of cleanups and fixes plus a few new
drivers.

The conversion to new style transfer delay will cause an issue with a
newly added staging driver which has a straightforward resolution in
-next.

- Completion of the conversion to new style transfer delay
configuration.
- Introduction and use of module_parport_driver() helper, merged here
as there's no parport tree.
- Support for Altera SoCs on DFL buses, NXP i.MX8DL, HiSilicon Kunpeng,
MediaTek MT8195,

----------------------------------------------------------------
Alain Volmat (2):
spi: stm32: avoid ifdef CONFIG_PM for pm callbacks
spi: stm32: Fix use-after-free on unbind

Alexander Sverdlin (1):
spi: omap2-mcspi: Activate pinctrl idle state during runtime suspend

Alexandru Ardelean (10):
spi: spi-axi-spi-engine: remove usage of delay_usecs
spi: bcm63xx-spi: don't check 'delay_usecs' field
spi: spi-bcm-qspi: replace 'delay_usecs' with 'delay.value' check
spi: spi-sh: replace 'delay_usecs' with 'delay.value' in pr_debug
spi: spi-tegra20-flash: don't check 'delay_usecs' field for spi transfer
staging: greybus: spilib: use 'spi_delay_to_ns' for getting xfer delay
spi: spi-falcon: remove check for 'delay_usecs'
spi: fsl-espi: remove usage of 'delay_usecs' field
spi: core: remove 'delay_usecs' field from spi_transfer
spi: docs: update info about 'delay_usecs'

Amit Kumar Mahapatra (1):
spi: spi-zynqmp-gqspi: Resolved slab-out-of-bounds bug

Andy Shevchenko (5):
parport: Introduce module_parport_driver() helper macro
spi: butterfly: Switch to use module_parport_driver()
spi: lm70llp: Switch to use module_parport_driver()
spi: Make error handling of gpiod_count() call cleaner
spi: Rename enable1 to activate in spi_set_cs()

Antonio Borneo (1):
spi: stm32: drop devres version of spi_register_master

Arnd Bergmann (2):
spi: rockchip: avoid objtool warning
spi: stm32-qspi: fix debug format string

Christophe JAILLET (1):
spi: fsi: add a missing of_node_put

Christophe Kerello (1):
spi: stm32-qspi: fix pm_runtime usage_count counter

Clark Wang (1):
spi: imx: add a check for speed_hz before calculating the clock

Colin Ian King (1):
spi: Fix spelling mistake "softwade" -> "software"

David Bauer (3):
spi: ath79: always call chipselect function
spi: ath79: remove spi-master setup and cleanup assignment
spi: sync up initial chipselect state

Dinghao Liu (2):
spi: spi-zynqmp-gqspi: Fix runtime PM imbalance in zynqmp_qspi_probe
spi: spi-zynqmp-gqspi: Fix runtime PM imbalance in zynqmp_qspi_probe

Eddie James (1):
spi: fsi: Remove multiple sequenced ops for restricted chips

Fabio Estevam (1):
spi: imx: Improve driver description

Han Xu (1):
spi: spi-nxp-fspi: Add imx8dxl driver support

Heikki Krogerus (4):
spi: Add support for software nodes
ARM: pxa: icontrol: Constify the software node
ARM: pxa: zeus: Constify the software node
spi: Remove support for dangling device properties

Heiko Schocher (2):
spi: fspi: enable fspi driver for on imx8mp
dt-bindings: spi: add compatible entry for imx8mp in FlexSPI controller

Jarkko Nikula (1):
spi: pxa2xx: Add support for Intel Alder Lake PCH-M

Jay Fang (14):
spi: cadence-quadspi: Silence shiftTooManyBitsSigned warning
spi: spi-topcliff-pch: Fix checkpatch spacing error
spi: sprd: Fix checkpatch spacing error
spi: pxa2xx: Fix checkpatch spacing errors
spi: omap-100k: Fix checkpatch spacing errors
spi: spi-mtk-nor: Fix checkpatch spacing error
spi: dln2: Fix open brace following function definitions go on the next line
spi: spi-bitbang: Fix open brace following function definitions go on the next line
spi: jcore: Fix trailing statements should be on next line
spi: spi-mem: Fix code indent should use tabs where possible
spi: rockchip: Fix code indent should use tabs where possible
spi: pl022: Fix trailing whitespace
spi: Add HiSilicon SPI Controller Driver for Kunpeng SoCs
spi: hisi-kunpeng: Fix Woverflow warning on conversion

Joe Burmeister (1):
spi: Handle SPI device setup callback failure.

Junlin Yang (1):
spi: cadence-quadspi: add missing of_node_put

Krzysztof Kozlowski (3):
spi: s3c64xx: simplify getting of_device_id match data
spi: s3c64xx: correct kerneldoc of s3c64xx_spi_port_config
spi: s3c64xx: constify driver/match data

Kuldeep Singh (4):
spi: spi-nxp-fspi: Add support for IP read only
spi: spi-nxp-fspi: Implement errata workaround for LS1028A
spi: spi-nxp-fspi: Add imx8dxl support
spi: Convert Freescale QSPI binding to json schema

Leilk Liu (4):
spi: update spi master bindings for MT8195 SoC
spi: update spi slave bindings for MT8195 SoC
spi: mediatek: add mtk_spi_compatible support
spi: mediatek: add mt8195 spi slave support

Linus Walleij (5):
spi: pl022: User more sensible defaults
spi: pl022: Drop custom per-chip cs_control
spi: pl022: Use GPIOs looked up by the core
spi: pl022: Convert to use GPIO descriptors
ARM/spi: spear: Drop PL022 num_chipselect

Mark Brown (11):
Merge existing fixes from spi/for-5.12
Merge series "parport: Introduce module_parport_driver() and use it" from Andy Shevchenko <[email protected]>:
Merge series "spi: finalize 'delay_usecs' removal/transition" from Alexandru Ardelean <[email protected]>:
Merge series "spi: Adding support for software nodes" from Heikki Krogerus <[email protected]>:
Merge series "enable flexspi support on imx8mp" from Heiko Schocher <[email protected]>:
Merge series "Convert Cadence QSPI bindings to yaml" from Pratyush Yadav <[email protected]>:
Merge series "spi: spi-zynqmp-gpspi: fix some issues" from [email protected] Quanyang Wang <[email protected]>:
Merge series "Minor updates for hisi-sfc-v3xx" from Yicong Yang <[email protected]>:
Merge branch 'for-5.12' of https://git.kernel.org/pub/scm/linux/kernel/git/broonie/spi into spi-5.13
Merge series "spi: stm32-qspi: Fix and update" from <[email protected]> Patrice Chotard <[email protected]>:
Merge series "spi: altera: Add DFL bus support for Altera SPI" from [email protected] Matthew Gerlach <[email protected]>:

Mason Zhang (1):
spi: mt6779: update spi document

Matthew Gerlach (2):
spi: altera: separate core code from platform code
spi: altera: Add DFL bus driver for Altera API Controller

Muhammad Usama Anjum (1):
spi: orion: set devdata properly as it is being used later

Patrice Chotard (3):
spi: stm32-qspi: Trigger DMA only if more than 4 bytes to transfer
spi: stm32-qspi: Add dirmap support
spi: stm32-qspi: Fix compilation warning in ARM64

Quanyang Wang (9):
spi: spi-zynqmp-gqspi: use wait_for_completion_timeout to make zynqmp_qspi_exec_op not interruptible
spi: spi-zynqmp-gqspi: add mutex locking for exec_op
spi: spi-zynqmp-gqspi: transmit dummy circles by using the controller's internal functionality
spi: spi-zynqmp-gqspi: fix incorrect operating mode in zynqmp_qspi_read_op
spi: spi-zynqmp-gqspi: fix clk_enable/disable imbalance issue
spi: spi-zynqmp-gqspi: fix hang issue when suspend/resume
spi: spi-zynqmp-gqspi: fix use-after-free in zynqmp_qspi_exec_op
spi: spi-zynqmp-gqspi: return -ENOMEM if dma_map_single fails
spi: tools: make a symbolic link to the header file spi.h

Rafał Miłecki (1):
spi: brcm,spi-bcm-qspi: convert to the json-schema

Ramuthevar Vadivel Murugan (1):
spi: Convert cadence-quadspi.txt to cadence-quadspi.yaml

Seiya Wang (1):
dt-bindings: spi: Add compatible for Mediatek MT8195

Shivamurthy Shastri (1):
spidev: Add Micron SPI NOR Authenta device compatible

Tian Tao (3):
spi: orion: Use device_get_match_data() helper
spi: simplify devm_spi_register_controller
spi: davinci: Use device_get_match_data() helper

Tudor Ambarus (3):
spi: spi-ti-qspi: Free DMA resources
spi: spi-ti-qspi: Free DMA resources
spi: atmel: Drop unused variable

Wan Jiabing (1):
spi: Remove repeated struct declaration

Wang Li (2):
spi: fsl-lpspi: Fix PM reference leak in lpspi_prepare_xfer_hardware()
spi: qup: fix PM reference leak in spi_qup_remove()

Wei Yongjun (3):
spi: dln2: Fix reference leak to master
spi: omap-100k: Fix reference leak to master
spi: spi-zynqmp-gqspi: Fix missing unlock on error in zynqmp_qspi_exec_op()

William A. Kennington III (1):
spi: Fix use-after-free with devm_spi_alloc_*

Yang Yingliang (1):
spi: fsl: add missing iounmap() on error in of_fsl_spi_probe()

Yicong Yang (2):
spi: hisi-sfc-v3xx: fix potential irq race condition
spi: hisi-sfc-v3xx: drop unnecessary ACPI_PTR and related ifendif protection

Álvaro Fernández Rojas (4):
spi: bcm63xx-spi: fix pm_runtime
spi: bcm63xx-hsspi: fix pm_runtime
spi: bcm63xx-spi: fix pm_runtime
spi: bcm63xx-hsspi: fix pm_runtime

.../devicetree/bindings/spi/brcm,spi-bcm-qspi.txt | 245 ----------
.../devicetree/bindings/spi/brcm,spi-bcm-qspi.yaml | 198 ++++++++
.../devicetree/bindings/spi/cadence-quadspi.txt | 68 ---
.../devicetree/bindings/spi/cdns,qspi-nor.yaml | 143 ++++++
.../devicetree/bindings/spi/fsl,spi-fsl-qspi.yaml | 96 ++++
.../bindings/spi/mediatek,spi-mtk-nor.yaml | 1 +
.../devicetree/bindings/spi/spi-fsl-qspi.txt | 66 ---
.../devicetree/bindings/spi/spi-mt65xx.txt | 2 +
.../devicetree/bindings/spi/spi-nxp-fspi.txt | 2 +
.../devicetree/bindings/spi/spi-slave-mt27xx.txt | 1 +
Documentation/spi/spi-summary.rst | 7 +-
MAINTAINERS | 10 +-
arch/arm/mach-pxa/icontrol.c | 12 +-
arch/arm/mach-pxa/zeus.c | 6 +-
arch/arm/mach-spear/spear320.c | 2 -
arch/arm/mach-spear/spear3xx.c | 10 -
drivers/spi/Kconfig | 28 +-
drivers/spi/Makefile | 5 +-
drivers/spi/{spi-altera.c => spi-altera-core.c} | 166 +------
drivers/spi/spi-altera-dfl.c | 204 +++++++++
drivers/spi/spi-altera-platform.c | 172 +++++++
drivers/spi/spi-ath79.c | 3 +-
drivers/spi/spi-atmel.c | 4 -
drivers/spi/spi-axi-spi-engine.c | 12 +-
drivers/spi/spi-bcm-qspi.c | 2 +-
drivers/spi/spi-bcm63xx-hsspi.c | 7 +-
drivers/spi/spi-bcm63xx.c | 8 +-
drivers/spi/spi-bitbang.c | 9 +-
drivers/spi/spi-butterfly.c | 13 +-
drivers/spi/spi-cadence-quadspi.c | 8 +-
drivers/spi/spi-davinci.c | 9 +-
drivers/spi/spi-dln2.c | 5 +-
drivers/spi/spi-falcon.c | 2 +-
drivers/spi/spi-fsi.c | 31 +-
drivers/spi/spi-fsl-espi.c | 17 +-
drivers/spi/spi-fsl-lpspi.c | 2 +-
drivers/spi/spi-fsl-spi.c | 23 +-
drivers/spi/spi-hisi-kunpeng.c | 505 +++++++++++++++++++++
drivers/spi/spi-hisi-sfc-v3xx.c | 7 +-
drivers/spi/spi-imx.c | 39 +-
drivers/spi/spi-jcore.c | 3 +-
drivers/spi/spi-lm70llp.c | 13 +-
drivers/spi/spi-mem.c | 6 +-
drivers/spi/spi-mtk-nor.c | 2 +-
drivers/spi/spi-nxp-fspi.c | 115 ++++-
drivers/spi/spi-omap-100k.c | 14 +-
drivers/spi/spi-omap2-mcspi.c | 24 +-
drivers/spi/spi-orion.c | 5 +-
drivers/spi/spi-pl022.c | 108 +----
drivers/spi/spi-pxa2xx-pci.c | 2 +-
drivers/spi/spi-pxa2xx.c | 6 +-
drivers/spi/spi-qup.c | 2 +-
drivers/spi/spi-rockchip.c | 19 +-
drivers/spi/spi-s3c64xx.c | 31 +-
drivers/spi/spi-sh.c | 4 +-
drivers/spi/spi-slave-mt27xx.c | 36 +-
drivers/spi/spi-sprd-adi.c | 2 +-
drivers/spi/spi-stm32-qspi.c | 106 ++++-
drivers/spi/spi-stm32.c | 39 +-
drivers/spi/spi-tegra20-sflash.c | 3 +-
drivers/spi/spi-ti-qspi.c | 20 +-
drivers/spi/spi-topcliff-pch.c | 3 +-
drivers/spi/spi-zynqmp-gqspi.c | 178 ++++----
drivers/spi/spi.c | 97 ++--
drivers/spi/spidev.c | 1 +
drivers/staging/greybus/spilib.c | 5 +-
include/linux/amba/pl022.h | 10 -
include/linux/parport.h | 12 +-
include/linux/spi/altera.h | 21 +
include/linux/spi/spi.h | 23 +-
tools/spi/Makefile | 5 +-
71 files changed, 2003 insertions(+), 1062 deletions(-)
delete mode 100644 Documentation/devicetree/bindings/spi/brcm,spi-bcm-qspi.txt
create mode 100644 Documentation/devicetree/bindings/spi/brcm,spi-bcm-qspi.yaml
delete mode 100644 Documentation/devicetree/bindings/spi/cadence-quadspi.txt
create mode 100644 Documentation/devicetree/bindings/spi/cdns,qspi-nor.yaml
create mode 100644 Documentation/devicetree/bindings/spi/fsl,spi-fsl-qspi.yaml
delete mode 100644 Documentation/devicetree/bindings/spi/spi-fsl-qspi.txt
rename drivers/spi/{spi-altera.c => spi-altera-core.c} (56%)
create mode 100644 drivers/spi/spi-altera-dfl.c
create mode 100644 drivers/spi/spi-altera-platform.c
create mode 100644 drivers/spi/spi-hisi-kunpeng.c


Attachments:
(No filename) (15.37 kB)
signature.asc (499.00 B)
Download all attachments

2021-04-26 16:24:37

by Greg KH

[permalink] [raw]
Subject: Re: linux-next: build failure after merge of the staging tree

On Mon, Apr 26, 2021 at 03:41:21PM +0100, Mark Brown wrote:
> On Mon, Mar 29, 2021 at 08:14:50AM +0200, Greg KH wrote:
>
> > Thanks for the fix, looks correct to me.
>
> Here's the SPI pull request if you want to pull it in:
>
> The following changes since commit a38fd8748464831584a19438cbb3082b5a2dab15:
>
> Linux 5.12-rc2 (2021-03-05 17:33:41 -0800)
>
> are available in the Git repository at:
>
> https://git.kernel.org/pub/scm/linux/kernel/git/broonie/spi.git tags/spi-v5.13
>
> for you to fetch changes up to 86527bcbc88922ea40df05d28189ee15489d2cf1:
>
> spi: Rename enable1 to activate in spi_set_cs() (2021-04-23 15:36:18 +0100)

I don't think I want to pull the full SPI merge into my staging tree at
this point in time, is that wise?

I already submitted a pull request to Linus for the staging tree as-is,
if there are problems we can work to address them then.

thanks,

greg k-h

2021-04-26 16:42:22

by Mark Brown

[permalink] [raw]
Subject: Re: linux-next: build failure after merge of the staging tree

On Mon, Apr 26, 2021 at 06:23:29PM +0200, Greg KH wrote:
> On Mon, Apr 26, 2021 at 03:41:21PM +0100, Mark Brown wrote:

> > spi: Rename enable1 to activate in spi_set_cs() (2021-04-23 15:36:18 +0100)

> I don't think I want to pull the full SPI merge into my staging tree at
> this point in time, is that wise?

I don't either, just putting it there in case you wanted it.

> I already submitted a pull request to Linus for the staging tree as-is,
> if there are problems we can work to address them then.

If it were a new API I'd have expected a cross tree merge of a tag with
just the API being added in it, though in this case since the API was
already there in mainline I'd been expecting it to get cleaned up with
Stephen's patch or something similar as part of the work in staging.


Attachments:
(No filename) (808.00 B)
signature.asc (499.00 B)
Download all attachments

2021-04-26 22:11:52

by Stephen Rothwell

[permalink] [raw]
Subject: Re: linux-next: build failure after merge of the staging tree

Hi all,

On Mon, 29 Mar 2021 16:55:25 +1100 Stephen Rothwell <[email protected]> wrote:
>
> After merging the staging tree, today's linux-next build (x86_64
> allmodconfig) failed like this:
>
> drivers/iio/adc/ti-ads131e08.c: In function 'ads131e08_read_reg':
> drivers/iio/adc/ti-ads131e08.c:180:5: error: 'struct spi_transfer' has no member named 'delay_usecs'
> 180 | .delay_usecs = st->sdecode_delay_us,
> | ^~~~~~~~~~~
> drivers/iio/adc/ti-ads131e08.c: In function 'ads131e08_write_reg':
> drivers/iio/adc/ti-ads131e08.c:206:5: error: 'struct spi_transfer' has no member named 'delay_usecs'
> 206 | .delay_usecs = st->sdecode_delay_us,
> | ^~~~~~~~~~~
>
> Caused by commit
>
> d935eddd2799 ("iio: adc: Add driver for Texas Instruments ADS131E0x ADC family")
>
> interacting with commit
>
> 3ab1cce55337 ("spi: core: remove 'delay_usecs' field from spi_transfer")
>
> from the spi tree.
>
> I have applied the following merge fix patch.
>
> From: Stephen Rothwell <[email protected]>
> Date: Mon, 29 Mar 2021 16:51:22 +1100
> Subject: [PATCH] iio: adc: merge fix for "spi: core: remove 'delay_usecs'
> field from spi_transfer"
>
> Signed-off-by: Stephen Rothwell <[email protected]>
> ---
> drivers/iio/adc/ti-ads131e08.c | 10 ++++++++--
> 1 file changed, 8 insertions(+), 2 deletions(-)
>
> diff --git a/drivers/iio/adc/ti-ads131e08.c b/drivers/iio/adc/ti-ads131e08.c
> index 0060d5f0abb0..764dab087b41 100644
> --- a/drivers/iio/adc/ti-ads131e08.c
> +++ b/drivers/iio/adc/ti-ads131e08.c
> @@ -177,7 +177,10 @@ static int ads131e08_read_reg(struct ads131e08_state *st, u8 reg)
> {
> .tx_buf = &st->tx_buf,
> .len = 2,
> - .delay_usecs = st->sdecode_delay_us,
> + .delay = {
> + .value = st->sdecode_delay_us,
> + .unit = SPI_DELAY_UNIT_USECS,
> + },
> }, {
> .rx_buf = &st->rx_buf,
> .len = 1,
> @@ -203,7 +206,10 @@ static int ads131e08_write_reg(struct ads131e08_state *st, u8 reg, u8 value)
> {
> .tx_buf = &st->tx_buf,
> .len = 3,
> - .delay_usecs = st->sdecode_delay_us,
> + .delay = {
> + .value = st->sdecode_delay_us,
> + .unit = SPI_DELAY_UNIT_USECS,
> + },
> }
> };
>
> --
> 2.30.0

This is now a conflict between the spi (and spi-fixes) tree and Linus' tree.

--
Cheers,
Stephen Rothwell


Attachments:
(No filename) (499.00 B)
OpenPGP digital signature