Thread (1 message) 1 message, 1 author, 2022-01-04

Re: [PATCH 16/34] brcmfmac: acpi: Add support for fetching Apple ACPI properties

From: Hector Martin <hidden>
Date: 2022-01-04 05:23:13
Also in: linux-acpi, linux-devicetree, linux-wireless, lkml

On 2022/01/04 7:50, Andy Shevchenko wrote:
    >     +       status = acpi_evaluate_object(adev->handle, "RWCV",
    NULL, &buf);
    >     +       o = buf.pointer;
    >     +       if (!ACPI_FAILURE(status) && o && o->type ==
    ACPI_TYPE_BUFFER &&
    >     +           o->buffer.length >= 2) {
    >     +               char *antenna_sku = devm_kzalloc(dev, 3,
    GFP_KERNEL);
    >     +
    >     +               memcpy(antenna_sku, o->buffer.pointer, 2);
    >
    >
    > NIH devm_kmemdup()?

    Not *quite*. I take the first two bytes of the returned buffer and turn
    them into a null-terminated 3-byte string. kmemdup wouldn't
    null-terminate or would copy too much, depending on length.



devm_kstrndup() then?

 
That doesn't seem to be a thing.


-- 
Hector Martin (marcan@marcan.st)
Public Key: https://mrcn.st/pub
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help