mbox series

[0/4] Add initial support for RoseapplePi SBC

Message ID cover.1592123160.git.cristian.ciocaltea@gmail.com
Headers show
Series Add initial support for RoseapplePi SBC | expand

Message

Cristian Ciocaltea June 15, 2020, midnight UTC
This patchset enables basic support for RoseapplePi, relying exclusively
on the existing infrastructure for the Actions Semi Sxx SoCs (thank you
Andreas and Manni for making this possible).

The SBC is powered by the Actions Semi S500 SoC and comes with 2GB RAM,
uSD slot and optional eMMC storage. For more details, please check:
http://roseapplepi.org/index.php/spec/

The upcoming patches will improve this initial support by adding the
missing bits and pieces to the S500 clock management unit, which is a
prerequisite for providing an S500 pinctrl and gpio driver, in order to
eventually enable access to additional functionality like I2C and MMC.

Thanks and regards,
Cristian Ciocaltea

Cristian Ciocaltea (4):
  arm: dts: owl-s500: Fix incorrect PPI interrupt specifiers
  dt-bindings: Add vendor prefix for RoseapplePi.org
  dt-bindings: arm: actions: Document RoseapplePi
  arm: dts: owl-s500: Add RoseapplePi

 .../devicetree/bindings/arm/actions.yaml      |  1 +
 .../devicetree/bindings/vendor-prefixes.yaml  |  2 +
 arch/arm/boot/dts/Makefile                    |  1 +
 arch/arm/boot/dts/owl-s500-roseapplepi.dts    | 47 +++++++++++++++++++
 arch/arm/boot/dts/owl-s500.dtsi               |  6 +--
 5 files changed, 54 insertions(+), 3 deletions(-)
 create mode 100644 arch/arm/boot/dts/owl-s500-roseapplepi.dts

Comments

Peter Korsgaard Aug. 27, 2020, 6:29 a.m. UTC | #1
>>>>> "Cristian" == Cristian Ciocaltea <cristian.ciocaltea@gmail.com> writes:

 > The PPI interrupts for cortex-a9 were incorrectly specified, fix them.
 > Fixes: fdfe7f4f9d85 ("ARM: dts: Add Actions Semi S500 and LeMaker Guitar")
 > Signed-off-by: Cristian Ciocaltea <cristian.ciocaltea@gmail.com>

Reviewed-by: Peter Korsgaard <peter@korsgaard.com>
Peter Korsgaard Aug. 27, 2020, 6:44 a.m. UTC | #2
>>>>> "Cristian" == Cristian Ciocaltea <cristian.ciocaltea@gmail.com> writes:

 > Add a Device Tree for the RoseapplePi SBC.
 > Signed-off-by: Cristian Ciocaltea <cristian.ciocaltea@gmail.com>

Reviewed-by: Peter Korsgaard <peter@korsgaard.com>

On a related note: There is now an owl-mmc driver for the s900. From a
quick look at the datasheet it looks compatible with the controller on
the s500. Did you have a look at hooking that up?
Cristian Ciocaltea Aug. 27, 2020, 10:37 a.m. UTC | #3
On Thu, Aug 27, 2020 at 08:29:06AM +0200, Peter Korsgaard wrote:
> >>>>> "Cristian" == Cristian Ciocaltea <cristian.ciocaltea@gmail.com> writes:
> 
>  > The PPI interrupts for cortex-a9 were incorrectly specified, fix them.
>  > Fixes: fdfe7f4f9d85 ("ARM: dts: Add Actions Semi S500 and LeMaker Guitar")
>  > Signed-off-by: Cristian Ciocaltea <cristian.ciocaltea@gmail.com>
> 
> Reviewed-by: Peter Korsgaard <peter@korsgaard.com>
> 
> -- 
> Bye, Peter Korsgaard

Thanks,
Cristi
Cristian Ciocaltea Aug. 27, 2020, 2:15 p.m. UTC | #4
Hi Peter,

Thanks for the review!

On Thu, Aug 27, 2020 at 08:44:40AM +0200, Peter Korsgaard wrote:
> >>>>> "Cristian" == Cristian Ciocaltea <cristian.ciocaltea@gmail.com> writes:
> 
>  > Add a Device Tree for the RoseapplePi SBC.
>  > Signed-off-by: Cristian Ciocaltea <cristian.ciocaltea@gmail.com>
> 
> Reviewed-by: Peter Korsgaard <peter@korsgaard.com>
> 
> On a related note: There is now an owl-mmc driver for the s900. From a
> quick look at the datasheet it looks compatible with the controller on
> the s500. Did you have a look at hooking that up?

Yes, please see:
https://lore.kernel.org/lkml/cover.1593124368.git.cristian.ciocaltea@gmail.com/

The clock related patches have been already applied to v5.9 and
hopefully the pinctrl driver will follow in v5.10.

> -- 
> Bye, Peter Korsgaard

Regards,
Cristi
Peter Korsgaard Aug. 27, 2020, 5:17 p.m. UTC | #5
>>>>> "Cristian" == Cristian Ciocaltea <cristian.ciocaltea@gmail.com> writes:

 > Hi Peter,
 > Thanks for the review!

 > On Thu, Aug 27, 2020 at 08:44:40AM +0200, Peter Korsgaard wrote:
 >> >>>>> "Cristian" == Cristian Ciocaltea <cristian.ciocaltea@gmail.com> writes:
 >> 
 >> > Add a Device Tree for the RoseapplePi SBC.
 >> > Signed-off-by: Cristian Ciocaltea <cristian.ciocaltea@gmail.com>
 >> 
 >> Reviewed-by: Peter Korsgaard <peter@korsgaard.com>
 >> 
 >> On a related note: There is now an owl-mmc driver for the s900. From a
 >> quick look at the datasheet it looks compatible with the controller on
 >> the s500. Did you have a look at hooking that up?

 > Yes, please see:
 > https://lore.kernel.org/lkml/cover.1593124368.git.cristian.ciocaltea@gmail.com/

 > The clock related patches have been already applied to v5.9 and
 > hopefully the pinctrl driver will follow in v5.10.

Cool, I must have missed that series Great, thanks!