[PATCH 2/2] fs: move fs_driver operations to struct file_operations

Jonas Rebmann jre at pengutronix.de
Mon Oct 13 04:20:04 PDT 2025


Hi

On 2025-10-01 10:06, Sascha Hauer wrote:
> The fs_driver operations really belong to struct file_operations. Move
> them there and clear a long standing TODO item.
> For the filesystems using the legacy layer move the file_operations to
> the legacy operations.

This patch (applied to next as ee6855706447) breaks CI: test/py/test_fit.py:

>       assert of_get_property(barebox, "/chosen/barebox-version") == f'"{ver}"', \
                "/chosen/barebox-version suggests we did not chainload"
E       AssertionError: /chosen/barebox-version suggests we did not chainload
E       assert False == '"barebox-2025.09.0-00109-gee6855706447"'
E        +  where False = of_get_property(BareboxDriver(target=Target(name='main', env=Environment(config_file='test/arm/virt at multi_v8_defconfig.yaml')), name=N...ebox@[^:]+:[^ ]+ ', autoboot='stop autoboot', interrupt='\n', bootstring='commandline:', password='', login_timeout=60), '/chosen/barebox-version')

test/py/test_fit.py:48: AssertionError

This is for all platforms supporting testfs.

This seems unrelated to CI failing on test_tlv as long as the CI container
hasn't been rebuilt with 9703b653fb09

Regards,
Jonas

-- 
Pengutronix e.K.                           | Jonas Rebmann               |
Steuerwalder Str. 21                       | http://www.pengutronix.de/  |
31137 Hildesheim, Germany                  | Phone: +49-5121-206917-0    |
Amtsgericht Hildesheim, HRA 2686           | Fax:   +49-5121-206917-9    |



More information about the barebox mailing list