Patchwork [0/4] arm: vt8500: Add support for Wondermedia WM8750/WM8850

login
register
mail settings
Submitter Tony Prisk
Date Jan. 11, 2013, 8:12 p.m.
Message ID <1357935162-7672-1-git-send-email-linux@prisktech.co.nz>
Download mbox
Permalink /patch/211415/
State New
Headers show

Pull-request

git://server.prisktech.co.nz/git/linuxwmt.git tags/armsoc-3.9

Comments

Tony Prisk - Jan. 11, 2013, 8:12 p.m.
The following changes since commit a49f0d1ea3ec94fc7cf33a7c36a16343b74bd565:

  Linux 3.8-rc1 (2012-12-21 17:19:00 -0800)

are available in the git repository at:

  git://server.prisktech.co.nz/git/linuxwmt.git tags/armsoc-3.9

for you to fetch changes up to 82c8f175662873f7f7f4c78698888ee963d7d967:

  arm: vt8500: Remove remaining mach includes (2012-12-28 21:05:19 +1300)

----------------------------------------------------------------
arm: vt8500: Add support for WM8750/WM8850. Cleanup multiplatform changes

This patchset adds support for the WM8750 (ARMv6) and WM8850 (ARMv7), and
cleans up changes for multiplatform configuration.

Single platform Kconfig options are removed, along with the remaining
mach/includes.

The debug-macro.s is moved to arm/include/debug/ to allow DEBUG_LL on
multi-platform.

Signed-off-by: Tony Prisk <linux@prisktech.co.nz>

----------------------------------------------------------------
Tony Prisk (4):
      arm: vt8500: Add support for Wondermedia WM8750/WM8850
      arm: vt8500: Remove single platform Kconfig options
      arm: vt8500: Convert debug-macro.S to be multiplatform friendly
      arm: vt8500: Remove remaining mach includes

 Documentation/devicetree/bindings/arm/vt8500.txt   |    8 +++++
 arch/arm/Kconfig                                   |   16 ---------
 arch/arm/Kconfig.debug                             |    8 +++++
 .../mach/debug-macro.S => include/debug/vt8500.S}  |   24 ++++++++-----
 arch/arm/mach-vt8500/Kconfig                       |   26 ++++++++++++--
 arch/arm/mach-vt8500/include/mach/timex.h          |   26 --------------
 arch/arm/mach-vt8500/include/mach/uncompress.h     |   37 --------------------
 arch/arm/mach-vt8500/vt8500.c                      |    2 ++
 8 files changed, 56 insertions(+), 91 deletions(-)
 rename arch/arm/{mach-vt8500/include/mach/debug-macro.S => include/debug/vt8500.S} (56%)
 delete mode 100644 arch/arm/mach-vt8500/include/mach/timex.h
 delete mode 100644 arch/arm/mach-vt8500/include/mach/uncompress.h
Olof Johansson - Jan. 11, 2013, 9:56 p.m.
Hi,

On Sat, Jan 12, 2013 at 09:12:38AM +1300, Tony Prisk wrote:
> The following changes since commit a49f0d1ea3ec94fc7cf33a7c36a16343b74bd565:
> 
>   Linux 3.8-rc1 (2012-12-21 17:19:00 -0800)
> 
> are available in the git repository at:
> 
>   git://server.prisktech.co.nz/git/linuxwmt.git tags/armsoc-3.9
> 
> for you to fetch changes up to 82c8f175662873f7f7f4c78698888ee963d7d967:
> 
>   arm: vt8500: Remove remaining mach includes (2012-12-28 21:05:19 +1300)
> 
> ----------------------------------------------------------------
> arm: vt8500: Add support for WM8750/WM8850. Cleanup multiplatform changes
> 
> This patchset adds support for the WM8750 (ARMv6) and WM8850 (ARMv7), and
> cleans up changes for multiplatform configuration.
> 
> Single platform Kconfig options are removed, along with the remaining
> mach/includes.
> 
> The debug-macro.s is moved to arm/include/debug/ to allow DEBUG_LL on
> multi-platform.
> 
> Signed-off-by: Tony Prisk <linux@prisktech.co.nz>
> 
> ----------------------------------------------------------------
> Tony Prisk (4):
>       arm: vt8500: Add support for Wondermedia WM8750/WM8850
>       arm: vt8500: Remove single platform Kconfig options
>       arm: vt8500: Convert debug-macro.S to be multiplatform friendly
>       arm: vt8500: Remove remaining mach includes

The first patch adds some Kconfig entries, and the second one removes them.
Also, all but the first patch are really about multiplatform enablement.

I suggest that you split off the last three (and rebase them to be
independent), and apply them to a multiplatform branch for vt8500. Then the
last one is a soc branch that goes on top of the multiplatform branch (i.e. you
build it on top of multiplatform). That way we can pull in the multiplatform
changes together with those for other platforms, and then take the wm8{7,8}50
branch goes in with the other new-soc support from other platforms.

Does that make sense? I'll be happy to provide more explanation if it doesn't.

-Olof
Tony Prisk - Jan. 12, 2013, 3:07 a.m.
> I suggest that you split off the last three (and rebase them to be
> independent), and apply them to a multiplatform branch for vt8500. Then the
> last one is a soc branch that goes on top of the multiplatform branch (i.e. you
> build it on top of multiplatform). That way we can pull in the multiplatform
> changes together with those for other platforms, and then take the wm8{7,8}50
> branch goes in with the other new-soc support from other platforms.
> 
> Does that make sense? I'll be happy to provide more explanation if it doesn't.
> 
> -Olof

I think I understand - pull requests to follow.

Tony P
Olof Johansson - Jan. 12, 2013, 5:16 p.m.
On Sat, Jan 12, 2013 at 04:07:24PM +1300, Tony Prisk wrote:
> > I suggest that you split off the last three (and rebase them to be
> > independent), and apply them to a multiplatform branch for vt8500. Then the
> > last one is a soc branch that goes on top of the multiplatform branch (i.e. you
> > build it on top of multiplatform). That way we can pull in the multiplatform
> > changes together with those for other platforms, and then take the wm8{7,8}50
> > branch goes in with the other new-soc support from other platforms.
> > 
> > Does that make sense? I'll be happy to provide more explanation if it doesn't.
> > 
> > -Olof
> 
> I think I understand - pull requests to follow.

Looks good, thanks for fixing it up. Next time feel free to change to
fresh subjects on the pull requests. :)

I don't see your PGP key uploaded to the public key servers, and you
likely don't have it signed by anyone else. If you have a chance, please
try to get it signed by a few people. Ideal time to do that is if you're
going to any conferences, but there are also local people around the
world that can meet up sometimes.


-Olof