lists.openwall.net | lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening PHC | |
Open Source and information security mailing list archives
| ||
|
Date: Thu, 30 Apr 2020 18:18:18 -0700 From: Joe Perches <joe@...ches.com> To: Guru Das Srinagesh <gurus@...eaurora.org>, Lee Jones <lee.jones@...aro.org> Cc: devicetree@...r.kernel.org, linux-arm-msm <linux-arm-msm@...r.kernel.org>, Rob Herring <robh+dt@...nel.org>, Subbaraman Narayanamurthy <subbaram@...eaurora.org>, David Collins <collinsd@...eaurora.org>, linux-kernel@...r.kernel.org Subject: Re: [PATCH v1 2/2] mfd: Introduce QTI I2C PMIC controller On Thu, 2020-04-30 at 18:13 -0700, Guru Das Srinagesh wrote: > On Wed, Apr 29, 2020 at 08:50:10AM +0100, Lee Jones wrote: > > On Tue, 28 Apr 2020, Guru Das Srinagesh wrote: > > > The Qualcomm Technologies, Inc. I2C PMIC Controller is used by > > > multi-function PMIC devices which communicate over the I2C bus. The > > > controller enumerates all child nodes as platform devices, and > > > instantiates a regmap interface for them to communicate over the I2C > > > bus. [] > > > diff --git a/drivers/mfd/qcom-i2c-pmic.c b/drivers/mfd/qcom-i2c-pmic.c [] > > Please don't role your own debug helpers. > > > > The ones the kernel provides are suitably proficient. > > Sure. Would this be acceptable instead, with the custom string replaced by a > macro that the kernel provides? > > #define pr_fmt(fmt) "%s: %s: " fmt, KBUILD_MODNAME, __func__ trivia: It's almost always smaller object code to use the KBUILD_MODNAME as a fixed string instead of as a printf argument. #define pr_fmt(fmt) KBUILD_MODNAME ": %s: " fmt, __func__
Powered by blists - more mailing lists