On Sat, Sep 10, 2011 at 09:08:16AM +0530, Shilimkar, Santosh wrote: > IOn Sat, Sep 10, 2011 at 5:04 AM, Shawn Guo <shawn.guo@xxxxxxxxxxxxx> wrote: [...] > > Also, IMO, lable "l2x_clean_inv" should be put after the "bne do_WFI". > > Otherwise, my original statement (it seems l2x_clean_inv will be > > called for case "2") stands correct :) > > > It's just a label. All L2 related code and checks for the valid state is > kept under that by purpose. > So for some case, you enter l2x_clean_inv but do not actually clean and invalidate L2. I do not see this label being used anywhere, so you may want to remove it to save the confusion to stupid people like me. Just my opinion. -- Regards, Shawn -- To unsubscribe from this list: send the line "unsubscribe linux-omap" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
