You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@nuttx.apache.org by ac...@apache.org on 2020/06/12 20:51:34 UTC

[incubator-nuttx] 01/02: drivers/: Fix some drivers mode priviliges. It's not used but it's better to get it right.

This is an automated email from the ASF dual-hosted git repository.

acassis pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/incubator-nuttx.git

commit ebfc12d3d293dc7a8b1fe6ccdda831bfedaf171b
Author: Ouss4 <ab...@gmail.com>
AuthorDate: Fri Jun 12 15:41:38 2020 +0100

    drivers/: Fix some drivers mode priviliges.  It's not used but it's
    better to get it right.
---
 drivers/analog/dac.c     | 2 +-
 drivers/leds/ncp5623c.c  | 2 +-
 drivers/leds/pca9635pw.c | 2 +-
 3 files changed, 3 insertions(+), 3 deletions(-)

diff --git a/drivers/analog/dac.c b/drivers/analog/dac.c
index 305110c..d2911e9 100644
--- a/drivers/analog/dac.c
+++ b/drivers/analog/dac.c
@@ -541,5 +541,5 @@ int dac_register(FAR const char *path, FAR struct dac_dev_s *dev)
 
   dev->ad_ops->ao_reset(dev);
 
-  return register_driver(path, &dac_fops, 0555, dev);
+  return register_driver(path, &dac_fops, 0222, dev);
 }
diff --git a/drivers/leds/ncp5623c.c b/drivers/leds/ncp5623c.c
index c1b1ea6..1576bfb 100644
--- a/drivers/leds/ncp5623c.c
+++ b/drivers/leds/ncp5623c.c
@@ -295,7 +295,7 @@ int ncp5623c_register(FAR const char *devpath, FAR struct i2c_master_s *i2c,
 
   /* Register the character driver */
 
-  int const ret = register_driver(devpath, &g_ncp5623c_fileops, 666, priv);
+  int const ret = register_driver(devpath, &g_ncp5623c_fileops, 0666, priv);
   if (ret != OK)
     {
       lcderr("ERROR: Failed to register driver: %d\n", ret);
diff --git a/drivers/leds/pca9635pw.c b/drivers/leds/pca9635pw.c
index 6b8d472..d3dfebf 100644
--- a/drivers/leds/pca9635pw.c
+++ b/drivers/leds/pca9635pw.c
@@ -363,7 +363,7 @@ int pca9635pw_register(FAR const char *devpath, FAR struct i2c_master_s *i2c,
 
   /* Register the character driver */
 
-  int const ret = register_driver(devpath, &g_pca9635pw_fileops, 666, priv);
+  int const ret = register_driver(devpath, &g_pca9635pw_fileops, 0666, priv);
   if (ret != OK)
     {
       lcderr("ERROR: Failed to register driver: %d\n", ret);