[U-Boot] [PATCH] watchdog: dm: Support manual relocation for watchdogs

[Simon Glass]

Hi,

On 19 July 2018 at 09:47, Tom Rini <trini at konsulko.com> wrote:
> On Thu, Jul 19, 2018 at 09:21:39AM -0600, Simon Glass wrote:
>> +Tom
>>
>> Hi Michal,
>>
>> On 19 July 2018 at 00:52, Michal Simek <michal.simek at xilinx.com> wrote:
>> > Hi Simon,
>> >
>> > On 19.7.2018 03:31, Simon Glass wrote:
>> >> Hi Michal,
>> >>
>> >> On 15 July 2018 at 23:34, Michal Simek <michal.simek at xilinx.com> wrote:
>> >>> On 15.7.2018 23:21, Simon Glass wrote:
>> >>>> Hi Michal,
>> >>>>
>> >>>> On 11 July 2018 at 23:47, Michal Simek <michal.simek at xilinx.com> wrote:
>> >>>>> On 11.7.2018 22:13, Simon Glass wrote:
>> >>>>>> On 11 July 2018 at 08:41, Michal Simek <michal.simek at xilinx.com> wrote:
>> >>>>>>> Relocate watchdog ops as was done by:
>> >>>>>>> "dm: Add support for all targets which requires MANUAL_RELOC"
>> >>>>>>> (sha1: 484fdf5ba058b07be5ca82763aa2b72063540ef3)
>> >>>>>>>
>> >>>>>>> Signed-off-by: Michal Simek <michal.simek at xilinx.com>
>> >>>>>>> ---
>> >>>>>>>
>> >>>>>>> based on https://lists.denx.de/pipermail/u-boot/2018-July/334227.html
>> >>>>>>>
>> >>>>>>> ---
>> >>>>>>>  drivers/watchdog/wdt-uclass.c | 23 +++++++++++++++++++++++
>> >>>>>>>  1 file changed, 23 insertions(+)
>> >>>>>>>
>> >>>>>>
>> >>>>>> Reviewed-by: Simon Glass <sjg at chromium.org>
>> >>>>>>
>> >>>>>> When will the toolchain be fixed?
>> >>>>>
>> >>>>> It is really several years back when I have looked at it last time but I
>> >>>>> think that toolchain is fixed for quite some time and only changes in
>> >>>>> microblaze u-boot code are needed but really I would have to check and
>> >>>>> start to play with it.
>> >>>>
>> >>>> I think someone should sort this out. It would be good to remove this
>> >>>> code. Is there a toolchain group at Xilinx?
>> >>>
>> >>> Xilinx has a toolchain group. I just looked a I was playing with it in
>> >>> January 2015 but didn't finish that. It is still on my long todo list.
>> >>> Will see when I have a time to look at it.
>> >>
>> >> Hoe about next week? -:) I think this is pretty important.
>> >
>> > will see but I need to do some Linux work first. Based on grep I see
>> > that m68k is also enabling CONFIG_NEEDS_MANUAL_RELOC.
>> > Is m68k going to be removed soon?
>>
>> I am not sure about that. Tom, do you know?
>
> m68k is still active, so no, it's not being removed and we can't drop
> CONFIG_NEEDS_MANUAL_RELOC just yet.

Thanks Tom.

That begs the question - how is relocation support coming along with m68k?

Regards,
Simon