public inbox for linux-arm-kernel@lists.infradead.org 
 help / color / mirror / Atom feed
From: Shashank Balaji <shashank.mahadasyam@sony•com>
To: "Suzuki K Poulose" <suzuki.poulose@arm•com>,
	"James Clark" <james.clark@linaro•org>,
	"Alexander Shishkin" <alexander.shishkin@linux•intel.com>,
	"Greg Kroah-Hartman" <gregkh@linuxfoundation•org>,
	"Rafael J. Wysocki" <rafael@kernel•org>,
	"Danilo Krummrich" <dakr@kernel•org>,
	"Miguel Ojeda" <ojeda@kernel•org>,
	"Boqun Feng" <boqun@kernel•org>, "Gary Guo" <gary@garyguo•net>,
	"Björn Roy Baron" <bjorn3_gh@protonmail•com>,
	"Benno Lossin" <lossin@kernel•org>,
	"Andreas Hindborg" <a.hindborg@kernel•org>,
	"Alice Ryhl" <aliceryhl@google•com>,
	"Trevor Gross" <tmgross@umich•edu>,
	"Jonathan Corbet" <corbet@lwn•net>,
	"Shuah Khan" <skhan@linuxfoundation•org>,
	"Luis Chamberlain" <mcgrof@kernel•org>,
	"Petr Pavlu" <petr.pavlu@suse•com>,
	"Daniel Gomez" <da.gomez@kernel•org>,
	"Sami Tolvanen" <samitolvanen@google•com>,
	"Aaron Tomlin" <atomlin@atomlin•com>,
	"Mike Leach" <mike.leach@arm•com>, "Leo Yan" <leo.yan@arm•com>,
	"Thierry Reding" <thierry.reding@kernel•org>,
	"Jonathan Hunter" <jonathanh@nvidia•com>,
	"Mike Leach" <mike.leach@arm•com>
Cc: Rahul Bukte <rahul.bukte@sony•com>,
	 Shashank Balaji <shashank.mahadasyam@sony•com>,
	 linux-kernel@vger•kernel.org, coresight@lists•linaro.org,
	 linux-arm-kernel@lists•infradead.org,
	driver-core@lists•linux.dev,  rust-for-linux@vger•kernel.org,
	linux-doc@vger•kernel.org,
	 Daniel Palmer <daniel.palmer@sony•com>,
	Tim Bird <tim.bird@sony•com>,
	 linux-modules@vger•kernel.org, linux-tegra@vger•kernel.org,
	 Sumit Gupta <sumitg@nvidia•com>
Subject: [PATCH v5 0/4] Enable sysfs module symlink for more built-in drivers
Date: Mon, 18 May 2026 19:19:56 +0900	[thread overview]
Message-ID: <20260518-acpi_mod_name-v5-0-705ccc430885@sony.com> (raw)
In-Reply-To: <20260427-acpi_mod_name-v4-0-22b42240c9bf@sony.com>

struct device_driver's mod_name is not set by a number of bus' driver registration
functions. Without that, built-in drivers don't have the module symlink in sysfs.
We want this to go from unbound driver name -> module name -> kernel config name.
This is useful on embedded platforms to minimize kernel config, reduce kernel size,
and reduce boot time.

In order to achieve this, mod_name has to be set to KBUILD_MODNAME, and this has
to be done for all buses which don't yet do this.

Here are some treewide stats:
- 110 registration functions across all bus types
- 20 of them set mod_name
- Remaining 90 do not set mod_name:
    1. 36 functions under pattern 1:
        They have a __register function + register macro. KBUILD_MODNAME needs to
        be passed and the function needs to take mod_name as input.
    2. 42 functions under pattern 2:
        These have no macro wrapper. They need a double-underscore rename + macro
        wrapper to make them similar to pattern 1.
    3. Remaining 12 do not have such a clean registration interface. More analysis
       is required.

We plan to start with pattern 1, since it's the easiest category of changes.
Within that, for now we're only sending the platform patch. If we get the go-ahead
on that, we'll send the remaining ones.

Patch 4 depends on patches 1, 2, and 3.

Co-developed-by: Rahul Bukte <rahul.bukte@sony•com>
Signed-off-by: Rahul Bukte <rahul.bukte@sony•com>
Signed-off-by: Shashank Balaji <shashank.mahadasyam@sony•com>
---
Changes in v5:
- Move tegra cbb driver init to core_initcall and module_kset init to pure_initcall (Gary)
- Rename coresight_init_driver() to coresight_init_driver_with_owner(), and
  declare function prototype before use (Leo)
- Merge doc change patch with code change patch, so that __platform_register_drivers()
  prototype is changed everywhere in the same patch
- Link to v4: https://patch.msgid.link/20260427-acpi_mod_name-v4-0-22b42240c9bf@sony.com

Changes in v4:
- Initialize module_kset in do_basic_setup() before do_initcalls() (Gary)
- Add commit body to the documentation patch (Greg)
- Link to v3: https://patch.msgid.link/20260422-acpi_mod_name-v3-0-a184eff9ff6f@sony.com

Changes in v3:
- Initialize module_kset on-demand (Greg)
- Make coresight driver registration happen through a macro (Greg)
- Split up the patch adding mod_name to platform driver registrations (Greg)
- Link to v2: https://patch.msgid.link/20260421-acpi_mod_name-v2-0-e73f9310dad3@sony.com

Changes in v2:
- Drop acpi patch, send platform instead (Rafael)
- Link to v1: https://patch.msgid.link/20260416-acpi_mod_name-v1-0-1a4d96fd86c9@sony.com

To: Suzuki K Poulose <suzuki.poulose@arm•com>
To: Mike Leach <mike.leach@linaro•org>
To: James Clark <james.clark@linaro•org>
To: Alexander Shishkin <alexander.shishkin@linux•intel.com>
To: Greg Kroah-Hartman <gregkh@linuxfoundation•org>
To: "Rafael J. Wysocki" <rafael@kernel•org>
To: Danilo Krummrich <dakr@kernel•org>
To: Miguel Ojeda <ojeda@kernel•org>
To: Boqun Feng <boqun@kernel•org>
To: Gary Guo <gary@garyguo•net>
To: Björn Roy Baron <bjorn3_gh@protonmail•com>
To: Benno Lossin <lossin@kernel•org>
To: Andreas Hindborg <a.hindborg@kernel•org>
To: Alice Ryhl <aliceryhl@google•com>
To: Trevor Gross <tmgross@umich•edu>
To: Jonathan Corbet <corbet@lwn•net>
To: Shuah Khan <skhan@linuxfoundation•org>
To: Luis Chamberlain <mcgrof@kernel•org>
To: Petr Pavlu <petr.pavlu@suse•com>
To: Daniel Gomez <da.gomez@kernel•org>
To: Sami Tolvanen <samitolvanen@google•com>
To: Aaron Tomlin <atomlin@atomlin•com>
To: Mike Leach <mike.leach@arm•com>
To: Leo Yan <leo.yan@arm•com>
To: Thierry Reding <thierry.reding@kernel•org>
To: Jonathan Hunter <jonathanh@nvidia•com>
Cc: linux-kernel@vger•kernel.org
Cc: coresight@lists•linaro.org
Cc: linux-arm-kernel@lists•infradead.org
Cc: driver-core@lists•linux.dev
Cc: rust-for-linux@vger•kernel.org
Cc: linux-doc@vger•kernel.org
Cc: Shashank Balaji <shashank.mahadasyam@sony•com>
Cc: Rahul Bukte <rahul.bukte@sony•com>
Cc: Daniel Palmer <daniel.palmer@sony•com>
Cc: Tim Bird <tim.bird@sony•com>
Cc: linux-modules@vger•kernel.org
Cc: linux-tegra@vger•kernel.org
Cc: Sumit Gupta <sumitg@nvidia•com>

---
Shashank Balaji (4):
      soc/tegra: cbb: Move driver registration from pure_initcall to core_initcall
      kernel: param: initialize module_kset in a pure_initcall
      coresight: pass THIS_MODULE implicitly through a macro
      driver core: platform: set mod_name in driver registration

 Documentation/driver-api/driver-model/platform.rst |  3 ++-
 drivers/base/platform.c                            | 21 ++++++++++++++-------
 drivers/hwtracing/coresight/coresight-catu.c       |  2 +-
 drivers/hwtracing/coresight/coresight-core.c       |  9 +++++----
 drivers/hwtracing/coresight/coresight-cpu-debug.c  |  3 +--
 drivers/hwtracing/coresight/coresight-funnel.c     |  3 +--
 drivers/hwtracing/coresight/coresight-replicator.c |  3 +--
 drivers/hwtracing/coresight/coresight-stm.c        |  2 +-
 drivers/hwtracing/coresight/coresight-tmc-core.c   |  2 +-
 drivers/hwtracing/coresight/coresight-tnoc.c       |  2 +-
 drivers/hwtracing/coresight/coresight-tpdm.c       |  3 +--
 drivers/hwtracing/coresight/coresight-tpiu.c       |  2 +-
 drivers/soc/tegra/cbb/tegra194-cbb.c               |  2 +-
 drivers/soc/tegra/cbb/tegra234-cbb.c               |  2 +-
 include/linux/coresight.h                          |  7 +++++--
 include/linux/platform_device.h                    | 17 +++++++++--------
 kernel/params.c                                    |  2 +-
 rust/kernel/platform.rs                            |  4 +++-
 18 files changed, 50 insertions(+), 39 deletions(-)
---
base-commit: 5200f5f493f79f14bbdc349e402a40dfb32f23c8
change-id: 20260416-acpi_mod_name-f645a76e337b

Best regards,
--  
Shashank Balaji <shashank.mahadasyam@sony•com>



  parent reply	other threads:[~2026-05-18 10:20 UTC|newest]

Thread overview: 45+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2026-04-21  6:02 [PATCH v2 0/2] Enable sysfs module symlink for more built-in drivers Shashank Balaji
2026-04-21  6:02 ` [PATCH v2 1/2] kernel: param: handle NULL module_kset in lookup_or_create_module_kobject() Shashank Balaji
2026-04-21  6:27   ` Greg Kroah-Hartman
2026-04-21 14:59     ` Shashank Balaji
2026-04-21 15:20       ` Greg Kroah-Hartman
2026-04-21  6:02 ` [PATCH v2 2/2] driver core: platform: set mod_name in driver registration Shashank Balaji
2026-04-21  6:24   ` Greg Kroah-Hartman
2026-04-22  9:49 ` [PATCH v3 0/4] Enable sysfs module symlink for more built-in drivers Shashank Balaji
2026-04-22  9:49   ` [PATCH v3 1/4] kernel: param: initialize module_kset on-demand Shashank Balaji
2026-04-24  9:16     ` Shashank Balaji
2026-04-24 19:26     ` Gary Guo
2026-04-22  9:49   ` [PATCH v3 2/4] coresight: pass THIS_MODULE implicitly through a macro Shashank Balaji
2026-04-22  9:49   ` [PATCH v3 3/4] driver core: platform: set mod_name in driver registration Shashank Balaji
2026-04-22  9:49   ` [PATCH v3 4/4] docs: driver-api: add mod_name argument to __platform_register_drivers() Shashank Balaji
2026-04-22 11:27     ` Greg Kroah-Hartman
2026-04-27  2:41   ` [PATCH v4 0/4] Enable sysfs module symlink for more built-in drivers Shashank Balaji
2026-04-27  2:41     ` [PATCH v4 1/4] kernel: param: initialize module_kset before do_initcalls() Shashank Balaji
2026-04-27 13:29       ` Gary Guo
2026-04-28  0:37         ` Shashank Balaji
2026-04-28 11:10           ` Gary Guo
2026-04-28 13:07             ` Shashank Balaji
2026-05-12  2:12               ` Shashank Balaji
2026-05-12  8:55                 ` Jon Hunter
2026-05-12 12:14                   ` Sumit Gupta
2026-05-12 13:32                     ` Shashank Balaji
2026-04-27  2:41     ` [PATCH v4 2/4] coresight: pass THIS_MODULE implicitly through a macro Shashank Balaji
2026-04-27 10:49       ` Leo Yan
2026-04-27  2:41     ` [PATCH v4 3/4] driver core: platform: set mod_name in driver registration Shashank Balaji
2026-04-27  2:41     ` [PATCH v4 4/4] docs: driver-api: add mod_name argument to __platform_register_drivers() Shashank Balaji
2026-05-18 10:19     ` Shashank Balaji [this message]
2026-05-18 10:19       ` [PATCH v5 1/4] soc/tegra: cbb: Move driver registration from pure_initcall to core_initcall Shashank Balaji
2026-05-27 11:19         ` Gary Guo
2026-05-28 21:09         ` Thierry Reding
2026-05-18 10:19       ` [PATCH v5 2/4] kernel: param: initialize module_kset in a pure_initcall Shashank Balaji
2026-05-22 13:06         ` Petr Pavlu
2026-05-23  3:24           ` Shashank Balaji
2026-06-01 10:19             ` [PATCH v6] " Shashank Balaji
2026-06-01 15:48               ` Danilo Krummrich
2026-05-27 11:19         ` [PATCH v5 2/4] " Gary Guo
2026-05-18 10:19       ` [PATCH v5 3/4] coresight: pass THIS_MODULE implicitly through a macro Shashank Balaji
2026-05-18 10:20       ` [PATCH v5 4/4] driver core: platform: set mod_name in driver registration Shashank Balaji
2026-05-18 10:27         ` Greg Kroah-Hartman
2026-05-18 10:32           ` Shashank Balaji
2026-05-18 10:49             ` Greg Kroah-Hartman
2026-05-28 21:19       ` [PATCH v5 0/4] Enable sysfs module symlink for more built-in drivers Danilo Krummrich

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=20260518-acpi_mod_name-v5-0-705ccc430885@sony.com \
    --to=shashank.mahadasyam@sony$(echo .)com \
    --cc=a.hindborg@kernel$(echo .)org \
    --cc=alexander.shishkin@linux$(echo .)intel.com \
    --cc=aliceryhl@google$(echo .)com \
    --cc=atomlin@atomlin$(echo .)com \
    --cc=bjorn3_gh@protonmail$(echo .)com \
    --cc=boqun@kernel$(echo .)org \
    --cc=corbet@lwn$(echo .)net \
    --cc=coresight@lists$(echo .)linaro.org \
    --cc=da.gomez@kernel$(echo .)org \
    --cc=dakr@kernel$(echo .)org \
    --cc=daniel.palmer@sony$(echo .)com \
    --cc=driver-core@lists$(echo .)linux.dev \
    --cc=gary@garyguo$(echo .)net \
    --cc=gregkh@linuxfoundation$(echo .)org \
    --cc=james.clark@linaro$(echo .)org \
    --cc=jonathanh@nvidia$(echo .)com \
    --cc=leo.yan@arm$(echo .)com \
    --cc=linux-arm-kernel@lists$(echo .)infradead.org \
    --cc=linux-doc@vger$(echo .)kernel.org \
    --cc=linux-kernel@vger$(echo .)kernel.org \
    --cc=linux-modules@vger$(echo .)kernel.org \
    --cc=linux-tegra@vger$(echo .)kernel.org \
    --cc=lossin@kernel$(echo .)org \
    --cc=mcgrof@kernel$(echo .)org \
    --cc=mike.leach@arm$(echo .)com \
    --cc=ojeda@kernel$(echo .)org \
    --cc=petr.pavlu@suse$(echo .)com \
    --cc=rafael@kernel$(echo .)org \
    --cc=rahul.bukte@sony$(echo .)com \
    --cc=rust-for-linux@vger$(echo .)kernel.org \
    --cc=samitolvanen@google$(echo .)com \
    --cc=skhan@linuxfoundation$(echo .)org \
    --cc=sumitg@nvidia$(echo .)com \
    --cc=suzuki.poulose@arm$(echo .)com \
    --cc=thierry.reding@kernel$(echo .)org \
    --cc=tim.bird@sony$(echo .)com \
    --cc=tmgross@umich$(echo .)edu \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html
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