From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org Received: from vger.kernel.org (vger.kernel.org [23.128.96.18]) by smtp.lore.kernel.org (Postfix) with ESMTP id BFEEFEE14C3 for ; Thu, 7 Sep 2023 03:28:58 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S231859AbjIGD3A convert rfc822-to-8bit (ORCPT ); Wed, 6 Sep 2023 23:29:00 -0400 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:55708 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S229782AbjIGD27 (ORCPT ); Wed, 6 Sep 2023 23:28:59 -0400 Received: from hsmtpd-def.xspmail.jp (hsmtpd-def.xspmail.jp [202.238.198.238]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id 646BF132 for ; Wed, 6 Sep 2023 20:28:54 -0700 (PDT) X-Country-Code: JP Received: from sakura.ysato.name (ik1-413-38519.vs.sakura.ne.jp [153.127.30.23]) by hsmtpd-out-2.asahinet.cluster.xspmail.jp (Halon) with ESMTPA id 54d4489a-4701-4c6e-a12e-8fda9f9c7d9e; Thu, 07 Sep 2023 12:28:52 +0900 (JST) Received: from SIOS1075.ysato.ml (al128006.dynamic.ppp.asahi-net.or.jp [111.234.128.6]) by sakura.ysato.name (Postfix) with ESMTPSA id 1C8281C0079; Thu, 7 Sep 2023 12:28:51 +0900 (JST) Date: Thu, 07 Sep 2023 12:28:50 +0900 Message-ID: <874jk6vez1.wl-ysato@users.sourceforge.jp> From: Yoshinori Sato To: John Paul Adrian Glaubitz Cc: Geert Uytterhoeven , linux-sh@vger.kernel.org Subject: Re: [RESEND RFC PATCH 01/12] sh: Add OF target boards. In-Reply-To: <994b6714a478ee8d33f5938e8b59685c692885b0.camel@physik.fu-berlin.de> References: <95d8b86480c60012252b37b9b13e5f709a2ec177.1693444193.git.ysato@users.sourceforge.jp> <87bkefvi9i.wl-ysato@users.sourceforge.jp> <994b6714a478ee8d33f5938e8b59685c692885b0.camel@physik.fu-berlin.de> User-Agent: Wanderlust/2.15.9 (Almost Unreal) SEMI-EPG/1.14.7 (Harue) FLIM-LB/1.14.9 (=?ISO-8859-4?Q?Goj=F2?=) APEL-LB/10.8 EasyPG/1.0.0 Emacs/28.2 (x86_64-pc-linux-gnu) MULE/6.0 (HANACHIRUSATO) MIME-Version: 1.0 (generated by SEMI-EPG 1.14.7 - "Harue") Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8BIT Precedence: bulk List-ID: X-Mailing-List: linux-sh@vger.kernel.org On Wed, 06 Sep 2023 17:12:31 +0900, John Paul Adrian Glaubitz wrote: > > Hi Yoshinori! > > On Wed, 2023-09-06 at 17:05 +0900, Yoshinori Sato wrote: > > On Fri, 01 Sep 2023 23:26:10 +0900, > > Geert Uytterhoeven wrote: > > > > > > Hi Sato-san, > > > > > > On Thu, Aug 31, 2023 at 5:19 AM Yoshinori Sato > > > wrote: > > > > --- a/arch/sh/Kconfig > > > > +++ b/arch/sh/Kconfig > > > > > > > @@ -702,7 +707,7 @@ config BUILTIN_DTB_SOURCE > > > > config ZERO_PAGE_OFFSET > > > > hex > > > > default "0x00010000" if PAGE_SIZE_64KB || SH_RTS7751R2D || \ > > > > - SH_7751_SOLUTION_ENGINE > > > > + SH_7751_SOLUTION_ENGINE || SH_RTS7751R2D_OF > > > > > > This is the only user of SH_RTS7751R2D_OF. Can we get rid of it? > > > > > > > default "0x00004000" if PAGE_SIZE_16KB || SH_SH03 > > > > default "0x00002000" if PAGE_SIZE_8KB > > > > default "0x00001000" > > > > > > > --- a/arch/sh/boards/Kconfig > > > > +++ b/arch/sh/boards/Kconfig > > > > @@ -161,6 +166,17 @@ config SH_RTS7751R2D > > > > Select RTS7751R2D if configuring for a Renesas Technology > > > > Sales SH-Graphics board. > > > > > > > > +config SH_RTS7751R2D_OF > > > > + bool "RTS7751R2D (DeviceTree)" > > > > + depends on CPU_SUBTYPE_SH7751R > > > > + select HAVE_PCI > > > > + select IO_TRAPPED if MMU > > > > + select SH_DEVICE_TREE > > > > + select COMMON_CLK > > > > + help > > > > + Select RTS7751R2D if configuring for a Renesas Technology > > > > + Sales SH-Graphics board. (Use DeviceTree) > > > > + > > > > config SH_RSK > > > > bool "Renesas Starter Kit" > > > > depends on CPU_SUBTYPE_SH7201 || CPU_SUBTYPE_SH7203 || \ > > > > @@ -300,8 +316,20 @@ config SH_LANDISK > > > > bool "LANDISK" > > > > depends on CPU_SUBTYPE_SH7751R > > > > select HAVE_PCI > > > > + select SYS_SUPPORTS_PCI > > > > + help > > > > + I-O DATA DEVICE, INC. "LANDISK Series" support. > > > > + > > > > +config SH_LANDISK_OF > > > > + bool "LANDISK (DeviceTree)" > > > > + depends on CPU_SUBTYPE_SH7751R > > > > + select HAVE_PCI > > > > + select SYS_SUPPORTS_PCI > > > > + select SH_DEVICE_TREE > > > > + select COMMON_CLK > > > > help > > > > I-O DATA DEVICE, INC. "LANDISK Series" support. > > > > + Use Device Tree. > > > > > > > > config SH_TITAN > > > > bool "TITAN" > > > > > > Apart from the above, there are no users of the "_OF" symbols. > > > Do we need them? > > Did you accidentally sent this mail before commenting? Ouch Yes. I'll put it out again. > Adrian > > -- > .''`. John Paul Adrian Glaubitz > : :' : Debian Developer > `. `' Physicist > `- GPG: 62FF 8A75 84E0 2956 9546 0006 7426 3B37 F5B5 F913 -- Yosinori Sato