PHP - Llenado de combo

<?php

	class Rol{

		private $idRol;

		private $descripcion;

		public function __construct($idRol="", $descripcion=""){

			$this->idRol=$idRol;

			$this->descripcion=$descripcion;

		}

		public static function selectRol($id=""){

			$mysql=new BaseDatos();

			$info="";

			$sql="select idRol, descripcion from rol;";

			$listado=$mysql->ejecutarSQL($sql);

			if(!empty($listado) && $mysql->getError()==""){

				foreach ($listado as $campo){

					$info.= "<option value='".$campo->idRol."'";

					if($id!="" && $id==$campo->idRol){

						$info.=" selected>";

					}else{

						$info.=">";

					}

					$info.=$campo->descripcion."</option>";

				}

			}else{

				$info=$mysql->getError();;

			}

			return $info;

		}

		//METODOS SET Y GET

		//Parametros	$propiedad->__set("nombre","Prueba");

		public function __set($nombreAtributo,$valor){

			if(property_exists($this, $nombreAtributo)){

				$this->$nombreAtributo=$valor;

			}

			else

			{

				echo "No existe el atributo $nombreAtributo";

			}

		}

		//Parametros	$propiedad->__get("nombre");

		public function __get($nombreAtributo){

			if(property_exists($this, $nombreAtributo)){

				return $this->$nombreAtributo;

			}

			else

			{

				echo "No existe el atributo $nombreAtributo";

			}

		}

		public function set_get($nombreAtributo,$value=null){

			if ($value!=null){

				$this->$nombreAtributo=$value;

			}else{

				return $this->$nombreAtributo;

			}

		}

	}

?>

y en la parte gráfica tengo esto en la parte del combo:

<fieldset>

	<legend>Rol</legend>

	<select name="Rol">

	<?php echo Rol::seleccionarRoles($propiedad->idRol);?>

	</select>

</fieldset>