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@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    |

Reply via email to