2014-11-18 09:24:07

by Paul Bolle

[permalink] [raw]
Subject: ARM: shmobile: ARCH_R8A7793?

Hi Geert,

Your commit 7a2071c58f36 ("ARM: shmobile: Add early debugging support
using SCIF(A)") is included in today's linux-next (ie, next-20141118).
It adds an (optional) dependency on the Kconfig symbol ARCH_R8A7793. But
that symbol isn't part of next-20141118.

Is a patch that adds this symbol queued somewhere?


Paul Bolle


2014-11-18 09:37:48

by Geert Uytterhoeven

[permalink] [raw]
Subject: Re: ARM: shmobile: ARCH_R8A7793?

Hi Paul,

On Tue, Nov 18, 2014 at 10:24 AM, Paul Bolle <[email protected]> wrote:
> Your commit 7a2071c58f36 ("ARM: shmobile: Add early debugging support
> using SCIF(A)") is included in today's linux-next (ie, next-20141118).
> It adds an (optional) dependency on the Kconfig symbol ARCH_R8A7793. But
> that symbol isn't part of next-20141118.
>
> Is a patch that adds this symbol queued somewhere?

Not yet. I was just a bit pro-active ;-)

However, there are already references to r8a7793 in DT binding documentation,
and in of_device_id arrays in drivers.

Gr{oetje,eeting}s,

Geert

--
Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- [email protected]

In personal conversations with technical people, I call myself a hacker. But
when I'm talking to journalists I just say "programmer" or something like that.
-- Linus Torvalds

2014-11-18 10:12:11

by Paul Bolle

[permalink] [raw]
Subject: Re: ARM: shmobile: ARCH_R8A7793?

On Tue, 2014-11-18 at 10:36 +0100, Geert Uytterhoeven wrote:
> On Tue, Nov 18, 2014 at 10:24 AM, Paul Bolle <[email protected]> wrote:
> > Your commit 7a2071c58f36 ("ARM: shmobile: Add early debugging support
> > using SCIF(A)") is included in today's linux-next (ie, next-20141118).
> > It adds an (optional) dependency on the Kconfig symbol ARCH_R8A7793. But
> > that symbol isn't part of next-20141118.
> >
> > Is a patch that adds this symbol queued somewhere?
>
> Not yet. I was just a bit pro-active ;-)

My script will stay silent (as might future, proper, bots) if you don't
add references to Kconfig symbols before adding the symbol itself.
(Whether or not it's worth it to remove this dependency here and add it
to the queue that adds this symbol is up to you, I guess.)

Unless the symbol takes a long time to land I'll try not to bother you
again.

> However, there are already references to r8a7793 in DT binding documentation,
> and in of_device_id arrays in drivers.

I see.


Paul Bolle

2015-04-28 15:53:24

by Valentin Rothberg

[permalink] [raw]
Subject: Re: ARM: shmobile: ARCH_R8A7793?

Hi Geert,

I am cleaning up some Kconfig issues in ARM
(./scripts/checkkconfgisymbols.py) at the moment and came across
CONFIG_ARCH_R8A7793:

arch/arm/Kconfig.debug: depends on ARCH_R8A7790 ||
ARCH_R8A7791 || ARCH_R8A7793
drivers/clk/shmobile/Makefile:obj-$(CONFIG_ARCH_R8A7793)
+= clk-rcar-gen2.o

Do you know when this option will be added to Kconfig (see email below)?

Kind regards,
Valentin

On Tue, Nov 18, 2014 at 10:36 AM, Geert Uytterhoeven
<[email protected]> wrote:
> Hi Paul,
>
> On Tue, Nov 18, 2014 at 10:24 AM, Paul Bolle <[email protected]> wrote:
>> Your commit 7a2071c58f36 ("ARM: shmobile: Add early debugging support
>> using SCIF(A)") is included in today's linux-next (ie, next-20141118).
>> It adds an (optional) dependency on the Kconfig symbol ARCH_R8A7793. But
>> that symbol isn't part of next-20141118.
>>
>> Is a patch that adds this symbol queued somewhere?
>
> Not yet. I was just a bit pro-active ;-)
>
> However, there are already references to r8a7793 in DT binding documentation,
> and in of_device_id arrays in drivers.
>
> Gr{oetje,eeting}s,
>
> Geert
>
> --
> Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- [email protected]
>
> In personal conversations with technical people, I call myself a hacker. But
> when I'm talking to journalists I just say "programmer" or something like that.
> -- Linus Torvalds

2015-04-28 16:53:19

by Geert Uytterhoeven

[permalink] [raw]
Subject: Re: ARM: shmobile: ARCH_R8A7793?

Hi Valentin,

On Tue, Apr 28, 2015 at 5:52 PM, Valentin Rothberg
<[email protected]> wrote:
> I am cleaning up some Kconfig issues in ARM
> (./scripts/checkkconfgisymbols.py) at the moment and came across
> CONFIG_ARCH_R8A7793:
>
> arch/arm/Kconfig.debug: depends on ARCH_R8A7790 ||
> ARCH_R8A7791 || ARCH_R8A7793
> drivers/clk/shmobile/Makefile:obj-$(CONFIG_ARCH_R8A7793)
> += clk-rcar-gen2.o
>
> Do you know when this option will be added to Kconfig (see email below)?

A first patch series was sent last month, cfr. ("[PATCH 00/11] r8a7793 SoC,
Gose board support", http://www.spinics.net/lists/linux-sh/msg40781.html).

I assume an update, taking account review comments, will be posted shortly.

> On Tue, Nov 18, 2014 at 10:36 AM, Geert Uytterhoeven
> <[email protected]> wrote:
>> Hi Paul,
>>
>> On Tue, Nov 18, 2014 at 10:24 AM, Paul Bolle <[email protected]> wrote:
>>> Your commit 7a2071c58f36 ("ARM: shmobile: Add early debugging support
>>> using SCIF(A)") is included in today's linux-next (ie, next-20141118).
>>> It adds an (optional) dependency on the Kconfig symbol ARCH_R8A7793. But
>>> that symbol isn't part of next-20141118.
>>>
>>> Is a patch that adds this symbol queued somewhere?
>>
>> Not yet. I was just a bit pro-active ;-)
>>
>> However, there are already references to r8a7793 in DT binding documentation,
>> and in of_device_id arrays in drivers.

Gr{oetje,eeting}s,

Geert

--
Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- [email protected]

In personal conversations with technical people, I call myself a hacker. But
when I'm talking to journalists I just say "programmer" or something like that.
-- Linus Torvalds