From: Biju Das <biju.das.jz@bp.renesas.com>
To: Andy Shevchenko <andriy.shevchenko@linux.intel.com>,
Daniel Scally <djrscally@gmail.com>,
Heikki Krogerus <heikki.krogerus@linux.intel.com>,
Sakari Ailus <sakari.ailus@linux.intel.com>,
Greg Kroah-Hartman <gregkh@linuxfoundation.org>,
Alexandre Belloni <alexandre.belloni@bootlin.com>,
Jonathan Cameron <jic23@kernel.org>,
"Rafael J. Wysocki" <rafael@kernel.org>
Cc: Biju Das <biju.das.jz@bp.renesas.com>,
linux-acpi@vger.kernel.org,
Dmitry Torokhov <dmitry.torokhov@gmail.com>,
Andi Shyti <andi.shyti@kernel.org>, Wolfram Sang <wsa@kernel.org>,
Geert Uytterhoeven <geert+renesas@glider.be>,
linux-rtc@vger.kernel.org, linux-iio@vger.kernel.org,
linux-i2c@vger.kernel.org, linux-renesas-soc@vger.kernel.org
Subject: [PATCH v7 0/4] Extend device_get_match_data() to struct bus_type
Date: Fri, 4 Aug 2023 17:17:24 +0100 [thread overview]
Message-ID: <20230804161728.394920-1-biju.das.jz@bp.renesas.com> (raw)
This patch series extend device_get_match_data() to struct bus_type,
so that buses like I2C can get matched data.
There is a plan to replace i2c_get_match_data()->device_get_match_data()
later, once this patch hits mainline as it is redundant.
v6->v7:
* Added ack from Greg Kroah-Hartman for patch#1
* Swapped patch#2 and patch#3 from v6.
* Added Rb tag from Andy for patch#2 and patch#4
* Updated commit description of patch#2 by removing unnecessary wrapping.
* Updated typo in commit description struct bus_type()->struct bus_type.
v5->v6:
* Cced linux-rtc and linux-iio as these subsytems uses i2c_get_match_
data() and this function become redundant once this patch series hits
mainline.
* Added Rb tag from Sakari for patch#1.
* Moved patch#3 from v5 to patch#2 and patch#2 from v5 to patch#4.
* Added Rb tag from Andy for patch#2
* Separate patch#3 to prepare for better difference for
i2c_match_id() changes.
* Merged patch#4 from v5 with patch#4.
v4->v5:
* Added const struct device_driver variable 'drv' in i2c_device_get_match
_data().
* For code readability and maintenance perspective, added separate NULL
check for drv and client variable and added comment for NULL check for
drv variable.
* Created separate patch for converting i2c_of_match_device_sysfs() to
non-static.
* Removed export symbol for i2c_of_match_device_sysfs().
* Replaced 'dev->driver'->'drv'.
* Replaced return value data->NULL to avoid (potentially) stale pointers,
if there is no match.
v3->v4:
* Documented corner case for device_get_match_data()
* Dropped struct i2c_driver parameter from i2c_get_match_data_helper()
* Split I2C sysfs handling in separate patch(patch#3)
* Added space after of_device_id for i2c_of_match_device_sysfs()
* Added const parameter for struct i2c_client, to prevent overriding it's
pointer.
* Moved declaration from public i2c.h->i2c-core.h
v2->v3:
* Added Rb tag from Andy for patch#1.
* Extended to support i2c_of_match_device() as suggested by Andy.
* Changed i2c_of_match_device_sysfs() as non-static function as it is
needed for i2c_device_get_match_data().
* Added a TODO comment to use i2c_verify_client() when it accepts const
pointer.
* Added multiple returns to make code path for device_get_match_data()
faster in i2c_get_match_data().
RFC v1->v2:
* Replaced "Signed-off-by"->"Suggested-by" tag for Dmitry.
* Documented device_get_match_data().
* Added multiple returns to make code path for generic fwnode-based
lookup faster.
* Fixed build warnings reported by kernel test robot <lkp@intel.com>
* Added const qualifier to return type and parameter struct i2c_driver
in i2c_get_match_data_helper().
* Added const qualifier to struct i2c_driver in i2c_get_match_data()
* Dropped driver variable from i2c_device_get_match_data()
* Replaced to_i2c_client with logic for assigning verify_client as it
returns non const pointer.
Biju Das (4):
drivers: fwnode: Extend device_get_match_data() to struct bus_type
i2c: Enhance i2c_get_match_data()
i2c: i2c-core-of: Convert i2c_of_match_device_sysfs() to non-static
i2c: Add i2c_device_get_match_data() callback
drivers/base/property.c | 27 ++++++++++++++++-
drivers/i2c/i2c-core-base.c | 60 ++++++++++++++++++++++++++++++-------
drivers/i2c/i2c-core-of.c | 4 +--
drivers/i2c/i2c-core.h | 9 ++++++
include/linux/device/bus.h | 3 ++
5 files changed, 90 insertions(+), 13 deletions(-)
--
2.25.1
next reply other threads:[~2023-08-04 16:17 UTC|newest]
Thread overview: 29+ messages / expand[flat|nested] mbox.gz Atom feed top
2023-08-04 16:17 Biju Das [this message]
2023-08-04 16:17 ` [PATCH v7 1/4] drivers: fwnode: Extend device_get_match_data() to struct bus_type Biju Das
2023-08-04 16:17 ` [PATCH v7 2/4] i2c: Enhance i2c_get_match_data() Biju Das
2023-08-04 16:17 ` [PATCH v7 3/4] i2c: i2c-core-of: Convert i2c_of_match_device_sysfs() to non-static Biju Das
2023-08-04 16:17 ` [PATCH v7 4/4] i2c: Add i2c_device_get_match_data() callback Biju Das
2023-08-05 16:40 ` [PATCH v7 0/4] Extend device_get_match_data() to struct bus_type Jonathan Cameron
2023-08-05 17:42 ` Biju Das
2023-08-06 13:29 ` Jonathan Cameron
2023-08-06 16:27 ` Biju Das
2023-08-07 14:54 ` Andy Shevchenko
2023-08-07 19:45 ` Jonathan Cameron
2023-08-08 12:14 ` Andy Shevchenko
2023-08-07 20:37 ` Dmitry Torokhov
2023-08-08 12:18 ` Andy Shevchenko
[not found] ` <20230809182551.7eca502e@jic23-huawei>
2023-08-10 9:05 ` Biju Das
2023-08-10 15:11 ` Andy Shevchenko
2023-08-11 13:27 ` Biju Das
2023-08-11 14:30 ` Andy Shevchenko
2023-08-11 14:46 ` Biju Das
2023-08-14 13:12 ` Geert Uytterhoeven
2023-08-14 13:17 ` Biju Das
2023-08-28 13:01 ` Jonathan Cameron
2023-08-15 6:44 ` Andy Shevchenko
2023-08-15 6:58 ` Biju Das
2023-08-15 7:06 ` Biju Das
2023-08-15 9:42 ` Andy Shevchenko
2023-08-10 15:04 ` Andy Shevchenko
2023-08-08 15:16 ` Jonathan Cameron
2023-08-14 13:01 ` Geert Uytterhoeven
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20230804161728.394920-1-biju.das.jz@bp.renesas.com \
--to=biju.das.jz@bp.renesas.com \
--cc=alexandre.belloni@bootlin.com \
--cc=andi.shyti@kernel.org \
--cc=andriy.shevchenko@linux.intel.com \
--cc=djrscally@gmail.com \
--cc=dmitry.torokhov@gmail.com \
--cc=geert+renesas@glider.be \
--cc=gregkh@linuxfoundation.org \
--cc=heikki.krogerus@linux.intel.com \
--cc=jic23@kernel.org \
--cc=linux-acpi@vger.kernel.org \
--cc=linux-i2c@vger.kernel.org \
--cc=linux-iio@vger.kernel.org \
--cc=linux-renesas-soc@vger.kernel.org \
--cc=linux-rtc@vger.kernel.org \
--cc=rafael@kernel.org \
--cc=sakari.ailus@linux.intel.com \
--cc=wsa@kernel.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).