Re: [PATCH 0/7] omap3: serial code for 3630 and zoom2/3 fixes

2010-03-22 Thread Tony Lindgren
* Sergio Aguirre  [100315 15:59]:
> Hi,
> 
> This series contains fixes for omap2/3/4 serial code, and are
> fixing:
>  - Avoid doing ioremap of a zero-based physical address.
>(causing a kernel panic during early init on 3630boards)
>  - Unproper omap_revision check during uart globals setup.
>(omap_revision is not yet filled at that point)
>  - Don't try to enable all uarts generically for zoom2/3
>and 3630sdp.
>  - Make zoom2/3 just register one serial port (ttyS0).

Thanks, I've pulled this into omap-fixes-for-linus.

Regards,

Tony

 
> This is a subset of the original series found here: [1]
> 
> Thanks to:
>  - Vikram Pandita
>  - Paul Walmsley
>  - Kevin Hilman
>  - Manjunath Kondaiah
>  - Felipe Balbi
>  - Tony Lindgreen
> 
> For all the feedback recieved so far.
> 
> Regards,
> Sergio
> 
> Detailed changelog:
> 
> Sergio Aguirre (7):
>   OMAP3: serial: Check for zero-based physical addr
>   OMAP3: serial: Use dev_* macros instead of printk
>   omap2/3/4: serial: Remove condition for getting uart4_phys
>   omap3: zoom2/3 / 3630sdp: Don't init always all uarts
>   omap3: 3630sdp: Explicitly enable all UARTs
>   omap3: zoom 2/3: Change debugboard serial port id
>   omap3: zoom2/3: Register only 1 8250 port
> 
>  arch/arm/configs/omap_zoom2_defconfig|2 +-
>  arch/arm/configs/omap_zoom3_defconfig|2 +-
>  arch/arm/mach-omap2/board-3630sdp.c  |1 +
>  arch/arm/mach-omap2/board-zoom-debugboard.c  |2 +-
>  arch/arm/mach-omap2/board-zoom-peripherals.c |1 -
>  arch/arm/mach-omap2/serial.c |   35 +
>  6 files changed, 22 insertions(+), 21 deletions(-)
> 
> [1] http://marc.info/?l=linux-omap&m=126826639903105&w=2
> --
> To unsubscribe from this list: send the line "unsubscribe linux-omap" in
> the body of a message to majord...@vger.kernel.org
> More majordomo info at  http://vger.kernel.org/majordomo-info.html
--
To unsubscribe from this list: send the line "unsubscribe linux-omap" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html


RE: [PATCH 0/7] omap3: serial code for 3630 and zoom2/3 fixes

2010-03-15 Thread Aguirre, Sergio
From: Tony Lindgren [mailto:t...@atomide.com]
Sent: Monday, March 15, 2010 5:49 PM



> 
> OK, thanks. One more thing: Please repost your fixes one more
> time with LAKML Cc'd for review. Then assuming no more comments, I'll
> merge them into omap-fixes-for-linus.

Done.

> 
> Meanwhile, I'll merge it into l-o master branch for testing.

Ok, thanks.

Regards,
Sergio

> 
> Tony
--
To unsubscribe from this list: send the line "unsubscribe linux-omap" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html


Re: [PATCH 0/7] omap3: serial code for 3630 and zoom2/3 fixes

2010-03-15 Thread Tony Lindgren
* Aguirre, Sergio  [100315 14:47]:
> 
> 
> From: Tony Lindgren [mailto:t...@atomide.com]
> Sent: Monday, March 15, 2010 4:15 PM
> > * Aguirre, Sergio  [100315 13:33]:
> > > Tony,
> > >
> > > Please find attached pull request for your convenience.
> > 
> > 
> > 
> > > The following changes since commit
> > a842b5f9ce70e1b738eabb4d719860070180ed1c:
> > >   Tony Lindgren (1):
> > > Revert "omap: Add DSI regulator supply to OMAP3EVM board file"
> > >
> > > are available in the git repository at:
> > >
> > >   git://dev.omapzoom.org/pub/scm/saaguirre/linux-omap-camera.git omap-
> > fixes
> > 
> > Please base all the branches against the recent tag in Linus' tree,
> > in this case against v2.6.34-rc1. What we have in the linux-omap master
> > branch is a merge of various development branches, and we don't want
> > to merge that history to the mainline tree.
> > 
> > If the recent mainline tag won't work as the base, then please
> > rebase it on omap-fixes-for-linus or omap-for-linus. In this case,
> > omap-fixes-for-linus at commit 29b2ee5af5f3a02846bd38a1e2121d62ee5f6aca.
> > The omap-for-linus branch will contain things going into the next
> > merge window, currently it's empty.
> > 
> > In general, the commits ins omap-fixes-for-linus and omap-for-linus
> > should stay around unless something goes wrong. But please use
> > the recent mainline tag where possible, that's the most flexible
> > option for merging.
> 
> Understood,
> 
> Please find updated pull request, rebased on omap-fixes-for-linux.
> 
> I tried it on master, but they don't apply clean, since "OMAP2: serial.c: Fix 
> number of uarts in early_init" is not upstream, and there's a small cleanup 
> change in it, which makes patch 0002 don't apply (printk to dev_* patch).

OK, thanks. One more thing: Please repost your fixes one more
time with LAKML Cc'd for review. Then assuming no more comments, I'll
merge them into omap-fixes-for-linus.

Meanwhile, I'll merge it into l-o master branch for testing.

Tony
--
To unsubscribe from this list: send the line "unsubscribe linux-omap" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html


RE: [PATCH 0/7] omap3: serial code for 3630 and zoom2/3 fixes

2010-03-15 Thread Aguirre, Sergio


From: Tony Lindgren [mailto:t...@atomide.com]
Sent: Monday, March 15, 2010 4:15 PM
> * Aguirre, Sergio  [100315 13:33]:
> > Tony,
> >
> > Please find attached pull request for your convenience.
> 
> 
> 
> > The following changes since commit
> a842b5f9ce70e1b738eabb4d719860070180ed1c:
> >   Tony Lindgren (1):
> > Revert "omap: Add DSI regulator supply to OMAP3EVM board file"
> >
> > are available in the git repository at:
> >
> >   git://dev.omapzoom.org/pub/scm/saaguirre/linux-omap-camera.git omap-
> fixes
> 
> Please base all the branches against the recent tag in Linus' tree,
> in this case against v2.6.34-rc1. What we have in the linux-omap master
> branch is a merge of various development branches, and we don't want
> to merge that history to the mainline tree.
> 
> If the recent mainline tag won't work as the base, then please
> rebase it on omap-fixes-for-linus or omap-for-linus. In this case,
> omap-fixes-for-linus at commit 29b2ee5af5f3a02846bd38a1e2121d62ee5f6aca.
> The omap-for-linus branch will contain things going into the next
> merge window, currently it's empty.
> 
> In general, the commits ins omap-fixes-for-linus and omap-for-linus
> should stay around unless something goes wrong. But please use
> the recent mainline tag where possible, that's the most flexible
> option for merging.

Understood,

Please find updated pull request, rebased on omap-fixes-for-linux.

I tried it on master, but they don't apply clean, since "OMAP2: serial.c: Fix 
number of uarts in early_init" is not upstream, and there's a small cleanup 
change in it, which makes patch 0002 don't apply (printk to dev_* patch).

Thanks for your attention.

Regards,
Sergio

> 
> Regards,
> 
> Tony
The following changes since commit 29b2ee5af5f3a02846bd38a1e2121d62ee5f6aca:
  Andrew Clayton (1):
ARM/OMAP: Remove the +x bit from a couple of source files

are available in the git repository at:

  git://dev.omapzoom.org/pub/scm/saaguirre/linux-omap-camera.git 
omap-fixes-for-tony

Sergio Aguirre (7):
  OMAP3: serial: Check for zero-based physical addr
  OMAP3: serial: Use dev_* macros instead of printk
  omap2/3/4: serial: Remove condition for getting uart4_phys
  omap3: zoom2/3 / 3630sdp: Don't init always all uarts
  omap3: 3630sdp: Explicitly enable all UARTs
  omap3: zoom 2/3: Change debugboard serial port id
  omap3: zoom2/3: Register only 1 8250 port

 arch/arm/configs/omap_zoom2_defconfig|2 +-
 arch/arm/configs/omap_zoom3_defconfig|2 +-
 arch/arm/mach-omap2/board-3630sdp.c  |1 +
 arch/arm/mach-omap2/board-zoom-debugboard.c  |2 +-
 arch/arm/mach-omap2/board-zoom-peripherals.c |1 -
 arch/arm/mach-omap2/serial.c |   35 +
 6 files changed, 22 insertions(+), 21 deletions(-)


Re: [PATCH 0/7] omap3: serial code for 3630 and zoom2/3 fixes

2010-03-15 Thread Tony Lindgren
* Aguirre, Sergio  [100315 13:33]:
> Tony,
> 
> Please find attached pull request for your convenience.



> The following changes since commit a842b5f9ce70e1b738eabb4d719860070180ed1c:
>   Tony Lindgren (1):
> Revert "omap: Add DSI regulator supply to OMAP3EVM board file"
> 
> are available in the git repository at:
> 
>   git://dev.omapzoom.org/pub/scm/saaguirre/linux-omap-camera.git omap-fixes

Please base all the branches against the recent tag in Linus' tree,
in this case against v2.6.34-rc1. What we have in the linux-omap master
branch is a merge of various development branches, and we don't want
to merge that history to the mainline tree.

If the recent mainline tag won't work as the base, then please
rebase it on omap-fixes-for-linus or omap-for-linus. In this case,
omap-fixes-for-linus at commit 29b2ee5af5f3a02846bd38a1e2121d62ee5f6aca.
The omap-for-linus branch will contain things going into the next
merge window, currently it's empty.

In general, the commits ins omap-fixes-for-linus and omap-for-linus
should stay around unless something goes wrong. But please use
the recent mainline tag where possible, that's the most flexible
option for merging.

Regards,

Tony
--
To unsubscribe from this list: send the line "unsubscribe linux-omap" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html


RE: [PATCH 0/7] omap3: serial code for 3630 and zoom2/3 fixes

2010-03-15 Thread Aguirre, Sergio
Tony,

Please find attached pull request for your convenience.

Regards,
Sergio

> -Original Message-
> From: Aguirre, Sergio
> Sent: Friday, March 12, 2010 2:00 PM
> To: Tony Lindgren
> Cc: linux-omap@vger.kernel.org; Kevin Hilman; Pandita, Vikram; Paul
> Walmsley; Felipe Balbi; Aguirre, Sergio
> Subject: [PATCH 0/7] omap3: serial code for 3630 and zoom2/3 fixes
> 
> Hi,
> 
> This series contains fixes for omap2/3/4 serial code, and are
> fixing:
>  - Avoid doing ioremapp of a zero-based physical address.
>(causing a kernel panic during early init on 3630boards)
>  - Unproper omap_revision check during uart globals setup.
>(omap_revision is not yet filled at that point)
>  - Don't try to enable all uarts generically for zoom2/3
>and 3630sdp.
>  - Make zoom2/3 just register one serial port (ttyS0).
> 
> This is a subset of the original series found here: [1]
> 
> Thanks to:
>  - Vikram Pandita
>  - Paul Walmsley
>  - Kevin Hilman
>  - Manjunath Kondaiah
>  - Felipe Balbi
>  - Tony Lindgreen
> 
> For all the feedback recieved so far.
> 
> Regards,
> Sergio
> 
> Sergio Aguirre (7):
>   OMAP3: serial: Check for zero-based physical addr
>   OMAP3: serial: Use dev_* macros instead of printk
>   omap2/3/4: serial: Remove condition for getting uart4_phys
>   omap3: zoom2/3 / 3630sdp: Don't init always all uarts
>   omap3: 3630sdp: Explicitly enable all UARTs
>   omap3: zoom 2/3: Change debugboard serial port id
>   omap3: zoom2/3: Register only 1 8250 port
> 
>  arch/arm/configs/omap_zoom2_defconfig|2 +-
>  arch/arm/configs/omap_zoom3_defconfig|2 +-
>  arch/arm/mach-omap2/board-3630sdp.c  |1 +
>  arch/arm/mach-omap2/board-zoom-debugboard.c  |2 +-
>  arch/arm/mach-omap2/board-zoom-peripherals.c |1 -
>  arch/arm/mach-omap2/serial.c |   35 +
> 
>  6 files changed, 22 insertions(+), 21 deletions(-)
> 
> [1] http://marc.info/?l=linux-omap&m=126826639903105&w=2

The following changes since commit a842b5f9ce70e1b738eabb4d719860070180ed1c:
  Tony Lindgren (1):
Revert "omap: Add DSI regulator supply to OMAP3EVM board file"

are available in the git repository at:

  git://dev.omapzoom.org/pub/scm/saaguirre/linux-omap-camera.git omap-fixes

Sergio Aguirre (7):
  OMAP3: serial: Check for zero-based physical addr
  OMAP3: serial: Use dev_* macros instead of printk
  omap2/3/4: serial: Remove condition for getting uart4_phys
  omap3: zoom2/3 / 3630sdp: Don't init always all uarts
  omap3: 3630sdp: Explicitly enable all UARTs
  omap3: zoom 2/3: Change debugboard serial port id
  omap3: zoom2/3: Register only 1 8250 port

 arch/arm/configs/omap_zoom2_defconfig|2 +-
 arch/arm/configs/omap_zoom3_defconfig|2 +-
 arch/arm/mach-omap2/board-3630sdp.c  |1 +
 arch/arm/mach-omap2/board-zoom-debugboard.c  |2 +-
 arch/arm/mach-omap2/board-zoom-peripherals.c |1 -
 arch/arm/mach-omap2/serial.c |   35 +
 6 files changed, 22 insertions(+), 21 deletions(-)