2013-06-03 10:13:28

by Michal Simek

[permalink] [raw]
Subject: [PATCH v4 0/7] xilinxfb changes


Attachments:
(No filename) (1.39 kB)
(No filename) (198.00 B)
Download all attachments
Subject: Re: [PATCH v4 0/7] xilinxfb changes

On 12:13 Mon 03 Jun , Michal Simek wrote:
> Hi,
>

Arnd can you take look on it again please

I'll take a look on it next week

Best Regards,
J.
> I have done more changes in the driver to support probing
> on little and big endian system where detection is done
> directly on the hardware.
> I have also done some cleanups to get it to the better shape.
>
> Thanks for your review,
> Michal
>
> Changes in v4:
> - Acked by Arnd
> - Remove "video: xilinxfb: Fix sparse warnings"
> patch because it is trying to fix incorrect API
> usage and sparse should warn about it.
>
> Changes in v3:
> - fix commit message
> - Remove out_be IO name from function name
> - Change patch subject from "Do not use out_be32 IO function"
> to "Do not name out_be32 in function name"
> - New patch in this patchset based on discussions
> - New patch in this patchset based on discussions
> - New patch in this patchset
> - New patch in this patchset based on discussions
>
> Changes in v2:
> - use of_property_read_u32 helper function
>
> Michal Simek (7):
> video: xilinxfb: Fix OF probing on little-endian systems
> video: xilinxfb: Do not name out_be32 in function name
> video: xilinxfb: Rename PLB_ACCESS_FLAG to BUS_ACCESS_FLAG
> video: xilinxfb: Use drvdata->regs_phys instead of physaddr
> video: xilinxfb: Group bus initialization
> video: xilinxfb: Add support for little endian accesses
> video: xilinxfb: Use driver for Xilinx ARM Zynq
>
> drivers/video/Kconfig | 2 +-
> drivers/video/xilinxfb.c | 135 +++++++++++++++++++++++------------------------
> 2 files changed, 68 insertions(+), 69 deletions(-)
>
> --
> 1.8.2.3
>

2013-06-17 05:24:05

by Michal Simek

[permalink] [raw]
Subject: Re: [PATCH v4 0/7] xilinxfb changes

On 06/06/2013 06:23 PM, Jean-Christophe PLAGNIOL-VILLARD wrote:
> On 12:13 Mon 03 Jun , Michal Simek wrote:
>> Hi,
>>
>
> Arnd can you take look on it again please
>
> I'll take a look on it next week

Any update on this?

Thanks,
Michal

--
Michal Simek, Ing. (M.Eng), OpenPGP -> KeyID: FE3D1F91
w: http://www.monstr.eu p: +42-0-721842854
Maintainer of Linux kernel - Microblaze cpu - http://www.monstr.eu/fdt/
Maintainer of Linux kernel - Xilinx Zynq ARM architecture
Microblaze U-BOOT custodian and responsible for u-boot arm zynq platform



Attachments:
signature.asc (263.00 B)
OpenPGP digital signature

2013-06-17 09:02:16

by Michal Simek

[permalink] [raw]
Subject: Re: [PATCH v4 0/7] xilinxfb changes

On 06/17/2013 10:56 AM, Jean-Christophe PLAGNIOL-VILLARD wrote:
> On 07:23 Mon 17 Jun , Michal Simek wrote:
>> On 06/06/2013 06:23 PM, Jean-Christophe PLAGNIOL-VILLARD wrote:
>>> On 12:13 Mon 03 Jun , Michal Simek wrote:
>>>> Hi,
>>>>
>>>
>>> Arnd can you take look on it again please
>>>
>>> I'll take a look on it next week
>>
>> Any update on this?
> look ok but I want the Ack from Arnd

ok.
Arnd?

Thanks,
Michal



--
Michal Simek, Ing. (M.Eng), OpenPGP -> KeyID: FE3D1F91
w: http://www.monstr.eu p: +42-0-721842854
Maintainer of Linux kernel - Microblaze cpu - http://www.monstr.eu/fdt/
Maintainer of Linux kernel - Xilinx Zynq ARM architecture
Microblaze U-BOOT custodian and responsible for u-boot arm zynq platform



Attachments:
signature.asc (263.00 B)
OpenPGP digital signature
Subject: Re: [PATCH v4 0/7] xilinxfb changes

On 07:23 Mon 17 Jun , Michal Simek wrote:
> On 06/06/2013 06:23 PM, Jean-Christophe PLAGNIOL-VILLARD wrote:
> > On 12:13 Mon 03 Jun , Michal Simek wrote:
> >> Hi,
> >>
> >
> > Arnd can you take look on it again please
> >
> > I'll take a look on it next week
>
> Any update on this?
look ok but I want the Ack from Arnd

Best Regards,
J.
>
> Thanks,
> Michal
>
> --
> Michal Simek, Ing. (M.Eng), OpenPGP -> KeyID: FE3D1F91
> w: http://www.monstr.eu p: +42-0-721842854
> Maintainer of Linux kernel - Microblaze cpu - http://www.monstr.eu/fdt/
> Maintainer of Linux kernel - Xilinx Zynq ARM architecture
> Microblaze U-BOOT custodian and responsible for u-boot arm zynq platform
>
>

2013-06-17 21:10:24

by Arnd Bergmann

[permalink] [raw]
Subject: Re: [PATCH v4 0/7] xilinxfb changes

On Monday 17 June 2013, Michal Simek wrote:
> On 06/17/2013 10:56 AM, Jean-Christophe PLAGNIOL-VILLARD wrote:
> > On 07:23 Mon 17 Jun , Michal Simek wrote:
> >> On 06/06/2013 06:23 PM, Jean-Christophe PLAGNIOL-VILLARD wrote:
> >>> On 12:13 Mon 03 Jun , Michal Simek wrote:
> >>>> Hi,
> >>>>
> >>>
> >>> Arnd can you take look on it again please
> >>>
> >>> I'll take a look on it next week
> >>
> >> Any update on this?
> > look ok but I want the Ack from Arnd

Sorry for the delay, everything looks good to me.

Acked-by: Arnd Bergmann <[email protected]>

2013-06-26 10:02:25

by Michal Simek

[permalink] [raw]
Subject: Re: [PATCH v4 0/7] xilinxfb changes

On 06/17/2013 11:07 PM, Arnd Bergmann wrote:
> On Monday 17 June 2013, Michal Simek wrote:
>> On 06/17/2013 10:56 AM, Jean-Christophe PLAGNIOL-VILLARD wrote:
>>> On 07:23 Mon 17 Jun , Michal Simek wrote:
>>>> On 06/06/2013 06:23 PM, Jean-Christophe PLAGNIOL-VILLARD wrote:
>>>>> On 12:13 Mon 03 Jun , Michal Simek wrote:
>>>>>> Hi,
>>>>>>
>>>>>
>>>>> Arnd can you take look on it again please
>>>>>
>>>>> I'll take a look on it next week
>>>>
>>>> Any update on this?
>>> look ok but I want the Ack from Arnd
>
> Sorry for the delay, everything looks good to me.
>
> Acked-by: Arnd Bergmann <[email protected]>

Jean-Christophe: Will you apply this series?
Or should I take it through my microblaze tree?

Thanks,
Michal


--
Michal Simek, Ing. (M.Eng), OpenPGP -> KeyID: FE3D1F91
w: http://www.monstr.eu p: +42-0-721842854
Maintainer of Linux kernel - Microblaze cpu - http://www.monstr.eu/fdt/
Maintainer of Linux kernel - Xilinx Zynq ARM architecture
Microblaze U-BOOT custodian and responsible for u-boot arm zynq platform



Attachments:
signature.asc (263.00 B)
OpenPGP digital signature

2013-06-26 11:42:11

by Tomi Valkeinen

[permalink] [raw]
Subject: Re: [PATCH v4 0/7] xilinxfb changes

Hi,

On 26/06/13 13:02, Michal Simek wrote:
> On 06/17/2013 11:07 PM, Arnd Bergmann wrote:
>> On Monday 17 June 2013, Michal Simek wrote:
>>> On 06/17/2013 10:56 AM, Jean-Christophe PLAGNIOL-VILLARD wrote:
>>>> On 07:23 Mon 17 Jun , Michal Simek wrote:
>>>>> On 06/06/2013 06:23 PM, Jean-Christophe PLAGNIOL-VILLARD wrote:
>>>>>> On 12:13 Mon 03 Jun , Michal Simek wrote:
>>>>>>> Hi,
>>>>>>>
>>>>>>
>>>>>> Arnd can you take look on it again please
>>>>>>
>>>>>> I'll take a look on it next week
>>>>>
>>>>> Any update on this?
>>>> look ok but I want the Ack from Arnd
>>
>> Sorry for the delay, everything looks good to me.
>>
>> Acked-by: Arnd Bergmann <[email protected]>
>
> Jean-Christophe: Will you apply this series?
> Or should I take it through my microblaze tree?

I've added this to my fbdev-3.11 branch.

Tomi



Attachments:
signature.asc (901.00 B)
OpenPGP digital signature

2013-06-26 12:51:45

by Michal Simek

[permalink] [raw]
Subject: Re: [PATCH v4 0/7] xilinxfb changes

On 06/26/2013 01:41 PM, Tomi Valkeinen wrote:
> Hi,
>
> On 26/06/13 13:02, Michal Simek wrote:
>> On 06/17/2013 11:07 PM, Arnd Bergmann wrote:
>>> On Monday 17 June 2013, Michal Simek wrote:
>>>> On 06/17/2013 10:56 AM, Jean-Christophe PLAGNIOL-VILLARD wrote:
>>>>> On 07:23 Mon 17 Jun , Michal Simek wrote:
>>>>>> On 06/06/2013 06:23 PM, Jean-Christophe PLAGNIOL-VILLARD wrote:
>>>>>>> On 12:13 Mon 03 Jun , Michal Simek wrote:
>>>>>>>> Hi,
>>>>>>>>
>>>>>>>
>>>>>>> Arnd can you take look on it again please
>>>>>>>
>>>>>>> I'll take a look on it next week
>>>>>>
>>>>>> Any update on this?
>>>>> look ok but I want the Ack from Arnd
>>>
>>> Sorry for the delay, everything looks good to me.
>>>
>>> Acked-by: Arnd Bergmann <[email protected]>
>>
>> Jean-Christophe: Will you apply this series?
>> Or should I take it through my microblaze tree?
>
> I've added this to my fbdev-3.11 branch.

Thanks,
Michal

--
Michal Simek, Ing. (M.Eng), OpenPGP -> KeyID: FE3D1F91
w: http://www.monstr.eu p: +42-0-721842854
Maintainer of Linux kernel - Microblaze cpu - http://www.monstr.eu/fdt/
Maintainer of Linux kernel - Xilinx Zynq ARM architecture
Microblaze U-BOOT custodian and responsible for u-boot arm zynq platform



Attachments:
signature.asc (263.00 B)
OpenPGP digital signature