Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]

Groups > linux.kernel > #1510295

Re: [PATCH v3] ARM: davinci: da8xx: Fix some redefined symbol warnings

Show all headers | View raw

On 10/27/2016 01:54 PM, Sekhar Nori wrote:
> On Thursday 27 October 2016 05:13 PM, Sekhar Nori wrote:
>> On Wednesday 26 October 2016 06:09 PM, Alexandre Bailon wrote:
>>> Some macro for DA8xx CFGCHIP are defined in usb-davinci.h,
>>> but da8xx-cfgchip.h intend to replace them.
>>> The usb-da8xx.c is using both headers, causing redefined symbol warnings.
>>
>> Looks like this is not true for v4.9-rc2 and so I don't see any warnings
> 
> Ah, just noticed that _this_ is the patch that introduces
> da8xx-cfgchip.h into usb-da8xx.c. So this is the patch that introduces
> the warnings (and fixes them).
> 
> I can queue this for v4.10 (with Greg's ack) if you change the
> description to make it about cleaning up duplicated defines between
> da8xx-cfgchip.h and usb-davinci.h and not talk about "redefined symbol
> warnings".
I will do it.
> 
> Also, when adding a header file, can you please keep it sorted in
> alphabetical order.
Ok.
> 
> Thanks,
> Sekhar
> 
Thanks,
Alexandre

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

Thread

[PATCH v3] Fix some potential warnings Alexandre Bailon <abailon@baylibre.com> - 2016-10-26 14:40 +0200
  [PATCH v3] ARM: davinci: da8xx: Fix some redefined symbol warnings Alexandre Bailon <abailon@baylibre.com> - 2016-10-26 14:50 +0200
    Re: [PATCH v3] ARM: davinci: da8xx: Fix some redefined symbol  warnings Sekhar Nori <nsekhar@ti.com> - 2016-10-27 16:00 +0200
      Re: [PATCH v3] ARM: davinci: da8xx: Fix some redefined symbol  warnings Alexandre Bailon <abailon@baylibre.com> - 2016-10-27 16:40 +0200
    Re: [PATCH v3] ARM: davinci: da8xx: Fix some redefined symbol  warnings Sekhar Nori <nsekhar@ti.com> - 2016-10-27 16:50 +0200

csiph-web