I think that we must only call inode_setattr() if yaffs_SetAttributes() succeed.

It seems logical and it's done like this in other filesystems.