Does Vega S95 support external HDD suspend?

[artouiros]

Hi, I have 3 android tv boxes in my home atm, AND ALL OF THEM DONT SUPPORT HDD suspend. I mean: when turning box off (sleep) the HDD spins forever, because it still get power. But it's not about the power to usb, it's about how driver is designed. A regular windows PC just sends some commands to hdd to tell it to spin down/sleep. Anyone can confirm vega s95 has this feature? I really don't wantto wate another 60$. It been not a total waste of money, I found these boxes useful, but I really need to connect my HDD.

[RISCy Dude]

Hello, I don't think so. Not sure if it can, but doesn't or if it is just a limitation of the platform. Good day.

[artouiros]

Hello, I don't think so. Not sure if it can, but doesn't or if it is just a limitation of the platform. Good day.

that's sad. the only android box I know that supports HDD power down is Nvidia Shield, but 200 euro for a tv box is too much.

[freeroc]

It's not difficult. We did this before in a allwinner product.

[artouiros]

It's not difficult. We did this before in a allwinner product.

how did you acomplish this?

[freeroc]

how did you acomplish this?

modify the code, and rebuild firmware.
If you have the SDK of your firmware , you can read following for reference.
diff --git a/DirectVolume.cpp b/DirectVolume.cpp
index 16ac2d8..b8ed136 100755
--- a/DirectVolume.cpp
+++ b/DirectVolume.cpp
@@ -30,8 +30,178 @@
#include "VolumeManager.h"
#include "ResponseCode.h"
#include "cryptfs.h"
-
+#include <stdlib.h>
+#include <pthread.h>
+#include<sys/types.h>
+#include<sys/stat.h>
+#include<fcntl.h>
+#include <linux/hdreg.h>
// #define PARTITION_DEBUG
+int sleep_sata(char *argv)
+{
+ if(!argv)
+ {
+ SLOGE("[sleep_sata]Invalid argument\n");
+ return -1;
+ }
+ int fd,ret;
+ unsigned char args[4] = { WIN_SETFEATURES, 0, 0, 0 };
+ printf("[sleep_sata] argv = %s\n",argv);
+ args[0] = WIN_SLEEPNOW1;
+ fd = open(argv,O_RDONLY | O_NONBLOCK,0666);
+ if(fd < 0)
+ {
+ printf("fail to open %s \n",argv);
+ return -1;
+ }
+ ret = ioctl(fd, HDIO_DRIVE_CMD, args);
+ if(ret != 0)
+ {
+ printf("fail to ioctl \n");
+ return -1;
+ }
+ else
+ {
+ printf("sucess ioctl\n");
+ }
+ return 0;
+}
+void * SleepSataDisk(void * p)
+{
+ char * dp = (char *)p;
+ char * temp1 = strstr(dp,"sd");
+ char * temp2 = strstr(temp1,"/");
+ if(!temp1 || !temp2)
+ {
+ SLOGE("faile to find disk info\n");
+ return NULL;
+ }
+ char file[256] = {0};
+ strncpy(file,temp1,temp2-temp1);
+ int fp;
+ char state[1024*2];
+ bool isfind = false;
+ bool issleep = false;
+ int last_iotime = 0;
+ int no_operate_time = 10;
+ int sleep_times = 0;
+ SLOGI("SleepSataDisk begin loop...");
+ while(1)
+ {
+ sleep(no_operate_time);//wait for check sata disk status
+ if(!(fp = open("/proc/diskstats",O_RDONLY)))
+ {
+ SLOGE("Error!!! can't open /proc/diskstats\n");
+ return NULL;
+ }
+ if(read(fp,(void*)state,sizeof(state)))
+ {
+ char devstate[1024]= {0};
+ //SLOGI("read diskstats = %s\n",state);
+ char * temp,*last = state ;
+ while((temp = strstr(last,"\n")))
+ {
+ //SLOGI("find sd* : %d %d\n",temp,last);
+ strncpy(devstate,last,temp-last);
+ if(strstr(devstate,file))
+ {
+ char * p1,*p2 = strstr(devstate,file);
+ isfind = true;
+ //SLOGI("find sd* : %s\n",p2);
+ char iotime[128] = {0};
+ for(int i = 0; i < 10;i++)
+ {
+ p1 = strstr(p2," ");
+ p2 = p1 + 1;
+ }
+ p1 = strstr(p2," ");
+ strncpy(iotime,p2,p1-p2);
+ if(0 == (atoi(iotime) - last_iotime) && false == issleep)
+ {
+ //set sata disk to sleep
+ SLOGI("set sata disk to sleep\n");
+ char cmd[256];
+ sprintf(cmd, "/dev/block/%s",file);
+ SLOGI("set sata to sleep!!!!!!! %s\n",cmd);
+ int ret = sleep_sata(cmd);
+ if(0 == ret)
+ {
+ issleep = true;
+ int sleepstatus_fp = open("/proc/sleepstatus",O_RDONLY);
+ if(!sleepstatus_fp)
+ {
+ SLOGE("open sleepstatus fail!!!!!!\n");
+ }else
+ {
+ read(sleepstatus_fp,&sleep_times,sizeof(sleep_time s));
+ SLOGI("fist time sleep sata the system sleep_times = %d \n",sleep_times);
+ close(sleepstatus_fp);
+ }
+ }
+ SLOGW("sata disk has sleep... %d\n",ret);
+
+ }
+ else
+ {
+ if(0 != (atoi(iotime) - last_iotime))
+ {
+ issleep = false;
+ }
+ if(issleep)//debug
+ {
+ // char cmd[256];
+ // sprintf(cmd, "busybox hdparm -C /dev/block/%s",file);
+ // SLOGI("debug cmd = %s\n",cmd);
+ // int ret = system(cmd);
+ //if(0 == ret)
+ //issleep = true;
+ int sleep_times_temp = 0;
+ int sleepstatus_fp = open("/proc/sleepstatus",O_RDONLY);
+ if(!sleepstatus_fp)
+ {
+ SLOGE("open sleepstatus fail!!!!!!\n");
+ }else
+ {
+ int read_ret = 0;
+ read_ret = read(sleepstatus_fp,&sleep_times_temp,sizeof(sleep _times_temp));
+ SLOGI("read_ret = %d sleep_times = %d;sleep_times_temp = %d \n",read_ret,sleep_times,sleep_times_temp);
+ if((sleep_times_temp - sleep_times) != 0 && read_ret >= 0)
+ {
+ char cmd[256];
+ sprintf(cmd, "/dev/block/%s",file);
+ SLOGI("system has sleep ,send cmd to set sata to sleep: %s\n",cmd);
+ sleep_sata(cmd);
+ sleep_times = sleep_times_temp;
+ }
+ close(sleepstatus_fp);
+ }
+ SLOGI("debug sata sleep ...\n");
+ }
+ last_iotime = atoi(iotime);
+ //SLOGI("find iotime : %d\n",last_iotime);
+ }
+ break;
+ }
+ memset(devstate,0,sizeof(devstate));
+ //if it is not target dev
+ last = temp + 1;
+ }
+ if(!isfind)
+ {
+ SLOGI("can't get target disk status,exit thread.\n");
+ return NULL;
+ }
+ isfind = false;
+ }else
+ {
+ SLOGI("exit sleep sata thread!\n");
+ return NULL;
+ }
+ memset(state,0,sizeof(state));
+ close(fp);
+
+ }
+}

DirectVolume:irectVolume(VolumeManager *vm, const char *label,
const char *mount_point, int partIdx) :
@@ -133,6 +303,11 @@ int DirectVolume::handleBlockEvent(NetlinkEvent *evt) {
snprintf(msg, sizeof(msg),
"Volume %s %s disk inserted (%d:%d)", getLabel(),
getMountpoint(), mDiskMajor, mDiskMinor);
+ if(0 == strcmp(getLabel(),"sata"))
+ {
+ pthread_t thread_id;
+ pthread_create(&thread_id, NULL, SleepSataDisk, (void *)dp);
+ }
mVm->getBroadcaster()->sendBroadcast(ResponseCode::VolumeDiskInserted,
msg, false);
}

[spideymon]

Can this be made into an xposed framework addon?? That would rock...

Thanks