On Mon, May 11, 2020 at 11:35 AM Pragnesh Patel <[email protected]> wrote: > > >-----Original Message----- > >From: Jagan Teki <[email protected]> > >Sent: 10 May 2020 15:02 > >To: Pragnesh Patel <[email protected]> > >Cc: U-Boot-Denx <[email protected]>; Atish Patra > ><[email protected]>; Palmer Dabbelt <[email protected]>; Bin > >Meng <[email protected]>; Paul Walmsley <[email protected]>; > >Troy Benjegerdes <[email protected]>; Anup Patel > ><[email protected]>; Sagar Kadam <[email protected]>; Rick Chen > ><[email protected]> > >Subject: Re: [PATCH v7 19/22] sifive: dts: fu540: Enable L2 Cache in U-Boot > > > >[External Email] Do not click links or attachments unless you recognize the > >sender and know the content is safe > > > >On Sun, May 3, 2020 at 12:57 PM Pragnesh Patel > ><[email protected]> wrote: > >> > >> Hi jagan, > >> > >> >-----Original Message----- > >> >From: Jagan Teki <[email protected]> > >> >Sent: 02 May 2020 22:43 > >> >To: Pragnesh Patel <[email protected]> > >> >Cc: U-Boot-Denx <[email protected]>; Atish Patra > >> ><[email protected]>; Palmer Dabbelt <[email protected]>; > >Bin > >> >Meng <[email protected]>; Paul Walmsley > ><[email protected]>; > >> >Troy Benjegerdes <[email protected]>; Anup Patel > >> ><[email protected]>; Sagar Kadam <[email protected]>; Rick > >Chen > >> ><[email protected]> > >> >Subject: Re: [PATCH v7 19/22] sifive: dts: fu540: Enable L2 Cache in > >> >U-Boot > >> > > >> >[External Email] Do not click links or attachments unless you > >> >recognize the sender and know the content is safe > >> > > >> >On Sat, May 2, 2020 at 10:12 PM Pragnesh Patel > >> ><[email protected]> > >> >wrote: > >> >> > >> >> Hi Jagan, > >> >> > >> >> >-----Original Message----- > >> >> >From: Jagan Teki <[email protected]> > >> >> >Sent: 02 May 2020 21:49 > >> >> >To: Pragnesh Patel <[email protected]> > >> >> >Cc: U-Boot-Denx <[email protected]>; Atish Patra > >> >> ><[email protected]>; Palmer Dabbelt > ><[email protected]>; > >> >Bin > >> >> >Meng <[email protected]>; Paul Walmsley > >> ><[email protected]>; > >> >> >Troy Benjegerdes <[email protected]>; Anup Patel > >> >> ><[email protected]>; Sagar Kadam <[email protected]>; Rick > >> >Chen > >> >> ><[email protected]> > >> >> >Subject: Re: [PATCH v7 19/22] sifive: dts: fu540: Enable L2 Cache > >> >> >in U-Boot > >> >> > > >> >> >[External Email] Do not click links or attachments unless you > >> >> >recognize the sender and know the content is safe > >> >> > > >> >> >On Sat, May 2, 2020 at 3:39 PM Pragnesh Patel > >> >> ><[email protected]> > >> >> >wrote: > >> >> >> > >> >> >> Add L2 cache node to enable cache ways from U-Boot > >> >> > > >> >> >This and 20/22 doesn't relate to SPL MMC boot?, if yes please send > >> >> >them separately. > >> >> > >> >> This series is for replacing FSBL and all the patches are related to > >> >> that. > >> >> IMHO it's better to add all FSBL functionality in one series. > >> > > >> >You mean does it break existing FSBL flow? if yes add proper commit > >> >message, but I am able to boot SPL MMC w/o this? > >> > >> Cache ways are enabled by FSBL also and if I will send cache ways > >> patches separately then it will a duplicate way of enabling cache ways if > >someone using FSBL. > > > >Sorry I didn't get you. > > > >If we cannot include these changes does U-Boot SPL break existing FSBL? > > No, U-Boot SPL does not break without this. > > As of now, we also want to support FSBL flow and FSBL also enabled the Cache > ways for U-Boot > proper and if someone use this patches of L2 cache enable ways will FSBL then > it will be a duplicate > work of cache enable ways.
My question is what if we don't add this change at all? Jagan.
