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 > 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 Silicon/Maxim/Library/Ds1307RtcLib/Ds1307RtcLib.dec > create mode 100644 Silicon/Maxim/Library/Ds1307RtcLib/Ds1307RtcLib.inf > create mode 100644 Silicon/NXP/Chassis/Chassis.c > create mode 100644 Silicon/NXP/Chassis/Chassis.h > create mode 100644 Silicon/NXP/Chassis/Chassis2/Chassis2.dec > create mode 100644 Silicon/NXP/Chassis/Chassis2/SerDes.h > create mode 100644 Silicon/NXP/Chassis/Chassis2/Soc.c > create mode 100644 Silicon/NXP/Chassis/Chassis2/Soc.h > create mode 100644 Silicon/NXP/Chassis/LS1043aSocLib.inf > create mode 100644 Silicon/NXP/Chassis/SerDes.c > create mode 100644 Silicon/NXP/LS1043A/Include/SocSerDes.h > create mode 100644 Silicon/NXP/LS1043A/LS1043A.dec > create mode 100644 Silicon/NXP/LS1043A/LS1043A.dsc > > v1: > Following patches will add support of NXP SoCs in edk2-platforms. > > Our directory structure will be: > > edk2-platforms/ > |-- Platform > | |-- NXP > | | |-- build.sh > | | |-- Drivers > | | |-- Env.cshrc > | | |-- Include > | | | `-- Library > | | | `-- Drivers > | | |-- Library > | | |-- LS1043aRdbPkg > | | | |-- Drivers > | | | |-- Include > | | | | `-- Library > | | | | `-- Drivers > | | | |-- Library > | | | |-- LS1043aRdbPkg.dec > | | | |-- LS1043aRdbPkg.dsc > | | | `-- LS1043aRdbPkg.fdf > | | |-- NxpQoriqLs.dec > | | |-- NxpQoriqLs.dsc > | | `-- Readme.md > `-- Silicon > |-- NXP > |-- Chassis > | |-- Chassis2 > `-- LS1043A > |-- Include > |-- LS1043A.dec > `-- LS1043A.dsc > > In Silicon/NXP, we are keeping our SoC specific information and remaining > code will be kept in Platform/NXP. > > Platform/NXP/LS1043aRdbPkg will host .dsc and .fdf files to support > compilation for LS1043A RDB board. > > In next series of patches we will be adding support for LS2088 and LS1046 > board. > > > Looking forward for your kind support in upstreaming our board in > edk2-platforms. > > > Meenakshi Aggarwal (10): > Platform/NXP: Library to provide helper functions. > Platform/NXP: Add support for system reset library > Platform/NXP: Add support for Big Endian Mmio APIs > Platform/NXP : Add support for Watchdog driver > Platform/NXP : Add support for DUART library > Platform/NXP: Add support for I2c operations library > Platform/NXP : Add support for DS1307 RTC library > Platform/NXP: Add support for ArmPlatformLib > SocLib : Add support for initialization of peripherals > Compilation : Add the fdf, dsc and dec files. > > Platform/NXP/Drivers/WatchDog/WatchDog.c | 386 +++++++++++++++ > Platform/NXP/Drivers/WatchDog/WatchDog.h | 37 ++ > Platform/NXP/Drivers/WatchDog/WatchDogDxe.inf | 47 ++ > Platform/NXP/Env.cshrc | 75 +++ > Platform/NXP/Include/Bitops.h | 179 +++++++ > Platform/NXP/Include/Library/BeIoLib.h | 332 +++++++++++++ > Platform/NXP/Include/Library/I2c.h | 125 +++++ > Platform/NXP/Include/Library/Utils.h | 137 +++++ > Platform/NXP/LS1043aRdbPkg/LS1043aRdbPkg.dec | 29 ++ > Platform/NXP/LS1043aRdbPkg/LS1043aRdbPkg.dsc | 74 +++ > Platform/NXP/LS1043aRdbPkg/LS1043aRdbPkg.fdf | 279 +++++++++++ > .../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 | 334 +++++++++++++ > Platform/NXP/Library/DUartPortLib/DUartPortLib.inf | 39 ++ > Platform/NXP/Library/Ds1307RtcLib/Ds1307Rtc.h | 40 ++ > Platform/NXP/Library/Ds1307RtcLib/Ds1307RtcLib.c | 226 +++++++++ > Platform/NXP/Library/Ds1307RtcLib/Ds1307RtcLib.inf | 40 ++ > Platform/NXP/Library/I2cLib/I2cLib.c | 549 > +++++++++++++++++++++ > Platform/NXP/Library/I2cLib/I2cLib.h | 109 ++++ > Platform/NXP/Library/I2cLib/I2cLib.inf | 43 ++ > .../NXP/Library/ResetSystemLib/ResetSystemLib.c | 96 ++++ > .../NXP/Library/ResetSystemLib/ResetSystemLib.inf | 33 ++ > Platform/NXP/Library/UtilsLib/Utils.c | 97 ++++ > Platform/NXP/Library/UtilsLib/Utils.inf | 30 ++ > Platform/NXP/NxpQoriqLs.dec | 257 ++++++++++ > Platform/NXP/NxpQoriqLs.dsc | 453 +++++++++++++++++ > Platform/NXP/Readme.md | 14 + > Platform/NXP/build.sh | 100 ++++ > Silicon/NXP/Chassis/Chassis.c | 393 +++++++++++++++ > Silicon/NXP/Chassis/Chassis.h | 123 +++++ > Silicon/NXP/Chassis/Chassis2/Chassis2.dec | 19 + > Silicon/NXP/Chassis/Chassis2/SerDes.h | 82 +++ > Silicon/NXP/Chassis/Chassis2/Soc.c | 146 ++++++ > Silicon/NXP/Chassis/Chassis2/Soc.h | 376 ++++++++++++++ > Silicon/NXP/Chassis/LS1043aSocLib.inf | 48 ++ > Silicon/NXP/Chassis/SerDes.c | 253 ++++++++++ > Silicon/NXP/LS1043A/Include/SocSerDes.h | 55 +++ > Silicon/NXP/LS1043A/LS1043A.dec | 22 + > Silicon/NXP/LS1043A/LS1043A.dsc | 82 +++ > 45 files changed, 6715 insertions(+) > 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 100644 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/Include/Library/I2c.h > create mode 100644 Platform/NXP/Include/Library/Utils.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/Library/Ds1307RtcLib/Ds1307Rtc.h > create mode 100644 Platform/NXP/Library/Ds1307RtcLib/Ds1307RtcLib.c > create mode 100644 Platform/NXP/Library/Ds1307RtcLib/Ds1307RtcLib.inf > create mode 100644 Platform/NXP/Library/I2cLib/I2cLib.c > create mode 100644 Platform/NXP/Library/I2cLib/I2cLib.h > create mode 100644 Platform/NXP/Library/I2cLib/I2cLib.inf > create mode 100644 > Platform/NXP/Library/ResetSystemLib/ResetSystemLib.c > create mode 100644 > Platform/NXP/Library/ResetSystemLib/ResetSystemLib.inf > create mode 100644 Platform/NXP/Library/UtilsLib/Utils.c > create mode 100644 Platform/NXP/Library/UtilsLib/Utils.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/NXP/Chassis/Chassis.c create mode 100644 > Silicon/NXP/Chassis/Chassis.h create mode 100644 > Silicon/NXP/Chassis/Chassis2/Chassis2.dec > create mode 100644 Silicon/NXP/Chassis/Chassis2/SerDes.h > create mode 100644 Silicon/NXP/Chassis/Chassis2/Soc.c > create mode 100644 Silicon/NXP/Chassis/Chassis2/Soc.h > create mode 100644 Silicon/NXP/Chassis/LS1043aSocLib.inf > create mode 100644 Silicon/NXP/Chassis/SerDes.c create mode 100644 > Silicon/NXP/LS1043A/Include/SocSerDes.h > create mode 100644 Silicon/NXP/LS1043A/LS1043A.dec create mode > 100644 Silicon/NXP/LS1043A/LS1043A.dsc > > -- > 1.9.1 > _______________________________________________ edk2-devel mailing list edk2-devel@lists.01.org https://lists.01.org/mailman/listinfo/edk2-devel