[SLOF] [PATCH] virtio-serial: Remove superfluous serial-* words

Alexey Kardashevskiy aik at ozlabs.ru
Wed Jan 27 21:25:51 AEDT 2021



On 24/01/2021 18:41, Thomas Huth wrote:
> These likely were a blind copy-n-paste from hvterm.fs, but they
> simply do not make any sense in virtio-serial.fs. The hvterm.fs is
> always included from OF.fs, so the serial-* words are globally there.
> virtio-serial.fs is only used within the virtio-serial device tree
> nodes, to adding the serial-* words there is just superfluous.
> 
> Signed-off-by: Thomas Huth <thuth at redhat.com>

Hm looks like it. Still works with the patch so thanks, applied.

How did you find it? Or in other words why on earth you looked there? :)


> ---
>   board-qemu/slof/virtio-serial.fs | 4 ----
>   1 file changed, 4 deletions(-)
> 
> diff --git a/board-qemu/slof/virtio-serial.fs b/board-qemu/slof/virtio-serial.fs
> index 06bfe76..82868e2 100644
> --- a/board-qemu/slof/virtio-serial.fs
> +++ b/board-qemu/slof/virtio-serial.fs
> @@ -90,7 +90,3 @@ virtiodev virtio-serial-init drop
>   ;
>   setup-alias
>   
> -\ Override serial methods to make term-io.fs happy
> -: serial-emit virtio-serial-term-emit ;
> -: serial-key? virtio-serial-term-key? ;
> -: serial-key  virtio-serial-term-key  ;
> 

-- 
Alexey


More information about the SLOF mailing list