EmbeddedPkg/Drivers/SataSiI3132Dxe/SataSiI3132.c | 301 ++++++++++++++++----- EmbeddedPkg/Drivers/SataSiI3132Dxe/SataSiI3132.h | 17 ++ .../Drivers/SataSiI3132Dxe/SiI3132AtaPassThru.c | 4 +- 3 files changed, 252 insertions(+), 70 deletions(-)
This set of patches fixes an issue with 64-bit DMA and implements the missing exit boot event and driver stop functionality including memory/protocols cleanup procedure. Daniil Egranov (4): Drivers/SataSiI3132Dxe: Fixed PCI IO read and write operations Drivers/SataSiI3132Dxe: Allow 64-bit DMA transfer Drivers/SataSiI3132Dxe: Enable multi-controller support Drivers/SataSiI3132Dxe: Fixed startup and shutdown procedures EmbeddedPkg/Drivers/SataSiI3132Dxe/SataSiI3132.c | 301 ++++++++++++++++----- EmbeddedPkg/Drivers/SataSiI3132Dxe/SataSiI3132.h | 17 ++ .../Drivers/SataSiI3132Dxe/SiI3132AtaPassThru.c | 4 +- 3 files changed, 252 insertions(+), 70 deletions(-) -- 2.11.0 _______________________________________________ edk2-devel mailing list edk2-devel@lists.01.org https://lists.01.org/mailman/listinfo/edk2-devel
On 27 October 2017 at 06:33, Daniil Egranov <daniil.egranov@arm.com> wrote: > This set of patches fixes an issue with 64-bit DMA and implements > the missing exit boot event and driver stop functionality including > memory/protocols cleanup procedure. > > Daniil Egranov (4): > Drivers/SataSiI3132Dxe: Fixed PCI IO read and write operations > Drivers/SataSiI3132Dxe: Allow 64-bit DMA transfer > Drivers/SataSiI3132Dxe: Enable multi-controller support > Drivers/SataSiI3132Dxe: Fixed startup and shutdown procedures > > EmbeddedPkg/Drivers/SataSiI3132Dxe/SataSiI3132.c | 301 ++++++++++++++++----- > EmbeddedPkg/Drivers/SataSiI3132Dxe/SataSiI3132.h | 17 ++ > .../Drivers/SataSiI3132Dxe/SiI3132AtaPassThru.c | 4 +- > 3 files changed, 252 insertions(+), 70 deletions(-) > Hi Daniil, Thanks for taking the time to fix this driver. I will go ahead and push the first two patches, given that they are self-contained and obvious bug fixes. The remaining patches, please split them up, and please align more closely with what other upstream PCI drivers do. Regards, Ard. _______________________________________________ edk2-devel mailing list edk2-devel@lists.01.org https://lists.01.org/mailman/listinfo/edk2-devel
Hi, On 10/27/2017 12:33 AM, Daniil Egranov wrote: > This set of patches fixes an issue with 64-bit DMA and implements > the missing exit boot event and driver stop functionality including > memory/protocols cleanup procedure. > > Daniil Egranov (4): > Drivers/SataSiI3132Dxe: Fixed PCI IO read and write operations > Drivers/SataSiI3132Dxe: Allow 64-bit DMA transfer > Drivers/SataSiI3132Dxe: Enable multi-controller support > Drivers/SataSiI3132Dxe: Fixed startup and shutdown procedures > > EmbeddedPkg/Drivers/SataSiI3132Dxe/SataSiI3132.c | 301 ++++++++++++++++----- > EmbeddedPkg/Drivers/SataSiI3132Dxe/SataSiI3132.h | 17 ++ > .../Drivers/SataSiI3132Dxe/SiI3132AtaPassThru.c | 4 +- > 3 files changed, 252 insertions(+), 70 deletions(-) This is generally good, but there remain quite a number of "errors" in the command submission path as well as the completely unnecessary 4k IO alignment requirement which has been known to break older grubs/etc. A few of those "errors" were fixed in this patch set (1) as well, so might be worthwhile if you are looking at this driver to integrate those fixes as well. (1) https://lists.01.org/pipermail/edk2-devel/2017-March/008277.html Thanks, _______________________________________________ edk2-devel mailing list edk2-devel@lists.01.org https://lists.01.org/mailman/listinfo/edk2-devel
On 27 October 2017 at 17:57, Jeremy Linton <jeremy.linton@arm.com> wrote: > Hi, > > On 10/27/2017 12:33 AM, Daniil Egranov wrote: >> >> This set of patches fixes an issue with 64-bit DMA and implements >> the missing exit boot event and driver stop functionality including >> memory/protocols cleanup procedure. >> >> Daniil Egranov (4): >> Drivers/SataSiI3132Dxe: Fixed PCI IO read and write operations >> Drivers/SataSiI3132Dxe: Allow 64-bit DMA transfer >> Drivers/SataSiI3132Dxe: Enable multi-controller support >> Drivers/SataSiI3132Dxe: Fixed startup and shutdown procedures >> >> EmbeddedPkg/Drivers/SataSiI3132Dxe/SataSiI3132.c | 301 >> ++++++++++++++++----- >> EmbeddedPkg/Drivers/SataSiI3132Dxe/SataSiI3132.h | 17 ++ >> .../Drivers/SataSiI3132Dxe/SiI3132AtaPassThru.c | 4 +- >> 3 files changed, 252 insertions(+), 70 deletions(-) > > > This is generally good, but there remain quite a number of "errors" in the > command submission path as well as the completely unnecessary 4k IO > alignment requirement which has been known to break older grubs/etc. A few > of those "errors" were fixed in this patch set (1) as well, so might be > worthwhile if you are looking at this driver to integrate those fixes as > well. > Yes, please. And apologies for forgetting about thise patches. > (1) https://lists.01.org/pipermail/edk2-devel/2017-March/008277.html > > Thanks, _______________________________________________ edk2-devel mailing list edk2-devel@lists.01.org https://lists.01.org/mailman/listinfo/edk2-devel
© 2016 - 2024 Red Hat, Inc.