No ... non proprio. Il punto è che values() di Map ha come tipo di ritorno un Collection<V>. E non è affatto detto che la collezione restituita sia una implementazione di List! Ad esempio HashMap ha come values una collezione che estende AbstractCollection. Che non è un List!
Se vuoi fare la cosa in modo davvero "pulito" e sicuro devi semplicemente popolare il tuo nuovo LinkedList con la collezione fornita da values(). E non c'è bisogno di fare cicli o invocare metodi .... basta il costruttore di LinkedList:
codice:new LinkedList<Libro>(this.codice2libro.values())
(ovviamente inseriscilo nel tuo metodo.
P.S. non postare il codice colorato ... complica solo il quoting del messaggio.