Skip to content

OMAP3/4: Misc fixes and clean-up

Santosh Shilimkar linux-arm-kernel linux-omap 1 version New

Get series with b4
b4 am [email protected]

Versions newest first

  1. v1 Latest0 of 8 patches 0 repliesNew

Patches in v1 No applies recorded yet

None of the 8 patches the cover letter announces are in this archive.

Reviews trailers per patch

No Reviewed-by, Acked-by or Tested-by trailers for v1 yet.

Read the v1 thread

Cover letter [PATCH 0/8] OMAP3/4: Misc fixes and clean-up
Quoted 19 lines · Paul Walmsley

Hi

Quoted 15 lines · Santosh Shilimkar
Quoted 14 lines · Paul Walmsley

Hi Santosh

Quoted 7 lines · Santosh Shilimkar
Quoted 1 line · Santosh Shilimkar

I need your ack on below two patches so that I can re-base them and send a pull request to Tony.

OMAP4: clock: Add CPU local timer clock node.
OMAP: Fix sparse warnings in l3 error handler.

Are you targeting these for 3.1-rc ?

Quoted 8 lines
Quoted 3 lines

Nope. It's for 3.2 merge window.
As such, the series doesn't have any major regression fixes which should make it before 3.2

In that case, patches 1 and 7 should go in through me. The rest should go via Tony. So you can just drop 1 and 7 from your branch, I'll pull those off the list.

Will do.

Quoted 4 lines

Some comments though: Looks like you need to repost patch 1 due to comments from Beno?t? Also, have the autogeneration scripts been updated for patch 7?

Ok. Will post updated patch1.
For the patch 7, I will align with Benoit on the autogen script. It has some differences with mainline OMAP4 clock data file.

Quoted 9 lines

...

As far as patch 5 goes, you don't need my ack on that one, since I'm not the maintainer or original author of that code. But feel free to add a Reviewed-by:. I also tested the two L3 files with sparse and they are clean now. Thanks very much for taking the initiative on this; I personally appreciate it quite a bit, and I think having clean code increases our credibility with upstream maintainers.

Actually I wanted you to have a look to ensure that we addressed the all sparce warnings. Will add your reviewed by on that patch.

Couldn't agree more on the clean code remark.

Regards
Santosh