Re: [PATCH v6 2/3] eeprom: add driver for ST M24LR series RFID/NFC EEPROM chips

[Abd-Alrhman Masalkhi <abd.masalkhi@gmail.com>]

On Wed, Jul 16, 2025 at 13:53 +0200, Greg KH wrote:
> On Sun, Jul 06, 2025 at 10:53:10AM +0000, Abd-Alrhman Masalkhi wrote:
>> +/**
>> + * m24lr_parse_le_value - Parse hex string and convert to little-endian binary
>> + * @buf:	Input string buffer (hex format)
>> + * @reg_size:	Size of the register in bytes (must be 1, 2, 4, or 8)
>> + * @output:	Output buffer to store the value in little-endian format
>> + *
>> + * Converts a hexadecimal string to a numeric value of the given register size
>> + * and writes it in little-endian byte order into the provided buffer.
>> + *
>> + * Return: 0 on success, or negative error code on failure
>> + */
>> +static __maybe_unused int m24lr_parse_le_value(const char *buf, u32 reg_size,
>> +					       u8 *output)
>
> Your __maybe_unused marker here hid the fact that it is NOT used
> anywhere in this driver :(
>
> Please remove this function as it's not needed.
>
> thanks,
>
> greg k-h

Hello Greg,

Thank you for the review!

It is done.

-- 
Best Regards,
Abd-Alrhman