Re: [edk2] [PATCH v5 0/6] read-only UDF file system support

2017-09-08 Thread Paulo Alcantara
On 08/09/2017 05:35, Laszlo Ersek wrote: Paulo, On 09/08/17 02:56, Paulo Alcantara wrote: Ray, On 07/09/2017 20:13, Paulo Alcantara wrote: v5: - Fixed OVMF IA32 build. - Fixed a typo in UdfDriveBindingStop() ("This" -> "SimpleFs") which broke retrieval of private fs data from Sim

Re: [edk2] [PATCH v5 0/6] read-only UDF file system support

2017-09-08 Thread Laszlo Ersek
Paulo, On 09/08/17 02:56, Paulo Alcantara wrote: > Ray, > > On 07/09/2017 20:13, Paulo Alcantara wrote: >> v5: >> - Fixed OVMF IA32 build. >> - Fixed a typo in UdfDriveBindingStop() ("This" -> "SimpleFs") which >> broke retrieval of private fs data from SimpleFs protocol -- >> identifi

Re: [edk2] [PATCH v5 0/6] read-only UDF file system support

2017-09-07 Thread Ni, Ruiyu
Reviewed-by: Ruiyu Ni Thanks/Ray > -Original Message- > From: Paulo Alcantara [mailto:pca...@zytor.com] > Sent: Friday, September 8, 2017 8:56 AM > To: edk2-devel@lists.01.org > Cc: Laszlo Ersek ; Justen, Jordan L > ; Andrew Fish ; Kinney, > Michael D ; Gao, Liming > ; Zeng, Star ; Dong,

Re: [edk2] [PATCH v5 0/6] read-only UDF file system support

2017-09-07 Thread Paulo Alcantara
Hi, On 07/09/2017 23:41, Ni, Ruiyu wrote: Paulo, Did you mix the usage of "\r\n" and "\n"? because I saw a ^M in your diff. However, the diff content looks good to me. No, I didn't. My local repo just lacked the 'git config core.whitespace cr-at-eol' setting so 'git diff' showed the CRLFs as

Re: [edk2] [PATCH v5 0/6] read-only UDF file system support

2017-09-07 Thread Ni, Ruiyu
Paulo, Did you mix the usage of "\r\n" and "\n"? because I saw a ^M in your diff. However, the diff content looks good to me. Thanks/Ray > -Original Message- > From: Paulo Alcantara [mailto:pca...@zytor.com] > Sent: Friday, September 8, 2017 8:56 AM > To: edk2-devel@lists.01.org > Cc: Las

Re: [edk2] [PATCH v5 0/6] read-only UDF file system support

2017-09-07 Thread Paulo Alcantara
Ray, On 07/09/2017 20:13, Paulo Alcantara wrote: v5: - Fixed OVMF IA32 build. - Fixed a typo in UdfDriveBindingStop() ("This" -> "SimpleFs") which broke retrieval of private fs data from SimpleFs protocol -- identified by 'reconnect -r' command in UEFI shell. Follow the diff betwee

[edk2] [PATCH v5 0/6] read-only UDF file system support

2017-09-07 Thread Paulo Alcantara
Hi, This series introduces read-only UDF file system support in EDK2. As Laszlo (or Red Hat) seemed to be interested in such support, I'm posting it again after ~3 years. The idea is not replacing the default FAT file system, nor breaking any existing file system support, but extending EDK2 with