2021-10-26 01:23:20

by Prabhakar Mahadev Lad

[permalink] [raw]
Subject: [PATCH v2 0/7] Add SPI Multi I/O Bus Controller support for RZ/G2L

Hi All,

This patch series adds a couple of fixes for rpc-if driver and
adds support for RZ/G2L SoC, where the SPI Multi I/O Bus Controller
is identical to the RPC-IF block found on R-Car Gen3 SoC's.

Cheers,
Prabhakar

Changes for v2:
* Rebased the patches on linux-next
* Split patch 5 from v1
* Included RB tags
* Fixed review comments pointed by Wolfram

v1:
https://patchwork.kernel.org/project/linux-renesas-soc/cover/
[email protected]/

Lad Prabhakar (7):
dt-bindings: memory: renesas,rpc-if: Add support for the R9A07G044
dt-bindings: memory: renesas,rpc-if: Add optional interrupts property
spi: spi-rpc-if: Check return value of rpcif_sw_init()
mtd: hyperbus: rpc-if: Check return value of rpcif_sw_init()
memory: renesas-rpc-if: Return error in case devm_ioremap_resource()
fails
memory: renesas-rpc-if: Drop usage of RPCIF_DIRMAP_SIZE macro
memory: renesas-rpc-if: Add support for RZ/G2L

.../memory-controllers/renesas,rpc-if.yaml | 54 +++++++++----
drivers/memory/renesas-rpc-if.c | 80 +++++++++++++++----
drivers/mtd/hyperbus/rpc-if.c | 8 +-
drivers/spi/spi-rpc-if.c | 8 +-
include/memory/renesas-rpc-if.h | 8 +-
5 files changed, 124 insertions(+), 34 deletions(-)

--
2.17.1


2021-10-26 20:19:06

by Krzysztof Kozlowski

[permalink] [raw]
Subject: Re: [PATCH v2 0/7] Add SPI Multi I/O Bus Controller support for RZ/G2L

On 25/10/2021 22:56, Lad Prabhakar wrote:
> Hi All,
>
> This patch series adds a couple of fixes for rpc-if driver and
> adds support for RZ/G2L SoC, where the SPI Multi I/O Bus Controller
> is identical to the RPC-IF block found on R-Car Gen3 SoC's.
>
> Cheers,
> Prabhakar
>
> Changes for v2:
> * Rebased the patches on linux-next
> * Split patch 5 from v1
> * Included RB tags
> * Fixed review comments pointed by Wolfram
>
> v1:
> https://patchwork.kernel.org/project/linux-renesas-soc/cover/
> [email protected]/
>

Patches look good but you sent them too late for this cycle. I'll take
the memory controller parts after the merge window.


Best regards,
Krzysztof

2021-10-27 07:32:11

by Mark Brown

[permalink] [raw]
Subject: Re: (subset) [PATCH v2 0/7] Add SPI Multi I/O Bus Controller support for RZ/G2L

On Mon, 25 Oct 2021 21:56:24 +0100, Lad Prabhakar wrote:
> This patch series adds a couple of fixes for rpc-if driver and
> adds support for RZ/G2L SoC, where the SPI Multi I/O Bus Controller
> is identical to the RPC-IF block found on R-Car Gen3 SoC's.
>
> Cheers,
> Prabhakar
>
> [...]

Applied to

https://git.kernel.org/pub/scm/linux/kernel/git/broonie/spi.git for-next

Thanks!

[3/7] spi: spi-rpc-if: Check return value of rpcif_sw_init()
commit: 0b0a281ed7001d4c4f4c47bdc84680c4997761ca

All being well this means that it will be integrated into the linux-next
tree (usually sometime in the next 24 hours) and sent to Linus during
the next merge window (or sooner if it is a bug fix), however if
problems are discovered then the patch may be dropped or reverted.

You may get further e-mails resulting from automated or manual testing
and review of the tree, please engage with people reporting problems and
send followup patches addressing any issues that are reported if needed.

If any updates are required or you are submitting further changes they
should be sent as incremental updates against current git, existing
patches will not be replaced.

Please add any relevant lists and maintainers to the CCs when replying
to this mail.

Thanks,
Mark

2021-11-16 10:33:40

by Krzysztof Kozlowski

[permalink] [raw]
Subject: Re: [PATCH v2 0/7] Add SPI Multi I/O Bus Controller support for RZ/G2L

On 25/10/2021 22:56, Lad Prabhakar wrote:
> Hi All,
>
> This patch series adds a couple of fixes for rpc-if driver and
> adds support for RZ/G2L SoC, where the SPI Multi I/O Bus Controller
> is identical to the RPC-IF block found on R-Car Gen3 SoC's.
>
> Cheers,
> Prabhakar
>
> Changes for v2:
> * Rebased the patches on linux-next
> * Split patch 5 from v1
> * Included RB tags
> * Fixed review comments pointed by Wolfram
>
> v1:
> https://patchwork.kernel.org/project/linux-renesas-soc/cover/
> [email protected]/
>
> Lad Prabhakar (7):
> dt-bindings: memory: renesas,rpc-if: Add support for the R9A07G044
> dt-bindings: memory: renesas,rpc-if: Add optional interrupts property
> spi: spi-rpc-if: Check return value of rpcif_sw_init()
> mtd: hyperbus: rpc-if: Check return value of rpcif_sw_init()
> memory: renesas-rpc-if: Return error in case devm_ioremap_resource()
> fails
> memory: renesas-rpc-if: Drop usage of RPCIF_DIRMAP_SIZE macro
> memory: renesas-rpc-if: Add support for RZ/G2L
>

Applied parts 1, 2, 5 and 6. I think 7 is going to have a new version
due to Wolfram's comments?


Best regards,
Krzysztof

2021-11-16 10:41:14

by Lad, Prabhakar

[permalink] [raw]
Subject: Re: [PATCH v2 0/7] Add SPI Multi I/O Bus Controller support for RZ/G2L

Hi Krzysztof,

On Tue, Nov 16, 2021 at 10:33 AM Krzysztof Kozlowski
<[email protected]> wrote:
>
> On 25/10/2021 22:56, Lad Prabhakar wrote:
> > Hi All,
> >
> > This patch series adds a couple of fixes for rpc-if driver and
> > adds support for RZ/G2L SoC, where the SPI Multi I/O Bus Controller
> > is identical to the RPC-IF block found on R-Car Gen3 SoC's.
> >
> > Cheers,
> > Prabhakar
> >
> > Changes for v2:
> > * Rebased the patches on linux-next
> > * Split patch 5 from v1
> > * Included RB tags
> > * Fixed review comments pointed by Wolfram
> >
> > v1:
> > https://patchwork.kernel.org/project/linux-renesas-soc/cover/
> > [email protected]/
> >
> > Lad Prabhakar (7):
> > dt-bindings: memory: renesas,rpc-if: Add support for the R9A07G044
> > dt-bindings: memory: renesas,rpc-if: Add optional interrupts property
> > spi: spi-rpc-if: Check return value of rpcif_sw_init()
> > mtd: hyperbus: rpc-if: Check return value of rpcif_sw_init()
> > memory: renesas-rpc-if: Return error in case devm_ioremap_resource()
> > fails
> > memory: renesas-rpc-if: Drop usage of RPCIF_DIRMAP_SIZE macro
> > memory: renesas-rpc-if: Add support for RZ/G2L
> >
>
> Applied parts 1, 2, 5 and 6. I think 7 is going to have a new version
> due to Wolfram's comments?
>
Thank you for queuing up the patches, wrt patch 7/7 this can also be
picked up, after the internal discussion it was clear that we cannot
use the R-car hw manual for RZ/G2L (we will have to live with magic
values). Wolfram has agreed on this and has already Acked patch 7/7.

Cheers,
Prabhakar

>
> Best regards,
> Krzysztof

2021-11-16 11:11:34

by Krzysztof Kozlowski

[permalink] [raw]
Subject: Re: [PATCH v2 0/7] Add SPI Multi I/O Bus Controller support for RZ/G2L

On 16/11/2021 11:40, Lad, Prabhakar wrote:
> Hi Krzysztof,
>
> On Tue, Nov 16, 2021 at 10:33 AM Krzysztof Kozlowski
> <[email protected]> wrote:
>>
>> On 25/10/2021 22:56, Lad Prabhakar wrote:
>>> Hi All,
>>>
>>> This patch series adds a couple of fixes for rpc-if driver and
>>> adds support for RZ/G2L SoC, where the SPI Multi I/O Bus Controller
>>> is identical to the RPC-IF block found on R-Car Gen3 SoC's.
>>>
>>> Cheers,
>>> Prabhakar
>>>
>>> Changes for v2:
>>> * Rebased the patches on linux-next
>>> * Split patch 5 from v1
>>> * Included RB tags
>>> * Fixed review comments pointed by Wolfram
>>>
>>> v1:
>>> https://patchwork.kernel.org/project/linux-renesas-soc/cover/
>>> [email protected]/
>>>
>>> Lad Prabhakar (7):
>>> dt-bindings: memory: renesas,rpc-if: Add support for the R9A07G044
>>> dt-bindings: memory: renesas,rpc-if: Add optional interrupts property
>>> spi: spi-rpc-if: Check return value of rpcif_sw_init()
>>> mtd: hyperbus: rpc-if: Check return value of rpcif_sw_init()
>>> memory: renesas-rpc-if: Return error in case devm_ioremap_resource()
>>> fails
>>> memory: renesas-rpc-if: Drop usage of RPCIF_DIRMAP_SIZE macro
>>> memory: renesas-rpc-if: Add support for RZ/G2L
>>>
>>
>> Applied parts 1, 2, 5 and 6. I think 7 is going to have a new version
>> due to Wolfram's comments?
>>
> Thank you for queuing up the patches, wrt patch 7/7 this can also be
> picked up, after the internal discussion it was clear that we cannot
> use the R-car hw manual for RZ/G2L (we will have to live with magic
> values). Wolfram has agreed on this and has already Acked patch 7/7.
>

OK, applied now. Thanks!


Best regards,
Krzysztof