Re: [PATCH] ARM: S5PV210: Fix build breakage due to renaming of S3C_VA_USB_HSPHY

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On 8 June 2011 19:43, Kyungmin Park <kmpark@xxxxxxxxxxxxx> wrote:
> On Wed, Jun 8, 2011 at 10:53 PM, Marek Szyprowski
> <m.szyprowski@xxxxxxxxxxx> 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,

Hi Mr. Park,

>
> 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?

USB Phy region static mapping could be moved to driver but how do we
have handle SROMC? There is no particular driver as such for SROMC.
There are others such as COREPERI_BASE and DMC that do not have any
driver.

And most of the platform data and code are reusable across SoC since
we use common macros. For example, arch/arm/plat-samsung/dev-hsmmc.c.
If we use physical address in such cases, we will have to duplicate
it.

As long as we can reuse s3c prefix for s5p plaforms, probably we
should continue using them. Any new additions for S5P platforms should
use the s5p prefix. Or we could arrive at a common consensus and stick
to it. I prefer not to rename s3c prefixes which are reusable for s5p
platforms.

Thanks,
Thomas.

>
> Thank you,
> Kyungmin Park
>
>>>
>>> Signed-off-by: Thomas Abraham <thomas.ab@xxxxxxxxxxx>
>>> ---
>>>  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@xxxxxxxxxxxxxxx
>> More majordomo info at  http://vger.kernel.org/majordomo-info.html
>>
> --
> To unsubscribe from this list: send the line "unsubscribe linux-samsung-soc" in
> the body of a message to majordomo@xxxxxxxxxxxxxxx
> More majordomo info at  http://vger.kernel.org/majordomo-info.html
>
--
To unsubscribe from this list: send the line "unsubscribe linux-samsung-soc" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html


[Index of Archives]     [Linux SoC Development]     [Linux Rockchip Development]     [Linux USB Development]     [Video for Linux]     [Linux Audio Users]     [Linux SCSI]     [Yosemite News]

  Powered by Linux