No, a number of things
it's Access, not Accesss and isn't actually required - frm AS Form will be just as good
For the purposes of your function as written byVal is OK, but if the function is supposed to modify the form in some way, then it should be byRef
You are also not calling it correctly. It should be
FunctionA subformcontrolname.form