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

drivers/video/isx012.c: Make forbidden calls into CXD56xx code #438

Closed
patacongo opened this issue Mar 5, 2020 · 2 comments · Fixed by #456
Closed

drivers/video/isx012.c: Make forbidden calls into CXD56xx code #438

patacongo opened this issue Mar 5, 2020 · 2 comments · Fixed by #456
Assignees
Labels
modularity Needed to support modular architecture

Comments

@patacongo
Copy link
Contributor

Common drivers must never call directly in architecture specific code. That is FORBIDDEN.

The driver at drivers/video/isx012.c does just that:

$ grep cxd56 drivers/video/isx012.c
  ret = cxd56_cisifinit();
      imagererr("Fail cxd56_cisifinit %d\n", ret);
  ret = cxd56_cisifstopcapture();
      imagererr("Fail cxd56_cisifstopcapture %d\n", ret);
  ret = cxd56_cisiffinalize();
      imagererr("Fail cxd56_cisiffinalize %d\n", ret);
      ret = cxd56_cisifsetdmabuf(&sarea);
      ret = cxd56_cisifstartcapture(&cis_param, &sarea);
  ret =  cxd56_cisifstopcapture();

That is in complete violation of the requirements of the NuttX modular design. This driver must either be corrected so that it uses a standard upper/lower half design as all other drivers do, or it must be removed, or it must be moved to boards/arm/cxd56xx/drivers/

@patacongo patacongo added invalid This doesn't seem right and removed invalid This doesn't seem right labels Mar 5, 2020
@jerpelea
Copy link
Contributor

jerpelea commented Mar 5, 2020

I am looking into it ASAP

@patacongo
Copy link
Contributor Author

This was reported to me in a personal email from the same person that reported Issue #439.

@patacongo patacongo added the modularity Needed to support modular architecture label Mar 5, 2020
@xiaoxiang781216 xiaoxiang781216 linked a pull request Mar 7, 2020 that will close this issue
cwespressif pushed a commit to cwespressif/incubator-nuttx that referenced this issue Apr 3, 2024
…defconfig

resize spec stack size to 16384
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
modularity Needed to support modular architecture
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants