Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Update xuartlite_intr_example.c #249

Open
wants to merge 594 commits into
base: master
Choose a base branch
from

Conversation

ChatterjeePracheta
Copy link

In the previous code the Recv and Send commands were not placed properly but the code was running as uartlite ip is an rtos( it tries to execute tasks with same priority parallely). Also the ASCII values below 'A' are not printed properly in vitis serial terminal or tera term. So in this update Recv and Send commands are placed in proper sequential manner for better understanding and also documentation is also changed a bit keeping that in mind. The sendbuffer initial value now starts from A so that all the characters are printed properly.

Please do not submit a Pull Request via github. Our project makes use of mailing lists for patch submission and review. For more details please see https://xilinx-wiki.atlassian.net/wiki/spaces/A/pages/18842172/Create+and+Submit+a+Patch

praveen teja kundanala and others added 30 commits August 12, 2022 17:27
…glitch attack

Modified if check for XPuf_ChangeIroFreq to avoid returning XST_SUCCESS incase of
glitch attack

Signed-off-by: Praveen Teja Kundanala <[email protected]>
Acked-by: Mohan Marutirao Dhanawade <[email protected]>
This patch zeroizes the user key in XSecure_AesDecryptCmKat

Signed-off-by: Praveen Teja Kundanala <[email protected]>
Acked-by: Mohan Marutirao Dhanawade <[email protected]>
This patch remvoes the logic of deleting source directories from the
tcl.

Signed-off-by: Harsha Harsha <[email protected]>
Acked-by: Kalyani Akula <[email protected]>
Update tcl to fix issues with versal net platform

Signed-off-by: Siva Durga Prasad Paladugu <[email protected]>
Acked-by:  Mubin Sayyed <[email protected]>
Update arguments description for microblaze_register_exception_handler()
to fix documentation issue with it.

Signed-off-by: Siva Durga Prasad Paladugu <[email protected]>
Acked-by:  Mubin Sayyed <[email protected]>
When ROW37 protection bit is set in eFUSE Page 0, Row 0 then ROM expects
a non-zero value in EFUSE_CACHE_BOOT_ENV_CTRL and bit[31] should be set
for all TRIM_2, TRIM_6, TRIM_7 & TRIM_12.
So in xilnvm instead of checking for TRIM2, TRIM3 is being checked during
Cache reload.

Signed-off-by: Kalyani Akula <[email protected]>
Acked-by: Mohan Marutirao Dhanawade <[email protected]>
Change remote firmware to not change flag state based on APU message.
Mimic convention of other vendors' OpenAMP Remote firmwares and have these
remotes keep going in an infinite loop.

Note that there are multiple options for RPU after shutdown signal. Among
these can be to put RPU in deep sleep mode.

Signed-off-by: Ben Levinsky <[email protected]>
Acked-by: Sergei Korneichuk <[email protected]>
Usage of const keyword to aviod accidental overwrite for function pointers
in Versal psmfw.

Signed-off-by: HariBabu Gattem <[email protected]>
Acked-by: Ronak Jain <[email protected]>
Since PLD topology is now enabled through tools, PLPD and NPD
housecleaning is triggered during PLD init_start by PLM.
Thus, PLPD and NPD housecleaning commands from PL images are now redundant
and soon will be removed.

These prints are alarming to users but harmless overall, so
reduce their level to INFO which will suppress them.

Signed-off-by: Izhar Ameer Shaikh <[email protected]>
Acked-by: Nicole Baze <[email protected]>
This patch adds the following,

-Link training State Machine and its supported functions.
-Callback function for XDPTXSS_HANDLER_DP_FFE_PRESET_ADJUST event, to
 configure/modify ffe preset values which is required for TX  link training.
-Enables constants and DP2.1 TX and RX specific hardware AUX registers and
 DPCD registers.
-Removes the conditional protocol check of DP1.4 or DP2.1 in the driver
 wherever the logic is needed for both protocols.
-Updates MST Payload calculation for DP2.1 TX, XDp_TxCalculateTs
 function gets updated to calculate pbn values for dp2.1 as per specification.
-Updates Vfreq values specific for DP2.1 TX
-Updates RX Payload allocation for DP2.1 RX in XDp_RxAllocatePayloadStream
 function
-Removes unused variables
-Removes DP2.1 TX Remote I2C sideband message writes as some of branch
 devices like UCD400 is not supporting.
-Add initial default time delays required for DP2.1 RX Link training

Signed-off-by: Lakshmi Prasanna Eachuri <[email protected]>

Acked-for-series: Rajesh Gugulothu <[email protected]>
…tocol

This patch enables the following,

-Enables XDPTXSS_HANDLER_DP_FFE_PRESET_ADJUST callback functionality
 to adjust FFE preset values for link training
-Removes Protocol check for DP1.4/DP2.1 whereever the functionality
 needs for both protocols.
-Adds DP2.1 specific link rates in all condition checks whereever
 checking for DP1.4 Link rates.
-Changes MST topology discovery to happen after successful link training
-Add new sideband MST message EnumPathRequest to get to know the
 available PBN bandwidth from downstream receiver

Signed-off-by: Lakshmi Prasanna Eachuri <[email protected]>

Acked-for-series: Rajesh Gugulothu <[email protected]>
IPI tasks are set to low priority,to create
room for creating high priority tasks.

Signed-off-by: Sreedhar Kundella <[email protected]>
Acked-by: Bharath Mulagondla <[email protected]>
Add check to support backward compartibility for ddr4 memory ip

Signed-off-by: Ashwini Lomate <[email protected]>
Acked-by: Varalaxmi Bingi <[email protected]>
Update the tcl and fix the interrupt ID's in the xparameters_ps.h file.

Signed-off-by: Appana Durga Kedareswara rao <[email protected]>

Acked-for-series: Siva Durga Prasad Paladugu <[email protected]>
Update tcl to add support for VERSAL NET for interrupt wrapper code.

Signed-off-by: Appana Durga Kedareswara rao <[email protected]>

Acked-for-series: Siva Durga Prasad Paladugu <[email protected]>
Commit adds support for versal net.

Signed-off-by: Appana Durga Kedareswara rao <[email protected]>

Acked-for-series: Siva Durga Prasad Paladugu <[email protected]>
Commit adds support for cortexr52 processor.

Signed-off-by: Appana Durga Kedareswara rao <[email protected]>

Acked-for-series: Siva Durga Prasad Paladugu <[email protected]>
Added doxygen tags for doxygen documentation.

Signed-off-by: Appana Durga Kedareswara rao <[email protected]>

Acked-for-series: Siva Durga Prasad Paladugu <[email protected]>
Added support for gpio to monitor gtpowergood

Signed-off-by: nishantd <[email protected]>
Acked-by: Kapil Usgaonkar <[email protected]>
Currently PLM is not sending power up/down interrupts to PSM for request
power up/down power islands. Add logic to send power up/down request to PSM.

Signed-off-by: Jay Buddhabhatti <[email protected]>
Acked-by: Trung Tran<[email protected]>
Remove unused common xpm_device.c file from versal_net specific
directory.

Signed-off-by: Jay Buddhabhatti <[email protected]>
Acked-by: Trung Tran<[email protected]>
… API

This patch corrects the END label position in the
Xil_WaitForEventSet() API.

Signed-off-by: Appana Durga Kedareswara rao <[email protected]>
Signed-off-by: Appana Durga Kedareswara rao <[email protected]>
Acked-by: Siva Durga Prasad Paladugu <[email protected]>
LpdScanClear has missing update to status which could result in status
returning false success. Set status to failure during check for ScanClear
pass.

Signed-off-by: Nicole Baze <[email protected]>
Acked-by: Jyotheeswar Reddy Mutthareddyvari <[email protected]>
-This patch fixes Misra C violations in xilsem client

Signed-off-by: HKRNV Gupta <[email protected]>
Acked-by: Rama devi Veggalam <[email protected]>
This patch corrects the NumOfMSTStreams variable usage in the
Dp_Get_Topology function. It should be defined as NumOfMstStreams,
but used as NumOfMSTStreams which is wrong. Now it is corrected.

Signed-off-by: Lakshmi Prasanna Eachuri <[email protected]>
Acked-by:  Venkateshwar Rao Gannavarapu <[email protected]>
DP14 drivers compilation is failing as timer driver is not getting pulled into bsp.
This is because HDCP mdd file is not getting compiled instead it is just
pulled into bsp.

Adding tmrctr as dependent flag in DP14TXSS & DP14RXSS drivers resolved
this issue.

Signed-off-by: Prasad Gutti <[email protected]>

Acked-for-series: Sandip Kothari <[email protected]>
Updated addtogroup of DP14 drivers from older version to newer version
in their respective files.

Signed-off-by: Prasad Gutti <[email protected]>

Acked-for-series: Sandip Kothari <[email protected]>
This patch removes duplicate defintion of macros common for client and
server code and moves them in common file.

Signed-off-by: Harsha Harsha <[email protected]>

Acked-for-series: Kalyani Akula <[email protected]>
This patch adds support for PUF_RO_SWAP in examples.

Signed-off-by: Harsha Harsha <[email protected]>

Acked-for-series: Kalyani Akula <[email protected]>
The start address of Syndrome data in eFUSE cache is changed for Versal
Net. Hence this patch updates the definition of XPUF_EFUSE_SYN_ADD_INIT
macro.

Signed-off-by: Harsha Harsha <[email protected]>

Acked-for-series: Kalyani Akula <[email protected]>
Bharath Mulagondla and others added 27 commits September 19, 2022 12:27
This patch reduces the maximum number of partition PLM allows from 32
to 20. The maximum number of images is also reduced from 32 to 10.
This is for versal only. Versal_net is having both as 32.

Signed-off-by: Bharath Mulagondla <[email protected]>

Acked-for-series: Mounika Akula <[email protected]>
Scatterwrite commands are not required in versal. So moving them
to versal_net specific files to reduce size in versal plm.

Signed-off-by: Bharath Mulagondla <[email protected]>

Acked-for-series: Mounika Akula <[email protected]>
As part of the below commit the suspend-resume functionality got
broken.
Commit-Id= 9da81ab
"zynqmp_pmufw: Do not turn off FPD for ETH wakeup source"

The actual issue with this commit is, we are deriving invalid data
type from the ETH Slaves and the data type that we are expecting is
"PmWakeEventGicProxy".

Here we want to derive the PmWakeEventGicProxy data structure type,
but we are getting "PmWakeEventEth" because the GIC wake event is
registered in the Subwake variable of the PmWakeEventEth data
structure.

So to address this issue first we derived the PmWakeEventEth data
structure and then we derived the PmWakeEventGicProxy data
structure from it.

Signed-off-by: Ronak Jain <[email protected]>
Acked-by: Madhav Bhatt <[email protected]>
…vice

Security policy handling was based on incorrect assumptions during
request and release node path. Update it to be as follows:

                    |Slave = secure         |Slave = non-secure

Master = secure     |Allowed, cmd = S       |Not allowed

Master = non-secure |Allowed, cmd = S/NS    |Allowed, cmd = S/NS

In addition, when adding peripheral requirements for default and pmc
subsystems, add the slave security policy as REQ_ACCESS_SECURE. This
helps because any device with a *master* port can then be requested in
secure mode if the topology CDO has support for it.

Signed-off-by: Izhar Ameer Shaikh <[email protected]>

Acked-for-series: Jay Buddhabhatti <[email protected]>
…and TCM banks

PM_CAP_SECURE capability is applicable only for devices with a
TrustZone configurable master port - such as boot devices among other
few peripherals. Remove usage of this capability for non-applicable
devices such as PSM, DDR and TCM banks. PSM has fixed TrustZone setting,
DDR and TCM devices do not support configurable TrustZones.

Note:
For PSM TrustZone setting is configurable in versal_net but it is
deprecated in HW. Nonetheless, xilpm doesn't model trustzone
configuration registers for PSM so it can't be changed anyway.

Signed-off-by: Izhar Ameer Shaikh <[email protected]>

Acked-for-series: Jay Buddhabhatti <[email protected]>
Updated for changelog

Signed-off-by: Meena Paleti <[email protected]>

Acked-by : Siva Addepalli<[email protected]>
…master

Once the PLM to PLM communication is enabled, PLM uses SSIT sync event for synchronization between Master and Slave SLRs. While master is checking for the synchronization from Slave SLRs, it is not checking the SlavesMask and trying to read the memory of secondary SLR which is not present. This is causing the interconnect to hang and partial PDI load is failing. This patch fixes the same and checks for SlavesMask and reads the SSIT sync event pending status only from the required Slave SLRs.

Signed-off-by: Mounika Akula <[email protected]>
Acked-by: Sreedhar Kundella <[email protected]>
Updated the onsemi R3 RX register values

Signed-off-by: S Shreesh <[email protected]>

Acked-for-series: Kapil Usgaonkar <[email protected]>
Updated the onsemi R3 RX register values

Signed-off-by: S Shreesh <[email protected]>

Acked-for-series: Kapil Usgaonkar <[email protected]>
This patch optimizes the library by changing u8 to u32

Signed-off-by: VNSL Durga Challa <[email protected]>

Acked-for-series: Kalyani Akula <[email protected]>
This patch optimizes the code size by removing initializations

Signed-off-by: VNSL Durga Challa <[email protected]>

Acked-for-series: Kalyani Akula <[email protected]>
XPmDDRDevice_IsInSelfRefresh() would return success in the case where no
DDRMCs are enabled in the design, tricking that DRAMs are in self
refresh mode. This results in NoC Clock gating during PLPD power down
and NoC being inaccessible as a result. This is not a desired behavior.
Fix this by updating the function to return success when at least one
DDRMC is enabled in the design and all the active DRAMs are in
self-refresh mode.

HandleDDRDeviceState() now only supports the state transitions which are
explicitly defined in XPmDDRDevTransitions[].

Remove unnecessary error prints from XPmDDRDevice_EnterSelfRefresh()
and XPmDDRDevice_ExitSelfRefresh(). Error codes on failure are already
being printed in those functions. This will help with reducing the
code size by a little bit.

Add PmDbg() prints at entry/exit for PowerEvent() FSM handler since
it helps in debugging because of the recursive nature of this handler.
These are disabled by default, thus no impact on code size.

Signed-off-by: Izhar Ameer Shaikh <[email protected]>
Acked-by: Nicole Baze <[email protected]>
Version number needs to be updated.

Signed-off-by: Conall O'Griofa <conall.o'[email protected]>
Acked-by: Kadamathikuttiyil Karthikeyan Pillai, Anish <[email protected]>
Fixed race condition in the interrupt mask and id generation in case
multiple interrupt source port names having the same name (i.e
interrupt-driven from the same source pin using multiple slices and
concat blocks).

Signed-off-by: Appana Durga Kedareswara rao <[email protected]>
Acked-by: Siva Durga Prasad Paladugu <[email protected]>
As per current implementation the XRAM device is getting released
from PM_INIT_FINALIZE and because of that XRAM registers are not
accessible from Linux space.

Hence excluding the XRAM device from the PM_INIT_FINALIZE.

Signed-off-by: Ronak Jain <[email protected]>
Acked-by: Nicole Baze <[email protected]>
vprocss driver tcl file is not working properly when IP instances are
greater than 10 in a system. It creates and issue in g.c file as subcore
structure elements are not generated properly.

Fixed this issue in tcl file resolve this g.c file issue, and did the
following changes in driver.

1. TCL file is modified to resolve g.c issue
2. mdd file is modified to increase driver minor version.
3. addtogroup is updated from vprocss_v2_10 to vprocss_v2_11

Signed-off-by: Prasad Gutti <[email protected]>
Acked-by: Sandip Kothari <[email protected]>
Removed debug prints in a78 and r52 static files, which are
generated from IPCores file.

Signed-off-by: Ankush Mehtre <[email protected]>
Acked-by: VNSL Durga Challa <[email protected]>
Updated for changelog

Signed-off-by: Meena Paleti <[email protected]>

Acked-by : Siva Addepalli<[email protected]>
updated changelog for mipicsiss_v1_8 and other components

Signed-off-by: Meena Paleti <[email protected]>

Acked-by : Siva Addepalli<[email protected]>
updating the license file
Signed-off-by: Manikanta Sreeram <[email protected]>
Acked-by: Shireesha Kothakonda <[email protected]>
Made unlock and lock of eFuse controller be taken care by
the caller of the CacheReload function.

Signed-off-by: Kalyani Akula <[email protected]>
Acked-by: Mohan Marutirao Dhanawade <[email protected]>
Currently UCD400/UCD500 is not supporting remote i2c write side band
messages, Our driver keep waiting for reply for remote i2c writes and
later returns failure as it is not getting any reply from ucd. This leads to
video failure. So this support is removed from drivers and it doesn't
impact with any of other sink devices also.

Signed-off-by: Lakshmi Prasanna Eachuri <[email protected]>
Acked-by: Rajesh Gugulothu <[email protected]>
Update changelog

Signed-off-by: Meena Paleti <[email protected]>

Acked-by : Siva Addepalli<[email protected]>
There is a bug in AIE2 zeriozation function when polling for memory
zeroization complete. Currently the entire memory register is being
checked against zero but instead we need to check the bits specific
to the memory tiles. This patch updates the zeroization check by
adding a mask so that only the desired bits are checked for zero.

Signed-off-by: Nicole Baze <[email protected]>
Acked-by: Jesus De Haro <[email protected]>
A lot of people are trying to send pull requests via github but this is not
the right channel to use. Linux project is doing review over emails that's
why use [email protected] if you want to contribute changes to this
repository.

Signed-off-by: Siva Addepalli <[email protected]>
In the previous code the Recv and Send commands were not placed properly but the code was running as uartlite ip is an rtos( it tries to execute tasks with same priority parallely). Also the ASCII values below 'A' are not printed properly in vitis serial terminal or tera term. So in this update Recv and Send commands are placed in proper sequential manner for better understanding and also documentation is also changed a bit keeping that in mind. The sendbuffer initial value now starts from A so that all the characters are printed properly.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

8 participants