[PATCH] ARM: shmobile: koelsch: Conditionally select MICREL_PHY

[PATCH] ARM: shmobile: koelsch: Conditionally select MICREL_PHY

[Simon Horman]

The koelsch board uses has an SH ethernet controller which uses a Micrel
phy. Select MICREL_PHY for koelsch if SH_ETH is enabled to make use of the
Micrel-specific phy driver rather than relying on the generic phy driver.

Signed-off-by: Simon Horman <horms+renesas@verge.net.au>
---
 arch/arm/mach-shmobile/Kconfig | 3 +++
 1 file changed, 3 insertions(+)

diff --git a/arch/arm/mach-shmobile/Kconfig b/arch/arm/mach-shmobile/Kconfig
index f01e878..670238a 100644
--- a/arch/arm/mach-shmobile/Kconfig
+++ b/arch/arm/mach-shmobile/Kconfig
@@ -41,6 +41,7 @@ config MACH_GENMAI
 config MACH_KOELSCH
 	bool "Koelsch board"
 	depends on ARCH_R8A7791
+	select MICREL_PHY if SH_ETH
 
 config MACH_KZM9D
 	bool "KZM9D board"
@@ -269,11 +270,13 @@ config MACH_KOELSCH
 	bool "Koelsch board"
 	depends on ARCH_R8A7791
 	select USE_OF
+	select MICREL_PHY if SH_ETH
 
 config MACH_KOELSCH_REFERENCE
 	bool "Koelsch board - Reference Device Tree Implementation"
 	depends on ARCH_R8A7791
 	select USE_OF
+	select MICREL_PHY if SH_ETH
 	---help---
 	   Use reference implementation of Koelsch board support
 	   which makes use of device tree at the expense
-- 
1.8.4

[PATCH] ARM: shmobile: koelsch: Conditionally select MICREL_PHY

[Simon Horman]

The koelsch board uses has an SH ethernet controller which uses a Micrel
phy. Select MICREL_PHY for koelsch if SH_ETH is enabled to make use of the
Micrel-specific phy driver rather than relying on the generic phy driver.

Signed-off-by: Simon Horman <horms+renesas@verge.net.au>
---
 arch/arm/mach-shmobile/Kconfig | 3 +++
 1 file changed, 3 insertions(+)

diff --git a/arch/arm/mach-shmobile/Kconfig b/arch/arm/mach-shmobile/Kconfig
index f01e878..670238a 100644
--- a/arch/arm/mach-shmobile/Kconfig
+++ b/arch/arm/mach-shmobile/Kconfig
@@ -41,6 +41,7 @@ config MACH_GENMAI
 config MACH_KOELSCH
 	bool "Koelsch board"
 	depends on ARCH_R8A7791
+	select MICREL_PHY if SH_ETH
 
 config MACH_KZM9D
 	bool "KZM9D board"
@@ -269,11 +270,13 @@ config MACH_KOELSCH
 	bool "Koelsch board"
 	depends on ARCH_R8A7791
 	select USE_OF
+	select MICREL_PHY if SH_ETH
 
 config MACH_KOELSCH_REFERENCE
 	bool "Koelsch board - Reference Device Tree Implementation"
 	depends on ARCH_R8A7791
 	select USE_OF
+	select MICREL_PHY if SH_ETH
 	---help---
 	   Use reference implementation of Koelsch board support
 	   which makes use of device tree at the expense
-- 
1.8.4

Re: [PATCH] ARM: shmobile: koelsch: Conditionally select MICREL_PHY

[Sergei Shtylyov]

Hello.

On 16-12-2013 4:31, Simon Horman wrote:

> The koelsch board uses has an SH ethernet controller which uses a Micrel
> phy. Select MICREL_PHY for koelsch if SH_ETH is enabled to make use of the
> Micrel-specific phy driver rather than relying on the generic phy driver.

    Frankly speaking, I'm seeing no advantages to doing that.

> Signed-off-by: Simon Horman <horms+renesas@verge.net.au>

    Analogous changes are needed for BOCK-W and Lager boards since they all 
use the same KSZ8041RNLI PHY chip. I was going to submit such changes with the 
PHY IRQ patches but since no IRQ is generated still, this got delayed...

WBR, Sergei

[PATCH] ARM: shmobile: koelsch: Conditionally select MICREL_PHY

[Sergei Shtylyov]

Hello.

On 16-12-2013 4:31, Simon Horman wrote:

> The koelsch board uses has an SH ethernet controller which uses a Micrel
> phy. Select MICREL_PHY for koelsch if SH_ETH is enabled to make use of the
> Micrel-specific phy driver rather than relying on the generic phy driver.

    Frankly speaking, I'm seeing no advantages to doing that.

> Signed-off-by: Simon Horman <horms+renesas@verge.net.au>

    Analogous changes are needed for BOCK-W and Lager boards since they all 
use the same KSZ8041RNLI PHY chip. I was going to submit such changes with the 
PHY IRQ patches but since no IRQ is generated still, this got delayed...

WBR, Sergei

Re: [PATCH] ARM: shmobile: koelsch: Conditionally select MICREL_PHY

[Sergei Shtylyov]

On 12/16/2013 04:25 PM, I wrote:

>> The koelsch board uses has an SH ethernet controller which uses a Micrel
>> phy. Select MICREL_PHY for koelsch if SH_ETH is enabled to make use of the
>> Micrel-specific phy driver rather than relying on the generic phy driver.

>     Frankly speaking, I'm seeing no advantages to doing that.

    Although wait, Geert has reported the absence of Rx FIFO overflow messages 
with this PHY driver, IIRC.

>> Signed-off-by: Simon Horman <horms+renesas@verge.net.au>

>     Analogous changes are needed for BOCK-W and Lager boards since they all
> use the same KSZ8041RNLI PHY chip. I was going to submit such changes with the
> PHY IRQ patches but since no IRQ is generated still, this got delayed...

    Only I was going to do it via defconfigs...

WBR, Sergei

[PATCH] ARM: shmobile: koelsch: Conditionally select MICREL_PHY

[Sergei Shtylyov]

On 12/16/2013 04:25 PM, I wrote:

>> The koelsch board uses has an SH ethernet controller which uses a Micrel
>> phy. Select MICREL_PHY for koelsch if SH_ETH is enabled to make use of the
>> Micrel-specific phy driver rather than relying on the generic phy driver.

>     Frankly speaking, I'm seeing no advantages to doing that.

    Although wait, Geert has reported the absence of Rx FIFO overflow messages 
with this PHY driver, IIRC.

>> Signed-off-by: Simon Horman <horms+renesas@verge.net.au>

>     Analogous changes are needed for BOCK-W and Lager boards since they all
> use the same KSZ8041RNLI PHY chip. I was going to submit such changes with the
> PHY IRQ patches but since no IRQ is generated still, this got delayed...

    Only I was going to do it via defconfigs...

WBR, Sergei

Re: [PATCH] ARM: shmobile: koelsch: Conditionally select MICREL_PHY

[Simon Horman]

On Mon, Dec 16, 2013 at 05:25:42PM +0400, Sergei Shtylyov wrote:
> Hello.
> 
> On 16-12-2013 4:31, Simon Horman wrote:
> 
> >The koelsch board uses has an SH ethernet controller which uses a Micrel
> >phy. Select MICREL_PHY for koelsch if SH_ETH is enabled to make use of the
> >Micrel-specific phy driver rather than relying on the generic phy driver.
> 
>    Frankly speaking, I'm seeing no advantages to doing that.

I think there is an advantage to using the hardware-specific driver.
Even if it make any difference at run-time with the present code.

> >Signed-off-by: Simon Horman <horms+renesas@verge.net.au>
> 
>    Analogous changes are needed for BOCK-W and Lager boards since
> they all use the same KSZ8041RNLI PHY chip. I was going to submit
> such changes with the PHY IRQ patches but since no IRQ is generated
> still, this got delayed...

I believe that similar changes are needed for almost all
of the shmobile boards. If you would like to handle some or all
of them that is quite fine by me.

[PATCH] ARM: shmobile: koelsch: Conditionally select MICREL_PHY

[Simon Horman]

On Mon, Dec 16, 2013 at 05:25:42PM +0400, Sergei Shtylyov wrote:
> Hello.
> 
> On 16-12-2013 4:31, Simon Horman wrote:
> 
> >The koelsch board uses has an SH ethernet controller which uses a Micrel
> >phy. Select MICREL_PHY for koelsch if SH_ETH is enabled to make use of the
> >Micrel-specific phy driver rather than relying on the generic phy driver.
> 
>    Frankly speaking, I'm seeing no advantages to doing that.

I think there is an advantage to using the hardware-specific driver.
Even if it make any difference at run-time with the present code.

> >Signed-off-by: Simon Horman <horms+renesas@verge.net.au>
> 
>    Analogous changes are needed for BOCK-W and Lager boards since
> they all use the same KSZ8041RNLI PHY chip. I was going to submit
> such changes with the PHY IRQ patches but since no IRQ is generated
> still, this got delayed...

I believe that similar changes are needed for almost all
of the shmobile boards. If you would like to handle some or all
of them that is quite fine by me.

Re: [PATCH] ARM: shmobile: koelsch: Conditionally select MICREL_PHY

[Simon Horman]

On Tue, Dec 17, 2013 at 02:31:35AM +0300, Sergei Shtylyov wrote:
> On 12/16/2013 04:25 PM, I wrote:
> 
> >>The koelsch board uses has an SH ethernet controller which uses a Micrel
> >>phy. Select MICREL_PHY for koelsch if SH_ETH is enabled to make use of the
> >>Micrel-specific phy driver rather than relying on the generic phy driver.
> 
> >    Frankly speaking, I'm seeing no advantages to doing that.
> 
>    Although wait, Geert has reported the absence of Rx FIFO overflow
> messages with this PHY driver, IIRC.
> 
> >>Signed-off-by: Simon Horman <horms+renesas@verge.net.au>
> 
> >    Analogous changes are needed for BOCK-W and Lager boards since they all
> >use the same KSZ8041RNLI PHY chip. I was going to submit such changes with the
> >PHY IRQ patches but since no IRQ is generated still, this got delayed...
> 
>    Only I was going to do it via defconfigs...

That was my initial approach too. However I have been convinced
that Kconfig is a better way forwards.

[PATCH] ARM: shmobile: koelsch: Conditionally select MICREL_PHY

[Simon Horman]

On Tue, Dec 17, 2013 at 02:31:35AM +0300, Sergei Shtylyov wrote:
> On 12/16/2013 04:25 PM, I wrote:
> 
> >>The koelsch board uses has an SH ethernet controller which uses a Micrel
> >>phy. Select MICREL_PHY for koelsch if SH_ETH is enabled to make use of the
> >>Micrel-specific phy driver rather than relying on the generic phy driver.
> 
> >    Frankly speaking, I'm seeing no advantages to doing that.
> 
>    Although wait, Geert has reported the absence of Rx FIFO overflow
> messages with this PHY driver, IIRC.
> 
> >>Signed-off-by: Simon Horman <horms+renesas@verge.net.au>
> 
> >    Analogous changes are needed for BOCK-W and Lager boards since they all
> >use the same KSZ8041RNLI PHY chip. I was going to submit such changes with the
> >PHY IRQ patches but since no IRQ is generated still, this got delayed...
> 
>    Only I was going to do it via defconfigs...

That was my initial approach too. However I have been convinced
that Kconfig is a better way forwards.

Re: [PATCH] ARM: shmobile: koelsch: Conditionally select MICREL_PHY

[Geert Uytterhoeven]

On Tue, Dec 17, 2013 at 1:31 AM, Simon Horman <horms@verge.net.au> wrote:
> On Tue, Dec 17, 2013 at 02:31:35AM +0300, Sergei Shtylyov wrote:
>> On 12/16/2013 04:25 PM, I wrote:
>> >>The koelsch board uses has an SH ethernet controller which uses a Micrel
>> >>phy. Select MICREL_PHY for koelsch if SH_ETH is enabled to make use of the
>> >>Micrel-specific phy driver rather than relying on the generic phy driver.
>>
>> >    Frankly speaking, I'm seeing no advantages to doing that.
>>
>>    Although wait, Geert has reported the absence of Rx FIFO overflow
>> messages with this PHY driver, IIRC.

I spoke too soon: apparently the statistical relevance of my reporting was
too low. I still see them from time to time, but I think less than before (no
numbers available).

>> >>Signed-off-by: Simon Horman <horms+renesas@verge.net.au>
>>
>> >    Analogous changes are needed for BOCK-W and Lager boards since they all
>> >use the same KSZ8041RNLI PHY chip. I was going to submit such changes with the
>> >PHY IRQ patches but since no IRQ is generated still, this got delayed...
>>
>>    Only I was going to do it via defconfigs...
>
> That was my initial approach too. However I have been convinced
> that Kconfig is a better way forwards.

Kconfig has the advantage that the minimal defconfigs will be smaller, so
less churn there.
However, it does mean you cannot disable the option anymore via your .config.

In this case it's a bit difficult to decide, as sh_eth will work both
with and without
the specific PHY-driver.

If no users want to disable it, I'd go for Kconfig.

Gr{oetje,eeting}s,

                        Geert

--
Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert@linux-m68k.org

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

[PATCH] ARM: shmobile: koelsch: Conditionally select MICREL_PHY

[Geert Uytterhoeven]

On Tue, Dec 17, 2013 at 1:31 AM, Simon Horman <horms@verge.net.au> wrote:
> On Tue, Dec 17, 2013 at 02:31:35AM +0300, Sergei Shtylyov wrote:
>> On 12/16/2013 04:25 PM, I wrote:
>> >>The koelsch board uses has an SH ethernet controller which uses a Micrel
>> >>phy. Select MICREL_PHY for koelsch if SH_ETH is enabled to make use of the
>> >>Micrel-specific phy driver rather than relying on the generic phy driver.
>>
>> >    Frankly speaking, I'm seeing no advantages to doing that.
>>
>>    Although wait, Geert has reported the absence of Rx FIFO overflow
>> messages with this PHY driver, IIRC.

I spoke too soon: apparently the statistical relevance of my reporting was
too low. I still see them from time to time, but I think less than before (no
numbers available).

>> >>Signed-off-by: Simon Horman <horms+renesas@verge.net.au>
>>
>> >    Analogous changes are needed for BOCK-W and Lager boards since they all
>> >use the same KSZ8041RNLI PHY chip. I was going to submit such changes with the
>> >PHY IRQ patches but since no IRQ is generated still, this got delayed...
>>
>>    Only I was going to do it via defconfigs...
>
> That was my initial approach too. However I have been convinced
> that Kconfig is a better way forwards.

Kconfig has the advantage that the minimal defconfigs will be smaller, so
less churn there.
However, it does mean you cannot disable the option anymore via your .config.

In this case it's a bit difficult to decide, as sh_eth will work both
with and without
the specific PHY-driver.

If no users want to disable it, I'd go for Kconfig.

Gr{oetje,eeting}s,

                        Geert

--
Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert at linux-m68k.org

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

Re: [PATCH] ARM: shmobile: koelsch: Conditionally select MICREL_PHY

[Simon Horman]

On Tue, Dec 17, 2013 at 11:10:15AM +0100, Geert Uytterhoeven wrote:
> On Tue, Dec 17, 2013 at 1:31 AM, Simon Horman <horms@verge.net.au> wrote:
> > On Tue, Dec 17, 2013 at 02:31:35AM +0300, Sergei Shtylyov wrote:
> >> On 12/16/2013 04:25 PM, I wrote:
> >> >>The koelsch board uses has an SH ethernet controller which uses a Micrel
> >> >>phy. Select MICREL_PHY for koelsch if SH_ETH is enabled to make use of the
> >> >>Micrel-specific phy driver rather than relying on the generic phy driver.
> >>
> >> >    Frankly speaking, I'm seeing no advantages to doing that.
> >>
> >>    Although wait, Geert has reported the absence of Rx FIFO overflow
> >> messages with this PHY driver, IIRC.
> 
> I spoke too soon: apparently the statistical relevance of my reporting was
> too low. I still see them from time to time, but I think less than before (no
> numbers available).
> 
> >> >>Signed-off-by: Simon Horman <horms+renesas@verge.net.au>
> >>
> >> >    Analogous changes are needed for BOCK-W and Lager boards since they all
> >> >use the same KSZ8041RNLI PHY chip. I was going to submit such changes with the
> >> >PHY IRQ patches but since no IRQ is generated still, this got delayed...
> >>
> >>    Only I was going to do it via defconfigs...
> >
> > That was my initial approach too. However I have been convinced
> > that Kconfig is a better way forwards.
> 
> Kconfig has the advantage that the minimal defconfigs will be smaller, so
> less churn there.
> However, it does mean you cannot disable the option anymore via your .config.
> 
> In this case it's a bit difficult to decide, as sh_eth will work both
> with and without
> the specific PHY-driver.
> 
> If no users want to disable it, I'd go for Kconfig.

It is indeed a tricky choice. But I think we should try going for Kconfig.

[PATCH] ARM: shmobile: koelsch: Conditionally select MICREL_PHY

[Simon Horman]

On Tue, Dec 17, 2013 at 11:10:15AM +0100, Geert Uytterhoeven wrote:
> On Tue, Dec 17, 2013 at 1:31 AM, Simon Horman <horms@verge.net.au> wrote:
> > On Tue, Dec 17, 2013 at 02:31:35AM +0300, Sergei Shtylyov wrote:
> >> On 12/16/2013 04:25 PM, I wrote:
> >> >>The koelsch board uses has an SH ethernet controller which uses a Micrel
> >> >>phy. Select MICREL_PHY for koelsch if SH_ETH is enabled to make use of the
> >> >>Micrel-specific phy driver rather than relying on the generic phy driver.
> >>
> >> >    Frankly speaking, I'm seeing no advantages to doing that.
> >>
> >>    Although wait, Geert has reported the absence of Rx FIFO overflow
> >> messages with this PHY driver, IIRC.
> 
> I spoke too soon: apparently the statistical relevance of my reporting was
> too low. I still see them from time to time, but I think less than before (no
> numbers available).
> 
> >> >>Signed-off-by: Simon Horman <horms+renesas@verge.net.au>
> >>
> >> >    Analogous changes are needed for BOCK-W and Lager boards since they all
> >> >use the same KSZ8041RNLI PHY chip. I was going to submit such changes with the
> >> >PHY IRQ patches but since no IRQ is generated still, this got delayed...
> >>
> >>    Only I was going to do it via defconfigs...
> >
> > That was my initial approach too. However I have been convinced
> > that Kconfig is a better way forwards.
> 
> Kconfig has the advantage that the minimal defconfigs will be smaller, so
> less churn there.
> However, it does mean you cannot disable the option anymore via your .config.
> 
> In this case it's a bit difficult to decide, as sh_eth will work both
> with and without
> the specific PHY-driver.
> 
> If no users want to disable it, I'd go for Kconfig.

It is indeed a tricky choice. But I think we should try going for Kconfig.

Re: [PATCH] ARM: shmobile: koelsch: Conditionally select MICREL_PHY

[Simon Horman]

On Wed, Dec 18, 2013 at 09:53:26AM +0900, Simon Horman wrote:
> On Tue, Dec 17, 2013 at 11:10:15AM +0100, Geert Uytterhoeven wrote:
> > On Tue, Dec 17, 2013 at 1:31 AM, Simon Horman <horms@verge.net.au> wrote:
> > > On Tue, Dec 17, 2013 at 02:31:35AM +0300, Sergei Shtylyov wrote:
> > >> On 12/16/2013 04:25 PM, I wrote:
> > >> >>The koelsch board uses has an SH ethernet controller which uses a Micrel
> > >> >>phy. Select MICREL_PHY for koelsch if SH_ETH is enabled to make use of the
> > >> >>Micrel-specific phy driver rather than relying on the generic phy driver.
> > >>
> > >> >    Frankly speaking, I'm seeing no advantages to doing that.
> > >>
> > >>    Although wait, Geert has reported the absence of Rx FIFO overflow
> > >> messages with this PHY driver, IIRC.
> > 
> > I spoke too soon: apparently the statistical relevance of my reporting was
> > too low. I still see them from time to time, but I think less than before (no
> > numbers available).
> > 
> > >> >>Signed-off-by: Simon Horman <horms+renesas@verge.net.au>
> > >>
> > >> >    Analogous changes are needed for BOCK-W and Lager boards since they all
> > >> >use the same KSZ8041RNLI PHY chip. I was going to submit such changes with the
> > >> >PHY IRQ patches but since no IRQ is generated still, this got delayed...
> > >>
> > >>    Only I was going to do it via defconfigs...
> > >
> > > That was my initial approach too. However I have been convinced
> > > that Kconfig is a better way forwards.
> > 
> > Kconfig has the advantage that the minimal defconfigs will be smaller, so
> > less churn there.
> > However, it does mean you cannot disable the option anymore via your .config.
> > 
> > In this case it's a bit difficult to decide, as sh_eth will work both
> > with and without
> > the specific PHY-driver.
> > 
> > If no users want to disable it, I'd go for Kconfig.
> 
> It is indeed a tricky choice. But I think we should try going for Kconfig.

I have decided to queue this up.

[PATCH] ARM: shmobile: koelsch: Conditionally select MICREL_PHY

[Simon Horman]

On Wed, Dec 18, 2013 at 09:53:26AM +0900, Simon Horman wrote:
> On Tue, Dec 17, 2013 at 11:10:15AM +0100, Geert Uytterhoeven wrote:
> > On Tue, Dec 17, 2013 at 1:31 AM, Simon Horman <horms@verge.net.au> wrote:
> > > On Tue, Dec 17, 2013 at 02:31:35AM +0300, Sergei Shtylyov wrote:
> > >> On 12/16/2013 04:25 PM, I wrote:
> > >> >>The koelsch board uses has an SH ethernet controller which uses a Micrel
> > >> >>phy. Select MICREL_PHY for koelsch if SH_ETH is enabled to make use of the
> > >> >>Micrel-specific phy driver rather than relying on the generic phy driver.
> > >>
> > >> >    Frankly speaking, I'm seeing no advantages to doing that.
> > >>
> > >>    Although wait, Geert has reported the absence of Rx FIFO overflow
> > >> messages with this PHY driver, IIRC.
> > 
> > I spoke too soon: apparently the statistical relevance of my reporting was
> > too low. I still see them from time to time, but I think less than before (no
> > numbers available).
> > 
> > >> >>Signed-off-by: Simon Horman <horms+renesas@verge.net.au>
> > >>
> > >> >    Analogous changes are needed for BOCK-W and Lager boards since they all
> > >> >use the same KSZ8041RNLI PHY chip. I was going to submit such changes with the
> > >> >PHY IRQ patches but since no IRQ is generated still, this got delayed...
> > >>
> > >>    Only I was going to do it via defconfigs...
> > >
> > > That was my initial approach too. However I have been convinced
> > > that Kconfig is a better way forwards.
> > 
> > Kconfig has the advantage that the minimal defconfigs will be smaller, so
> > less churn there.
> > However, it does mean you cannot disable the option anymore via your .config.
> > 
> > In this case it's a bit difficult to decide, as sh_eth will work both
> > with and without
> > the specific PHY-driver.
> > 
> > If no users want to disable it, I'd go for Kconfig.
> 
> It is indeed a tricky choice. But I think we should try going for Kconfig.

I have decided to queue this up.

Re: [PATCH] ARM: shmobile: koelsch: Conditionally select MICREL_PHY

[Sergei Shtylyov]

Hello.

On 12/16/2013 03:31 AM, Simon Horman wrote:

> The koelsch board uses has an SH ethernet controller which uses a Micrel
> phy. Select MICREL_PHY for koelsch if SH_ETH is enabled to make use of the
> Micrel-specific phy driver rather than relying on the generic phy driver.

> Signed-off-by: Simon Horman <horms+renesas@verge.net.au>
> ---
>   arch/arm/mach-shmobile/Kconfig | 3 +++
>   1 file changed, 3 insertions(+)

> diff --git a/arch/arm/mach-shmobile/Kconfig b/arch/arm/mach-shmobile/Kconfig
> index f01e878..670238a 100644
> --- a/arch/arm/mach-shmobile/Kconfig
> +++ b/arch/arm/mach-shmobile/Kconfig
> @@ -41,6 +41,7 @@ config MACH_GENMAI
>   config MACH_KOELSCH
>   	bool "Koelsch board"
>   	depends on ARCH_R8A7791
> +	select MICREL_PHY if SH_ETH
>
>   config MACH_KZM9D
>   	bool "KZM9D board"

    Simon, did you initially remove that hunk when committing?

WBR, Sergei

[PATCH] ARM: shmobile: koelsch: Conditionally select MICREL_PHY

[Sergei Shtylyov]

Hello.

On 12/16/2013 03:31 AM, Simon Horman wrote:

> The koelsch board uses has an SH ethernet controller which uses a Micrel
> phy. Select MICREL_PHY for koelsch if SH_ETH is enabled to make use of the
> Micrel-specific phy driver rather than relying on the generic phy driver.

> Signed-off-by: Simon Horman <horms+renesas@verge.net.au>
> ---
>   arch/arm/mach-shmobile/Kconfig | 3 +++
>   1 file changed, 3 insertions(+)

> diff --git a/arch/arm/mach-shmobile/Kconfig b/arch/arm/mach-shmobile/Kconfig
> index f01e878..670238a 100644
> --- a/arch/arm/mach-shmobile/Kconfig
> +++ b/arch/arm/mach-shmobile/Kconfig
> @@ -41,6 +41,7 @@ config MACH_GENMAI
>   config MACH_KOELSCH
>   	bool "Koelsch board"
>   	depends on ARCH_R8A7791
> +	select MICREL_PHY if SH_ETH
>
>   config MACH_KZM9D
>   	bool "KZM9D board"

    Simon, did you initially remove that hunk when committing?

WBR, Sergei

Re: [PATCH] ARM: shmobile: koelsch: Conditionally select MICREL_PHY

[Simon Horman]

On Fri, Dec 20, 2013 at 02:05:16AM +0300, Sergei Shtylyov wrote:
> Hello.
> 
> On 12/16/2013 03:31 AM, Simon Horman wrote:
> 
> >The koelsch board uses has an SH ethernet controller which uses a Micrel
> >phy. Select MICREL_PHY for koelsch if SH_ETH is enabled to make use of the
> >Micrel-specific phy driver rather than relying on the generic phy driver.
> 
> >Signed-off-by: Simon Horman <horms+renesas@verge.net.au>
> >---
> >  arch/arm/mach-shmobile/Kconfig | 3 +++
> >  1 file changed, 3 insertions(+)
> 
> >diff --git a/arch/arm/mach-shmobile/Kconfig b/arch/arm/mach-shmobile/Kconfig
> >index f01e878..670238a 100644
> >--- a/arch/arm/mach-shmobile/Kconfig
> >+++ b/arch/arm/mach-shmobile/Kconfig
> >@@ -41,6 +41,7 @@ config MACH_GENMAI
> >  config MACH_KOELSCH
> >  	bool "Koelsch board"
> >  	depends on ARCH_R8A7791
> >+	select MICREL_PHY if SH_ETH
> >
> >  config MACH_KZM9D
> >  	bool "KZM9D board"
> 
>    Simon, did you initially remove that hunk when committing?

Thanks for noticing.

I will queue up the following:

From: Simon Horman <horms+renesas@verge.net.au>

ARM: shmobile: koelsch: Conditionally select MICREL_PHY for Multiplatform

The koelsch board uses has an SH ethernet controller which uses a Micrel
phy. Select MICREL_PHY for koelsch if SH_ETH is enabled to make use of the
Micrel-specific phy driver rather than relying on the generic phy driver.

Signed-off-by: Simon Horman <horms+renesas@verge.net.au>
---
 arch/arm/mach-shmobile/Kconfig | 1 +
 1 file changed, 1 insertion(+)

diff --git a/arch/arm/mach-shmobile/Kconfig b/arch/arm/mach-shmobile/Kconfig
index a127252..958ccf3d 100644
--- a/arch/arm/mach-shmobile/Kconfig
+++ b/arch/arm/mach-shmobile/Kconfig
@@ -45,6 +45,7 @@ config MACH_GENMAI
 config MACH_KOELSCH
 	bool "Koelsch board"
 	depends on ARCH_R8A7791
+	select MICREL_PHY if SH_ETH
 
 config MACH_KZM9D
 	bool "KZM9D board"
-- 
1.8.4

[PATCH] ARM: shmobile: koelsch: Conditionally select MICREL_PHY

[Simon Horman]

On Fri, Dec 20, 2013 at 02:05:16AM +0300, Sergei Shtylyov wrote:
> Hello.
> 
> On 12/16/2013 03:31 AM, Simon Horman wrote:
> 
> >The koelsch board uses has an SH ethernet controller which uses a Micrel
> >phy. Select MICREL_PHY for koelsch if SH_ETH is enabled to make use of the
> >Micrel-specific phy driver rather than relying on the generic phy driver.
> 
> >Signed-off-by: Simon Horman <horms+renesas@verge.net.au>
> >---
> >  arch/arm/mach-shmobile/Kconfig | 3 +++
> >  1 file changed, 3 insertions(+)
> 
> >diff --git a/arch/arm/mach-shmobile/Kconfig b/arch/arm/mach-shmobile/Kconfig
> >index f01e878..670238a 100644
> >--- a/arch/arm/mach-shmobile/Kconfig
> >+++ b/arch/arm/mach-shmobile/Kconfig
> >@@ -41,6 +41,7 @@ config MACH_GENMAI
> >  config MACH_KOELSCH
> >  	bool "Koelsch board"
> >  	depends on ARCH_R8A7791
> >+	select MICREL_PHY if SH_ETH
> >
> >  config MACH_KZM9D
> >  	bool "KZM9D board"
> 
>    Simon, did you initially remove that hunk when committing?

Thanks for noticing.

I will queue up the following:

From: Simon Horman <horms+renesas@verge.net.au>

ARM: shmobile: koelsch: Conditionally select MICREL_PHY for Multiplatform

The koelsch board uses has an SH ethernet controller which uses a Micrel
phy. Select MICREL_PHY for koelsch if SH_ETH is enabled to make use of the
Micrel-specific phy driver rather than relying on the generic phy driver.

Signed-off-by: Simon Horman <horms+renesas@verge.net.au>
---
 arch/arm/mach-shmobile/Kconfig | 1 +
 1 file changed, 1 insertion(+)

diff --git a/arch/arm/mach-shmobile/Kconfig b/arch/arm/mach-shmobile/Kconfig
index a127252..958ccf3d 100644
--- a/arch/arm/mach-shmobile/Kconfig
+++ b/arch/arm/mach-shmobile/Kconfig
@@ -45,6 +45,7 @@ config MACH_GENMAI
 config MACH_KOELSCH
 	bool "Koelsch board"
 	depends on ARCH_R8A7791
+	select MICREL_PHY if SH_ETH
 
 config MACH_KZM9D
 	bool "KZM9D board"
-- 
1.8.4