On 06-02 11:13, Mike Rapoport wrote:
> On Sat, 30 May 2026 22:19:28 +0000, Pasha Tatashin
> <[email protected]> wrote:
> > diff --git a/kernel/liveupdate/luo_flb.c b/kernel/liveupdate/luo_flb.c
> > index 8f5c5dd01cd0..c8dd30b41238 100644
> > --- a/kernel/liveupdate/luo_flb.c
> > +++ b/kernel/liveupdate/luo_flb.c
> > @@ -579,53 +565,18 @@ int __init luo_flb_setup_outgoing(void *fdt_out)
> > [ ... skip 18 lines ... ]
> > - offset = fdt_subnode_offset(fdt_in, 0, LUO_FDT_FLB_NODE_NAME);
> > - if (offset < 0) {
> > - pr_err("Unable to get FLB node [%s]\n", LUO_FDT_FLB_NODE_NAME);
> > -
> > - return -ENOENT;
> > + if (flbs_pa) {
>
> I like
>
> if (!flbs_pa)
> return;
>
> more
Ok.
>
> >
> > diff --git a/kernel/liveupdate/luo_session.c
> > b/kernel/liveupdate/luo_session.c
> > index 8d9201c25412..3b760fefa7b9 100644
> > --- a/kernel/liveupdate/luo_session.c
> > +++ b/kernel/liveupdate/luo_session.c
> > @@ -497,75 +494,34 @@ int luo_session_retrieve(const char *name, struct
> > file **filep)
> > [ ... skip 58 lines ... ]
> > + if (sessions_pa) {
> > + header_ser = phys_to_virt(sessions_pa);
> > + luo_session_global.incoming.header_ser = header_ser;
> > + luo_session_global.incoming.ser = (void *)(header_ser + 1);
> > + luo_session_global.incoming.active = true;
> > }
>
> Ditto
This functions get's re-written with early return later in the serires.
>
> --
> Sincerely yours,
> Mike.
>
