Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) client-ip=2620:137:e000::1:20;
IronPort-SDR: CT0VqATWlbiGZNAIg+u7pkdul2JuSf976Pk7TnOMgQvgOFM1lKkwG/mbFQhaNFz6E0WuS9EAsx
 gPduXD3kkUQRvqe85ldhg73uXtQEaDJahXRu3oXIRXSZTShbeb2c7qNoQSRFARI8NjkVeJuitN
 w0F1UXS9V1wbwzCNC4m0zG8hLUm5bc0OhpBD3sA0ghpIpRVPTViEwjpWhH9GOD88dxn1Tyr5h4
 +al3XQOG8AvTQT8XyUUrWwc2uaz1Um6DgXT07PK50eayiJW5344QrDUMYU/jnIIwoLTPaOcNVp
 +87wa29jylv2iY8Bcs57laF/
IronPort-SDR: 1kxxRZmiPR74U0twh4V7F/Ij0ehuOkaRyCqlTYNjWcReZvzox7KJ543slPqfbFZAbJsrXBjl4u
 exuXiMTMMMGc/0Zh0cLjCN5wzyQXi7Wa3+D0AVwLrfRPy2ZdQHz9x0JQ2Id2bQG++ExVzUirxL
 njksuh9aR/3VHwpC4H0S95qVy0rD0eKuzcHMu4LhbDkukbV3kWirFs7GHrRLK97UmNd4N6Bs4/
 myGEaC0fy5u3B3nvJbFC1o88qUKNdLyuQwUczvrylWuHLyEaGCb9RSspPtHnCndJlUVL7QXUNT
 iQ8=
WDCIronportException: Internal
Message-ID: <71cca7ed-6409-3066-8878-cdc647b17057@opensource.wdc.com>
Date:   Mon, 20 Jun 2022 20:07:23 +0900
MIME-Version: 1.0
User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:91.0) Gecko/20100101
 Thunderbird/91.10.0
Subject: Re: linux-next: build warning after merge of the libata tree
Content-Language: en-US
To:     Sergey Shtylyov <s.shtylyov@omp.ru>,
        Stephen Rothwell <sfr@canb.auug.org.au>
Cc:     Linux Kernel Mailing List <linux-kernel@vger.kernel.org>,
        Linux Next Mailing List <linux-next@vger.kernel.org>
References: <20220620103400.6b03f8d9@canb.auug.org.au>
 <20220620182108.5efb3107@canb.auug.org.au>
 <387885f0-999f-3746-0238-77052f10b25c@opensource.wdc.com>
 <8e0717c2-d3c0-c81c-3a33-afdc045d123c@omp.ru>
From:   Damien Le Moal <damien.lemoal@opensource.wdc.com>
Organization: Western Digital Research
In-Reply-To: <8e0717c2-d3c0-c81c-3a33-afdc045d123c@omp.ru>
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 7bit
Precedence: bulk

On 6/20/22 19:57, Sergey Shtylyov wrote:
> Hello!
> 
> On 6/20/22 11:53 AM, Damien Le Moal wrote:
> 
> [...]
>>>> After merging the libata tree, today's linux-next build (powercp
>>>> ppc64_defconfig) produced this warning:
>>>>
>>>> In file included from include/linux/device.h:15,
>>>>                  from arch/powerpc/include/asm/io.h:27,
>>>>                  from include/linux/io.h:13,
>>>>                  from include/linux/irq.h:20,
>>>>                  from arch/powerpc/include/asm/hardirq.h:6,
>>>>                  from include/linux/hardirq.h:11,
>>>>                  from include/linux/highmem.h:11,
>>>>                  from include/linux/bvec.h:10,
>>>>                  from include/linux/blk_types.h:10,
>>>>                  from include/linux/blkdev.h:9,
>>>>                  from drivers/ata/pata_macio.c:18:
>>>> drivers/ata/pata_macio.c: In function 'pmac_macio_calc_timing_masks':
>>>> drivers/ata/pata_macio.c:1031:28: warning: format '%lx' expects argument of type 'long unsigned int', but argument 4 has type 'unsigned int' [-Wformat=]
>>>>  1031 |         dev_dbg(priv->dev, "Supported masks: PIO=%lx, MWDMA=%lx, UDMA=%lx\n",
>>>
>>> 	.
>>> 	.
>>> 	.
>>>
>>>> Introduced by commit
>>>>
>>>>   f0a6d77b351c ("ata: make transfer mode masks *unsigned int*")
> 
>    Oops, sorry about that!>
>>> This turned into a build failure in my powerpc allyesconfig build, so I
>>> reverted the above commit for today.
>>
>> I pushed the fix already. It is in ata for-next branch.
> 
>    Thank you!
>    Note that your Fixes: tag doesn't look right -- the asterisks are missing
> in the summary...

Fixed. Copy-paste from the mailer (build failure report email), the
asterisks are gone and "unsigned int" shown with bold font. Please stop
with the markdown style emphasis. That creates problems.

> 
> MBR, Sergey


-- 
Damien Le Moal
Western Digital Research