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

[Tom Rini]

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.

-- 
Tom
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 819 bytes
Desc: not available
URL: <http://lists.denx.de/pipermail/u-boot/attachments/20180719/61ae9a61/attachment.sig>