[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <570EA09F.1070000@codeaurora.org>
Date: Wed, 13 Apr 2016 14:40:15 -0500
From: Shanker Donthineni <shankerd@...eaurora.org>
To: Timur Tabi <timur@...eaurora.org>,
kbuild test robot <lkp@...el.com>
Cc: kbuild-all@...org, netdev@...r.kernel.org,
linux-kernel@...r.kernel.org, devicetree@...r.kernel.org,
linux-arm-msm@...r.kernel.org, sdharia@...eaurora.org,
Greg Kroah-Hartman <greg@...ah.com>, vikrams@...eaurora.org,
cov@...eaurora.org, gavidov@...eaurora.org,
Rob Herring <robh+dt@...nel.org>, andrew@...n.ch,
bjorn.andersson@...aro.org, Mark Langsdorf <mlangsdo@...hat.com>,
Jon Masters <jcm@...hat.com>,
Andy Gross <agross@...eaurora.org>,
"David S. Miller" <davem@...emloft.net>
Subject: Re: [PATCH 1/2] [v4] net: emac: emac gigabit ethernet controller
driver
On 04/13/2016 02:31 PM, Timur Tabi wrote:
> kbuild test robot wrote:
>>
>> drivers/net/ethernet/qualcomm/emac/emac-mac.c: In function 'emac_mac_up':
>>>> >>drivers/net/ethernet/qualcomm/emac/emac-mac.c:1076:9: warning: large integer implicitly truncated to unsigned type [-Woverflow]
>> writel(~DIS_INT, adpt->base + EMAC_INT_STATUS);
>
> This doesn't happen on arm64, and I don't know how to fix it. DIS_INT is defined as:
>
> #define DIS_INT BIT(31)
>
Try with (1U<<31).
> It seems silly to add a typecast to DIS_INT.
>
--
Shanker Donthineni
Qualcomm Technologies, Inc. on behalf of Qualcomm Innovation Center, Inc.
Qualcomm Innovation Center, Inc. is a member of Code Aurora Forum, a Linux Foundation Collaborative Project
Powered by blists - more mailing lists