Re: [RESEND RFC PATCH 01/12] sh: Add OF target boards.

[Yoshinori Sato <ysato@users.sourceforge.jp>]

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
> > > <ysato@users.sourceforge.jp> 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 "<machine>_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