From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1752228Ab3EEUgW (ORCPT ); Sun, 5 May 2013 16:36:22 -0400 Received: from mail2.gnudd.com ([213.203.150.91]:42465 "EHLO mail.gnudd.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752101Ab3EEUgV (ORCPT ); Sun, 5 May 2013 16:36:21 -0400 X-Greylist: delayed 596 seconds by postgrey-1.27 at vger.kernel.org; Sun, 05 May 2013 16:36:20 EDT Date: Sun, 5 May 2013 22:24:22 +0200 From: Alessandro Rubini To: bp@alien8.de Cc: hpa@zytor.com, ciminaghi@gnudd.com, jaswinder.singh@linaro.org, will.deacon@arm.com, linux@arm.linux.org.uk, tglx@linutronix.de, mingo@redhat.com, djbw@fb.com, vinod.koul@intel.com, grant.likely@secretlab.ca, linus.walleij@linaro.org, wim@iguana.be, cjb@laptop.org, davidb@codeaurora.org, nico@fluxnic.net, ben-linux@fluff.org, viresh.linux@gmail.com, rajeev-dlh.kumar@st.com, linux-kernel@vger.kernel.org, linux-arm-kernel@lists.infradead.org, giancarlo.asnaghi@st.com Subject: Re: [PATCH v4 0/7] enable support for AMBA drivers under x86 Message-ID: <20130505202422.GA13758@mail.gnudd.com> MIME-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Disposition: inline Content-Transfer-Encoding: 8bit Organization: GnuDD, Device Drivers, Embedded Systems, Courses In-Reply-To: <20130505123633.GA14095@pd.tnic> References: <20130505123633.GA14095@pd.tnic> <50C93A83.8040106@zytor.com> <1355146956-6009-1-git-send-email-ciminaghi@gnudd.com> <50C93718.2090706@zytor.com> <20121213181449.GA15966@mail.gnudd.com> Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org > Did this got fixed in some tree in the meantime? Because I still see it on > current Linus + tip/master: > > arch/x86/pci/sta2x11-fixup.c:186:2: warning: initialization from incompatible pointer type [enabled by default] > arch/x86/pci/sta2x11-fixup.c:186:2: warning: (near initialization for ‘sta2x11_dma_ops.free’) [enabled by default] Last time I sent the patch was on Feb 22nd: The .free field of struct dma_map_ops features a prototype that is different from what lib/swiotlb.c offers. The new "attrs" argument in dma_ops.free is missing from the generic implementation. This removes the build warning by providing a function that just calls swiotlb_free_coherent() without passing "attrs", like others do. I got no reply. This is the link: http://lkml.org/lkml/2013/2/22/111 It is in patchwork here: https://patchwork.kernel.org/patch/2175551/ /alessandro