Re: [PATCH 1/6] rtc: generic: allow building on all architectures

From	Geert Uytterhoeven <geert@linux-m68k.org>
Newsgroups	linux.kernel
Subject	Re: [PATCH 1/6] rtc: generic: allow building on all architectures
Date	2016-03-02 10:40 +0100
Message-ID	<r8bqb-5KC-17@gated-at.bofh.it> (permalink)
References	<r7VY6-3zV-17@gated-at.bofh.it> <r7VY6-3zV-33@gated-at.bofh.it> <r8aNs-5fM-17@gated-at.bofh.it> <r8bgu-5FN-9@gated-at.bofh.it>
Organization	linux.* mail to news gateway

Show all headers | View raw

On Wed, Mar 2, 2016 at 10:28 AM, Andreas Schwab <schwab@linux-m68k.org> wrote:
> Geert Uytterhoeven <geert@linux-m68k.org> writes:
>
>>> +#else
>>> +#define generic_rtc_ops *(struct rtc_class_ops*)NULL
>>> +#endif
>>>
>>>  static int __init generic_rtc_probe(struct platform_device *dev)
>>>  {
>>>         struct rtc_device *rtc;
>>> +       const struct rtc_class_ops *ops;
>>> +
>>> +       ops = dev_get_platdata(&dev->dev);
>>> +       if (!ops)
>>> +               ops = &generic_rtc_ops;
>>
>> I hope no compiler version treats "&*(struct rtc_class_ops*)NULL" as
>> undefined behavior?
>
> Yes, that is guaranteed, the operations cancel each other (6.5.3.2#3: If
> the operand is the result of a unary * operator, neither that operator
> nor the & operator is evaluated and the result is as if both were
> omitted).

Thanks for confirming.

Gr{oetje,eeting}s,

                        Geert

--
Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert@linux-m68k.org

In personal conversations with technical people, I call myself a hacker. But
when I'm talking to journalists I just say "programmer" or something like that.
                                -- Linus Torvalds

Back to linux.kernel | Previous | Next — Previous in thread | Find similar | Unroll thread

Thread

[PATCH 1/6] rtc: generic: allow building on all architectures Arnd Bergmann <arnd@arndb.de> - 2016-03-01 18:10 +0100
  Re: [PATCH 1/6] rtc: generic: allow building on all architectures Geert Uytterhoeven <geert@linux-m68k.org> - 2016-03-02 10:00 +0100
    Re: [PATCH 1/6] rtc: generic: allow building on all architectures Arnd Bergmann <arnd@arndb.de> - 2016-03-02 10:10 +0100
    Re: [PATCH 1/6] rtc: generic: allow building on all architectures Andreas Schwab <schwab@linux-m68k.org> - 2016-03-02 10:30 +0100
      Re: [PATCH 1/6] rtc: generic: allow building on all architectures Geert Uytterhoeven <geert@linux-m68k.org> - 2016-03-02 10:40 +0100

csiph-web