On Fri 2010-03-26 16:55:19, Daniel Walker wrote:
> This cleans up codeing style. There are no run time changes.
>
> Signed-off-by: Daniel Walker <dwalker@xxxxxxxxxxxxxx>
ACK.
> @@ -904,9 +902,9 @@ static irqreturn_t smsm_irq_handler(int irq, void *data)
>
> if (msm_smd_debug_mask & MSM_SMSM_DEBUG)
> pr_info("<SM %08x %08x>\n", apps, modm);
> - if (modm & SMSM_RESET) {
> + if (modm & SMSM_RESET)
> handle_modem_crash();
> - }
> +
> do_smd_probe();
>
> spin_unlock_irqrestore(&smem_lock, flags);
Even through I do not think this is an improvement. Especially
considering the if() immediately before that.
Pavel
--
(english) http://www.livejournal.com/~pavelmachek
(cesky, pictures) http://atrey.karlin.mff.cuni.cz/~pavel/picture/horses/blog.html
--
To unsubscribe from this list: send the line "unsubscribe linux-arm-msm" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
