Separar produtos por categorias

<?php
include("conexao.php");

$db= "banco";
$tabela = "tb_veiculos";

$banco = mysql_select_db($db);
$sql=mysql_query("SELECT * FROM $tabela WHERE montadora NOT IN ('Peças Novas', 'Peças Usadas', 'Peças Recondicionadas', 'ESTOQUE') ORDER BY montadora ASC") or die(mysql_error());

// adicionei o mysql_error para facilitar um pouco na hora que der pau

//$total2 = mysql_num_rows($sql);

?>
<? //echo " Total de <font face=verdana size=2 color=yellow>    $total2   </font> Veículos Cadastrados " ;?>
<table width="98%" border="0" cellpadding="0" cellspacing="0" style="border-collapse: collapse">
 <tr>
 <td height="20" align="right" background="../images/bg bar.jpg" bgcolor="#333333"><span class="Estilo2">
 <?php
include("usuariosonline.php");
echo "Usuários on-line : $usuariosonline";
?>
     </span>          <b>  Visitas  </b>  
 <?php include("contador.php");?>
                         </td>
 </tr>
</table>

<?php
echo "<table width=100 border=0>";
$total = mysql_num_rows($sql);
?>

<?php $colunas = "2"; ?>

<?php

if ($total>0) {

for($i=0;$i<$total;$i++) {

if (($i%$colunas)==0) {

echo "</tr>"; //oque é isto?
echo "<tr>";
}

?>

<?php
$dados = mysql_fetch_array($sql);
$id = $dados["id"];
$montadora = $dados["montadora"];
$modelo = $dados["modelo"];
$ano = $dados["ano"];
$valor = $dados["valor"];
$placa = $dados["placa"];
$miniatura = $dados["imagem"];
list ($foto1) = split('[;]',$miniatura);

	if($foto1 == "veiculos/")
 {
 $foto1="images/nocarro.jpg";
 }

	$montadora_atual=$vdados["montadora"];//atribui a montadora à variavel montadora atual
	$vconsulta2=mysql_query("SELECT * FROM montadora WHERE montadora like '$montadora'");
	$vdados2=mysql_fetch_array($vconsulta2);
	$fotoo=$vdados2["foto"];
	$montadora=$vdados2["montadora"];

print "<td><table width=\"280\" align=\"center\" border=\"1\" cellspacing=\"2\" cellpadding=\"2\" bordercolor=\"#CCCCCC\" style=\"border-collapse: collapse\" >
<tr>";
print "";

print "

<td width=\"150\" align=\"center\"> <a href=javascript:; onClick= javascript:poptastic('detalhes_do_veiculo.php?id=$id')> <img src='gera_thumb.inc.php?imagem=$foto1&x=150' style='filter:progid:DXImageTransform.Microsoft.Shadow(color=#333333, direction=135, strength=4)' border=\"0\" align='left'></td>

<td width=\"150\" align=\"left\" valign=\"top\" background=\"images/boxx.jpg\">
<table width=\"140\" height=\"25\" border=\"0\" cellspacing=\"0\" cellpadding=\"0\">
<tr>
<td width=\"135\" bgcolor=\"#000000\" align=\"center\"> <b> $montadora </b> </td>
</tr>

</table>

<br><a href=javascript:; onClick= javascript:poptastic('detalhes_do_veiculo.php?id=$id')<strong><font face=verdana size=1 color=yellow>Modelo :</font> <br>$modelo <br> <strong> <font face=verdana size=1 color=yellow>Ano: </font> $ano <br> <strong><font face=verdana size=1 color=yellow>Lote:</font> $valor <br> <strong><font face=verdana size=1 color=yellow>Procedência:</font> <br>$placa</a></td>

</tr>
</table></td> ";

}

} else {

echo "Nenhum registro encontrado";
}
?>