Re: [edk2] [PATCH 1/1] MdePkg/IoLib: Add support for big-endian MMIO

2017-12-04 Thread Kinney, Michael D
Biesheuvel > > Subject: Re: [edk2] [PATCH 1/1] MdePkg/IoLib: Add support > for big-endian MMIO > > Hi Mike, > > This separate library would only be necessary > specifically because > an additional library to the default platform IoLib was > needed. > So while it&

Re: [edk2] [PATCH 1/1] MdePkg/IoLib: Add support for big-endian MMIO

2017-12-04 Thread Leif Lindholm
; > > I am in favor of adding new APIs that perform the > > > byte swap operations. > > > > > > Mike > > > > > > > -Original Message- > > > > From: Udit Kumar [mailto:udit.ku...@nxp.com] > > > > Sent: Frida

Re: [edk2] [PATCH 1/1] MdePkg/IoLib: Add support for big-endian MMIO

2017-12-04 Thread Kinney, Michael D
ndholm [mailto:leif.lindh...@linaro.org] > Sent: Monday, December 4, 2017 4:36 AM > To: Kinney, Michael D > Cc: Udit Kumar ; Meenakshi Aggarwal > ; Varun Sethi > ; edk2-devel@lists.01.org; Gao, Liming > ; Ard Biesheuvel > > Subject: Re: [edk2] [PATCH 1/1] MdePkg/IoLib: Add

Re: [edk2] [PATCH 1/1] MdePkg/IoLib: Add support for big-endian MMIO

2017-12-04 Thread Leif Lindholm
> > > > Cc: Kinney, Michael D ; edk2- > > de...@lists.01.org; Gao, Liming ; > > Ard Biesheuvel > > Subject: RE: [edk2] [PATCH 1/1] MdePkg/IoLib: Add support > > for big-endian MMIO > > > > Thanks Leif, > > > > > It may require a

Re: [edk2] [PATCH 1/1] MdePkg/IoLib: Add support for big-endian MMIO

2017-12-03 Thread Meenakshi Aggarwal
arwal > ; Varun Sethi ; Kinney, > Michael D > Cc: edk2-devel@lists.01.org; Gao, Liming ; Ard > Biesheuvel > Subject: RE: [edk2] [PATCH 1/1] MdePkg/IoLib: Add support for big-endian > MMIO > > Udit, > > I agree with your concern. > > I am in favor of addi

Re: [edk2] [PATCH 1/1] MdePkg/IoLib: Add support for big-endian MMIO

2017-12-01 Thread Kinney, Michael D
n Sethi > > Cc: Kinney, Michael D ; edk2- > de...@lists.01.org; Gao, Liming ; > Ard Biesheuvel > Subject: RE: [edk2] [PATCH 1/1] MdePkg/IoLib: Add support > for big-endian MMIO > > Thanks Leif, > > > It may require a little bit more of up-front work, but > th

Re: [edk2] [PATCH 1/1] MdePkg/IoLib: Add support for big-endian MMIO

2017-12-01 Thread Udit Kumar
rom: edk2-devel [mailto:edk2-devel-boun...@lists.01.org] On Behalf Of Leif > Lindholm > Sent: Friday, December 01, 2017 4:28 PM > To: Meenakshi Aggarwal > Cc: Kinney, Michael D ; edk2-devel@lists.01.org; > Gao, Liming ; Ard Biesheuvel > > Subject: Re: [edk2] [PATCH 1/1] MdePkg/IoLi

Re: [edk2] [PATCH 1/1] MdePkg/IoLib: Add support for big-endian MMIO

2017-12-01 Thread Leif Lindholm
:leif.lindh...@linaro.org] > > Sent: Thursday, November 30, 2017 1:19 AM > > To: Kinney, Michael D > > Cc: Meenakshi Aggarwal ; Gao, Liming > > ; edk2-devel@lists.01.org; Ard Biesheuvel > > > > Subject: Re: [edk2] [PATCH 1/1] MdePkg/IoLib: Add support for big-endian &

Re: [edk2] [PATCH 1/1] MdePkg/IoLib: Add support for big-endian MMIO

2017-11-29 Thread Meenakshi Aggarwal
ent: Thursday, November 30, 2017 1:19 AM > To: Kinney, Michael D > Cc: Meenakshi Aggarwal ; Gao, Liming > ; edk2-devel@lists.01.org; Ard Biesheuvel > > Subject: Re: [edk2] [PATCH 1/1] MdePkg/IoLib: Add support for big-endian > MMIO > > I guess there is no strict rule about a drive

Re: [edk2] [PATCH 1/1] MdePkg/IoLib: Add support for big-endian MMIO

2017-11-29 Thread Leif Lindholm
gt; To: Meenakshi Aggarwal ; > > Gao, Liming > > Cc: Kinney, Michael D ; > > edk2-devel@lists.01.org; Ard Biesheuvel > > > > Subject: Re: [edk2] [PATCH 1/1] MdePkg/IoLib: Add > > support for big-endian MMIO > > > > Hi Meenakshi, > > &g

Re: [edk2] [PATCH 1/1] MdePkg/IoLib: Add support for big-endian MMIO

2017-11-29 Thread Kinney, Michael D
t we can't go with this approach (reason mentioned > by Udit). > > > > So please suggest if we should keep this library > under Platform/NXP > > or I send a new patch moving this library in MdePkg. > > > > But we have to keep a different name for Big Endian > M

Re: [edk2] [PATCH 1/1] MdePkg/IoLib: Add support for big-endian MMIO

2017-11-29 Thread Leif Lindholm
t; > Sent: Monday, October 23, 2017 12:38 PM > > To: Gao, Liming ; Meenakshi Aggarwal > > ; Ard Biesheuvel > > ; Kinney, Michael D > > ; edk2-devel@lists.01.org > > Subject: RE: [edk2] [PATCH 1/1] MdePkg/IoLib: Add support for big-endian > > MMIO > > > >

Re: [edk2] [PATCH 1/1] MdePkg/IoLib: Add support for big-endian MMIO

2017-11-27 Thread Leif Lindholm
On Mon, Nov 27, 2017 at 06:06:33AM +, Meenakshi Aggarwal wrote: > Hi Leif, > > This is regarding Big-Endian Library patch ([PATCH v2 1/9] > Platform/NXP: Add support for Big Endian Mmio APIs) > > We have started this discussion before and the suggestion was to > create a separate .inf file ke

Re: [edk2] [PATCH 1/1] MdePkg/IoLib: Add support for big-endian MMIO

2017-11-26 Thread Meenakshi Aggarwal
Udit Kumar > Sent: Monday, October 23, 2017 12:38 PM > To: Gao, Liming ; Meenakshi Aggarwal > ; Ard Biesheuvel > ; Kinney, Michael D > ; edk2-devel@lists.01.org > Subject: RE: [edk2] [PATCH 1/1] MdePkg/IoLib: Add support for big-endian > MMIO > > Hi Meenakshi/Liming, >

Re: [edk2] [PATCH 1/1] MdePkg/IoLib: Add support for big-endian MMIO

2017-10-23 Thread Udit Kumar
al ; Ard Biesheuvel > ; Kinney, Michael D ; > edk2-devel@lists.01.org > Subject: Re: [edk2] [PATCH 1/1] MdePkg/IoLib: Add support for big-endian > MMIO > > Meenakshi: > I suggest to introduce new IoLib library instance, not to add new IoLib > APIs. > New IoLib library instan

Re: [edk2] [PATCH 1/1] MdePkg/IoLib: Add support for big-endian MMIO

2017-10-15 Thread Gao, Liming
y, October 13, 2017 2:07 PM >To: Ard Biesheuvel ; Kinney, Michael D >; edk2-devel@lists.01.org; Gao, Liming > >Subject: RE: [edk2] [PATCH 1/1] MdePkg/IoLib: Add support for big-endian >MMIO > >Hi All, > > >It’s a pretty old discussion, we have left the upstreaming of NX

Re: [edk2] [PATCH 1/1] MdePkg/IoLib: Add support for big-endian MMIO

2017-10-12 Thread Meenakshi Aggarwal
; > Meenakshi Aggarwal > Subject: RE: [edk2] [PATCH 1/1] MdePkg/IoLib: Add support for big-endian > MMIO > > Hi Ard, > > > -Original Message- > > From: Ard Biesheuvel [mailto:ard.biesheu...@linaro.org] > > Sent: Monday, October 17, 2016 1:12 PM >

Re: [edk2] [PATCH 1/1] MdePkg/IoLib: Add support for big-endian MMIO

2016-10-17 Thread Bhupesh Sharma
Hi Ard, > -Original Message- > From: Ard Biesheuvel [mailto:ard.biesheu...@linaro.org] > Sent: Monday, October 17, 2016 1:12 PM > To: Kinney, Michael D > Cc: Gao, Liming ; Bhupesh Sharma > ; edk2-de...@ml01.01.org > Subject: Re: [edk2] [PATCH 1/1] MdePkg/IoLib

Re: [edk2] [PATCH 1/1] MdePkg/IoLib: Add support for big-endian MMIO

2016-10-17 Thread Ard Biesheuvel
On 17 October 2016 at 05:10, Kinney, Michael D wrote: > Bhupesh, > > It is also possible to add an ARM specific PCD to select endianness and update > MdePkg/Library/BaseIoLibIntrinsic/IoLibArm.c to use that PCD in > MmioRead/Write() > APIs in that file to support both endian types. You can use t

Re: [edk2] [PATCH 1/1] MdePkg/IoLib: Add support for big-endian MMIO

2016-10-16 Thread Kinney, Michael D
un...@lists.01.org] On Behalf Of > > Bhupesh Sharma > > Sent: Friday, October 14, 2016 5:34 PM > > To: edk2-de...@ml01.01.org > > Subject: [edk2] [PATCH 1/1] MdePkg/IoLib: Add support for big-endian > > MMIO > > > > Various IPs on NXP/FSL SoCs having ARM6

Re: [edk2] [PATCH 1/1] MdePkg/IoLib: Add support for big-endian MMIO

2016-10-16 Thread Gao, Liming
MmioClearBitsBe16() --> MmioAnd16() Thanks Liming > -Original Message- > From: edk2-devel [mailto:edk2-devel-boun...@lists.01.org] On Behalf Of > Bhupesh Sharma > Sent: Friday, October 14, 2016 5:34 PM > To: edk2-de...@ml01.01.org > Subject: [edk2] [PATCH 1/1] MdePkg/Io

Re: [edk2] [PATCH 1/1] MdePkg/IoLib: Add support for big-endian MMIO

2016-10-14 Thread Laszlo Ersek
On 10/14/16 15:17, Bhupesh Sharma wrote: > Hi Laszlo, > >> From: Laszlo Ersek [mailto:ler...@redhat.com] >> Sent: Friday, October 14, 2016 5:37 PM >> >> On 10/14/16 11:33, Bhupesh Sharma wrote: >>> Various IPs on NXP/FSL SoCs having ARM64 cores have big-endian MMIO >>> interfaces. >>> >>> This imp

Re: [edk2] [PATCH 1/1] MdePkg/IoLib: Add support for big-endian MMIO

2016-10-14 Thread Bhupesh Sharma
Hi Laszlo, > From: Laszlo Ersek [mailto:ler...@redhat.com] > Sent: Friday, October 14, 2016 5:37 PM > > On 10/14/16 11:33, Bhupesh Sharma wrote: > > Various IPs on NXP/FSL SoCs having ARM64 cores have big-endian MMIO > > interfaces. > > > > This implies that a byte-swap operation is needed to rea

Re: [edk2] [PATCH 1/1] MdePkg/IoLib: Add support for big-endian MMIO

2016-10-14 Thread Laszlo Ersek
On 10/14/16 11:33, Bhupesh Sharma wrote: > Various IPs on NXP/FSL SoCs having ARM64 cores have big-endian > MMIO interfaces. > > This implies that a byte-swap operation is needed to read/write > such BE MMIO registers from the LE ARM64 cores. > > This patch adds the support for the same. > > Con

[edk2] [PATCH 1/1] MdePkg/IoLib: Add support for big-endian MMIO

2016-10-14 Thread Bhupesh Sharma
Various IPs on NXP/FSL SoCs having ARM64 cores have big-endian MMIO interfaces. This implies that a byte-swap operation is needed to read/write such BE MMIO registers from the LE ARM64 cores. This patch adds the support for the same. Contributed-under: TianoCore Contribution Agreement 1.0 Signed