From mboxrd@z Thu Jan 1 00:00:00 1970 From: Igor Grinberg Date: Thu, 14 Jul 2011 17:57:16 +0300 Subject: [U-Boot] [PATCH 1/3] arm: add CONFIG_MACH_TYPE option and documentation In-Reply-To: <4E1EFB36.201@aribaud.net> References: <1309770021-9908-1-git-send-email-grinberg@compulab.co.il> <4E14AF14.5020104@aribaud.net> <4E14C020.8030109@compulab.co.il> <4E15D9B5.5000001@aribaud.net> <4E15E41E.3090007@compulab.co.il> <4E15F0E1.3080209@aribaud.net> <4E161FD9.2050205@compulab.co.il> <4E1D3287.1030407@compulab.co.il> <4E1EF8C3.5030707@aribaud.net> <4E1EFB36.201@aribaud.net> Message-ID: <4E1F03CC.5040401@compulab.co.il> List-Id: MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: u-boot@lists.denx.de On 07/14/11 17:20, Albert ARIBAUD wrote: > Hi again Igor, > > Le 14/07/2011 16:10, Albert ARIBAUD a ?crit : > >> Pragmatism against perfection: let's go for 2, then. Change the boards >> you intended to change, and from now on reviewers for any change to a >> board should point out the move to CONFIG_MACH_TYPE as mandatory. > > Forgot to mention: > > That makes patches 2/3 and 3/3 ok for acceptance, but can you post a V2 of patch 1/3 in which documentation of CONFIG_MACH_TYPE describes it as mandatory (and forbids direct setting of bi_arch_number) for board configs with a single machine-type? Ok. I have to make it v2 anyway, because the setting of bi_arch_number in the board.c has to be moved to board_init_f() - before the init_sequence[] array is run. -- Regards, Igor.