On Tue, Aug 02, 2022 at 08:48:51AM +0000, Dmytro Firsov wrote: > Can moderators, please, take a look on this patch?
What about it? https://patchwork.ozlabs.org/project/uboot/patch/20220704120533.42168-1-dmytro_fir...@epam.com/ > > On 20.07.22 23:43, Nastya Vicodin wrote: > Reviewed-by: Anastasiia Lukianenko > <vicooo...@gmail.com<mailto:vicooo...@gmail.com>> > > On Mon, Jul 4, 2022 at 3:05 PM Dmytro Firsov > <dmytro_fir...@epam.com<mailto:dmytro_fir...@epam.com>> wrote: > Some setups do not use Xen hypervisor console for logging, e.g. they > use emulated PL011 hardware or shared peripherals (real UART). In such > cases Xen HVC will be disabled on a build time and will cause issues in > current driver implementation. > > This commit fixes build issues in Xen event channel driver, caused > by absense of console event channel, that is not available when console > config is disabled. Now console related code will be removed when > Xen HVC is turned off. > > Signed-off-by: Dmytro Firsov > <dmytro_fir...@epam.com<mailto:dmytro_fir...@epam.com>> > --- > drivers/xen/events.c | 5 +++++ > 1 file changed, 5 insertions(+) > > diff --git a/drivers/xen/events.c b/drivers/xen/events.c > index 5e90a65846..532216fece 100644 > --- a/drivers/xen/events.c > +++ b/drivers/xen/events.c > @@ -23,7 +23,9 @@ > #include <xen/events.h> > #include <xen/hvm.h> > > +#if CONFIG_IS_ENABLED(XEN_SERIAL) > extern u32 console_evtchn; > +#endif /* CONFIG_IS_ENABLED(XEN_SERIAL) */ > > #define NR_EVS 1024 > > @@ -51,8 +53,11 @@ void unbind_all_ports(void) > struct vcpu_info *vcpu_info = &s->vcpu_info[cpu]; > > for (i = 0; i < NR_EVS; i++) { > +#if CONFIG_IS_ENABLED(XEN_SERIAL) > if (i == console_evtchn) > continue; > +#endif /* CONFIG_IS_ENABLED(XEN_SERIAL) */ > + > if (test_and_clear_bit(i, bound_ports)) { > printf("port %d still bound!\n", i); > unbind_evtchn(i); > -- > 2.25.1 -- Tom
signature.asc
Description: PGP signature
