[PATCH] ARM: S5PV210: Fix build breakage due to renaming of S3C_VA_USB_HSPHY
Kyungmin Park
kmpark at infradead.org
Wed Jun 8 10:13:52 EDT 2011
On Wed, Jun 8, 2011 at 10:53 PM, Marek Szyprowski
<m.szyprowski at samsung.com> wrote:
> Hello,
>
> There is already the patch that fixes this issue available on
> kgene/s5p_fixes_for_linus branch. Please check commit 08115a139 from
> git://git.kernel.org/pub/scm/linux/kernel/git/kgene/linux-samsung.git
>
> On Wednesday, June 08, 2011 1:26 PM Thomas Abraham wrote:
>
>> Commit 8f1d169f999fea892c3fcbf5a79ae8525a477572
>> ("ARM: EXYNOS4: Add usb host phy control") renamed S3C_VA_USB_HSPHY
>> to S5P_VA_USB_HSPHY in s5p-map.h file. Fix build for S5PV210 platform.
Hi Thomas & Marek,
We lived for long time with mismatch prefix.
So how about to clean up the mismatch prefix, S3C_* and S5P_* at this time?
One method is that it just passes the physical address and driver
should ioremap at driver instead of static mapping.
How do you think?
Thank you,
Kyungmin Park
>>
>> Signed-off-by: Thomas Abraham <thomas.ab at samsung.com>
>> ---
>> arch/arm/mach-s5pv210/cpu.c | 2 +-
>> 1 files changed, 1 insertions(+), 1 deletions(-)
>>
>> diff --git a/arch/arm/mach-s5pv210/cpu.c b/arch/arm/mach-s5pv210/cpu.c
>> index 61e6c24..b343582 100644
>> --- a/arch/arm/mach-s5pv210/cpu.c
>> +++ b/arch/arm/mach-s5pv210/cpu.c
>> @@ -91,7 +91,7 @@ static struct map_desc s5pv210_iodesc[] __initdata = {
>> .length = SZ_4K,
>> .type = MT_DEVICE,
>> }, {
>> - .virtual = (unsigned long)S3C_VA_USB_HSPHY,
>> + .virtual = (unsigned long)S5P_VA_USB_HSPHY,
>> .pfn =__phys_to_pfn(S5PV210_PA_HSPHY),
>> .length = SZ_4K,
>> .type = MT_DEVICE,
>> --
>> 1.6.6.rc2
>
> S3C_VA_USB_HSPHY is the correct name, it is also used on s3c64xx and
> s3c-hsotg driver (see drivers/usb/gadget).
>
> Best regards
> --
> Marek Szyprowski
> Samsung Poland R&D Center
>
>
> --
> To unsubscribe from this list: send the line "unsubscribe linux-samsung-soc" in
> the body of a message to majordomo at vger.kernel.org
> More majordomo info at http://vger.kernel.org/majordomo-info.html
>
More information about the linux-arm-kernel
mailing list