Не совсем понятно, что за значения находятся в полях - firma, tip. Это первое.
Второе, зачем такие сложности с объединением, если запрос идет только к одной таблице.

И вообще, тут надо искать ошибку не в самом запросе, а в структуре базы данных.
Что нужно сделать?
Это - создать две таблицы, где в одной будет храниться перечень товаров (tovar), а во второй – перечень фирм (firms). При этом, в первой - должно быть поле, которое каждую строку товара будет однозначно привязывать к какой-то фирме. Например – idfirm. Тогда при составлении запроса, именно по этому полю и будет происходить объединение данных из этих таблиц. ( ON firms.idfirm = tovar idfirm и конструкция - WHERE, для этой цели - будет уже не нужна)

Дальше, на форме нужно иметь, кроме комбобокса с переченем фирм – или еще один комбобокс, куда будет выводиться список товаров выбранной фирмы или, для тех же целей – Grid. Т.е. идея – очень проста, в одном комбобоксе выбираем – фирму, во втором комбобоксе (или Gridе) выводим список товаров, которые ей соответствуют.

При выполнении того, что я написал, у тебя значительно упроститься как сам запрос, так и вывод его результата.