[PATCH 0/9] sh_eth: make driver build on any platform

[PATCH 0/9] sh_eth: make driver build on any platform

[Sergei Shtylyov]

On 06/08/2013 04:10 AM, Sergei Shtylyov wrote:
> Hello.
>
> On 06/08/2013 03:51 AM, Sergei Shtylyov wrote:
>
>>     This series of 9 patches are against Dave's 'net-next.git' 
>> repository.
>>     The series deals with the #ifdef'fery around SoC support 
>> code/data in the
>> 'sh_eth' driver. It doesn't yet get rid of all the #ifdef's: 3 are 
>> left, 1 of
>> them still hinders ARM multiplatform build -- I'll try to deal with 
>> it next
>> week.
>>
>> [1/9] sh_eth: create initial ID table
>> [2/9] sh_eth: get SH771x support out of #ifdef
>> [3/9] sh_eth: get SH7619 support out of #ifdef
>> [4/9] sh_eth: get R8A7740 support out of #ifdef
>> [5/9] sh_eth: get SH77{34|63} support out of #ifdef
>> [6/9] sh_eth: get SH7757 support out of #ifdef
>> [7/9] sh_eth: get SH7724 support out of #ifdef
>> [8/9] sh_eth: get R8A777x support out of #ifdef
>> [9/9] sh_eth: remove dependencies from Kconfig
>>
>
>    Sorry, Paul, forgot to include you into the CC list on most of the 
> patches
> and the cover letter. Please look for the patches on linux-sh and ACK 
> them
> if you deem necessary.

    Also, forgot to CC Simon, Magnus, Russell and LAKML on some patches.
Too much haste, forgot to run scripts/get_maintainer.pl before sending 
out patches...

WBR, Sergei


>
> WBR, Serge
>

[PATCH 0/9] sh_eth: make driver build on any platform

[David Miller]

From: Sergei Shtylyov <sergei.shtylyov@cogentembedded.com>
Date: Sat, 08 Jun 2013 04:15:35 +0400

> On 06/08/2013 04:10 AM, Sergei Shtylyov wrote:
>> Hello.
>>
>> On 06/08/2013 03:51 AM, Sergei Shtylyov wrote:
>>
>>>     This series of 9 patches are against Dave's 'net-next.git' repository.
>>>     The series deals with the #ifdef'fery around SoC support code/data in
>>>     the
>>> 'sh_eth' driver. It doesn't yet get rid of all the #ifdef's: 3 are
>>> left, 1 of
>>> them still hinders ARM multiplatform build -- I'll try to deal with it
>>> next
>>> week.
>>>
>>> [1/9] sh_eth: create initial ID table
>>> [2/9] sh_eth: get SH771x support out of #ifdef
>>> [3/9] sh_eth: get SH7619 support out of #ifdef
>>> [4/9] sh_eth: get R8A7740 support out of #ifdef
>>> [5/9] sh_eth: get SH77{34|63} support out of #ifdef
>>> [6/9] sh_eth: get SH7757 support out of #ifdef
>>> [7/9] sh_eth: get SH7724 support out of #ifdef
>>> [8/9] sh_eth: get R8A777x support out of #ifdef
>>> [9/9] sh_eth: remove dependencies from Kconfig
>>>
>>
>>    Sorry, Paul, forgot to include you into the CC list on most of the
>>    patches
>> and the cover letter. Please look for the patches on linux-sh and ACK
>> them
>> if you deem necessary.
> 
>    Also, forgot to CC Simon, Magnus, Russell and LAKML on some patches.
> Too much haste, forgot to run scripts/get_maintainer.pl before sending
> out patches...

I'll apply this solely because it finally allows me to build test this
driver on x86-64.

:-)

[PATCH 0/9] sh_eth: make driver build on any platform

[Sergei Shtylyov]

Hello.

On 08-06-2013 10:39, David Miller wrote:

>>>>      This series of 9 patches are against Dave's 'net-next.git' repository.
>>>>      The series deals with the #ifdef'fery around SoC support code/data in
>>>>      the
>>>> 'sh_eth' driver. It doesn't yet get rid of all the #ifdef's: 3 are
>>>> left, 1 of
>>>> them still hinders ARM multiplatform build -- I'll try to deal with it
>>>> next
>>>> week.

>>>> [1/9] sh_eth: create initial ID table
>>>> [2/9] sh_eth: get SH771x support out of #ifdef
>>>> [3/9] sh_eth: get SH7619 support out of #ifdef
>>>> [4/9] sh_eth: get R8A7740 support out of #ifdef
>>>> [5/9] sh_eth: get SH77{34|63} support out of #ifdef
>>>> [6/9] sh_eth: get SH7757 support out of #ifdef
>>>> [7/9] sh_eth: get SH7724 support out of #ifdef
>>>> [8/9] sh_eth: get R8A777x support out of #ifdef
>>>> [9/9] sh_eth: remove dependencies from Kconfig

>>>     Sorry, Paul, forgot to include you into the CC list on most of the
>>>     patches
>>> and the cover letter. Please look for the patches on linux-sh and ACK
>>> them
>>> if you deem necessary.

>>     Also, forgot to CC Simon, Magnus, Russell and LAKML on some patches.
>> Too much haste, forgot to run scripts/get_maintainer.pl before sending
>> out patches...

> I'll apply this solely because it finally allows me to build test this
> driver on x86-64.

    Hold on please. It just occured to me that the series won't be 
bisectable on anything but R-Car CPUs (I checked it only on R8A7779) 
since I forgot that 'sh_eth_my_cpu_data' variable should be always 
available in any configuuration until the last patch (so the patch #1 is 
somewhat incomplete). I'll respin the series today and resend.

WBR, Sergei

[PATCH 0/9] sh_eth: make driver build on any platform

[Sergei Shtylyov]

On 08-06-2013 17:11, Sergei Shtylyov wrote:

>>>>>      This series of 9 patches are against Dave's 'net-next.git'
>>>>> repository.
>>>>>      The series deals with the #ifdef'fery around SoC support
>>>>> code/data in
>>>>>      the
>>>>> 'sh_eth' driver. It doesn't yet get rid of all the #ifdef's: 3 are
>>>>> left, 1 of
>>>>> them still hinders ARM multiplatform build -- I'll try to deal with it
>>>>> next
>>>>> week.

>>>>> [1/9] sh_eth: create initial ID table
>>>>> [2/9] sh_eth: get SH771x support out of #ifdef
>>>>> [3/9] sh_eth: get SH7619 support out of #ifdef
>>>>> [4/9] sh_eth: get R8A7740 support out of #ifdef
>>>>> [5/9] sh_eth: get SH77{34|63} support out of #ifdef
>>>>> [6/9] sh_eth: get SH7757 support out of #ifdef
>>>>> [7/9] sh_eth: get SH7724 support out of #ifdef
>>>>> [8/9] sh_eth: get R8A777x support out of #ifdef
>>>>> [9/9] sh_eth: remove dependencies from Kconfig

>>>>     Sorry, Paul, forgot to include you into the CC list on most of the
>>>>     patches
>>>> and the cover letter. Please look for the patches on linux-sh and ACK
>>>> them
>>>> if you deem necessary.

>>>     Also, forgot to CC Simon, Magnus, Russell and LAKML on some patches.
>>> Too much haste, forgot to run scripts/get_maintainer.pl before sending
>>> out patches...

>> I'll apply this solely because it finally allows me to build test this
>> driver on x86-64.

>     Hold on please. It just occured to me that the series won't be
> bisectable on anything but R-Car CPUs (I checked it only on R8A7779)
> since I forgot that 'sh_eth_my_cpu_data' variable should be always
> available in any configuuration until the last patch (so the patch #1 is
> somewhat incomplete). I'll respin the series today and resend.

    Oh, you've already applied it! Well, at least the result is 
buildable on all arches. :-)
    You're sometimes too quick though. :-)

WBR, Sergei

[PATCH 0/9] sh_eth: make driver build on any platform

[David Miller]

From: Sergei Shtylyov <sergei.shtylyov@cogentembedded.com>
Date: Sat, 08 Jun 2013 17:11:18 +0400

>    Hold on please. It just occured to me that the series won't be
>    bisectable on anything but R-Car CPUs (I checked it only on R8A7779)
>    since I forgot that 'sh_eth_my_cpu_data' variable should be always
>    available in any configuuration until the last patch (so the patch #1
>    is somewhat incomplete). I'll respin the series today and resend.

Too late, it's already pushed to my tree and at that point it's permanent.