Re: [edk2] [PATCH edk2-platforms] [PATCH v3 0/9] Platform/NXP
On Mon, Nov 27, 2017 at 04:21:48PM +0530, Meenakshi Aggarwal wrote: > v3: > Added patch-prefix I only meant you could apply this in future. However, this new posting has ended up with ([PATCH ...][PATCH...]), which is not ideal. There is no need to edit the patch subject for each version - just specify (for a v3) git format-patch --subject-prefix="PATCH edk2-platforms" -v3 If there is a need for a v4, or for future submissions, please also look into https://github.com/tianocore/tianocore.github.io/wiki/Laszlo's-unkempt-git-guide-for-edk2-contributors-and-maintainers#contributor-workflow for how to configure your cloned git repository to generate summaries and diffs that are easier to review. > v2: > 1. Incorporated styling comments > 2. Removed/Rewrite function referred from linux > 3. Created DS1307 Library under Silicon/Maxim and make it i2c driver based. > 4. Created i2c driver > > Meenakshi Aggarwal (9): > Platform/NXP: Add support for Big Endian Mmio APIs > Platform/NXP : Add support for Watchdog driver > SocLib : Add support for initialization of peripherals > Platform/NXP : Add support for DUART library > Platform/NXP: Add support for I2c driver > Silicon/Maxim : Add support for DS1307 RTC library > Platform/NXP: Add support for ArmPlatformLib > Compilation : Add the fdf, dsc and dec files. > Build : Add build script and environment script > > Platform/NXP/Drivers/I2cDxe/I2cDxe.c | 728 > + > Platform/NXP/Drivers/I2cDxe/I2cDxe.h | 64 ++ > Platform/NXP/Drivers/I2cDxe/I2cDxe.inf | 57 ++ > Platform/NXP/Drivers/WatchDog/WatchDog.c | 421 > Platform/NXP/Drivers/WatchDog/WatchDog.h | 37 ++ > Platform/NXP/Drivers/WatchDog/WatchDogDxe.inf | 47 ++ > Platform/NXP/Env.cshrc | 77 +++ > Platform/NXP/Include/Bitops.h | 179 + > Platform/NXP/Include/Library/BeIoLib.h | 332 ++ > Platform/NXP/LS1043aRdbPkg/LS1043aRdbPkg.dec | 29 + > Platform/NXP/LS1043aRdbPkg/LS1043aRdbPkg.dsc | 77 +++ > Platform/NXP/LS1043aRdbPkg/LS1043aRdbPkg.fdf | 281 > .../Library/PlatformLib/ArmPlatformLib.c | 105 +++ > .../Library/PlatformLib/ArmPlatformLib.inf | 70 ++ > .../Library/PlatformLib/NxpQoriqLsHelper.S | 38 ++ > .../Library/PlatformLib/NxpQoriqLsMem.c| 184 ++ In particular these ..., meaning reviewers need to start guessing which files are affected. Best Regards, Leif > Platform/NXP/Library/BeIoLib/BeIoLib.c | 400 +++ > Platform/NXP/Library/BeIoLib/BeIoLib.inf | 31 + > Platform/NXP/Library/DUartPortLib/DUart.h | 128 > Platform/NXP/Library/DUartPortLib/DUartPortLib.c | 331 ++ > Platform/NXP/Library/DUartPortLib/DUartPortLib.inf | 39 ++ > Platform/NXP/NxpQoriqLs.dec| 248 +++ > Platform/NXP/NxpQoriqLs.dsc| 453 + > Platform/NXP/Readme.md | 15 + > Platform/NXP/build.sh | 103 +++ > Silicon/Maxim/Library/Ds1307RtcLib/Ds1307Rtc.h | 59 ++ > Silicon/Maxim/Library/Ds1307RtcLib/Ds1307RtcLib.c | 327 + > .../Maxim/Library/Ds1307RtcLib/Ds1307RtcLib.dec| 26 + > .../Maxim/Library/Ds1307RtcLib/Ds1307RtcLib.inf| 45 ++ > Silicon/NXP/Chassis/Chassis.c | 413 > Silicon/NXP/Chassis/Chassis.h | 144 > Silicon/NXP/Chassis/Chassis2/Chassis2.dec | 19 + > Silicon/NXP/Chassis/Chassis2/SerDes.h | 69 ++ > Silicon/NXP/Chassis/Chassis2/Soc.c | 145 > Silicon/NXP/Chassis/Chassis2/Soc.h | 376 +++ > Silicon/NXP/Chassis/LS1043aSocLib.inf | 47 ++ > Silicon/NXP/Chassis/SerDes.c | 254 +++ > Silicon/NXP/LS1043A/Include/SocSerDes.h| 55 ++ > Silicon/NXP/LS1043A/LS1043A.dec| 22 + > Silicon/NXP/LS1043A/LS1043A.dsc| 82 +++ > 40 files changed, 6557 insertions(+) > create mode 100644 Platform/NXP/Drivers/I2cDxe/I2cDxe.c > create mode 100644 Platform/NXP/Drivers/I2cDxe/I2cDxe.h > create mode 100644 Platform/NXP/Drivers/I2cDxe/I2cDxe.inf > create mode 100644 Platform/NXP/Drivers/WatchDog/WatchDog.c > create mode 100644 Platform/NXP/Drivers/WatchDog/WatchDog.h > create mode 100644 Platform/NXP/Drivers/WatchDog/WatchDogDxe.inf > create mode 100755 Platform/NXP/Env.cshrc > create mode 100644 Platform/NXP/Include/Bitops.h > create mode 100644 Platform/NXP/Include/Library/BeIoLib.h > create mode 100644 Platform/NXP/LS1043aRdbPkg/LS1043aRdbPkg.dec > create mode 100644 Platform/NXP/LS1043aRdbPkg/LS1043aRdbPkg.dsc > create mode 100644 Platform/NXP/LS1043aRdbPkg/LS1043aRdbPkg.fdf > create mode 100644 >
[edk2] [PATCH edk2-platforms] [PATCH v3 0/9] Platform/NXP
v3: Added patch-prefix v2: 1. Incorporated styling comments 2. Removed/Rewrite function referred from linux 3. Created DS1307 Library under Silicon/Maxim and make it i2c driver based. 4. Created i2c driver Meenakshi Aggarwal (9): Platform/NXP: Add support for Big Endian Mmio APIs Platform/NXP : Add support for Watchdog driver SocLib : Add support for initialization of peripherals Platform/NXP : Add support for DUART library Platform/NXP: Add support for I2c driver Silicon/Maxim : Add support for DS1307 RTC library Platform/NXP: Add support for ArmPlatformLib Compilation : Add the fdf, dsc and dec files. Build : Add build script and environment script Platform/NXP/Drivers/I2cDxe/I2cDxe.c | 728 + Platform/NXP/Drivers/I2cDxe/I2cDxe.h | 64 ++ Platform/NXP/Drivers/I2cDxe/I2cDxe.inf | 57 ++ Platform/NXP/Drivers/WatchDog/WatchDog.c | 421 Platform/NXP/Drivers/WatchDog/WatchDog.h | 37 ++ Platform/NXP/Drivers/WatchDog/WatchDogDxe.inf | 47 ++ Platform/NXP/Env.cshrc | 77 +++ Platform/NXP/Include/Bitops.h | 179 + Platform/NXP/Include/Library/BeIoLib.h | 332 ++ Platform/NXP/LS1043aRdbPkg/LS1043aRdbPkg.dec | 29 + Platform/NXP/LS1043aRdbPkg/LS1043aRdbPkg.dsc | 77 +++ Platform/NXP/LS1043aRdbPkg/LS1043aRdbPkg.fdf | 281 .../Library/PlatformLib/ArmPlatformLib.c | 105 +++ .../Library/PlatformLib/ArmPlatformLib.inf | 70 ++ .../Library/PlatformLib/NxpQoriqLsHelper.S | 38 ++ .../Library/PlatformLib/NxpQoriqLsMem.c| 184 ++ Platform/NXP/Library/BeIoLib/BeIoLib.c | 400 +++ Platform/NXP/Library/BeIoLib/BeIoLib.inf | 31 + Platform/NXP/Library/DUartPortLib/DUart.h | 128 Platform/NXP/Library/DUartPortLib/DUartPortLib.c | 331 ++ Platform/NXP/Library/DUartPortLib/DUartPortLib.inf | 39 ++ Platform/NXP/NxpQoriqLs.dec| 248 +++ Platform/NXP/NxpQoriqLs.dsc| 453 + Platform/NXP/Readme.md | 15 + Platform/NXP/build.sh | 103 +++ Silicon/Maxim/Library/Ds1307RtcLib/Ds1307Rtc.h | 59 ++ Silicon/Maxim/Library/Ds1307RtcLib/Ds1307RtcLib.c | 327 + .../Maxim/Library/Ds1307RtcLib/Ds1307RtcLib.dec| 26 + .../Maxim/Library/Ds1307RtcLib/Ds1307RtcLib.inf| 45 ++ Silicon/NXP/Chassis/Chassis.c | 413 Silicon/NXP/Chassis/Chassis.h | 144 Silicon/NXP/Chassis/Chassis2/Chassis2.dec | 19 + Silicon/NXP/Chassis/Chassis2/SerDes.h | 69 ++ Silicon/NXP/Chassis/Chassis2/Soc.c | 145 Silicon/NXP/Chassis/Chassis2/Soc.h | 376 +++ Silicon/NXP/Chassis/LS1043aSocLib.inf | 47 ++ Silicon/NXP/Chassis/SerDes.c | 254 +++ Silicon/NXP/LS1043A/Include/SocSerDes.h| 55 ++ Silicon/NXP/LS1043A/LS1043A.dec| 22 + Silicon/NXP/LS1043A/LS1043A.dsc| 82 +++ 40 files changed, 6557 insertions(+) create mode 100644 Platform/NXP/Drivers/I2cDxe/I2cDxe.c create mode 100644 Platform/NXP/Drivers/I2cDxe/I2cDxe.h create mode 100644 Platform/NXP/Drivers/I2cDxe/I2cDxe.inf create mode 100644 Platform/NXP/Drivers/WatchDog/WatchDog.c create mode 100644 Platform/NXP/Drivers/WatchDog/WatchDog.h create mode 100644 Platform/NXP/Drivers/WatchDog/WatchDogDxe.inf create mode 100755 Platform/NXP/Env.cshrc create mode 100644 Platform/NXP/Include/Bitops.h create mode 100644 Platform/NXP/Include/Library/BeIoLib.h create mode 100644 Platform/NXP/LS1043aRdbPkg/LS1043aRdbPkg.dec create mode 100644 Platform/NXP/LS1043aRdbPkg/LS1043aRdbPkg.dsc create mode 100644 Platform/NXP/LS1043aRdbPkg/LS1043aRdbPkg.fdf create mode 100644 Platform/NXP/LS1043aRdbPkg/Library/PlatformLib/ArmPlatformLib.c create mode 100644 Platform/NXP/LS1043aRdbPkg/Library/PlatformLib/ArmPlatformLib.inf create mode 100644 Platform/NXP/LS1043aRdbPkg/Library/PlatformLib/NxpQoriqLsHelper.S create mode 100644 Platform/NXP/LS1043aRdbPkg/Library/PlatformLib/NxpQoriqLsMem.c create mode 100644 Platform/NXP/Library/BeIoLib/BeIoLib.c create mode 100644 Platform/NXP/Library/BeIoLib/BeIoLib.inf create mode 100644 Platform/NXP/Library/DUartPortLib/DUart.h create mode 100644 Platform/NXP/Library/DUartPortLib/DUartPortLib.c create mode 100644 Platform/NXP/Library/DUartPortLib/DUartPortLib.inf create mode 100644 Platform/NXP/NxpQoriqLs.dec create mode 100644 Platform/NXP/NxpQoriqLs.dsc create mode 100644 Platform/NXP/Readme.md create mode 100755 Platform/NXP/build.sh create mode 100644 Silicon/Maxim/Library/Ds1307RtcLib/Ds1307Rtc.h create mode 100644 Silicon/Maxim/Library/Ds1307RtcLib/Ds1307RtcLib.c create mode 100644