* Re: Re: [PATCH 05/32] lib: devres: don't enclose pcim_*() functions in CONFIG_HAS_IOPORT
@ 2013-03-06 6:28 Jingoo Han
2013-03-06 8:26 ` Thomas Petazzoni
0 siblings, 1 reply; 2+ messages in thread
From: Jingoo Han @ 2013-03-06 6:28 UTC (permalink / raw)
To: Arnd Bergmann, Thomas Petazzoni
Cc: Lior Amsalem, Andrew Lunn, Russell King - ARM Linux, Jason Cooper,
Tawfik Bayouk, Stephen Warren, linux-pci@vger.kernel.org,
Thierry Reding, Paul Gortmaker, linux-kernel@vger.kernel.org,
Jesse Barnes, Eran Ben-Avi, Nadav Haklai, Maen Suleiman,
Shadi Ammouri, Bjorn Helgaas, Gregory Clement, Yinghai Lu,
linux-arm-kernel@lists.infradead.org, Jason Gunthorpe, Jingoo Han
[-- Warning: decoded text below may be mangled, UTF-8 assumed --]
[-- Attachment #1: Type: text/plain; charset=euc-kr, Size: 1406 bytes --]
On Tuesday, March 05, 2013 5:30 AM, Arnd Bergmann wrote:
>
> On Monday 04 March 2013, Thomas Petazzoni wrote:
> > FWIW, a patch that is doing what I was initially proposing has been
> > merged for 3.9, and it doesn't contain the
> > IS_ENABLED(CONFIG_HAS_IOPORT) test you were proposing (and which I
> > think was correct). See:
> >
> > commit 9ed8a30f3471347c1b763bd062fa78ae80f18eae
> > Author: Jingoo Han <jg1.han@samsung.com>
> > Date: Wed Feb 27 17:02:42 2013 -0800
> >
>
> Sigh.
>
> I'll take it as an additional incentive to finally clean up the logic behind
> CONFIG_HAS_IOPORT by introducing a CONFIG_HAS_IOPORT_MAP symbol to replace it.
>
> Thanks for the heads up.
Hi Thomas Petazzoni
Sorry, I did not know that you submitted the patch.
Like you, I am developing PCIe Host driver.
Also, I experienced the annoying build error related to
CONFIG_HAS_IOPORT.
Hi Arnd Bergmann,
I have just read the mailing thread.
If you resolve this situation properly, it will be very helpful.
Thank you.
Best regards,
Jingoo Han
>
> Arnd
> --
> To unsubscribe from this list: send the line "unsubscribe linux-pci" in
> the body of a message to majordomo@vger.kernel.org
> More majordomo info at http://vger.kernel.org/majordomo-info.html
ÿôèº{.nÇ+·®+%Ëÿ±éݶ\x17¥wÿº{.nÇ+·¥{±þG«éÿ{ayº\x1dÊÚë,j\a¢f£¢·hïêÿêçz_è®\x03(éÝ¢j"ú\x1a¶^[m§ÿÿ¾\a«þG«éÿ¢¸?¨èÚ&£ø§~á¶iOæ¬z·vØ^\x14\x04\x1a¶^[m§ÿÿÃ\fÿ¶ìÿ¢¸?I¥
^ permalink raw reply [flat|nested] 2+ messages in thread
* Re: [PATCH 05/32] lib: devres: don't enclose pcim_*() functions in CONFIG_HAS_IOPORT
2013-03-06 6:28 Re: [PATCH 05/32] lib: devres: don't enclose pcim_*() functions in CONFIG_HAS_IOPORT Jingoo Han
@ 2013-03-06 8:26 ` Thomas Petazzoni
0 siblings, 0 replies; 2+ messages in thread
From: Thomas Petazzoni @ 2013-03-06 8:26 UTC (permalink / raw)
To: jg1.han
Cc: Arnd Bergmann, Lior Amsalem, Andrew Lunn,
Russell King - ARM Linux, Jason Cooper, Tawfik Bayouk,
Stephen Warren, linux-pci@vger.kernel.org, Thierry Reding,
Paul Gortmaker, linux-kernel@vger.kernel.org, Jesse Barnes,
Eran Ben-Avi, Nadav Haklai, Maen Suleiman, Shadi Ammouri,
Bjorn Helgaas, Gregory Clement, Yinghai Lu,
linux-arm-kernel@lists.infradead.org, Jason Gunthorpe
Dear Jingoo Han,
On Wed, 06 Mar 2013 06:28:08 +0000 (GMT), Jingoo Han wrote:
> Sorry, I did not know that you submitted the patch.
No problem, I'm happy to have one less patch to carry in my PCIe patch
set :)
> Like you, I am developing PCIe Host driver.
Just curious, do you already have some code? Thierry Reding and myself
have been looking at each other's PCIe host driver since a while in
order to make some consistent choices where possible. It would be good
to see your code as well.
Best regards,
Thomas
--
Thomas Petazzoni, Free Electrons
Kernel, drivers, real-time and embedded Linux
development, consulting, training and support.
http://free-electrons.com
^ permalink raw reply [flat|nested] 2+ messages in thread
end of thread, other threads:[~2013-03-06 8:26 UTC | newest]
Thread overview: 2+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2013-03-06 6:28 Re: [PATCH 05/32] lib: devres: don't enclose pcim_*() functions in CONFIG_HAS_IOPORT Jingoo Han
2013-03-06 8:26 ` Thomas Petazzoni
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox