|To:||Aaro Koskinen <firstname.lastname@example.org>|
|Subject:||Re: [PATCH v2 1/2] MIPS: cavium-octeon: cvmx-helper-board: print unknown board warning only once|
|From:||David Daney <email@example.com>|
|Date:||Mon, 24 Jun 2013 16:24:06 -0700|
|Cc:||Ralf Baechle <firstname.lastname@example.org>, email@example.com|
|Dkim-signature:||v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20120113; h=message-id:date:from:user-agent:mime-version:to:cc:subject :references:in-reply-to:content-type:content-transfer-encoding; bh=LdgGCDHevXWPwRhfwldfh41c5nLKPzKcW73G9WxRwnA=; b=KWn9aGZNhZJnckk+849B+bh4IvB35nch+BS4D7SPBbW6ZIzZz9/mlo2MeZ522j0+vF DcOKr+Km8K1ol0jbEdumpo7/DDlsPji+oKAndWTVyYlFr7zF4FSOecJaMfhoGWrkkQsj PZtIY3cCVkW3ryqHMrwSNDtXKg40IUYKyz+Kw818Pgm3INCNnLkHM55yc3F+K3NkboRq ql/J6JU98ViWopuYnZBggUtfTMceeXvWcZZkcinLykt5crtDHF7gUCqv4VhZu+1zpEXa DF0+WCtaSTrq54wy3SKJhDYvkZVnIo9lU6wPqFUfJMKr0O1EnYTw4rXuDuslXSRy0NaW qt3A==|
|List-software:||Ecartis version 1.0.0|
|References:||<firstname.lastname@example.org> <51C89567.email@example.com> <20130624220429.GB20703@blackmetal.musicnaut.iki.fi> <51C8C940.firstname.lastname@example.org> <20130624230639.GC20703@blackmetal.musicnaut.iki.fi>|
|User-agent:||Mozilla/5.0 (X11; Linux x86_64; rv:17.0) Gecko/20130514 Thunderbird/17.0.6|
On 06/24/2013 04:06 PM, Aaro Koskinen wrote:
On Mon, Jun 24, 2013 at 03:33:36PM -0700, David Daney wrote:On 06/24/2013 03:04 PM, Aaro Koskinen wrote:On Mon, Jun 24, 2013 at 11:52:23AM -0700, David Daney wrote:On 06/23/2013 02:38 PM, Aaro Koskinen wrote:When booting a new board for the first time, the console is flooded with "Unknown board" messages. This is not really helpful. Board type is not going to change after the boot, so it's sufficient to print the warning only once. Signed-off-by: Aaro Koskinen <email@example.com>I don't think we need this patch. In 2/2 you add the board type for the board you have, so you shouldn't be getting any messages, and this is unneeded. I don't mind spamming people with all the messages, if people see these messages, they have bigger problems than too many messages.I guess this patch can be dropped, but whoever tries to improve the support for the next new Octeon board will get annoyed by these same messages...I would hope that the "next new Octeon board" would have a bootloader that supplies a device tree. That way most of this would never be used, and there would be no messages.Yes, actually I was wondering if MIPS kernel could offer something similar to ARM's APPENDED_DTB, i.e. provide some mechanism to pass the dtb if the bootloader support is missing.
Well the octeon port does supply its own device tree if one isn't passed from the bootloader. And your patch 2/2 supplies the information used populate it.
But that is not quite the same I think. It would be nice to be able to pass on the command line something. Then you could load a blob from the boot ROM or similar and get something sensible.
|<Prev in Thread]||Current Thread||[Next in Thread>|
|Previous by Date:|
|Next by Date:||Re: [PATCH] gpio MIPS/OCTEON: Add a driver for OCTEON's on-chip GPIO pins., David Daney|
|Previous by Thread:|
|Next by Thread:||Re: [PATCH v2 1/2] MIPS: cavium-octeon: cvmx-helper-board: print unknown board warning only once, Ralf Baechle|
|Indexes:||[Date] [Thread] [Top] [All Lists]|