Re: [PATCH 00/22] x86: Move some boards to text environment
On Monday 07 August 2023 15:30:25 Michael Nazzareno Trimarchi wrote: > Hi Andy > > On Mon, Aug 7, 2023 at 3:04 PM Andy Shevchenko > wrote: > > > > On Mon, Aug 07, 2023 at 02:44:31PM +0200, Michael Nazzareno Trimarchi wrote: > > > Hi Pali > > > > > > Can you just filter emails on your side? > > > > Independently on the question is default setup is good or not > > (from _this_ point of view, I *disagree* with Pali), we have to > > have a possibility to filter on _our_ side the email addresses > > to make people happy. If Pali by some reasons does not want to > > see, it must be easy to keep some deny list in the repository. > > > > What you are suggesting is not polite I believe. > > I understand what you mean, I never consider emails to me as a problem > if I'm working > on an opensource project and mostly of the time I'm happy to receive them No, I'm really not happy to receive emails from people who either implicity or explicitly said that will ignore all my contributions there and trying to convince me to go away and at the same time they want from me to do review on their own changes in my free time. I was patient, I wait one year to see if something happen - but absolutely nothing. So why you cannot understand that if you are not interested in my patches, then I'm obviously not interested in yours? Or you are continuing trying to say that I'm idiot who should do exactly what you wrote? No! You would have to find another fool who would follow these your ideas. > Michael > > > > > > On Mon, Aug 7, 2023 at 2:18 PM Pali Rohár wrote: > > > > > > > > So remove me from that list of dram.c file. I'm not interested to > > > > receive emails from people who are ignoring me about unrelated things. > > > > > > > > On Monday 07 August 2023 09:43:01 Bin Meng wrote: > > > > > Hi Pali, > > > > > > > > > > On Sun, Aug 6, 2023 at 11:55 PM Pali Rohár wrote: > > > > > > > > > > > > On Sunday 06 August 2023 08:39:43 Simon Glass wrote: > > > > > > > Hi Pali, > > > > > > > > > > > > > > On Sun, 6 Aug 2023 at 04:51, Pali Rohár wrote: > > > > > > > > > > > > > > > > I'm not x86 maintainer, and I'm not going to review changes. So > > > > > > > > please > > > > > > > > do not send me these emails. I have expressed it many times. > > > > > > > > > > > > > > You were sent one patch (and the cover letter) because you are the > > > > > > > second committer on arch/x86/cpu/qemu/dram.c > > > > > > > > > > > > I'm not maintainer of arch/x86/cpu/qemu/dram.c. How many times I > > > > > > have to > > > > > > repeat it? You do not understand? Or what you are trying to do now? > > > > > > > > > > I believe this cc list comes from patman which calls get_maintainer.pl > > > > > to get the cc list. > > > > > get_maintainer.pl determines the person names from 1. MAINTAINERS of > > > > > the changed file 2. git commit history of the changed file. > > > > > > > > > > I can see the philosophy was that someone who touched the changed file > > > > > should be copied for review. > > > > > We certainly could argue that and just get the list solely from the > > > > > MAINTAINERS file. > > > > > > > > > > Regards, > > > > > Bin > > > > > > > > > > > > -- > > > Michael Nazzareno Trimarchi > > > Co-Founder & Chief Executive Officer > > > M. +39 347 913 2170 > > > mich...@amarulasolutions.com > > > __ > > > > > > Amarula Solutions BV > > > Joop Geesinkweg 125, 1114 AB, Amsterdam, NL > > > T. +31 (0)85 111 9172 > > > i...@amarulasolutions.com > > > www.amarulasolutions.com > > > > -- > > With Best Regards, > > Andy Shevchenko > > > > > > > -- > Michael Nazzareno Trimarchi > Co-Founder & Chief Executive Officer > M. +39 347 913 2170 > mich...@amarulasolutions.com > __ > > Amarula Solutions BV > Joop Geesinkweg 125, 1114 AB, Amsterdam, NL > T. +31 (0)85 111 9172 > i...@amarulasolutions.com > www.amarulasolutions.com
Re: [PATCH 00/22] x86: Move some boards to text environment
On Mon, Aug 07, 2023 at 03:30:25PM +0200, Michael Nazzareno Trimarchi wrote: > On Mon, Aug 7, 2023 at 3:04 PM Andy Shevchenko > wrote: > > On Mon, Aug 07, 2023 at 02:44:31PM +0200, Michael Nazzareno Trimarchi wrote: > > > Hi Pali > > > > > > Can you just filter emails on your side? > > > > Independently on the question is default setup is good or not > > (from _this_ point of view, I *disagree* with Pali), we have to > > have a possibility to filter on _our_ side the email addresses > > to make people happy. If Pali by some reasons does not want to > > see, it must be easy to keep some deny list in the repository. > > > > What you are suggesting is not polite I believe. > > I understand what you mean, I never consider emails to me as a problem > if I'm working > on an opensource project and mostly of the time I'm happy to receive them It really depends. I agree with Bin on that, but we should respect someone's wishes (in case it's done automatically by heuristics [git history] rather than letter of law [MAINTAINERS database]). > > > On Mon, Aug 7, 2023 at 2:18 PM Pali Rohár wrote: > > > > > > > > So remove me from that list of dram.c file. I'm not interested to > > > > receive emails from people who are ignoring me about unrelated things. > > > > > > > > On Monday 07 August 2023 09:43:01 Bin Meng wrote: > > > > > Hi Pali, > > > > > > > > > > On Sun, Aug 6, 2023 at 11:55 PM Pali Rohár wrote: > > > > > > > > > > > > On Sunday 06 August 2023 08:39:43 Simon Glass wrote: > > > > > > > Hi Pali, > > > > > > > > > > > > > > On Sun, 6 Aug 2023 at 04:51, Pali Rohár wrote: > > > > > > > > > > > > > > > > I'm not x86 maintainer, and I'm not going to review changes. So > > > > > > > > please > > > > > > > > do not send me these emails. I have expressed it many times. > > > > > > > > > > > > > > You were sent one patch (and the cover letter) because you are the > > > > > > > second committer on arch/x86/cpu/qemu/dram.c > > > > > > > > > > > > I'm not maintainer of arch/x86/cpu/qemu/dram.c. How many times I > > > > > > have to > > > > > > repeat it? You do not understand? Or what you are trying to do now? > > > > > > > > > > I believe this cc list comes from patman which calls get_maintainer.pl > > > > > to get the cc list. > > > > > get_maintainer.pl determines the person names from 1. MAINTAINERS of > > > > > the changed file 2. git commit history of the changed file. > > > > > > > > > > I can see the philosophy was that someone who touched the changed file > > > > > should be copied for review. > > > > > We certainly could argue that and just get the list solely from the > > > > > MAINTAINERS file. -- With Best Regards, Andy Shevchenko
Re: [PATCH 00/22] x86: Move some boards to text environment
Hi Andy On Mon, Aug 7, 2023 at 3:04 PM Andy Shevchenko wrote: > > On Mon, Aug 07, 2023 at 02:44:31PM +0200, Michael Nazzareno Trimarchi wrote: > > Hi Pali > > > > Can you just filter emails on your side? > > Independently on the question is default setup is good or not > (from _this_ point of view, I *disagree* with Pali), we have to > have a possibility to filter on _our_ side the email addresses > to make people happy. If Pali by some reasons does not want to > see, it must be easy to keep some deny list in the repository. > > What you are suggesting is not polite I believe. I understand what you mean, I never consider emails to me as a problem if I'm working on an opensource project and mostly of the time I'm happy to receive them Michael > > > On Mon, Aug 7, 2023 at 2:18 PM Pali Rohár wrote: > > > > > > So remove me from that list of dram.c file. I'm not interested to > > > receive emails from people who are ignoring me about unrelated things. > > > > > > On Monday 07 August 2023 09:43:01 Bin Meng wrote: > > > > Hi Pali, > > > > > > > > On Sun, Aug 6, 2023 at 11:55 PM Pali Rohár wrote: > > > > > > > > > > On Sunday 06 August 2023 08:39:43 Simon Glass wrote: > > > > > > Hi Pali, > > > > > > > > > > > > On Sun, 6 Aug 2023 at 04:51, Pali Rohár wrote: > > > > > > > > > > > > > > I'm not x86 maintainer, and I'm not going to review changes. So > > > > > > > please > > > > > > > do not send me these emails. I have expressed it many times. > > > > > > > > > > > > You were sent one patch (and the cover letter) because you are the > > > > > > second committer on arch/x86/cpu/qemu/dram.c > > > > > > > > > > I'm not maintainer of arch/x86/cpu/qemu/dram.c. How many times I have > > > > > to > > > > > repeat it? You do not understand? Or what you are trying to do now? > > > > > > > > I believe this cc list comes from patman which calls get_maintainer.pl > > > > to get the cc list. > > > > get_maintainer.pl determines the person names from 1. MAINTAINERS of > > > > the changed file 2. git commit history of the changed file. > > > > > > > > I can see the philosophy was that someone who touched the changed file > > > > should be copied for review. > > > > We certainly could argue that and just get the list solely from the > > > > MAINTAINERS file. > > > > > > > > Regards, > > > > Bin > > > > > > > > -- > > Michael Nazzareno Trimarchi > > Co-Founder & Chief Executive Officer > > M. +39 347 913 2170 > > mich...@amarulasolutions.com > > __ > > > > Amarula Solutions BV > > Joop Geesinkweg 125, 1114 AB, Amsterdam, NL > > T. +31 (0)85 111 9172 > > i...@amarulasolutions.com > > www.amarulasolutions.com > > -- > With Best Regards, > Andy Shevchenko > > -- Michael Nazzareno Trimarchi Co-Founder & Chief Executive Officer M. +39 347 913 2170 mich...@amarulasolutions.com __ Amarula Solutions BV Joop Geesinkweg 125, 1114 AB, Amsterdam, NL T. +31 (0)85 111 9172 i...@amarulasolutions.com www.amarulasolutions.com
Re: [PATCH 00/22] x86: Move some boards to text environment
On Mon, Aug 07, 2023 at 04:03:45PM +0300, Andy Shevchenko wrote: > On Mon, Aug 07, 2023 at 02:44:31PM +0200, Michael Nazzareno Trimarchi wrote: > > Hi Pali > > > > Can you just filter emails on your side? > > Independently on the question is default setup is good or not > (from _this_ point of view, I *disagree* with Pali), we have to > have a possibility to filter on _our_ side the email addresses > to make people happy. If Pali by some reasons does not want to > see, it must be easy to keep some deny list in the repository. > > What you are suggesting is not polite I believe. I have to admit I didn't notice ".get_maintainer.ignore" until now. So yes, it sounds like we can do that too. > > > On Mon, Aug 7, 2023 at 2:18 PM Pali Rohár wrote: > > > > > > So remove me from that list of dram.c file. I'm not interested to > > > receive emails from people who are ignoring me about unrelated things. > > > > > > On Monday 07 August 2023 09:43:01 Bin Meng wrote: > > > > Hi Pali, > > > > > > > > On Sun, Aug 6, 2023 at 11:55 PM Pali Rohár wrote: > > > > > > > > > > On Sunday 06 August 2023 08:39:43 Simon Glass wrote: > > > > > > Hi Pali, > > > > > > > > > > > > On Sun, 6 Aug 2023 at 04:51, Pali Rohár wrote: > > > > > > > > > > > > > > I'm not x86 maintainer, and I'm not going to review changes. So > > > > > > > please > > > > > > > do not send me these emails. I have expressed it many times. > > > > > > > > > > > > You were sent one patch (and the cover letter) because you are the > > > > > > second committer on arch/x86/cpu/qemu/dram.c > > > > > > > > > > I'm not maintainer of arch/x86/cpu/qemu/dram.c. How many times I have > > > > > to > > > > > repeat it? You do not understand? Or what you are trying to do now? > > > > > > > > I believe this cc list comes from patman which calls get_maintainer.pl > > > > to get the cc list. > > > > get_maintainer.pl determines the person names from 1. MAINTAINERS of > > > > the changed file 2. git commit history of the changed file. > > > > > > > > I can see the philosophy was that someone who touched the changed file > > > > should be copied for review. > > > > We certainly could argue that and just get the list solely from the > > > > MAINTAINERS file. > > > > > > > > Regards, > > > > Bin > > > > > > > > -- > > Michael Nazzareno Trimarchi > > Co-Founder & Chief Executive Officer > > M. +39 347 913 2170 > > mich...@amarulasolutions.com > > __ > > > > Amarula Solutions BV > > Joop Geesinkweg 125, 1114 AB, Amsterdam, NL > > T. +31 (0)85 111 9172 > > i...@amarulasolutions.com > > www.amarulasolutions.com > > -- > With Best Regards, > Andy Shevchenko > > -- Tom signature.asc Description: PGP signature
Re: [PATCH 00/22] x86: Move some boards to text environment
On Mon, Aug 07, 2023 at 02:44:31PM +0200, Michael Nazzareno Trimarchi wrote: > Hi Pali > > Can you just filter emails on your side? Independently on the question is default setup is good or not (from _this_ point of view, I *disagree* with Pali), we have to have a possibility to filter on _our_ side the email addresses to make people happy. If Pali by some reasons does not want to see, it must be easy to keep some deny list in the repository. What you are suggesting is not polite I believe. > On Mon, Aug 7, 2023 at 2:18 PM Pali Rohár wrote: > > > > So remove me from that list of dram.c file. I'm not interested to > > receive emails from people who are ignoring me about unrelated things. > > > > On Monday 07 August 2023 09:43:01 Bin Meng wrote: > > > Hi Pali, > > > > > > On Sun, Aug 6, 2023 at 11:55 PM Pali Rohár wrote: > > > > > > > > On Sunday 06 August 2023 08:39:43 Simon Glass wrote: > > > > > Hi Pali, > > > > > > > > > > On Sun, 6 Aug 2023 at 04:51, Pali Rohár wrote: > > > > > > > > > > > > I'm not x86 maintainer, and I'm not going to review changes. So > > > > > > please > > > > > > do not send me these emails. I have expressed it many times. > > > > > > > > > > You were sent one patch (and the cover letter) because you are the > > > > > second committer on arch/x86/cpu/qemu/dram.c > > > > > > > > I'm not maintainer of arch/x86/cpu/qemu/dram.c. How many times I have to > > > > repeat it? You do not understand? Or what you are trying to do now? > > > > > > I believe this cc list comes from patman which calls get_maintainer.pl > > > to get the cc list. > > > get_maintainer.pl determines the person names from 1. MAINTAINERS of > > > the changed file 2. git commit history of the changed file. > > > > > > I can see the philosophy was that someone who touched the changed file > > > should be copied for review. > > > We certainly could argue that and just get the list solely from the > > > MAINTAINERS file. > > > > > > Regards, > > > Bin > > > > -- > Michael Nazzareno Trimarchi > Co-Founder & Chief Executive Officer > M. +39 347 913 2170 > mich...@amarulasolutions.com > __ > > Amarula Solutions BV > Joop Geesinkweg 125, 1114 AB, Amsterdam, NL > T. +31 (0)85 111 9172 > i...@amarulasolutions.com > www.amarulasolutions.com -- With Best Regards, Andy Shevchenko
Re: [PATCH 00/22] x86: Move some boards to text environment
Hi Pali Can you just filter emails on your side? Michael On Mon, Aug 7, 2023 at 2:18 PM Pali Rohár wrote: > > So remove me from that list of dram.c file. I'm not interested to > receive emails from people who are ignoring me about unrelated things. > > On Monday 07 August 2023 09:43:01 Bin Meng wrote: > > Hi Pali, > > > > On Sun, Aug 6, 2023 at 11:55 PM Pali Rohár wrote: > > > > > > On Sunday 06 August 2023 08:39:43 Simon Glass wrote: > > > > Hi Pali, > > > > > > > > On Sun, 6 Aug 2023 at 04:51, Pali Rohár wrote: > > > > > > > > > > I'm not x86 maintainer, and I'm not going to review changes. So please > > > > > do not send me these emails. I have expressed it many times. > > > > > > > > You were sent one patch (and the cover letter) because you are the > > > > second committer on arch/x86/cpu/qemu/dram.c > > > > > > I'm not maintainer of arch/x86/cpu/qemu/dram.c. How many times I have to > > > repeat it? You do not understand? Or what you are trying to do now? > > > > I believe this cc list comes from patman which calls get_maintainer.pl > > to get the cc list. > > get_maintainer.pl determines the person names from 1. MAINTAINERS of > > the changed file 2. git commit history of the changed file. > > > > I can see the philosophy was that someone who touched the changed file > > should be copied for review. > > We certainly could argue that and just get the list solely from the > > MAINTAINERS file. > > > > Regards, > > Bin -- Michael Nazzareno Trimarchi Co-Founder & Chief Executive Officer M. +39 347 913 2170 mich...@amarulasolutions.com __ Amarula Solutions BV Joop Geesinkweg 125, 1114 AB, Amsterdam, NL T. +31 (0)85 111 9172 i...@amarulasolutions.com www.amarulasolutions.com
Re: [PATCH 00/22] x86: Move some boards to text environment
So remove me from that list of dram.c file. I'm not interested to receive emails from people who are ignoring me about unrelated things. On Monday 07 August 2023 09:43:01 Bin Meng wrote: > Hi Pali, > > On Sun, Aug 6, 2023 at 11:55 PM Pali Rohár wrote: > > > > On Sunday 06 August 2023 08:39:43 Simon Glass wrote: > > > Hi Pali, > > > > > > On Sun, 6 Aug 2023 at 04:51, Pali Rohár wrote: > > > > > > > > I'm not x86 maintainer, and I'm not going to review changes. So please > > > > do not send me these emails. I have expressed it many times. > > > > > > You were sent one patch (and the cover letter) because you are the > > > second committer on arch/x86/cpu/qemu/dram.c > > > > I'm not maintainer of arch/x86/cpu/qemu/dram.c. How many times I have to > > repeat it? You do not understand? Or what you are trying to do now? > > I believe this cc list comes from patman which calls get_maintainer.pl > to get the cc list. > get_maintainer.pl determines the person names from 1. MAINTAINERS of > the changed file 2. git commit history of the changed file. > > I can see the philosophy was that someone who touched the changed file > should be copied for review. > We certainly could argue that and just get the list solely from the > MAINTAINERS file. > > Regards, > Bin
Re: [PATCH 00/22] x86: Move some boards to text environment
Hi Pali, On Sun, Aug 6, 2023 at 11:55 PM Pali Rohár wrote: > > On Sunday 06 August 2023 08:39:43 Simon Glass wrote: > > Hi Pali, > > > > On Sun, 6 Aug 2023 at 04:51, Pali Rohár wrote: > > > > > > I'm not x86 maintainer, and I'm not going to review changes. So please > > > do not send me these emails. I have expressed it many times. > > > > You were sent one patch (and the cover letter) because you are the > > second committer on arch/x86/cpu/qemu/dram.c > > I'm not maintainer of arch/x86/cpu/qemu/dram.c. How many times I have to > repeat it? You do not understand? Or what you are trying to do now? I believe this cc list comes from patman which calls get_maintainer.pl to get the cc list. get_maintainer.pl determines the person names from 1. MAINTAINERS of the changed file 2. git commit history of the changed file. I can see the philosophy was that someone who touched the changed file should be copied for review. We certainly could argue that and just get the list solely from the MAINTAINERS file. Regards, Bin
Re: [PATCH 00/22] x86: Move some boards to text environment
On Sunday 06 August 2023 08:39:43 Simon Glass wrote: > Hi Pali, > > On Sun, 6 Aug 2023 at 04:51, Pali Rohár wrote: > > > > I'm not x86 maintainer, and I'm not going to review changes. So please > > do not send me these emails. I have expressed it many times. > > You were sent one patch (and the cover letter) because you are the > second committer on arch/x86/cpu/qemu/dram.c I'm not maintainer of arch/x86/cpu/qemu/dram.c. How many times I have to repeat it? You do not understand? Or what you are trying to do now? > > On Sunday 30 July 2023 21:01:43 Simon Glass wrote: > > > This series starts the process of moving x86 boards to a text environment. > > > > > > This includes conversions for about half the boards. > > > > > > The rest (including the more complicated Chromebooks) can come in a later > > > series. > > > > > > > > > Simon Glass (22): > > > doc: Explain how to avoid the distro-boot scripts > > > env: Use include/env for text-environment includes > > > env: Explain how to use #include files in text environment > > > x86: Drop CFG_SYS_STACK_SIZE > > > x86: i8254: Include required ibmpc.h header > > > x86: edison: Drop inclusion of ibmpc.h > > > x86: Drop inclusion of ibmpc.h > > > x86: Drop unused distro settings > > > x86: Add a common include for environment settings > > > x86: coreboot: Convert to text environment > > > x86: crownbay: Convert to text environment > > > x86: bayleybay: Convert to text environment > > > x86: galileo: Convert to text environment > > > x86: edison: Convert to text environment > > > x86: cherryhill: Convert to text environment > > > x86: cougarcanyon2: Convert to text environment > > > x86: minnowmax: Convert to text environment > > > x86: slimbootloader: Convert to text environment > > > x86: efi-x86_app: Convert to text environment > > > x86: efi-x86_payload: Convert to text environment > > > x86: qemu: Add required linux/sizes.h include > > > x86: qemu-x86: Convert to text environment > > > > > > arch/x86/cpu/qemu/dram.c | 1 + > > > arch/x86/cpu/qemu/e820.c | 1 + > > > arch/x86/lib/i8254.c | 1 + > > > arch/x86/lib/physmem.c| 3 +- > > > board/coreboot/coreboot/coreboot.env | 9 + > > > board/efi/efi-x86_app/efi-x86_app.env | 6 > > > board/efi/efi-x86_payload/efi-x86_payload.env | 6 > > > board/emulation/qemu-x86/qemu-x86.env | 6 > > > board/intel/bayleybay/bayleybay.env | 9 + > > > board/intel/cherryhill/cherryhill.env | 9 + > > > board/intel/cougarcanyon2/cougarcanyon2.env | 6 > > > board/intel/crownbay/crownbay.env | 6 > > > board/intel/edison/edison.env | 6 > > > board/intel/galileo/galileo.env | 11 ++ > > > board/intel/minnowmax/minnowmax.env | 11 ++ > > > board/intel/slimbootloader/slimbootloader.env | 25 + > > > board/keymile/km83xx/km83xx.env | 4 +-- > > > board/keymile/kmcent2/kmcent2.env | 2 +- > > > .../keymile/pg-wcom-ls102xa/pg-wcom-expu1.env | 2 +- > > > .../keymile/pg-wcom-ls102xa/pg-wcom-seli8.env | 2 +- > > > board/siemens/iot2050/iot2050.env | 2 +- > > > board/ti/am62ax/am62ax.env| 4 +-- > > > board/ti/am62x/am62x.env | 4 +-- > > > board/ti/am64x/am64x.env | 6 ++-- > > > board/ti/am65x/am65x.env | 8 ++--- > > > board/ti/j721e/j721e.env | 10 +++--- > > > board/ti/j721s2/j721s2.env| 10 +++--- > > > board/ti/ks2_evm/k2e_evm.env | 4 +-- > > > board/ti/ks2_evm/k2g_evm.env | 6 ++-- > > > board/ti/ks2_evm/k2hk_evm.env | 4 +-- > > > board/ti/ks2_evm/k2l_evm.env | 4 +-- > > > doc/usage/environment.rst | 8 - > > > include/configs/am335x_evm.h | 2 +- > > > include/configs/am43xx_evm.h | 2 +- > > > include/configs/am57xx_evm.h | 2 +- > > > include/configs/am62ax_evm.h | 4 +-- > > > include/configs/am62x_evm.h | 2 +- > > > include/configs/am64x_evm.h | 4 +-- > > > include/configs/am65x_evm.h | 6 ++-- > > > include/configs/bayleybay.h | 17 - > > > include/configs/cherryhill.h | 13 --- > > > include/configs/coreboot.h| 20 --- > > > include/configs/cougarcanyon2.h | 13 --- > > > include/configs/crownbay.h| 17 - > > > include/configs/da850evm.h| 2 +- > > > include/configs/dra7xx_evm.h | 2 +- > > > include/configs/edison.h | 11 -- > > >
Re: [PATCH 00/22] x86: Move some boards to text environment
Hi Pali, On Sun, 6 Aug 2023 at 04:51, Pali Rohár wrote: > > I'm not x86 maintainer, and I'm not going to review changes. So please > do not send me these emails. I have expressed it many times. You were sent one patch (and the cover letter) because you are the second committer on arch/x86/cpu/qemu/dram.c > On Sunday 30 July 2023 21:01:43 Simon Glass wrote: > > This series starts the process of moving x86 boards to a text environment. > > > > This includes conversions for about half the boards. > > > > The rest (including the more complicated Chromebooks) can come in a later > > series. > > > > > > Simon Glass (22): > > doc: Explain how to avoid the distro-boot scripts > > env: Use include/env for text-environment includes > > env: Explain how to use #include files in text environment > > x86: Drop CFG_SYS_STACK_SIZE > > x86: i8254: Include required ibmpc.h header > > x86: edison: Drop inclusion of ibmpc.h > > x86: Drop inclusion of ibmpc.h > > x86: Drop unused distro settings > > x86: Add a common include for environment settings > > x86: coreboot: Convert to text environment > > x86: crownbay: Convert to text environment > > x86: bayleybay: Convert to text environment > > x86: galileo: Convert to text environment > > x86: edison: Convert to text environment > > x86: cherryhill: Convert to text environment > > x86: cougarcanyon2: Convert to text environment > > x86: minnowmax: Convert to text environment > > x86: slimbootloader: Convert to text environment > > x86: efi-x86_app: Convert to text environment > > x86: efi-x86_payload: Convert to text environment > > x86: qemu: Add required linux/sizes.h include > > x86: qemu-x86: Convert to text environment > > > > arch/x86/cpu/qemu/dram.c | 1 + > > arch/x86/cpu/qemu/e820.c | 1 + > > arch/x86/lib/i8254.c | 1 + > > arch/x86/lib/physmem.c| 3 +- > > board/coreboot/coreboot/coreboot.env | 9 + > > board/efi/efi-x86_app/efi-x86_app.env | 6 > > board/efi/efi-x86_payload/efi-x86_payload.env | 6 > > board/emulation/qemu-x86/qemu-x86.env | 6 > > board/intel/bayleybay/bayleybay.env | 9 + > > board/intel/cherryhill/cherryhill.env | 9 + > > board/intel/cougarcanyon2/cougarcanyon2.env | 6 > > board/intel/crownbay/crownbay.env | 6 > > board/intel/edison/edison.env | 6 > > board/intel/galileo/galileo.env | 11 ++ > > board/intel/minnowmax/minnowmax.env | 11 ++ > > board/intel/slimbootloader/slimbootloader.env | 25 + > > board/keymile/km83xx/km83xx.env | 4 +-- > > board/keymile/kmcent2/kmcent2.env | 2 +- > > .../keymile/pg-wcom-ls102xa/pg-wcom-expu1.env | 2 +- > > .../keymile/pg-wcom-ls102xa/pg-wcom-seli8.env | 2 +- > > board/siemens/iot2050/iot2050.env | 2 +- > > board/ti/am62ax/am62ax.env| 4 +-- > > board/ti/am62x/am62x.env | 4 +-- > > board/ti/am64x/am64x.env | 6 ++-- > > board/ti/am65x/am65x.env | 8 ++--- > > board/ti/j721e/j721e.env | 10 +++--- > > board/ti/j721s2/j721s2.env| 10 +++--- > > board/ti/ks2_evm/k2e_evm.env | 4 +-- > > board/ti/ks2_evm/k2g_evm.env | 6 ++-- > > board/ti/ks2_evm/k2hk_evm.env | 4 +-- > > board/ti/ks2_evm/k2l_evm.env | 4 +-- > > doc/usage/environment.rst | 8 - > > include/configs/am335x_evm.h | 2 +- > > include/configs/am43xx_evm.h | 2 +- > > include/configs/am57xx_evm.h | 2 +- > > include/configs/am62ax_evm.h | 4 +-- > > include/configs/am62x_evm.h | 2 +- > > include/configs/am64x_evm.h | 4 +-- > > include/configs/am65x_evm.h | 6 ++-- > > include/configs/bayleybay.h | 17 - > > include/configs/cherryhill.h | 13 --- > > include/configs/coreboot.h| 20 --- > > include/configs/cougarcanyon2.h | 13 --- > > include/configs/crownbay.h| 17 - > > include/configs/da850evm.h| 2 +- > > include/configs/dra7xx_evm.h | 2 +- > > include/configs/edison.h | 11 -- > > include/configs/efi-x86_app.h | 11 -- > > include/configs/efi-x86_payload.h | 17 - > > include/configs/galileo.h | 19 -- > > include/configs/minnowmax.h | 16 - > > include/configs/omap3_evm.h | 2 +- > > include/configs/omapl138_lcdk.h | 2 +- > >
Re: [PATCH 00/22] x86: Move some boards to text environment
I'm not x86 maintainer, and I'm not going to review changes. So please do not send me these emails. I have expressed it many times. On Sunday 30 July 2023 21:01:43 Simon Glass wrote: > This series starts the process of moving x86 boards to a text environment. > > This includes conversions for about half the boards. > > The rest (including the more complicated Chromebooks) can come in a later > series. > > > Simon Glass (22): > doc: Explain how to avoid the distro-boot scripts > env: Use include/env for text-environment includes > env: Explain how to use #include files in text environment > x86: Drop CFG_SYS_STACK_SIZE > x86: i8254: Include required ibmpc.h header > x86: edison: Drop inclusion of ibmpc.h > x86: Drop inclusion of ibmpc.h > x86: Drop unused distro settings > x86: Add a common include for environment settings > x86: coreboot: Convert to text environment > x86: crownbay: Convert to text environment > x86: bayleybay: Convert to text environment > x86: galileo: Convert to text environment > x86: edison: Convert to text environment > x86: cherryhill: Convert to text environment > x86: cougarcanyon2: Convert to text environment > x86: minnowmax: Convert to text environment > x86: slimbootloader: Convert to text environment > x86: efi-x86_app: Convert to text environment > x86: efi-x86_payload: Convert to text environment > x86: qemu: Add required linux/sizes.h include > x86: qemu-x86: Convert to text environment > > arch/x86/cpu/qemu/dram.c | 1 + > arch/x86/cpu/qemu/e820.c | 1 + > arch/x86/lib/i8254.c | 1 + > arch/x86/lib/physmem.c| 3 +- > board/coreboot/coreboot/coreboot.env | 9 + > board/efi/efi-x86_app/efi-x86_app.env | 6 > board/efi/efi-x86_payload/efi-x86_payload.env | 6 > board/emulation/qemu-x86/qemu-x86.env | 6 > board/intel/bayleybay/bayleybay.env | 9 + > board/intel/cherryhill/cherryhill.env | 9 + > board/intel/cougarcanyon2/cougarcanyon2.env | 6 > board/intel/crownbay/crownbay.env | 6 > board/intel/edison/edison.env | 6 > board/intel/galileo/galileo.env | 11 ++ > board/intel/minnowmax/minnowmax.env | 11 ++ > board/intel/slimbootloader/slimbootloader.env | 25 + > board/keymile/km83xx/km83xx.env | 4 +-- > board/keymile/kmcent2/kmcent2.env | 2 +- > .../keymile/pg-wcom-ls102xa/pg-wcom-expu1.env | 2 +- > .../keymile/pg-wcom-ls102xa/pg-wcom-seli8.env | 2 +- > board/siemens/iot2050/iot2050.env | 2 +- > board/ti/am62ax/am62ax.env| 4 +-- > board/ti/am62x/am62x.env | 4 +-- > board/ti/am64x/am64x.env | 6 ++-- > board/ti/am65x/am65x.env | 8 ++--- > board/ti/j721e/j721e.env | 10 +++--- > board/ti/j721s2/j721s2.env| 10 +++--- > board/ti/ks2_evm/k2e_evm.env | 4 +-- > board/ti/ks2_evm/k2g_evm.env | 6 ++-- > board/ti/ks2_evm/k2hk_evm.env | 4 +-- > board/ti/ks2_evm/k2l_evm.env | 4 +-- > doc/usage/environment.rst | 8 - > include/configs/am335x_evm.h | 2 +- > include/configs/am43xx_evm.h | 2 +- > include/configs/am57xx_evm.h | 2 +- > include/configs/am62ax_evm.h | 4 +-- > include/configs/am62x_evm.h | 2 +- > include/configs/am64x_evm.h | 4 +-- > include/configs/am65x_evm.h | 6 ++-- > include/configs/bayleybay.h | 17 - > include/configs/cherryhill.h | 13 --- > include/configs/coreboot.h| 20 --- > include/configs/cougarcanyon2.h | 13 --- > include/configs/crownbay.h| 17 - > include/configs/da850evm.h| 2 +- > include/configs/dra7xx_evm.h | 2 +- > include/configs/edison.h | 11 -- > include/configs/efi-x86_app.h | 11 -- > include/configs/efi-x86_payload.h | 17 - > include/configs/galileo.h | 19 -- > include/configs/minnowmax.h | 16 - > include/configs/omap3_evm.h | 2 +- > include/configs/omapl138_lcdk.h | 2 +- > include/configs/phycore_am335x_r2.h | 4 +-- > include/configs/qemu-x86.h| 23 > include/configs/sifive-unleashed.h| 2 +- > include/configs/slimbootloader.h | 35 --- > include/configs/ti_omap4_common.h | 2 +- > include/configs/ti_omap5_common.h
Re: [PATCH 00/22] x86: Move some boards to text environment
On Thu, Aug 3, 2023 at 7:27 PM Bin Meng wrote: > > On Mon, Jul 31, 2023 at 4:12 PM Andy Shevchenko > wrote: > > > > On Sun, Jul 30, 2023 at 09:01:43PM -0600, Simon Glass wrote: > > > This series starts the process of moving x86 boards to a text environment. > > > > > > This includes conversions for about half the boards. > > > > > > The rest (including the more complicated Chromebooks) can come in a later > > > series. > > > > Patch wise it looks good to me > > Reviewed-by: Andy Shevchenko # Intel > > Edison > > Series LGTM too. > > Reviewed-by: Bin Meng > Edited "[18/22] x86: slimbootloader: Convert to text environment ", series applied to u-boot-x86, thanks!
Re: [PATCH 00/22] x86: Move some boards to text environment
On Mon, Jul 31, 2023 at 4:12 PM Andy Shevchenko wrote: > > On Sun, Jul 30, 2023 at 09:01:43PM -0600, Simon Glass wrote: > > This series starts the process of moving x86 boards to a text environment. > > > > This includes conversions for about half the boards. > > > > The rest (including the more complicated Chromebooks) can come in a later > > series. > > Patch wise it looks good to me > Reviewed-by: Andy Shevchenko # Intel > Edison Series LGTM too. Reviewed-by: Bin Meng > > Where can I read about this new edison.env file and what is expected to be > there? > Regards, Bin
Re: [PATCH 00/22] x86: Move some boards to text environment
Hi Andy, On Mon, 31 Jul 2023 at 02:09, Andy Shevchenko wrote: > > On Sun, Jul 30, 2023 at 09:01:43PM -0600, Simon Glass wrote: > > This series starts the process of moving x86 boards to a text environment. > > > > This includes conversions for about half the boards. > > > > The rest (including the more complicated Chromebooks) can come in a later > > series. > > Patch wise it looks good to me > Reviewed-by: Andy Shevchenko # Intel > Edison > > Where can I read about this new edison.env file and what is expected to be > there? Thanks. It is here: https://u-boot.readthedocs.io/en/latest/usage/environment.html#text-based-environment [..] Regards, Simon
Re: [PATCH 00/22] x86: Move some boards to text environment
On Sun, Jul 30, 2023 at 09:01:43PM -0600, Simon Glass wrote: > This series starts the process of moving x86 boards to a text environment. > > This includes conversions for about half the boards. > > The rest (including the more complicated Chromebooks) can come in a later > series. Patch wise it looks good to me Reviewed-by: Andy Shevchenko # Intel Edison Where can I read about this new edison.env file and what is expected to be there? > doc: Explain how to avoid the distro-boot scripts > env: Use include/env for text-environment includes > env: Explain how to use #include files in text environment > x86: Drop CFG_SYS_STACK_SIZE > x86: i8254: Include required ibmpc.h header > x86: edison: Drop inclusion of ibmpc.h > x86: Drop inclusion of ibmpc.h > x86: Drop unused distro settings > x86: Add a common include for environment settings > x86: coreboot: Convert to text environment > x86: crownbay: Convert to text environment > x86: bayleybay: Convert to text environment > x86: galileo: Convert to text environment > x86: edison: Convert to text environment > x86: cherryhill: Convert to text environment > x86: cougarcanyon2: Convert to text environment > x86: minnowmax: Convert to text environment > x86: slimbootloader: Convert to text environment > x86: efi-x86_app: Convert to text environment > x86: efi-x86_payload: Convert to text environment > x86: qemu: Add required linux/sizes.h include > x86: qemu-x86: Convert to text environment > > arch/x86/cpu/qemu/dram.c | 1 + > arch/x86/cpu/qemu/e820.c | 1 + > arch/x86/lib/i8254.c | 1 + > arch/x86/lib/physmem.c| 3 +- > board/coreboot/coreboot/coreboot.env | 9 + > board/efi/efi-x86_app/efi-x86_app.env | 6 > board/efi/efi-x86_payload/efi-x86_payload.env | 6 > board/emulation/qemu-x86/qemu-x86.env | 6 > board/intel/bayleybay/bayleybay.env | 9 + > board/intel/cherryhill/cherryhill.env | 9 + > board/intel/cougarcanyon2/cougarcanyon2.env | 6 > board/intel/crownbay/crownbay.env | 6 > board/intel/edison/edison.env | 6 > board/intel/galileo/galileo.env | 11 ++ > board/intel/minnowmax/minnowmax.env | 11 ++ > board/intel/slimbootloader/slimbootloader.env | 25 + > board/keymile/km83xx/km83xx.env | 4 +-- > board/keymile/kmcent2/kmcent2.env | 2 +- > .../keymile/pg-wcom-ls102xa/pg-wcom-expu1.env | 2 +- > .../keymile/pg-wcom-ls102xa/pg-wcom-seli8.env | 2 +- > board/siemens/iot2050/iot2050.env | 2 +- > board/ti/am62ax/am62ax.env| 4 +-- > board/ti/am62x/am62x.env | 4 +-- > board/ti/am64x/am64x.env | 6 ++-- > board/ti/am65x/am65x.env | 8 ++--- > board/ti/j721e/j721e.env | 10 +++--- > board/ti/j721s2/j721s2.env| 10 +++--- > board/ti/ks2_evm/k2e_evm.env | 4 +-- > board/ti/ks2_evm/k2g_evm.env | 6 ++-- > board/ti/ks2_evm/k2hk_evm.env | 4 +-- > board/ti/ks2_evm/k2l_evm.env | 4 +-- > doc/usage/environment.rst | 8 - > include/configs/am335x_evm.h | 2 +- > include/configs/am43xx_evm.h | 2 +- > include/configs/am57xx_evm.h | 2 +- > include/configs/am62ax_evm.h | 4 +-- > include/configs/am62x_evm.h | 2 +- > include/configs/am64x_evm.h | 4 +-- > include/configs/am65x_evm.h | 6 ++-- > include/configs/bayleybay.h | 17 - > include/configs/cherryhill.h | 13 --- > include/configs/coreboot.h| 20 --- > include/configs/cougarcanyon2.h | 13 --- > include/configs/crownbay.h| 17 - > include/configs/da850evm.h| 2 +- > include/configs/dra7xx_evm.h | 2 +- > include/configs/edison.h | 11 -- > include/configs/efi-x86_app.h | 11 -- > include/configs/efi-x86_payload.h | 17 - > include/configs/galileo.h | 19 -- > include/configs/minnowmax.h | 16 - > include/configs/omap3_evm.h | 2 +- > include/configs/omapl138_lcdk.h | 2 +- > include/configs/phycore_am335x_r2.h | 4 +-- > include/configs/qemu-x86.h| 23 > include/configs/sifive-unleashed.h| 2 +- > include/configs/slimbootloader.h | 35 --- > include/configs/ti_omap4_common.h | 2 +- >