Skip to content

Commit a47cb47

Browse files
fjtrujyfjtrujy
authored
Merge pull request #695 from uyjulian/io_dummy_impl_use_retval
Use original error return values for IO stub ops
2 parents 8c01ea0 + d44e012 commit a47cb47

File tree

35 files changed

+610
-539
lines changed

35 files changed

+610
-539
lines changed

‎ee/iopreboot/src/imgdrv/src/imgdrv.c‎

Lines changed: 8 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -39,14 +39,16 @@ typedef struct _iop_device_ops_short
3939
int (*lseek)(iop_file_t *, int, int);
4040
} iop_device_ops_short_t;
4141

42+
IOMAN_RETURN_VALUE_IMPL(0);
43+
4244
static iop_device_ops_short_t imgdrv_ops = {
43-
DUMMY_IMPLEMENTATION, // init
44-
DUMMY_IMPLEMENTATION, // deinit
45-
NOT_SUPPORTED, // format
46-
NOT_SUPPORTED, // open
47-
NOT_SUPPORTED, // close
45+
IOMAN_RETURN_VALUE(0), // init
46+
IOMAN_RETURN_VALUE(0), // deinit
47+
IOMAN_RETURN_VALUE(0), // format
48+
IOMAN_RETURN_VALUE(0), // open
49+
IOMAN_RETURN_VALUE(0), // close
4850
&imgdrv_read, // read
49-
NOT_SUPPORTED, // write
51+
IOMAN_RETURN_VALUE(0), // write
5052
&imgdrv_lseek, // lseek
5153
};
5254

‎iop/arcade/accdvd/src/cddrv.c‎

Lines changed: 11 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -19,24 +19,26 @@ static int cddrv_write(iop_file_t *io, void *buf, int cnt);
1919
static int cddrv_lseek(iop_file_t *io, int offset, int whence);
2020
static int cddrv_ioctl(iop_file_t *io, int cmd, void *arg);
2121

22+
IOMAN_RETURN_VALUE_IMPL(EINVAL);
23+
2224
static iop_device_ops_t Cddrv_ops = {
2325
&cddrv_adddrv, // init
2426
&cddrv_deldrv, // deinit
25-
NOT_SUPPORTED, // format
27+
IOMAN_RETURN_VALUE(EINVAL), // format
2628
&cddrv_open, // open
2729
&cddrv_close, // close
2830
&cddrv_read, // read
2931
&cddrv_write, // write
3032
&cddrv_lseek, // lseek
3133
&cddrv_ioctl, // ioctl
32-
NOT_SUPPORTED, // remove
33-
NOT_SUPPORTED, // mkdir
34-
NOT_SUPPORTED, // rmdir
35-
NOT_SUPPORTED, // dopen
36-
NOT_SUPPORTED, // dclose
37-
NOT_SUPPORTED, // dread
38-
NOT_SUPPORTED, // getstat
39-
NOT_SUPPORTED, // chstat
34+
IOMAN_RETURN_VALUE(EINVAL), // remove
35+
IOMAN_RETURN_VALUE(EINVAL), // mkdir
36+
IOMAN_RETURN_VALUE(EINVAL), // rmdir
37+
IOMAN_RETURN_VALUE(EINVAL), // dopen
38+
IOMAN_RETURN_VALUE(EINVAL), // dclose
39+
IOMAN_RETURN_VALUE(EINVAL), // dread
40+
IOMAN_RETURN_VALUE(EINVAL), // getstat
41+
IOMAN_RETURN_VALUE(EINVAL), // chstat
4042
};
4143

4244
static iop_device_t Cddrv = {"cdrom", 16u, 0u, "ATAPI_C/DVD-ROM", &Cddrv_ops};

‎iop/arcade/acuart/src/tty.c‎

Lines changed: 18 additions & 15 deletions
Original file line numberDiff line numberDiff line change
@@ -14,24 +14,27 @@ static int acuart_write(iop_file_t *f, void *buffer, int size) {
1414
return acUartWrite(buffer, size);
1515
}
1616

17+
IOMAN_RETURN_VALUE_IMPL(0);
18+
IOMAN_RETURN_VALUE_IMPL(ENOTSUP);
19+
1720
static iop_device_ops_t uart_ops = {
18-
DUMMY_IMPLEMENTATION, // init
19-
DUMMY_IMPLEMENTATION, // deinit
20-
NOT_SUPPORTED, // format
21-
NOT_SUPPORTED, // open
22-
NOT_SUPPORTED, // close
21+
IOMAN_RETURN_VALUE(0), // init
22+
IOMAN_RETURN_VALUE(0), // deinit
23+
IOMAN_RETURN_VALUE(ENOTSUP), // format
24+
IOMAN_RETURN_VALUE(ENOTSUP), // open
25+
IOMAN_RETURN_VALUE(ENOTSUP), // close
2326
&acuart_read, // read
2427
&acuart_write, // write
25-
NOT_SUPPORTED, // lseek
26-
NOT_SUPPORTED, // ioctl
27-
NOT_SUPPORTED, // remove
28-
NOT_SUPPORTED, // mkdir
29-
NOT_SUPPORTED, // rmdir
30-
NOT_SUPPORTED, // dopen
31-
NOT_SUPPORTED, // dclose
32-
NOT_SUPPORTED, // dread
33-
NOT_SUPPORTED, // getstat
34-
NOT_SUPPORTED, // chstat
28+
IOMAN_RETURN_VALUE(ENOTSUP), // lseek
29+
IOMAN_RETURN_VALUE(ENOTSUP), // ioctl
30+
IOMAN_RETURN_VALUE(ENOTSUP), // remove
31+
IOMAN_RETURN_VALUE(ENOTSUP), // mkdir
32+
IOMAN_RETURN_VALUE(ENOTSUP), // rmdir
33+
IOMAN_RETURN_VALUE(ENOTSUP), // dopen
34+
IOMAN_RETURN_VALUE(ENOTSUP), // dclose
35+
IOMAN_RETURN_VALUE(ENOTSUP), // dread
36+
IOMAN_RETURN_VALUE(ENOTSUP), // getstat
37+
IOMAN_RETURN_VALUE(ENOTSUP), // chstat
3538
};
3639

3740
#define DEVNAME "tty"

‎iop/cdvd/cdfs/src/main.c‎

Lines changed: 8 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -418,24 +418,26 @@ static int fio_getstat(iop_file_t *fd, const char *name, io_stat_t *stat)
418418
return ret;
419419
}
420420

421+
IOMAN_RETURN_VALUE_IMPL(EIO);
422+
421423
static iop_device_ops_t fio_ops = {
422424
&fio_init, // init
423425
&fio_deinit, // deinit
424-
NOT_SUPPORTED, // format
426+
IOMAN_RETURN_VALUE(EIO), // format
425427
&fio_open, // open
426428
&fio_close, // close
427429
&fio_read, // read
428430
&fio_write, // write
429431
&fio_lseek, // lseek
430-
NOT_SUPPORTED, // ioctl
431-
NOT_SUPPORTED, // remove
432-
NOT_SUPPORTED, // mkdir
433-
NOT_SUPPORTED, // rmdir
432+
IOMAN_RETURN_VALUE(EIO), // ioctl
433+
IOMAN_RETURN_VALUE(EIO), // remove
434+
IOMAN_RETURN_VALUE(EIO), // mkdir
435+
IOMAN_RETURN_VALUE(EIO), // rmdir
434436
&fio_openDir, // dopen
435437
&fio_closeDir, // dclose
436438
&fio_dread, // dread
437439
&fio_getstat, // getstat
438-
NOT_SUPPORTED, // chstat
440+
IOMAN_RETURN_VALUE(EIO), // chstat
439441
};
440442

441443
static iop_device_t fio_driver = {

‎iop/cdvd/xesdrv/src/xesdrv.c‎

Lines changed: 24 additions & 22 deletions
Original file line numberDiff line numberDiff line change
@@ -218,33 +218,35 @@ struct DevctlCmdTbl_t
218218
{0x5473, &esioctl2_func_10},
219219
};
220220

221+
IOMANX_RETURN_VALUE_IMPL(ENOTSUP);
222+
221223
static iomanX_iop_device_ops_t DvrFuncTbl = {
222224
&esdrv_df_init, // init
223225
&esdrv_df_exit, // deinit
224-
NOT_SUPPORTED, // format
225-
NOT_SUPPORTED, // open
226-
NOT_SUPPORTED, // close
227-
NOT_SUPPORTED, // read
228-
NOT_SUPPORTED, // write
229-
NOT_SUPPORTED, // lseek
226+
IOMANX_RETURN_VALUE(ENOTSUP), // format
227+
IOMANX_RETURN_VALUE(ENOTSUP), // open
228+
IOMANX_RETURN_VALUE(ENOTSUP), // close
229+
IOMANX_RETURN_VALUE(ENOTSUP), // read
230+
IOMANX_RETURN_VALUE(ENOTSUP), // write
231+
IOMANX_RETURN_VALUE(ENOTSUP), // lseek
230232
&esdrv_df_ioctl, // ioctl
231-
NOT_SUPPORTED, // remove
232-
NOT_SUPPORTED, // mkdir
233-
NOT_SUPPORTED, // rmdir
234-
NOT_SUPPORTED, // dopen
235-
NOT_SUPPORTED, // dclose
236-
NOT_SUPPORTED, // dread
237-
NOT_SUPPORTED, // getstat
238-
NOT_SUPPORTED, // chstat
239-
NOT_SUPPORTED, // rename
240-
NOT_SUPPORTED, // chdir
241-
NOT_SUPPORTED, // sync
242-
NOT_SUPPORTED, // mount
243-
NOT_SUPPORTED, // umount
244-
NOT_SUPPORTED_S64, // lseek64
233+
IOMANX_RETURN_VALUE(ENOTSUP), // remove
234+
IOMANX_RETURN_VALUE(ENOTSUP), // mkdir
235+
IOMANX_RETURN_VALUE(ENOTSUP), // rmdir
236+
IOMANX_RETURN_VALUE(ENOTSUP), // dopen
237+
IOMANX_RETURN_VALUE(ENOTSUP), // dclose
238+
IOMANX_RETURN_VALUE(ENOTSUP), // dread
239+
IOMANX_RETURN_VALUE(ENOTSUP), // getstat
240+
IOMANX_RETURN_VALUE(ENOTSUP), // chstat
241+
IOMANX_RETURN_VALUE(ENOTSUP), // rename
242+
IOMANX_RETURN_VALUE(ENOTSUP), // chdir
243+
IOMANX_RETURN_VALUE(ENOTSUP), // sync
244+
IOMANX_RETURN_VALUE(ENOTSUP), // mount
245+
IOMANX_RETURN_VALUE(ENOTSUP), // umount
246+
IOMANX_RETURN_VALUE_S64(ENOTSUP), // lseek64
245247
&esdrv_df_devctl, // devctl
246-
NOT_SUPPORTED, // symlink
247-
NOT_SUPPORTED, // readlink
248+
IOMANX_RETURN_VALUE(ENOTSUP), // symlink
249+
IOMANX_RETURN_VALUE(ENOTSUP), // readlink
248250
&esdrv_df_ioctl2, // ioctl2
249251
};
250252
static iomanX_iop_device_t ESDRV = {

‎iop/debug/iop_sbusdbg/src/sbus_tty.c‎

Lines changed: 12 additions & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -92,25 +92,27 @@ static int ttyfs_write(iop_file_t *file, void *ptr, int size) {
9292
return(bCount);
9393
}
9494

95+
IOMAN_RETURN_VALUE_IMPL(EPERM);
96+
9597
static iop_device_ops_t fsd_ops =
9698
{
9799
&ttyfs_init, // init
98100
&ttyfs_deinit, // deinit
99-
NOT_SUPPORTED, // format
101+
IOMAN_RETURN_VALUE(EPERM), // format
100102
&ttyfs_open, // open
101103
&ttyfs_close, // close
102-
NOT_SUPPORTED, // read
104+
IOMAN_RETURN_VALUE(EPERM), // read
103105
&ttyfs_write, // write
104-
NOT_SUPPORTED, // lseek
105-
NOT_SUPPORTED, // ioctl
106-
NOT_SUPPORTED, // remove
107-
NOT_SUPPORTED, // mkdir
108-
NOT_SUPPORTED, // rmdir
106+
IOMAN_RETURN_VALUE(EPERM), // lseek
107+
IOMAN_RETURN_VALUE(EPERM), // ioctl
108+
IOMAN_RETURN_VALUE(EPERM), // remove
109+
IOMAN_RETURN_VALUE(EPERM), // mkdir
110+
IOMAN_RETURN_VALUE(EPERM), // rmdir
109111
&ttyfs_dopen, // dopen
110112
&ttyfs_close, // dclose
111-
NOT_SUPPORTED, // dread
112-
NOT_SUPPORTED, // getstat
113-
NOT_SUPPORTED, // chstat
113+
IOMAN_RETURN_VALUE(EPERM), // dread
114+
IOMAN_RETURN_VALUE(EPERM), // getstat
115+
IOMAN_RETURN_VALUE(EPERM), // chstat
114116
};
115117

116118
static iop_device_t tty_fsd =

‎iop/debug/ppctty/src/tty.c‎

Lines changed: 17 additions & 14 deletions
Original file line numberDiff line numberDiff line change
@@ -72,25 +72,28 @@ static int ttyfs_write(iop_file_t *file, void *ptr, int size) {
7272
return(bCount);
7373
}
7474

75+
IOMAN_RETURN_VALUE_IMPL(0);
76+
IOMAN_RETURN_VALUE_IMPL(EPERM);
77+
7578
static iop_device_ops_t fsd_ops =
7679
{
7780
&ttyfs_init, // init
7881
&ttyfs_deinit, // deinit
79-
NOT_SUPPORTED, // format
80-
NOT_SUPPORTED, // open
81-
NOT_SUPPORTED, // close
82-
NOT_SUPPORTED, // read
82+
IOMAN_RETURN_VALUE(EPERM), // format
83+
IOMAN_RETURN_VALUE(0), // open
84+
IOMAN_RETURN_VALUE(0), // close
85+
IOMAN_RETURN_VALUE(EPERM), // read
8386
&ttyfs_write, // write
84-
NOT_SUPPORTED, // lseek
85-
NOT_SUPPORTED, // ioctl
86-
NOT_SUPPORTED, // remove
87-
NOT_SUPPORTED, // mkdir
88-
NOT_SUPPORTED, // rmdir
89-
NOT_SUPPORTED, // dopen
90-
NOT_SUPPORTED, // dclose
91-
NOT_SUPPORTED, // dread
92-
NOT_SUPPORTED, // getstat
93-
NOT_SUPPORTED, // chstat
87+
IOMAN_RETURN_VALUE(EPERM), // lseek
88+
IOMAN_RETURN_VALUE(EPERM), // ioctl
89+
IOMAN_RETURN_VALUE(EPERM), // remove
90+
IOMAN_RETURN_VALUE(EPERM), // mkdir
91+
IOMAN_RETURN_VALUE(EPERM), // rmdir
92+
IOMAN_RETURN_VALUE(0), // dopen
93+
IOMAN_RETURN_VALUE(0), // dclose
94+
IOMAN_RETURN_VALUE(EPERM), // dread
95+
IOMAN_RETURN_VALUE(EPERM), // getstat
96+
IOMAN_RETURN_VALUE(EPERM), // chstat
9497
};
9598

9699
static iop_device_t tty_fsd =

‎iop/dev9/dev9/src/ps2dev9.c‎

Lines changed: 28 additions & 26 deletions
Original file line numberDiff line numberDiff line change
@@ -146,35 +146,37 @@ static int dev9x_devctl(iop_file_t *f, const char *name, int cmd, void *args, un
146146
}
147147
}
148148

149+
IOMANX_RETURN_VALUE_IMPL(0);
150+
149151
static iop_device_ops_t dev9x_ops =
150152
{
151-
DUMMY_IMPLEMENTATION, // init
152-
DUMMY_IMPLEMENTATION, // deinit
153-
NOT_SUPPORTED, // format
154-
NOT_SUPPORTED, // open
155-
NOT_SUPPORTED, // close
156-
NOT_SUPPORTED, // read
157-
NOT_SUPPORTED, // write
158-
NOT_SUPPORTED, // lseek
159-
NOT_SUPPORTED, // ioctl
160-
NOT_SUPPORTED, // remove
161-
NOT_SUPPORTED, // mkdir
162-
NOT_SUPPORTED, // rmdir
163-
NOT_SUPPORTED, // dopen
164-
NOT_SUPPORTED, // dclose
165-
NOT_SUPPORTED, // dread
166-
NOT_SUPPORTED, // getstat
167-
NOT_SUPPORTED, // chstat
168-
NOT_SUPPORTED, // rename
169-
NOT_SUPPORTED, // chdir
170-
NOT_SUPPORTED, // sync
171-
NOT_SUPPORTED, // mount
172-
NOT_SUPPORTED, // umount
173-
NOT_SUPPORTED_S64, // lseek64
153+
IOMANX_RETURN_VALUE(0), // init
154+
IOMANX_RETURN_VALUE(0), // deinit
155+
IOMANX_RETURN_VALUE(0), // format
156+
IOMANX_RETURN_VALUE(0), // open
157+
IOMANX_RETURN_VALUE(0), // close
158+
IOMANX_RETURN_VALUE(0), // read
159+
IOMANX_RETURN_VALUE(0), // write
160+
IOMANX_RETURN_VALUE(0), // lseek
161+
IOMANX_RETURN_VALUE(0), // ioctl
162+
IOMANX_RETURN_VALUE(0), // remove
163+
IOMANX_RETURN_VALUE(0), // mkdir
164+
IOMANX_RETURN_VALUE(0), // rmdir
165+
IOMANX_RETURN_VALUE(0), // dopen
166+
IOMANX_RETURN_VALUE(0), // dclose
167+
IOMANX_RETURN_VALUE(0), // dread
168+
IOMANX_RETURN_VALUE(0), // getstat
169+
IOMANX_RETURN_VALUE(0), // chstat
170+
IOMANX_RETURN_VALUE(0), // rename
171+
IOMANX_RETURN_VALUE(0), // chdir
172+
IOMANX_RETURN_VALUE(0), // sync
173+
IOMANX_RETURN_VALUE(0), // mount
174+
IOMANX_RETURN_VALUE(0), // umount
175+
IOMANX_RETURN_VALUE_S64(0), // lseek64
174176
&dev9x_devctl, // devctl
175-
NOT_SUPPORTED, // symlink
176-
NOT_SUPPORTED, // readlink
177-
NOT_SUPPORTED, // ioctl2
177+
IOMANX_RETURN_VALUE(0), // symlink
178+
IOMANX_RETURN_VALUE(0), // readlink
179+
IOMANX_RETURN_VALUE(0), // ioctl2
178180
};
179181

180182
static iop_device_t dev9x_device =

‎iop/dvrp/dvr/src/dvr.c‎

Lines changed: 24 additions & 22 deletions
Original file line numberDiff line numberDiff line change
@@ -108,34 +108,36 @@ struct DevctlCmdTbl_t
108108
{0x5650, &dvrioctl2_rec_pictclip},
109109
};
110110

111+
IOMANX_RETURN_VALUE_IMPL(EUNSUP);
112+
111113
static iomanX_iop_device_ops_t DvrFuncTbl =
112114
{
113115
&dvr_df_init, // init
114116
&dvr_df_exit, // deinit
115-
NOT_SUPPORTED, // format
116-
NOT_SUPPORTED, // open
117-
NOT_SUPPORTED, // close
118-
NOT_SUPPORTED, // read
119-
NOT_SUPPORTED, // write
120-
NOT_SUPPORTED, // lseek
117+
IOMANX_RETURN_VALUE(EUNSUP), // format
118+
IOMANX_RETURN_VALUE(EUNSUP), // open
119+
IOMANX_RETURN_VALUE(EUNSUP), // close
120+
IOMANX_RETURN_VALUE(EUNSUP), // read
121+
IOMANX_RETURN_VALUE(EUNSUP), // write
122+
IOMANX_RETURN_VALUE(EUNSUP), // lseek
121123
&dvr_df_ioctl, // ioctl
122-
NOT_SUPPORTED, // remove
123-
NOT_SUPPORTED, // mkdir
124-
NOT_SUPPORTED, // rmdir
125-
NOT_SUPPORTED, // dopen
126-
NOT_SUPPORTED, // dclose
127-
NOT_SUPPORTED, // dread
128-
NOT_SUPPORTED, // getstat
129-
NOT_SUPPORTED, // chstat
130-
NOT_SUPPORTED, // rename
131-
NOT_SUPPORTED, // chdir
132-
NOT_SUPPORTED, // sync
133-
NOT_SUPPORTED, // mount
134-
NOT_SUPPORTED, // umount
135-
NOT_SUPPORTED_S64, // lseek64
124+
IOMANX_RETURN_VALUE(EUNSUP), // remove
125+
IOMANX_RETURN_VALUE(EUNSUP), // mkdir
126+
IOMANX_RETURN_VALUE(EUNSUP), // rmdir
127+
IOMANX_RETURN_VALUE(EUNSUP), // dopen
128+
IOMANX_RETURN_VALUE(EUNSUP), // dclose
129+
IOMANX_RETURN_VALUE(EUNSUP), // dread
130+
IOMANX_RETURN_VALUE(EUNSUP), // getstat
131+
IOMANX_RETURN_VALUE(EUNSUP), // chstat
132+
IOMANX_RETURN_VALUE(EUNSUP), // rename
133+
IOMANX_RETURN_VALUE(EUNSUP), // chdir
134+
IOMANX_RETURN_VALUE(EUNSUP), // sync
135+
IOMANX_RETURN_VALUE(EUNSUP), // mount
136+
IOMANX_RETURN_VALUE(EUNSUP), // umount
137+
IOMANX_RETURN_VALUE_S64(EUNSUP), // lseek64
136138
&dvr_df_devctl, // devctl
137-
NOT_SUPPORTED, // symlink
138-
NOT_SUPPORTED, // readlink
139+
IOMANX_RETURN_VALUE(EUNSUP), // symlink
140+
IOMANX_RETURN_VALUE(EUNSUP), // readlink
139141
&dvr_df_ioctl2, // ioctl2
140142
};
141143
char TEVENT_BUF[6144];

0 commit comments

Comments
 (0)