[PATCH] msm: fix debug-macro.S build failure

Daniel Walker dwalker at codeaurora.org
Fri Oct 29 19:07:18 EDT 2010


On Fri, 2010-10-29 at 18:53 -0400, Nicolas Pitre wrote:
> On Fri, 29 Oct 2010, Daniel Walker wrote:
> 
> > On Fri, 2010-10-29 at 18:17 -0400, Nicolas Pitre wrote:
> > 
> > > Could you at least comment the patch I posted for you?
> > 
> > I didn't see anything wrong with it, not from an MSM perspective.
> 
> So now you have two possibilities: my patch, or Russell's suggestions.  
> Personally I find my patch is slightly better in the sense that it 
> doesn't make use of a magic 0xfff00000 value with no obvious semantic.  
> But you have the choice.

Russell's because I think I can get that in right now, which is all I
care about.

> > I'd like to have MSM building before -rc1 , that's why I was suggesting
> > I go with my original revert then we would fix this issue during the -rc
> > cycle.
> 
> y
> Your "fix" is just as broken as the current compilation problem.  Sure 
> with your revert you may compile a kernel, but that kernel may as well 
> crash at run time instead.

The build issue is a little more dire I think, I never suggested my
revert fixed anything but the build issue (which was the point) .. It
was just suppose to bring back what was wrongfully removed.

Daniel

-- 
Sent by an consultant of the Qualcomm Innovation Center, Inc.
The Qualcomm Innovation Center, Inc. is a member of the Code Aurora
Forum.





More information about the linux-arm-kernel mailing list