Hi, On 4/21/23 13:37, Pavel Machek wrote: > Hi! > >> Linux Security components are under development and not published yet. >> The only linux component is the driver (hp bioscfg) at this time. >> Other published security components are under Windows. >> >> Signed-off-by: Jorge Lopez <jorge.lopez2@xxxxxx> > >> +/* >> + * ascii_to_utf16_unicode - Convert ascii string to UTF-16 unicode >> + * >> + * BIOS supports UTF-16 characters that are 2 bytes long. No variable >> + * multi-byte language supported. >> + * >> + * @p: Unicode buffer address >> + * @str: string to convert to unicode >> + * >> + * Returns a void pointer to the buffer containing unicode string >> + */ >> +void *ascii_to_utf16_unicode(u16 *p, const u8 *str) >> +{ > > Does this need to go to library somewhere? This has already been discussed in earlier submissions of the driver, the utf16 format is HP specific (prefixed with a 16 bit le lenght, and the 0 length string needs special encoding) so despite the name this is not generic. It should probably be prefixed with hp_ because of this though, to avoid potential symbol conflicts when builtin. (and the same applies to other generic functions). Regards, Hans