Re: [PATCH v4 0/11] at91: Support PMC clock bindings

[Ahmad Fatoum <a.fatoum@pengutronix.de>]

On 24/2/19 19:52, Sam Ravnborg wrote:
> Hi all.
> 
> The following patchset add supports for the PMC clk
> bindings for at91.
> The patchset was tested on:
> - at91sam9264ek - DT enabled
> - sama5d4-xplained - non DT
> 
> Ladislav and Ahmad did some additional testing on the boards they have available.
> 
> Please await feedback from Ahmad and Ladislav before applying to master.

I don't have the hardware to test these at the moment,
but from the changelog, I assume it should be okay.

> 
> 
> The patchset includes sama5d2.c which is not used today.
> I have a sama5d27-som1-ek board that I hope to find time to work
> with and then this will be needed.
> 
> Patch was made on top of barebox master:
> 3c2dee2e7d62a32ba85f7bf94bb7abf5b4f09dab ("doc: bootchooser: add retry handling to algorithm description")
> 
> On stray patch sneaked in - to kill a warning.
> 
> The patchset triggers some checkpatch warnings:
> - SPDX, the barebox variant of checkpatch is missing support for these
> - Too long lines. As code is copied from the kernel I left them as-is
> 
> Comments/review/testing very welcome!
> Note: I am at Embedded World Nürenberg for a few days, so no swift
> repsonses to feedback/comments.
> 
> v4:
> - added dt-compat required by sama5d3 (Ahmad)
> - added syscon_regmap_lookup_by_compatible (Ahmad)
> - Rebased on master
> - Added SOC_SAMA5D2 - as it is used later
>   Not used for now, but needed later
> 
> v3:
> - fixed build breakage in sama5d4 and clk-h32mx
> - added Tested-by: Ladislav Michl <ladis@linux-mips.org> to relevant patches
>   Not all, as only a subet was covered by the test
> 
> v2:
> - Split up in smaller patches
> - Fix builds in various configs
> - Reworked Kconfig changes
> - Fix checkpatch warnings (mostly whitespace)
> 
> v1:
> - Posted to mailing list as a WIP patch
> 
>         Sam
> 
> 
> Ahmad Fatoum (2):
>       regmap: Implement syscon_regmap_lookup_by_compatible()
>       clk: at91: add dt-compat to PMC bindings
> 
> Sam Ravnborg (9):
>       arm: at91: fix at91_configure_usart6 warning
>       clocksource: Do not mix depends and select for ATMEL_PIT
>       arm: at91: add SOC_SAMA5{D2,D3,D4}
>       clk: at91: add at91sam9260
>       clk: at91: add at91sam9rl
>       clk: at91: add at91sam9x5
>       clk: at91: add sama5d2
>       clk: at91: add sama5d4
>       clk: at91: update to PMC bindings
> 
>  arch/arm/mach-at91/Kconfig              |  37 +-
>  arch/arm/mach-at91/include/mach/board.h |   1 +
>  drivers/clk/at91/Makefile               |   6 +
>  drivers/clk/at91/at91sam9260.c          | 497 +++++++++++++++++
>  drivers/clk/at91/at91sam9rl.c           | 177 ++++++
>  drivers/clk/at91/at91sam9x5.c           | 315 +++++++++++
>  drivers/clk/at91/clk-generated.c        | 185 ++----
>  drivers/clk/at91/clk-h32mx.c            |  55 +-
>  drivers/clk/at91/clk-main.c             | 112 +---
>  drivers/clk/at91/clk-master.c           |  94 +---
>  drivers/clk/at91/clk-peripheral.c       |  82 +--
>  drivers/clk/at91/clk-pll.c              | 174 +-----
>  drivers/clk/at91/clk-plldiv.c           |  27 +-
>  drivers/clk/at91/clk-programmable.c     |  83 +--
>  drivers/clk/at91/clk-slow.c             |  33 +-
>  drivers/clk/at91/clk-smd.c              |  33 +-
>  drivers/clk/at91/clk-system.c           |  42 +-
>  drivers/clk/at91/clk-usb.c              |  94 +---
>  drivers/clk/at91/clk-utmi.c             | 100 ++--
>  drivers/clk/at91/dt-compat.c            | 961 ++++++++++++++++++++++++++++++++
>  drivers/clk/at91/pmc.c                  | 248 +++++++++
>  drivers/clk/at91/pmc.h                  | 169 ++++++
>  drivers/clk/at91/sama5d2.c              | 342 ++++++++++++
>  drivers/clk/at91/sama5d4.c              | 270 +++++++++
>  drivers/clocksource/Kconfig             |   1 -
>  drivers/mfd/syscon.c                    |  14 +
>  include/linux/clk.h                     |   7 +
>  include/mfd/syscon.h                    |   5 +
>  include/soc/at91/atmel-sfr.h            |  34 ++
>  29 files changed, 3220 insertions(+), 978 deletions(-)
> 

-- 
Pengutronix e.K.                           |                             |
Industrial Linux Solutions                 | http://www.pengutronix.de/  |
Peiner Str. 6-8, 31137 Hildesheim, Germany | Phone: +49-5121-206917-0    |
Amtsgericht Hildesheim, HRA 2686           | Fax:   +49-5121-206917-5555 |

_______________________________________________
barebox mailing list
barebox@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/barebox