Re: [PATCH i2c-tools v5] i2ctransfer: Add optional message modifier flags

[Wolfram Sang <wsa+renesas@sang-engineering.com>]

[-- Attachment #1: Type: text/plain, Size: 1163 bytes --]

Hi Benoît,

On Wed, Jun 10, 2026 at 09:49:01AM +0200, Benoît Monin wrote:
> Allow setting protocol mangling and repeated start elision flags of an i2c
> message with a set of optional command-line flags. These optional flags
> are parsed at the beginning of the DESC field up to a read or write flag.
> 
> For example, to read one byte from address 0x50 followed by a stop, then
> write two bytes at 0x54 on bus 0, one would call i2ctransfer as follow:
> 
>     i2ctransfer 0 pr1@0x50 w2@0x54 0x10 0x20
> 
> Since the new flags are optional, this patch preserves the compatibility
> of the i2ctransfer syntax.
> 
> Handling of the message flags is done in add_flag_if_supported(). This
> function checks if the flag is defined at compile time and if the adapter
> supports the required functionality to handle the flag at runtime.
> 
> Signed-off-by: Benoît Monin <benoit.monin@bootlin.com>

Applied, thank you!

It would be awesome if the PRINT_HEADER part could be extended to
display to the user the flags used per message, if any. As an
incremental patch. I don't know if you have time for that?

Happy hacking,

   Wolfram


[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 833 bytes --]