mail archive of the barebox mailing list
 help / color / mirror / Atom feed
From: Andrey Smirnov <andrew.smirnov@gmail.com>
To: Sam Ravnborg <sam@ravnborg.org>
Cc: Barebox List <barebox@lists.infradead.org>
Subject: Re: [PATCH v4 0/25] at91: move reset vector to board code
Date: Thu, 4 Jan 2018 09:12:03 -0800	[thread overview]
Message-ID: <CAHQ1cqGrWvbfY-Y=OouEYKorVLCMspz7uz+Ca7vGrjO+pVCoyQ@mail.gmail.com> (raw)
In-Reply-To: <20171231083407.GA16645@ravnborg.org>

On Sun, Dec 31, 2017 at 12:34 AM, Sam Ravnborg <sam@ravnborg.org> wrote:
> The following patchset moves the reset vector
> for at91 boards from mach-at91 to board code.
> This is similar to what other boards do and is done to
> prepare for multi-image and DT support.
>
> The patchset convert all boards and thus clean up the
> old code that resided in mach-at91/
>
> The first patch have minimal impact, but prepares for a
> board-by-board conversion.
> All temporary CONFIG symbols are gone at the end of this series.
>
> I kept Jean-Christophe PLAGNIOL-VILLARD and others
> copyright in all the created files and did not add
> copyright myself. The patchset does IMO not add any
> copyrightable material from my side as this is
> in reality just moving around code.
>
> With this patchset applied we no longer have any
> places where we call board code from mach-at91/
> thus allowing us to introduce multi-images.
> And we are also on the right track to introduce
> DT support for additional AT91 based boards.
>
> The patchset was successfully booted on a AT91SAM9263EK,
> and all defconfigs was built (expect the ones that failed
> to build before this patchset was applied).
>
> I consider the patchset ready for barebox-next.
>
>         Sam
>
> Changes v4 => v3
>   Updated at91sam926x_board_init.h based on feedback from Andrey Smirnov
>
> Changes v2 => v3
>   v2 was posted in July - but I have lacked spare time until now
>   to give it a proper respin
>   - respin of all patches on top of v2017.12.0
>   - fix bogus handling of at91sam926x_board_init
>   - Introduce new CONFIG symbol for at91sam926x_board_init to allow bisects
>
> Changes v1 => v2
>   - converted all boards over
>   - fixed author info
>


Reviewed-by: Andrey Smirnov <andrew.smirnov@gmail.com>

>
> Sam Ravnborg (25):
>       arm: at91: refactor lowlevel_init selection
>       arm: at91: drop unused at91sam9x5_lowlevel_init.c
>       arm: at91: add at91sam926x_board_init.h
>       arm: at91: code cleanup in at91sam926x_board_init
>       at91sam9263ek: move reset vector to board code
>       at91sam9261ek, at91sam9g10ek: move reset vector to board code
>       pm9261: move reset vector to board code
>       at91: drop unused at91sam9261_lowlevel_init
>       pm9263: move reset vector to board code
>       usb-a926x: move reset vector to board code
>       mmccpu: delete unused lowlevel_init
>       mmccpu: move reset vector to board code
>       tny-a926x: delete unused tny_a9263_lowlevel_init.c
>       tny-a926x: move reset vector to board code
>       qil-a926x: move reset vector to board code
>       haba-knx: move reset vector to board code
>       sama5d{3,4}{xek,xplained}: move reset vector to board code
>       at91sam9n12ek: move reset vector to board code
>       at91sam9260ek, at91sam9g20ek: move reset vector to board code
>       at91sam9m10g45ek, at91sam9m10ihd, pm9g45: move reset vector to board code
>       animeo: move reset vector to board code
>       telit-evk-pro3: move reset vector to board code
>       dss11: move reset vector to board code
>       at91rm9200ek: move reset vector to board code
>       arm: at91: remove leftovers from moving reset code in mach-at91
>
>
> # git diff --stat=120 -M -C HEAD~36..HEAD~11
> # Note options above to let git track copies/renames
>  arch/arm/boards/animeo_ip/Makefile                                          |   3 +
>  arch/arm/{mach-at91/sama5d3_lowlevel_init.c => boards/animeo_ip/lowlevel.c} |  17 +++--
>  arch/arm/boards/at91rm9200ek/Makefile                                       |   3 +
>  .../at91rm9200_lowlevel_init.c => boards/at91rm9200ek/lowlevel.c}           |  14 ++--
>  arch/arm/boards/at91sam9260ek/Makefile                                      |   3 +
>  .../{mach-at91/sama5d3_lowlevel_init.c => boards/at91sam9260ek/lowlevel.c}  |  17 +++--
>  arch/arm/boards/at91sam9261ek/Makefile                                      |   2 +-
>  arch/arm/boards/at91sam9261ek/lowlevel_init.c                               |  41 +++++++---
>  arch/arm/boards/at91sam9263ek/Makefile                                      |   2 +-
>  arch/arm/boards/at91sam9263ek/lowlevel_init.c                               |  43 ++++++++---
>  arch/arm/boards/at91sam9m10g45ek/Makefile                                   |   3 +
>  .../at91sam9g45_lowlevel_init.c => boards/at91sam9m10g45ek/lowlevel.c}      |  11 +--
>  arch/arm/boards/at91sam9m10ihd/Makefile                                     |   2 +
>  .../at91sam9x5_lowlevel_init.c => boards/at91sam9m10ihd/lowlevel.c}         |  19 +++--
>  arch/arm/boards/at91sam9n12ek/Makefile                                      |   3 +
>  .../at91sam9n12_lowlevel_init.c => boards/at91sam9n12ek/lowlevel.c}         |  13 ++--
>  arch/arm/boards/dss11/Makefile                                              |   2 +
>  arch/arm/{mach-at91/sama5d3_lowlevel_init.c => boards/dss11/lowlevel.c}     |  17 +++--
>  arch/arm/boards/haba-knx/Makefile                                           |   3 +
>  arch/arm/{mach-at91/sama5d3_lowlevel_init.c => boards/haba-knx/lowlevel.c}  |  17 +++--
>  arch/arm/boards/mmccpu/Makefile                                             |   2 +-
>  .../arm/{mach-at91/at91sam9n12_lowlevel_init.c => boards/mmccpu/lowlevel.c} |  19 +++--
>  arch/arm/boards/mmccpu/lowlevel_init.c                                      | 135 --------------------------------
>  arch/arm/boards/pm9261/Makefile                                             |   2 +-
>  arch/arm/boards/pm9261/lowlevel_init.c                                      |  39 +++++++---
>  arch/arm/boards/pm9263/Makefile                                             |   2 +-
>  arch/arm/boards/pm9263/lowlevel_init.c                                      |  43 ++++++++---
>  arch/arm/boards/pm9g45/Makefile                                             |   3 +
>  .../arm/{mach-at91/at91sam9g45_lowlevel_init.c => boards/pm9g45/lowlevel.c} |  13 ++--
>  arch/arm/boards/qil-a926x/Makefile                                          |   3 +
>  arch/arm/{mach-at91/sama5d3_lowlevel_init.c => boards/qil-a926x/lowlevel.c} |  17 +++--
>  arch/arm/boards/sama5d3_xplained/Makefile                                   |   1 +
>  .../sama5d3_lowlevel_init.c => boards/sama5d3_xplained/lowlevel.c}          |  10 +--
>  arch/arm/boards/sama5d3xek/Makefile                                         |   1 +
>  .../arm/{mach-at91/sama5d3_lowlevel_init.c => boards/sama5d3xek/lowlevel.c} |  10 +--
>  arch/arm/boards/sama5d4_xplained/Makefile                                   |   1 +
>  .../sama5d3_lowlevel_init.c => boards/sama5d4_xplained/lowlevel.c}          |  10 +--
>  arch/arm/boards/sama5d4ek/Makefile                                          |   1 +
>  arch/arm/{mach-at91/sama5d3_lowlevel_init.c => boards/sama5d4ek/lowlevel.c} |  10 +--
>  arch/arm/boards/telit-evk-pro3/Makefile                                     |   2 +
>  .../{mach-at91/sama5d3_lowlevel_init.c => boards/telit-evk-pro3/lowlevel.c} |  17 +++--
>  arch/arm/boards/tny-a926x/Makefile                                          |   8 +-
>  .../sama5d3_lowlevel_init.c => boards/tny-a926x/tny_a9260_lowlevel.c}       |  17 +++--
>  .../at91sam9n12_lowlevel_init.c => boards/tny-a926x/tny_a9263_lowlevel.c}   |  19 +++--
>  arch/arm/boards/tny-a926x/tny_a9263_lowlevel_init.c                         | 107 --------------------------
>  arch/arm/boards/usb-a926x/Makefile                                          |   8 +-
>  .../sama5d3_lowlevel_init.c => boards/usb-a926x/usb_a9260_lowlevel.c}       |  17 +++--
>  .../boards/usb-a926x/{usb_a9263_lowlevel_init.c => usb_a9263_lowlevel.c}    |  43 ++++++++---
>  arch/arm/mach-at91/Kconfig                                                  |  31 ++++----
>  arch/arm/mach-at91/Makefile                                                 |  14 +---
>  arch/arm/mach-at91/at91sam9260_lowlevel_init.c                              |  48 ------------
>  arch/arm/mach-at91/at91sam9261_lowlevel_init.c                              |  48 ------------
>  arch/arm/mach-at91/at91sam9263_lowlevel_init.c                              |  48 ------------
>  arch/arm/mach-at91/include/mach/at91_lowlevel_init.h                        |  46 -----------
>  .../{at91sam926x_lowlevel_init.c => include/mach/at91sam926x_board_init.h}  | 148 ++++++++++++++++--------------------
>  55 files changed, 422 insertions(+), 756 deletions(-)

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

      parent reply	other threads:[~2018-01-04 17:12 UTC|newest]

Thread overview: 35+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2017-12-31  8:34 Sam Ravnborg
2018-01-01  8:34 ` Mails lost towards barebox mailing list [Was: [PATCH v4 0/25] at91: move ...] Sam Ravnborg
2018-01-01 17:05   ` Sam Ravnborg
2018-01-03 18:19     ` Sam Ravnborg
2018-01-01 14:16 ` [PATCH 02/25] arm: at91: drop unused at91sam9x5_lowlevel_init.c Sam Ravnborg
2018-01-01 14:16 ` [PATCH 03/25] arm: at91: add at91sam926x_board_init.h Sam Ravnborg
2018-01-01 14:16 ` [PATCH 04/25] arm: at91: code cleanup in at91sam926x_board_init Sam Ravnborg
2018-01-01 14:16 ` [PATCH 05/25] at91sam9263ek: move reset vector to board code Sam Ravnborg
2018-01-01 14:16 ` [PATCH 06/25] at91sam9261ek, at91sam9g10ek: " Sam Ravnborg
2018-01-01 14:16 ` [PATCH 07/25] pm9261: " Sam Ravnborg
2018-01-01 14:16 ` [PATCH 08/25] at91: drop unused at91sam9261_lowlevel_init Sam Ravnborg
2018-01-01 14:16 ` [PATCH 09/25] pm9263: move reset vector to board code Sam Ravnborg
2018-01-01 14:18 ` [PATCH 11/25] mmccpu: delete unused lowlevel_init Sam Ravnborg
2018-01-05  9:14   ` Sascha Hauer
2018-01-06  9:33     ` Sam Ravnborg
2018-01-01 14:18 ` [PATCH 12/25] mmccpu: move reset vector to board code Sam Ravnborg
2018-01-01 14:18 ` [PATCH 13/25] tny-a926x: delete unused tny_a9263_lowlevel_init.c Sam Ravnborg
2018-01-01 14:19 ` [PATCH 17/25] sama5d{3, 4}{xek, xplained}: move reset vector to board code Sam Ravnborg
2018-01-05  9:27   ` Sascha Hauer
2018-01-06 20:06     ` Sam Ravnborg
2018-01-01 14:19 ` [PATCH 18/25] at91sam9n12ek: " Sam Ravnborg
2018-01-01 14:19 ` [PATCH 19/25] at91sam9260ek, at91sam9g20ek: " Sam Ravnborg
2018-01-01 14:19 ` [PATCH 20/25] at91sam9m10g45ek, at91sam9m10ihd, pm9g45: " Sam Ravnborg
2018-01-01 14:19 ` [PATCH 21/25] animeo: " Sam Ravnborg
2018-01-01 14:19 ` [PATCH 22/25] telit-evk-pro3: " Sam Ravnborg
2018-01-01 14:19 ` [PATCH 23/25] dss11: " Sam Ravnborg
2018-01-01 14:19 ` [PATCH 24/25] at91rm9200ek: " Sam Ravnborg
2018-01-01 14:19 ` [PATCH 25/25] arm: at91: remove leftovers from moving reset code in mach-at91 Sam Ravnborg
2018-01-01 16:48 ` [PATCH 01/25] arm: at91: refactor lowlevel_init selection Sam Ravnborg
2018-01-05  9:24   ` Sascha Hauer
2018-01-01 16:48 ` [PATCH 10/25] usb-a926x: move reset vector to board code Sam Ravnborg
2018-01-01 16:48 ` [PATCH 14/25] tny-a926x: " Sam Ravnborg
2018-01-01 16:48 ` [PATCH 15/25] qil-a926x: " Sam Ravnborg
2018-01-01 16:48 ` [PATCH 16/25] haba-knx: " Sam Ravnborg
2018-01-04 17:12 ` Andrey Smirnov [this message]

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to='CAHQ1cqGrWvbfY-Y=OouEYKorVLCMspz7uz+Ca7vGrjO+pVCoyQ@mail.gmail.com' \
    --to=andrew.smirnov@gmail.com \
    --cc=barebox@lists.infradead.org \
    --cc=sam@ravnborg.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox