>-----Original Message----- >From: Jagan Teki <[email protected]> >Sent: 11 May 2020 12:56 >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 12:37 PM Pragnesh Patel ><[email protected]> wrote: >> >> >-----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. > >So, FSBL flow would break if we add U-Boot SPL so this patch fixing by enabling >cache's explicitly to avoid that break. isn't it?
No, you are not getting this. Initially FSBL enable the cache ways before U-Boot SPL. https://github.com/sifive/freedom-u540-c000-bootloader/blob/master/fsbl/main.c#L428 Now U-Boot SPL doing the same in [v7 19/22] and [v7 20/22]. User can use U-Boot SPL or FSBL anyone at a time as a bootloader, so better to replace all FSBL stuff in one series. > >Jagan.
