Release 2020.2 #187
Open
Release 2020.2 #187
+2,202,806
−179,215
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
… node start Invalidate child nodes of a PLD when new RM corresponding to it is run. This is because the topology of the PLD's children is no longer valid and subject to change. As part of invalidation, the node is removed from tree and links (such as Parent, Child & NextPeer) set to NULL. This is done at init node start phase since that's the first command to run when an RM is loaded Signed-off-by: Amit Sunil Dhamne <[email protected]> Acked-for-series: Izhar Ameer Shaikh <[email protected]>
PLDs are created by pm_add_node cmd present in static image or RM. Currently if same RM is run, PM errors out as PLD corresponding to a nodeid is already present in database. However, we should allow this operation since re-running same RM is valid. This change is to allow addition of duplicate add node cmds without erroring out. This change optimizes memory in case the nodeid passed has an entry in database without allocating new memory. Re-addition for PLD0 is illegal as PLD0 represents static image and added as part of topology CDO Signed-off-by: Amit Sunil Dhamne <[email protected]> Acked-for-series: Izhar Ameer Shaikh <[email protected]>
As part of Add Node Parent API, nodeid of PLD and the nodeid of parent PLD args are passed. In code PLDs are represented as multi-level linkedlist, that has only one root node (PLD0). Add check to make sure Parent Node (if valid), PLD's parent is not NULL. This will help prevent creation of broken trees as there are uninitialized nodes present in database Signed-off-by: Amit Sunil Dhamne <[email protected]> Acked-for-series: Izhar Ameer Shaikh <[email protected]>
Scanf is expected to read data from stdin. For that to happen, __read needs to be customized in IAR port to always read data from stdin. Signed-off-by: Mubin Usman Sayyed <[email protected]> Acked-by: Siva Durga Prasad Paladugu <[email protected]>
This patch fix following violation "misra_c_2012_rule_8_4_violation:Object definition does not have a visible prototype" Fixed this violation by keeping the variable declaration in the header file. Signed-off-by: Srinivas Neeli <[email protected]> Acked-for-series: Srinivas Goud <[email protected]>
This patch fix following violation "misra_c_2012_rule_10_3_violation:Implicit conversion from essential type to different or narrower essential type "signed 32-bit int" Fixed this violation across the driver by explicitly typecasting at all the required/intended places. Signed-off-by: Srinivas Neeli <[email protected]> Acked-for-series: Srinivas Goud <[email protected]>
This patch fix following violation "misra_c_2012_rule_10_6_violation:Assigning composite expression of width 32 to a target of width 64" Fixed this violation across the driver files by changing the type. Signed-off-by: Srinivas Neeli <[email protected]> Acked-for-series: Srinivas Goud <[email protected]>
This patch fix following violation "misra_c_2012_rule_10_4_violation:Essential type of the left hand operand(float) is not the same as that of the right operand(unsigned)" Fixed this violation by adding "U" to unsigned integer constants at all required places. Signed-off-by: Srinivas Neeli <[email protected]> Acked-for-series: Srinivas Goud <[email protected]>
This patch fix following violation "misra_c_2012_directive_4_5_violation:Declaration with identifier "FuncPtr", which is ambiguous" Fixed this violation by changing the funciton argument name. Signed-off-by: Srinivas Neeli <[email protected]> Acked-for-series: Srinivas Goud <[email protected]>
To maintain common signature in all drivers removing const keyword. Signed-off-by: Srinivas Neeli <[email protected]> Acked-for-series: Srinivas Goud <[email protected]>
Updated changelog file for rtcpsu driver. Signed-off-by: Srinivas Neeli <[email protected]> Acked-for-series: Srinivas Goud <[email protected]>
Driver should only check the datapath mode for Gen 3 devices. Signed-off-by: Conall O'Griofa <conall.o'[email protected]> Acked-by: Anand Ashok Dumbre <[email protected]>
… to typecast Dp subsystem pointer Updated the Dprx_InterruptHandlerPayloadAlloc() in MST PT application to typecast Dp Subsystem pointer Signed-off-by: nishantd <[email protected]> Acked by: Kapil Usgaonkar<[email protected]>
When AXI IIC is used as slave, NAAS interrupt is used to recognize the end of current transaction. In high traffic master-slave communication with slow slave response, there is a chance of AAS and 1 byte received in FIFO even before NAAS interrupt handler is completed. If the flushing of RxFIFO happens, the one byte of next transaction is lost. To avoid this, flush the RxFIFO only after checking the status of controller. Signed-off-by: Raviteja Narayanam <[email protected]> Acked-by: Srinivas Goud <[email protected]>
MISRA C Directive 4.7 States: If a function returns error information, then that error information shall be tested This patch fixes MISRA C Directive 4.7 violation by checking for return statements for XST_SUCCESS/XST_FAILURE from functions. Signed-off-by: Tarun DTS <[email protected]> Acked-for-series: Vikram Sreenivasa Batchali <[email protected]>
MISRA C Rule 10.1 States: Operands shall not be of an inappropriate essential type This patch fixes MISRA C Rule 10.1 violation by using the proper type variables where required or typecasting if necessary Signed-off-by: Tarun DTS <[email protected]> Acked-for-series: Vikram Sreenivasa Batchali <[email protected]>
MISRA C Rule 10.3 States: The value of an expression shall not be assigned to an object with a narrower essential type or of a different essential type category This patch fixes MISRA C Rule 10.3 violation by using the proper type variables where required or typecasting if necessary Signed-off-by: Tarun DTS <[email protected]> Acked-for-series: Vikram Sreenivasa Batchali <[email protected]>
MISRA C Rule 10.4 States: Both operands of an operator in which the usual arithmetic conversionsare performed shall have the same essential type category This patch fixes MISRA C Rule 10.4 violation by using the same type variable for both operands where possible Signed-off-by: Tarun DTS <[email protected]> Acked-for-series: Vikram Sreenivasa Batchali <[email protected]>
MISRA C Rule 10.6 States: The value of a composite expression shall not be assigned to an object with wider essential type This patch fixes MISRA C Rule 10.6 violation by using the same type variables for composite expression Signed-off-by: Tarun DTS <[email protected]> Acked-for-series: Vikram Sreenivasa Batchali <[email protected]>
MISRA C Rule 10.7 States: If a composite expression is used as one operand of an operator in which the usual arithmetic conversions are performed then the other operand shall not have wider essential type This patch fixes MISRA C Rule 10.7 violation by using the same type variable types for both operands in an arithmetic expression Signed-off-by: Tarun DTS <[email protected]> Acked-for-series: Vikram Sreenivasa Batchali <[email protected]>
MISRA C Rule 11.9 States: The macro NULL shall be the only permitted form of integer null pointer constant This patch fixes MISRA C Rule 11.9 violation by using NULL instead of 0U for null poitner Signed-off-by: Tarun DTS <[email protected]> Acked-for-series: Vikram Sreenivasa Batchali <[email protected]>
MISRA C Rule 12.2 States: The right hand operand of a shift operator shall lie in the range zero to one less than the width in bits of the essential type of the left hand operand This patch fixes MISRA C Rule 12.2 violation by casting constants with higher type when shifting more than 7 bits Signed-off-by: Tarun DTS <[email protected]> Acked-for-series: Vikram Sreenivasa Batchali <[email protected]>
MISRA C Rule 13.2 States: The value of an expression and its persistent side effects shall be the same under all permitted evaluation orders This patch fixes MISRA C Rule 13.2 violation by using separate statements for function calls which include volatile variable access Signed-off-by: Tarun DTS <[email protected]> Acked-for-series: Vikram Sreenivasa Batchali <[email protected]>
For HD mode, bit 4 and 7 of byte 3 (bit 23 and 20) of ST352 payload determine the colorimetry per SMPTE 292-1:2018 Sec 9.5. The same thing is observed with Phabrix Qx for 3GB mode with payload 0x8A. Signed-off-by: Vishal Sagar <[email protected]> Acked-for-series: Sandip Kothari <[email protected]>
There is a possibility that incoming video stream parameters like EOTF function can change on the fly without the video getting unlocked and then locked again. This patch handles such a scenario by checking the incoming stream's ST352 payload against older value on vsync interrupt and updating the stream's parameters like format, frame rate and EOTF. A new interrupt handler is added for Vsync interrupt Signed-off-by: Vishal Sagar <[email protected]> Acked-for-series: Sandip Kothari <[email protected]>
This patch adds support for the new Vsync handler. Signed-off-by: Vishal Sagar <[email protected]> Acked-for-series: Sandip Kothari <[email protected]>
Fix no video output on changing resolution by updating the canonical definitions of UART Signed-off-by: Shruti Gupta <[email protected]> Acked by: Kapil Usgaonkar<[email protected]>
New APIs will enable user to configure thresholds for temperature and voltages Signed-off-by: Anand Ashok Dumbre <[email protected]>
New APIs will allow user to set alarm config and alarm modes Signed-off-by: Anand Ashok Dumbre <[email protected]>
The new interrupt handling structure allows user to set individual callbacks for individual supply and temperature alarms Signed-off-by: Anand Ashok Dumbre <[email protected]>
Updated for changelog 2020.2 xilsem Signed-off-by: Meena Paleti <[email protected]> Acked-by : Siva Addepalli<[email protected]>
Updated for changelog devcfg and avbuf Signed-off-by: Meena Paleti <[email protected]> Acked-by : Siva Addepalli<[email protected]>
Updated Changelog Signed-off-by: Conall O'Griofa <conall.o'[email protected]> Acked-by: Anand Ashok Dumbre <[email protected]>
Updated driver version to 2.4 Signed-off-by: Rohit Visavalia <[email protected]> Reviewed-by: Hyun Kwon <[email protected]>
use unsigned char data type to access individual bytes for unaligned memory access. Without this, if the unaligned data is signed value, the sign extension will corrupt the data before writing to the data memory. Signed-off-by: Tejus Siddagangaiah <[email protected]> Acked-by: Wendy Liang <[email protected]>
updated for addtogroup avbuf Signed-off-by: Meena Paleti <[email protected]> Acked-by : Siva Prasad Addepalli<[email protected]>
Updated changelog for axipcie Signed-off-by: Meena Paleti <[email protected]> Acked-by : Siva Addepalli<[email protected]>
This patch changes device names referenced within the code to match the external device names. Signed-off-by: Nicole Baze <[email protected]> Acked-for-series: Izhar Ameer Shaikh <[email protected]>
This patch changes the device names referenced within the code to match the external device names. Signed-off-by: Nicole Baze <[email protected]> Acked-for-series: Izhar Ameer Shaikh <[email protected]>
This patch fixes the below issues in encrypted boot: - Initializes Global Variation Filter in PufData structure to TRUE. This will fix PUF regeneration failure at PLM due to GVF mismatch. - Forces DpaCmCfg back to the value read from the PDI after comparison with EfuseDpaCm value. This will fix invalid input error for DPACM disabled boards. Signed-off-by: Harsha <[email protected]> Acked-by: Ramakrishna Ganeshu Poolla <[email protected]>
BSP changelog updated with standalone 7.3 updates. Signed-off-by: Mubin Usman Sayyed <[email protected]> Acked-by: Siva Durga Prasad Paladugu <[email protected]>
- ImageSelector was checking for a different string and not "ABUM" as identification string. The check was failing and Image Selector did a soft reset and CSU ROM booted the next available image, which was always ImageA irrespective of the values of persistent registers The patch rectifies the issue by checking the identification string with "ABUM" - The patch rectifies the checksum calculation logic by calculating checksum on the first 8 words of persistent register space Signed-off-by: Anantha Umapathi <[email protected]> Acked-by: Vikram Sreenivasa Batchali <[email protected]>
Updated for changelog cframe and others Signed-off-by: Meena Paleti <[email protected]> Acked-by : Siva Addepalli<[email protected]>
New rules include: - if distributing full rate clock, the souece tile must be a centre tile (ADC 1,2 or DAC 1,2) - no division of full rate clock - Full rate clock can not be passed from ADC to ADC Signed-off-by: Conall O'Griofa <conall.o'[email protected]> Acked-by: Matthew Roche <[email protected]>
New rules include: - if distributing full rate clock, the souece tile must be a centre tile (ADC 1,2 or DAC 1,2) - no division of full rate clock - Full rate clock can not be passed from ADC to ADC Signed-off-by: Conall O'Griofa <conall.o'[email protected]> Acked-by: Matthew Roche <[email protected]> Acked-by : Siva Addepalli<[email protected]>
…sw/embeddedsw into master-rel-2020.2
This patch fixes the issue by masking SSS register with corresponding inputsrc mask value instead of enum value Signed-off-by: Praveen Teja Kundanala <[email protected]> Acked-by: Sarat Chand Savitala <[email protected]>
The patch updates the text on GUI of image recovery to make it more generic. Signed-off-by: Vikram Sreenivasa Batchali <[email protected]> Acked-by: Mohan Marutirao Dhanawade <[email protected]>
… folder The patch checks in latest web.img which has changes corresponding to renaming of variables in index.js and index.htm Signed-off-by: Vikram Sreenivasa Batchali <[email protected]> Acked-by: Mohan Marutirao Dhanawade <[email protected]>
The patch updates text on Vitis GUI and image recovery webpage GUI to make the app more generic. Signed-off-by: Vikram Sreenivasa Batchali <[email protected]> Acked-by: Mohan Marutirao Dhanawade <[email protected]>
… libraries Signed-off-by: Siva Addepalli <[email protected]>
Signed-off-by: Siva Addepalli <[email protected]>
This patch adds support for versionless build support for PSMFW for 2020.2 release. Signed-off-by: Rajan Vaja <[email protected]> Acked-by: Nicole Baze <[email protected]>
There is a warning for "omitting copy of directory" as -r option was not used. As such there is no need to copy subdirectory, but to avoid warning use "cp -r". Signed-off-by: Ravi Patel <[email protected]> Acked-by: Amanda Nicole Baze <[email protected]>
|
hi @llr891011 - what is this PR for ? It says learn ? We donot accept any PRs , if you have anyspecific fix then please send the patch to [email protected] |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
learn