[PATCH v3 0/3] staging: vc04_services: Tweak module autoloading support
Kieran Bingham
kieran.bingham at ideasonboard.com
Thu Oct 19 07:02:20 PDT 2023
Quoting Umang Jain (2023-10-19 10:01:25)
> As explained by Greg KH in [1], we should not be using MODULE_ALIAS
> to auto load modules within VC04. Drop that, in favour of
> MODULE_DEVICE_TABLE. This series addresses the suggestion.
>
> Patch 1/3 adds vchiq_bus_device_id and supporting entries in
> script/mod/file2alias.c
>
> Patch 2/3 drops MODULE_ALIAS from bcm2835-audio in favour
> of MODULE_DEVICE_TABLE.
>
> Patch 3/3 drops MODULE_ALIAS from bcm2835-camera in favour
> of MODULE_DEVICE_TABLE.
>
> [1]: https://lore.kernel.org/linux-media/2023100955-stunt-equate-c6fa@gregkh/
Does this series now complete the task in
drivers/staging/vc04_services/interface/TODO? :
> * Get rid of all non essential global structures and create a proper
> per device structure
The series looks good to me though:
Reviewed-by: Kieran Bingham <kieran.bingham at ideasonboard.com>
>
> --
> Changes in v3:
> - Drop "_bus" suffix from all patches (and commit message)
> as pointed out in v2 review
>
> Changes in v2:
> - NULL terminate device_id_table arrays (in 2/3 and 3/3)
>
> Umang Jain (3):
> staging: vc04_services: Support module autoloading using
> MODULE_DEVICE_TABLE
> staging: vc04_services: bcm2835-audio: Drop MODULE_ALIAS
> staging: vc04_services: bcm2835-camera: Drop MODULE_ALIAS
>
> drivers/staging/vc04_services/bcm2835-audio/bcm2835.c | 8 +++++++-
> .../vc04_services/bcm2835-camera/bcm2835-camera.c | 8 +++++++-
> .../vc04_services/interface/vchiq_arm/vchiq_bus.c | 2 +-
> .../vc04_services/interface/vchiq_arm/vchiq_bus.h | 3 +++
> include/linux/mod_devicetable.h | 4 ++++
> scripts/mod/devicetable-offsets.c | 3 +++
> scripts/mod/file2alias.c | 9 +++++++++
> 7 files changed, 34 insertions(+), 3 deletions(-)
>
> --
> 2.40.1
>
More information about the linux-arm-kernel
mailing list