[PATCH 4/7] drm/pl111: Enable PL110 variant
Linus Walleij
linus.walleij at linaro.org
Fri Sep 1 02:22:19 PDT 2017
On Fri, Sep 1, 2017 at 10:23 AM, Emil Velikov <emil.l.velikov at gmail.com> wrote:
>>Eric E:
>>> +struct pl111_variant_data pl110_variant = {
>>
>> I think this (and `pl111_variant` below) can be `static const`, right?
>>
> Static - yes, const - I don't think so.
> Struct amba_id::data lacks the constness, so the const qualifier will
> get discarded.
I fixed it with a (void*) cast so we can have it right in the driver.
> In practise everyone considers/uses ::data as const, so that could be
> tweaked with separate patch.
I think it is not necessarily possible to make that const since the amba_id
is used in several dynamically generated board files. But I may
be wrong.
Yours,
Linus Walleij
More information about the linux-arm-kernel
mailing list