[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <20210118094859.GE1018@ninjato>
Date: Mon, 18 Jan 2021 10:48:59 +0100
From: Wolfram Sang <wsa@...nel.org>
To: Sakari Ailus <sakari.ailus@...ux.intel.com>
Cc: Daniel Scally <djrscally@...il.com>, linux-kernel@...r.kernel.org,
linux-acpi@...r.kernel.org, linux-gpio@...r.kernel.org,
linux-i2c@...r.kernel.org, platform-driver-x86@...r.kernel.org,
devel@...ica.org, rjw@...ysocki.net, lenb@...nel.org,
andy@...nel.org, mika.westerberg@...ux.intel.com,
linus.walleij@...aro.org, bgolaszewski@...libre.com,
lee.jones@...aro.org, hdegoede@...hat.com, mgross@...ux.intel.com,
robert.moore@...el.com, erik.kaneda@...el.com,
andriy.shevchenko@...ux.intel.com,
laurent.pinchart@...asonboard.com, kieran.bingham@...asonboard.com
Subject: Re: [PATCH v2 3/7] i2c: i2c-core-base: Use format macro in
i2c_dev_set_name()
> > if (adev) {
> > - dev_set_name(&client->dev, "i2c-%s", acpi_dev_name(adev));
> > + dev_set_name(&client->dev, I2C_DEV_NAME_FORMAT, acpi_dev_name(adev));
>
> Over 80, please wrap.
I am not that strict with the 80, and I don't think it will improve
readability here. It can stay this time IMO.
Download attachment "signature.asc" of type "application/pgp-signature" (834 bytes)
Powered by blists - more mailing lists