[meta-freescale] [linux-fslc-imx][4.1-2.0.x-imx][PATCH] imx: fix typo and simple assumptions in imx6sll support - Message was found to be spam: (100%) URL is in DNSBL,

[Otavio Salvador]

On Mon, Jun 19, 2017 at 1:01 PM, Alejandro Mery
<amery at hanoverdisplays.com> wrote:
> On 19/06/17 15:41, Otavio Salvador wrote:
>> On Mon, Jun 19, 2017 at 7:32 AM, Alejandro Mery
>> <amery at hanoverdisplays.com> wrote:
>>> On 19/06/17 11:18, Jacky Bai wrote:
>>>> Hi Alejandro,
>>>>
>>>> Can you tell me the steps for reproducing this build error? Thanks.
>>> http://sprunge.us/ReRQ is the defconfig I'm including via a
>>> linux-fslc-imx_4.1-2.0.x.bbappend pointing to git via EXTERNALSRC
>>>
>>> it was working fine by Otavio's "Merge tag 'v4.1.39' into 4.1-2.0.x-imx"
>>> but broke as described after v4.1.41's
>>>
>>>> arch/arm/mach-imx/built-in.o: In function `init_mmdc_lpddr2_settings':
>>>> platform-imx-dma.c:(.text+0x9938): undefined reference to
>>>> `imx6sll_lpddr2_freq_change'
>>>> platform-imx-dma.c:(.text+0x993c): undefined reference to
>>>> `imx6sll_lpddr2_freq_change'
>>>> arch/arm/mach-imx/built-in.o: In function `imx6sl_init_late':
>>>> platform-imx-dma.c:(.init.text+0xdef0): undefined reference to
>>>> `imx6sll_cpuidle_init'
>>> http://sprunge.us/fiMU is the .config that gets made during the build of
>>> the package
>>>
>>> I believe it's simply because CONFIG_SOC_IMX6SLL is not defined by my
>>> defconfig
>> Indeed; it does seem to be the case. You fix seem to be right as we
>> shouldn't be forced to enable it.
>>
>> Can you open a pull request with the fix?
>
>
> github didn't let me fork freescale/linux-fslc because I already have
> one fork with ancestry going back to torvalds/linux
> and then it didn't allow me to initiate a pull request from a non-fork
> but the url is there
>
>
>  The following changes since commit
> 5c7513c4866175f5057fbe1d039e53e00a2dba53:
>
>   Merge tag 'v4.1.41' into 4.1-2.0.x-imx (2017-06-17 10:40:10 -0300)
>
> are available in the git repository at:
>
>   https://github.com/amery/linux-fslc 4.1-2.0.x-imx-for-otavio
>
> for you to fetch changes up to 25a1fee6bf7f1dff303b2d840c9a5634be75b260:
>
>   imx: fix errors when CONFIG_SOC_IMX6SLL is not defined (2017-06-19
> 16:18:37 +0100)
>
> ----------------------------------------------------------------
> Alejandro Mery (1):
>       imx: fix errors when CONFIG_SOC_IMX6SLL is not defined
>
>  arch/arm/mach-imx/common.c  |  2 +-
>  arch/arm/mach-imx/cpuidle.h | 14 +++++++++-----
>  2 files changed, 10 insertions(+), 6 deletions(-)

Merged it :-)


-- 
Otavio Salvador                             O.S. Systems
http://www.ossystems.com.br        http://code.ossystems.com.br
Mobile: +55 (53) 9981-7854            Mobile: +1 (347) 903-9750