From: Yong <yong.deng@magewell.com>
To: Yong <yong.deng@magewell.com>
Cc: kbuild test robot <lkp@intel.com>,
kbuild-all@01.org,
Maxime Ripard <maxime.ripard@free-electrons.com>,
Mauro Carvalho Chehab <mchehab@kernel.org>,
Rob Herring <robh+dt@kernel.org>,
Mark Rutland <mark.rutland@arm.com>, Chen-Yu Tsai <wens@csie.org>,
"David S. Miller" <davem@davemloft.net>,
Greg Kroah-Hartman <gregkh@linuxfoundation.org>,
Hans Verkuil <hans.verkuil@cisco.com>,
Linus Walleij <linus.walleij@linaro.org>,
Randy Dunlap <rdunlap@infradead.org>,
Stanimir Varbanov <stanimir.varbanov@linaro.org>,
Hugues Fruchet <hugues.fruchet@st.com>,
Yannick Fertre <yannick.fertre@st.com>,
Philipp Zabel <p.zabel@pengutronix.de>,
Arnd Bergmann <arnd@arndb.de>,
Benjamin Gaignard <benjamin.gaignard@linaro.org>,
Ramesh Shanmugasundaram <ramesh.shanmugasundaram@bp>
Subject: Re: [PATCH v6 2/2] media: V3s: Add support for Allwinner CSI.
Date: Fri, 26 Jan 2018 11:00:41 +0800 [thread overview]
Message-ID: <20180126110041.f89848325b9ecfb07df387ca@magewell.com> (raw)
In-Reply-To: <20180126094658.aa70ed3f890464f6051e21e4@magewell.com>
Hi Maxime,
On Fri, 26 Jan 2018 09:46:58 +0800
Yong <yong.deng@magewell.com> wrote:
> Hi Maxime,
>
> Do you have any experience in solving this problem?
> It seems the PHYS_OFFSET maybe undeclared when the ARCH is not arm.
Got it.
Should I add 'depends on ARM' in Kconfig?
>
> On Fri, 26 Jan 2018 08:04:18 +0800
> kbuild test robot <lkp@intel.com> wrote:
>
> > Hi Yong,
> >
> > I love your patch! Yet something to improve:
> >
> > [auto build test ERROR on linuxtv-media/master]
> > [also build test ERROR on v4.15-rc9 next-20180119]
> > [if your patch is applied to the wrong git tree, please drop us a note to help improve the system]
> >
> > url: https://github.com/0day-ci/linux/commits/Yong-Deng/dt-bindings-media-Add-Allwinner-V3s-Camera-Sensor-Interface-CSI/20180126-054511
> > base: git://linuxtv.org/media_tree.git master
> > config: i386-allmodconfig (attached as .config)
> > compiler: gcc-7 (Debian 7.2.0-12) 7.2.1 20171025
> > reproduce:
> > # save the attached .config to linux build tree
> > make ARCH=i386
> >
> > All errors (new ones prefixed by >>):
> >
> > drivers/media/platform/sunxi/sun6i-csi/sun6i_csi.c: In function 'sun6i_csi_update_buf_addr':
> > >> drivers/media/platform/sunxi/sun6i-csi/sun6i_csi.c:567:31: error: 'PHYS_OFFSET' undeclared (first use in this function); did you mean 'PAGE_OFFSET'?
> > dma_addr_t bus_addr = addr - PHYS_OFFSET;
> > ^~~~~~~~~~~
> > PAGE_OFFSET
> > drivers/media/platform/sunxi/sun6i-csi/sun6i_csi.c:567:31: note: each undeclared identifier is reported only once for each function it appears in
> >
> > vim +567 drivers/media/platform/sunxi/sun6i-csi/sun6i_csi.c
> >
> > 562
> > 563 void sun6i_csi_update_buf_addr(struct sun6i_csi *csi, dma_addr_t addr)
> > 564 {
> > 565 struct sun6i_csi_dev *sdev = sun6i_csi_to_dev(csi);
> > 566 /* transform physical address to bus address */
> > > 567 dma_addr_t bus_addr = addr - PHYS_OFFSET;
> > 568
> > 569 regmap_write(sdev->regmap, CSI_CH_F0_BUFA_REG,
> > 570 (bus_addr + sdev->planar_offset[0]) >> 2);
> > 571 if (sdev->planar_offset[1] != -1)
> > 572 regmap_write(sdev->regmap, CSI_CH_F1_BUFA_REG,
> > 573 (bus_addr + sdev->planar_offset[1]) >> 2);
> > 574 if (sdev->planar_offset[2] != -1)
> > 575 regmap_write(sdev->regmap, CSI_CH_F2_BUFA_REG,
> > 576 (bus_addr + sdev->planar_offset[2]) >> 2);
> > 577 }
> > 578
> >
> > ---
> > 0-DAY kernel test infrastructure Open Source Technology Center
> > https://lists.01.org/pipermail/kbuild-all Intel Corporation
>
>
> Thanks,
> Yong
Thanks,
Yong
next prev parent reply other threads:[~2018-01-26 3:00 UTC|newest]
Thread overview: 31+ messages / expand[flat|nested] mbox.gz Atom feed top
2018-01-23 8:18 [PATCH v6 2/2] media: V3s: Add support for Allwinner CSI Yong Deng
2018-01-26 0:04 ` kbuild test robot
[not found] ` <201801260759.RyNhDZz4%fengguang.wu-ral2JQCrhuEAvxtiuMwx3w@public.gmane.org>
2018-01-26 1:46 ` Yong
2018-01-26 3:00 ` Yong [this message]
2018-01-26 8:10 ` Maxime Ripard
[not found] ` <20180126081000.hy7g57zp5dv6ug2g-ZC1Zs529Oq4@public.gmane.org>
2018-01-28 2:19 ` [linux-sunxi] " Yong
[not found] ` <20180128101903.fbaec083c787bda30aeb05ef-+3dxTMOEIRNWk0Htik3J/w@public.gmane.org>
2018-01-29 8:16 ` Maxime Ripard
[not found] ` <20180126110041.f89848325b9ecfb07df387ca-+3dxTMOEIRNWk0Htik3J/w@public.gmane.org>
2018-01-31 3:08 ` Liviu Dudau
2018-01-31 3:24 ` [linux-sunxi] " Chen-Yu Tsai
[not found] ` <20180131030807.GA19945-hOhETlTuV5niMG9XS5x8Mg@public.gmane.org>
2018-01-31 7:42 ` Maxime Ripard
[not found] ` <20180131074212.7hvb3nqkt22h2chg-ZC1Zs529Oq4@public.gmane.org>
2018-01-31 14:47 ` Liviu Dudau
2018-02-01 8:32 ` [linux-sunxi] " Maxime Ripard
2018-02-01 9:20 ` Arnd Bergmann
[not found] ` <CAK8P3a01DqmVJKt6J2i_okVoeELJCUW0voW0r52EzBY7iF74xQ-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org>
2018-02-01 11:34 ` Liviu Dudau
[not found] ` <20180201113442.GC19945-hOhETlTuV5niMG9XS5x8Mg@public.gmane.org>
2018-02-01 15:54 ` Maxime Ripard
2018-01-27 15:51 ` kbuild test robot
[not found] ` <1516695531-23349-1-git-send-email-yong.deng-+3dxTMOEIRNWk0Htik3J/w@public.gmane.org>
2018-01-27 16:14 ` Linus Walleij
[not found] ` <CACRpkdan52UB7HOyH1gnHWg4CDke_VQxAdq8cBgwUroibE59Ow-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org>
2018-01-28 2:39 ` Yong
2018-01-29 8:25 ` Maxime Ripard
2018-01-29 8:28 ` [linux-sunxi] " icenowy
[not found] ` <20180129082533.6edmqgbauo6q5dgz-ZC1Zs529Oq4@public.gmane.org>
2018-01-29 9:25 ` Linus Walleij
[not found] ` <CACRpkdYAGwUjr2C-w5U+WuG48pZAOUcnxFjznLbdF6Lmy1uZuQ-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org>
2018-01-29 14:34 ` Arnd Bergmann
2018-01-30 7:54 ` Maxime Ripard
[not found] ` <20180130075441.rqxzkwero6sdfak6-ZC1Zs529Oq4@public.gmane.org>
2018-01-30 9:24 ` Arnd Bergmann
2018-01-30 9:59 ` Thierry Reding
2018-01-30 10:01 ` Thierry Reding
2018-01-31 7:29 ` Maxime Ripard
[not found] ` <20180131072910.ajp3jc5dmetsjtf2-ZC1Zs529Oq4@public.gmane.org>
2018-01-31 9:37 ` Arnd Bergmann
[not found] ` <CAK8P3a0X2bpLjKE6xKehG1junZoG1N_DjepOBQ+SZetKf6sgfA-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org>
2018-02-01 15:29 ` Maxime Ripard
2018-02-01 15:36 ` Arnd Bergmann
2018-02-01 16:31 ` Robin Murphy
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20180126110041.f89848325b9ecfb07df387ca@magewell.com \
--to=yong.deng@magewell.com \
--cc=arnd@arndb.de \
--cc=benjamin.gaignard@linaro.org \
--cc=davem@davemloft.net \
--cc=gregkh@linuxfoundation.org \
--cc=hans.verkuil@cisco.com \
--cc=hugues.fruchet@st.com \
--cc=kbuild-all@01.org \
--cc=linus.walleij@linaro.org \
--cc=lkp@intel.com \
--cc=mark.rutland@arm.com \
--cc=maxime.ripard@free-electrons.com \
--cc=mchehab@kernel.org \
--cc=p.zabel@pengutronix.de \
--cc=ramesh.shanmugasundaram@bp \
--cc=rdunlap@infradead.org \
--cc=robh+dt@kernel.org \
--cc=stanimir.varbanov@linaro.org \
--cc=wens@csie.org \
--cc=yannick.fertre@st.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).