Нет, неправильно.
БЭМ нужен во избежание вложенностей. Правильно так
.header {
$parent: &;
&__top {
}
&__logotype {
}
#{$parent}_link {
}
#{$parent}_image {
}
}
UPD:
Короче говоря по бэму должно быть в scss что-то вроде этого.
Хотя тут svg - не по БЭМ, но это потому что при подгрузке новых svg я не хочу залазить в каждый и добавлять класс элемента.