On 08/03/2015 06:42 PM, Sergei Shtylyov wrote:

    Please use pr_err() instead. And "0x%d" makes no sense at all, please
use "%#x" instead.

Yeah, not sure what I was drinking before writing this 0x%d thing...

Regarding the pr_err() - it is not used at all in this file, and
printk() is used instead.

Didn't notice before: the patch subject should start with the driver name and a colon, "sats_sx4: ".

--Tomer

MBR, Sergei

--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

Reply via email to