>-----Original Message----- >From: Jagan Teki <[email protected]> >Sent: 11 May 2020 12:25 >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 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?
U-Boot SPL will work without L2 cache enable patches but why we want to do this. This series is not just for SPL mmc booting but also replacing FSBL functionality so better to cover all FSBL stuff in one series. I am not in favour of pulling out L2 cache patches out of this series. > >Jagan.
