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

Fixed incorrect asm keyword syntax #106

Open
wants to merge 3,955 commits into
base: master
Choose a base branch
from
Open

Conversation

sdshlanta
Copy link

Fixed problem described in issue #105 which prevents compilation under clang.

b.vikram@xilinx.com and others added 30 commits June 5, 2019 17:17
DDRMC_MAIN, NOC_NIR and DDRMC_NOC block values are modified to ensure
that they are in sync with CDO v2 spec. CDO v1 spec has been modified as
well.

Signed-off-by: Vikram Sreenivasa Batchali <bvikram@xilinx.com>
Acked-by: Mounika Grace Akula <makula@xilinx.com>

Acked-by: Mounika Grace Akula <makula@xilinx.com>
Signed-off-by: Mubin Usman Sayyed <mubin.usman.sayyed@xilinx.com>

Acked-for-series: Anirudha Sarangi <anirudh@xilinx.com>
Pass subsystem id instead of index

Signed-off-by: Jolly Shah <jollys@xilinx.com>

Acked-for-series: Rajan Vaja <rajan.vaja@xilinx.com>
Use subsystem id passed by PLM for CDO commands

Signed-off-by: Jolly Shah <jollys@xilinx.com>

Acked-for-series: Rajan Vaja <rajan.vaja@xilinx.com>
Added routines within pre-processor directives to for GTYE5 flow.
Removed ERR_IRQ from the driver.

Signed-off-by: Eric Bong <nyongnyi@xilinx.com>
Acked-by: Gilbert Magnaye <gmagnay@xilinx.com>

Acked-by: Gilbert Magnaye <gmagnay@xilinx.com>
This patch updates the index.html and readme.txt files
with the information related to newly supported example
drivers

Signed-off-by: Anurag Kumar Vulisha <anurag.kumar.vulisha@xilinx.com>
Updated xwdttb_selftest_example.c & xwdttb_winwdt_example.c files with versal support.
Added xwdttb_gwdt_selftest_example.c & xwdttb_gwdt_example.c examples for Versal.

Signed-off-by: Srinivas Neeli <sneeli@xilinx.com>

Acked-for-series: Srinivas Goud <srinivas.goud@xilinx.com>
Existing R5 freertos port does not have support for hard float.

Signed-off-by: Anirudha Sarangi <anirudh@xilinx.com>
This patch fix MISRA-C mandatory violations in below files.
69D
Variable used before assignment
MISRA-C:2012/AMD1/ADD2: R.9.1

Signed-off-by: Sivaji Boddupilli <sivajib@xilinx.com>
Acked-by: Vikram Sreenivasa Batchali <Vikram.Sreenivasa.Batchali@xilinx.com>

Acked-by: Vikram Sreenivasa Batchali <Vikram.Sreenivasa.Batchali@xilinx.com>
By default this was taking lower page only, but for soft reset usecases,
the DDR initialization was failing due to incorrect SPD data being
read from EEPROM. This patch fixes the same.

Signed-off-by: Manish Narani <manish.narani@xilinx.com>
Acked-by: Vikram Sreenivasa Batchali <Vikram.Sreenivasa.Batchali@xilinx.com>

Acked-by: Vikram Sreenivasa Batchali <Vikram.Sreenivasa.Batchali@xilinx.com>
BYTE_ORDER is defined in microblaze compiler header so we get a warning
when it's redefined in lwip arch/cpu.h. To address it add a check before
redefining it. Fixes below compiler warning.

../../../include/arch/cpu.h:41: warning: "BYTE_ORDER" redefined
In file included from /include/sys/types.h:67,
                 from /include/stdio.h:61,
                 from contrib/ports/xilinx/netif/xemacliteif.c:47:
<install_path>/microblaze-xilinx-elf/include/machine/endian.h:20: note:
this is the location of the previous definition

Signed-off-by: Radhey Shyam Pandey <radhey.shyam.pandey@xilinx.com>

Acked-for-series: Harini Katakam <harini.katakam@xilinx.com>
Remove unused variables and also make IP address parameter constant in
line with the output function type definition. Fixes below gcc warnings-

xemacliteif.c: In function 'configure_IEEE_phy_speed_emaclite':
xemacliteif.c:769:6: warning: unused variable 'phylinkspeed' [-Wunused-variable]
  u16 phylinkspeed;
      ^~~~~~~~~~~~
xemacliteif.c:768:6: warning: unused variable 'status' [-Wunused-variable]
  u16 status;
      ^~~~~~
xemacliteif.c: In function 'xemacliteif_init':
xemacliteif.c:842:16: warning: assignment to 'netif_output_fn' {aka 'signed char
(*)(struct netif *, struct pbuf *, const struct ip4_addr *)'} from incompatible
 pointer type 'err_t (*)(struct netif *, struct pbuf *, ip_addr_t *)' {aka
 'signed char (*)(struct netif *, struct pbuf *, struct ip4_addr *)'}
 [-Wincompatible-pointer-types]
  netif->output = xemacliteif_output;

Signed-off-by: Radhey Shyam Pandey <radhey.shyam.pandey@xilinx.com>

Acked-for-series: Harini Katakam <harini.katakam@xilinx.com>
For freertos configuration, when the multiple packets arrive at the same
time binary semaphore sem_rx_data_available is incremented/signaled only
once. So to ensure that all packets are received before next semaphore
event loop through low_level_input().

Signed-off-by: Radhey Shyam Pandey <radhey.shyam.pandey@xilinx.com>

Acked-for-series: Harini Katakam <harini.katakam@xilinx.com>
Return 0 when NOC_TYPE1_NCR bit is set to 0, else return 1;

Signed-off-by: Vikram Sreenivasa Batchali <bvikram@xilinx.com>
Acked-by: Mounika Grace Akula <makula@xilinx.com>

Acked-by: Mounika Grace Akula <makula@xilinx.com>
This patch fix MISRA-C violations in below files.
69D
Variable used before assignment
MISRA-C:2012/AMD1/ADD2: R.9.1

Signed-off-by: Sivaji Boddupilli <sivajib@xilinx.com>
Acked-by: Vikram Sreenivasa Batchali <Vikram.Sreenivasa.Batchali@xilinx.com>

Acked-by: Vikram Sreenivasa Batchali <Vikram.Sreenivasa.Batchali@xilinx.com>
This patch fix MISRA-C mandatory violations in below files.
69D
Variable used before assignment
MISRA-C:2012/AMD1/ADD2: R.9.1

Signed-off-by: Sivaji Boddupilli <sivajib@xilinx.com>
Acked-by: Vikram Sreenivasa Batchali <Vikram.Sreenivasa.Batchali@xilinx.com>

Acked-by: Vikram Sreenivasa Batchali <Vikram.Sreenivasa.Batchali@xilinx.com>
This patch adds the support for modified versal IP names
for canfd, spips, ospipsv, qspipsu, uartsbsa, iicps.

Signed-off-by: Naga Sureshkumar Relli <naga.sureshkumar.relli@xilinx.com>

Acked-for-series: Srinivas Goud <srinivas.goud@xilinx.com>

Acked-for-series: Srinivas Goud <srinivas.goud@xilinx.com>
Add support for versal IP name.

Signed-off-by: Nava kishore Manne <nava.manne@xilinx.com>
Acked-by: Appana Durga Kedareswara rao <appana.durga.rao@xilinx.com>

Acked-by: Appana Durga Kedareswara rao <appana.durga.rao@xilinx.com>
Add support for versal IP name.

Signed-off-by: Nava kishore Manne <nava.manne@xilinx.com>
Acked-by: Appana Durga Kedareswara rao <appana.durga.rao@xilinx.com>

Acked-by: Appana Durga Kedareswara rao <appana.durga.rao@xilinx.com>
The macro returns the ALARM flag needed for a
particular supply

Signed-off-by: Anand Ashok Dumbre <anand.ashok.dumbre@xilinx.com>

Acked-for-series: Conall O'Griofa <conallo@xilinx.com>
Added API to be able to set a particular supply to
a New Data interrupt

Signed-off-by: Anand Ashok Dumbre <anand.ashok.dumbre@xilinx.com>

Acked-for-series: Conall O'Griofa <conallo@xilinx.com>
Added extern C to all header files to enable compilation in C++

Signed-off-by: Nicole Baze <nicoleb@xilinx.com>
Acked-by: Jyotheeswar Reddy Mutthareddyvari <jyothee@xilinx.com>

Acked-by: Jyotheeswar Reddy Mutthareddyvari <jyothee@xilinx.com>
Added extern C to resolve compilation errors occuring for C++

Signed-off-by: Nicole Baze <nicoleb@xilinx.com>
Acked-by: Jyotheeswar Reddy Mutthareddyvari <jyothee@xilinx.com>

Acked-by: Jyotheeswar Reddy Mutthareddyvari <jyothee@xilinx.com>
Signed-off-by: Ben Levinsky <ben.levinsky@xilinx.com>

Acked-for-series: Wendy Liang <wendy.liang@xilinx.com>
…U id

Use different shared memory based on RPU id.

Signed-off-by: Ben Levinsky <ben.levinsky@xilinx.com>

Acked-for-series: Wendy Liang <wendy.liang@xilinx.com>
… on RPU id

Use different shared memory based on RPU id.

Signed-off-by: Ben Levinsky <ben.levinsky@xilinx.com>

Acked-for-series: Wendy Liang <wendy.liang@xilinx.com>
Use different shared memory based on RPU id.

Signed-off-by: Ben Levinsky <ben.levinsky@xilinx.com>

Acked-for-series: Wendy Liang <wendy.liang@xilinx.com>
Added length validations in XilSKey_Efuse_ValidateKey()
functions

Signed-off-by: Archana Obannagari <archanao@xilinx.com>
Acked-by: VNSL Durga Challa <vnsldurg@xilinx.com>

Acked-by: VNSL Durga Challa <vnsldurg@xilinx.com>
Signed-off-by: Eric Bong <nyongnyi@xilinx.com>
Acked-by: Yunhai Qiao <yunhaiq@xilinx.com>

Acked-by: Yunhai Qiao <yunhaiq@xilinx.com>
As part of refactoring hardware offsets are placed
into the corresponding platform folders.
Splited common xsecure_hw.h file into xsecure_sha_hw.h,
xsecure_aes_hw.h and xsecure_rsa_hw.h.
Also common functionalities like SSS configurations and
hardware read and write APIs are placed in utils

Signed-off-by: VNSL Durga Challa <vnsl.durga.challa@xilinx.com>

Acked-for-series: Mohan Dhanawade <mohan.dhanawade@xilinx.com>
mnarani and others added 20 commits June 5, 2019 21:33
Eye Tests have memory range to be tested of 1MB. So this was not a
problem for split between Lower and Upper DDR testing together. This
patch handles the case when User might select the Start Address just
before the end of the lower DDR Range.

Signed-off-by: Manish Narani <manish.narani@xilinx.com>
Acked-By: Srinivas Goud <srinivas.goud@xilinx.com>

Acked-By: Srinivas Goud <srinivas.goud@xilinx.com>
Signed-off-by: Shruti Gupta <shrutigu@xilinx.com>

Acked by: Kapil Usgaonkar<kapilu@xilinx.com>
…an 32 bit value.

Signed-off-by: Eric Bong <nyongnyi@xilinx.com>
Acked-by: Yunhai Qiao <yunhaiq@xilinx.com>

Acked-by: Yunhai Qiao <yunhaiq@xilinx.com>
…ot works. Fix Made in application files to retrain RX properly

Signed-off-by: Shruti Gupta <shrutigu@xilinx.com>

Acked by: Kapil Usgaonkar<kapilu@xilinx.com>
…o PSV

This patch updates the PS IP definition macros available in the
xparameters_ps.h file from PSU to PSV.

Signed-off-by: Appana Durga Kedareswara rao <appana.durga.rao@xilinx.com>

Acked-for-series: Anirudha Sarangi <anirudha.sarangi@xilinx.com>
Existing implementation fetches subsystem_id from IPI mask only if
current subsystem_id is set as INVALID_SUBSYTEMID. The CDO sets
subsystem_id during start to process other CDO commands in LibPM.
So after CDO load, every command from IPI is treated for current
subsystem_id not from IPI mask which is wrong.

As a solution of above, CDO needs to invalidate the subsystem_id
at the end of CDO, but set_current_subsystem() API doesn't allow
INVALID_SUBSYTEMID to set.

Allow INVALID_SUBSYTEMID to be set from CDO through
set_current_subsystem() API.

Signed-off-by: Ravi Patel <ravi.patel@xilinx.com>
Acked-by: Jolly Shah <jolly.shah@xilinx.com>

Acked-by: Jolly Shah <jolly.shah@xilinx.com>
Added frequently used functions in common area, so that other
modules can make use of it.

Signed-off-by: Mohan Marutirao Dhanawade <mohan.dhanawade@xilinx.com>

Acked-for-series: Appana Durga Kedareswara rao <appana.durga.rao@xilinx.com>
This patch fixes psu->psv as per xparameters.h

Signed-off-by: VNSL Durga Challa <vnsl.durga.challa@xilinx.com>
Acked-by: Sarat Chand Savitala <saratcha@xilinx.com>

Acked-by: Sarat Chand Savitala <saratcha@xilinx.com>
armclang archiver needs "-" prefix for archiver flags.
Update xilpm makefile accordingly.

Signed-off-by: Ravi Patel <ravi.patel@xilinx.com>
Acked-by:  Mubin Sayyed <mubin.usman.sayyed@xilinx.com>

Acked-by:  Mubin Sayyed <mubin.usman.sayyed@xilinx.com>
… and update for MST application

Signed-off-by: Kapil Usgaonkar <kapilu@xilinx.com>

Acked by : Pankaj Kumbhare <pankajk@xilinx.com>
Updated xgpiops_polled_example.c file with versal support.

Signed-off-by: Srinivas Neeli <srinivas.neeli@xilinx.com>

Acked-for-series: Anirudha Sarangi <anirudha.sarangi@xilinx.com>
updated GPIO Makefile for IAR compiler.

Signed-off-by: Srinivas Neeli <srinivas.neeli@xilinx.com>

Acked-for-series: Anirudha Sarangi <anirudha.sarangi@xilinx.com>
armcc compiler was not accepting declaring typedef variable in for
statement. So moved typedef variable definition at start of the function.

Signed-off-by: Mohan Marutirao Dhanawade <mohan.dhanawade@xilinx.com>

BRANCH:2019.1
CR-1030701
Acked-by: Srikanth Vemula <srikanth.vemula@xilinx.com>

Acked-by: Srikanth Vemula <srikanth.vemula@xilinx.com>
This patch renames the library instance name from xilmbox to xilmailbox.

Signed-off-by: Appana Durga Kedareswara rao <appana.durga.rao@xilinx.com>
Acked-by: Nava kishore Manne <nava.manne@xilinx.com>
Signed-off-by: Siva Addepalli <saddepal@xilinx.com>
Signed-off-by: Siva Addepalli <saddepal@xilinx.com>
This patch removes Xilinx copyright and license text from FatFS source
files.

Signed-off-by: Manish Narani <manish.narani@xilinx.com>
Signed-off-by: Shireesha Kothakonda <shireesh@xilinx.com>
Signed-off-by: Siva Addepalli <saddepal@xilinx.com>
Fixed problem described in this issue Xilinx#105
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.

9 participants