All of lore.kernel.org
 help / color / mirror / Atom feed
* [U-Boot] [GIT] Pull request: u-boot-dfu (04.01.2019)
@ 2019-01-04  7:01 Lukasz Majewski
  2019-01-04 15:07 ` Marek Vasut
  0 siblings, 1 reply; 6+ messages in thread
From: Lukasz Majewski @ 2019-01-04  7:01 UTC (permalink / raw)
  To: u-boot

Dear Marek,

The following changes since commit
7436f5e54d35bcad53befec90e2e67288071f74e:

  Merge tag 'for-master-20190103' of git://git.denx.de/u-boot-rockchip
  (2019-01-03 08:39:44 -0500)

are available in the git repository at:

  git://git.denx.de/u-boot-dfu.git 

for you to fetch changes up to deb73d0aea6ca09ecb56ea3431d3151427fff4a4:

  dm: usb: gadget: Fix boot breakage on sunxi platforms (2019-01-03
  22:43:02 +0100)

----------------------------------------------------------------
Jean-Jacques Hiblot (3):
      dm: usb: udc: Use SEQ_ALIAS to index the USB gadget ports
      ARM: dts: define USB aliases for all omap5 platforms
      dm: usb: gadget: Fix boot breakage on sunxi platforms

 arch/arm/dts/omap5-u-boot.dtsi      | 5 +++++
 drivers/usb/gadget/udc/Makefile     | 3 ++-
 drivers/usb/gadget/udc/udc-uclass.c | 7 +++++--
 3 files changed, 12 insertions(+), 3 deletions(-)


Travis-Ci: https://travis-ci.org/lmajewski/u-boot-dfu/builds/475053011


Best regards,

Lukasz Majewski

--

DENX Software Engineering GmbH,      Managing Director: Wolfgang Denk
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: (+49)-8142-66989-59 Fax: (+49)-8142-66989-80 Email: lukma at denx.de
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 488 bytes
Desc: OpenPGP digital signature
URL: <http://lists.denx.de/pipermail/u-boot/attachments/20190104/f7660fdc/attachment.sig>

^ permalink raw reply	[flat|nested] 6+ messages in thread

* [U-Boot] [GIT] Pull request: u-boot-dfu (04.01.2019)
  2019-01-04  7:01 [U-Boot] [GIT] Pull request: u-boot-dfu (04.01.2019) Lukasz Majewski
@ 2019-01-04 15:07 ` Marek Vasut
  2019-01-04 15:17   ` Jean-Jacques Hiblot
  0 siblings, 1 reply; 6+ messages in thread
From: Marek Vasut @ 2019-01-04 15:07 UTC (permalink / raw)
  To: u-boot

On 1/4/19 8:01 AM, Lukasz Majewski wrote:
> Dear Marek,
> 
> The following changes since commit
> 7436f5e54d35bcad53befec90e2e67288071f74e:
> 
>   Merge tag 'for-master-20190103' of git://git.denx.de/u-boot-rockchip
>   (2019-01-03 08:39:44 -0500)
> 
> are available in the git repository at:
> 
>   git://git.denx.de/u-boot-dfu.git 
> 
> for you to fetch changes up to deb73d0aea6ca09ecb56ea3431d3151427fff4a4:
> 
>   dm: usb: gadget: Fix boot breakage on sunxi platforms (2019-01-03
>   22:43:02 +0100)
> 
> ----------------------------------------------------------------
> Jean-Jacques Hiblot (3):
>       dm: usb: udc: Use SEQ_ALIAS to index the USB gadget ports
>       ARM: dts: define USB aliases for all omap5 platforms
>       dm: usb: gadget: Fix boot breakage on sunxi platforms

Why is udc-uclass.c compiled in when CONFIG_(SPL_)DM is enabled ?
This doesn't make sense, I don't want any UDC code in builds which
enable just DM .

-- 
Best regards,
Marek Vasut

^ permalink raw reply	[flat|nested] 6+ messages in thread

* [U-Boot] [GIT] Pull request: u-boot-dfu (04.01.2019)
  2019-01-04 15:07 ` Marek Vasut
@ 2019-01-04 15:17   ` Jean-Jacques Hiblot
  2019-01-04 18:51     ` Marek Vasut
  0 siblings, 1 reply; 6+ messages in thread
From: Jean-Jacques Hiblot @ 2019-01-04 15:17 UTC (permalink / raw)
  To: u-boot


On 04/01/2019 16:07, Marek Vasut wrote:
> On 1/4/19 8:01 AM, Lukasz Majewski wrote:
>> Dear Marek,
>>
>> The following changes since commit
>> 7436f5e54d35bcad53befec90e2e67288071f74e:
>>
>>    Merge tag 'for-master-20190103' of git://git.denx.de/u-boot-rockchip
>>    (2019-01-03 08:39:44 -0500)
>>
>> are available in the git repository at:
>>
>>    git://git.denx.de/u-boot-dfu.git
>>
>> for you to fetch changes up to deb73d0aea6ca09ecb56ea3431d3151427fff4a4:
>>
>>    dm: usb: gadget: Fix boot breakage on sunxi platforms (2019-01-03
>>    22:43:02 +0100)
>>
>> ----------------------------------------------------------------
>> Jean-Jacques Hiblot (3):
>>        dm: usb: udc: Use SEQ_ALIAS to index the USB gadget ports
>>        ARM: dts: define USB aliases for all omap5 platforms
>>        dm: usb: gadget: Fix boot breakage on sunxi platforms
> Why is udc-uclass.c compiled in when CONFIG_(SPL_)DM is enabled ?
> This doesn't make sense, I don't want any UDC code in builds which
> enable just DM .

It only adds the UCLASS_DRIVER definition for the USB gadget. The rest 
is compiled out inside udc-uclass.c



>

^ permalink raw reply	[flat|nested] 6+ messages in thread

* [U-Boot] [GIT] Pull request: u-boot-dfu (04.01.2019)
  2019-01-04 15:17   ` Jean-Jacques Hiblot
@ 2019-01-04 18:51     ` Marek Vasut
  2019-01-07 14:51       ` Jean-Jacques Hiblot
  0 siblings, 1 reply; 6+ messages in thread
From: Marek Vasut @ 2019-01-04 18:51 UTC (permalink / raw)
  To: u-boot

On 1/4/19 4:17 PM, Jean-Jacques Hiblot wrote:
> 
> On 04/01/2019 16:07, Marek Vasut wrote:
>> On 1/4/19 8:01 AM, Lukasz Majewski wrote:
>>> Dear Marek,
>>>
>>> The following changes since commit
>>> 7436f5e54d35bcad53befec90e2e67288071f74e:
>>>
>>>    Merge tag 'for-master-20190103' of git://git.denx.de/u-boot-rockchip
>>>    (2019-01-03 08:39:44 -0500)
>>>
>>> are available in the git repository at:
>>>
>>>    git://git.denx.de/u-boot-dfu.git
>>>
>>> for you to fetch changes up to deb73d0aea6ca09ecb56ea3431d3151427fff4a4:
>>>
>>>    dm: usb: gadget: Fix boot breakage on sunxi platforms (2019-01-03
>>>    22:43:02 +0100)
>>>
>>> ----------------------------------------------------------------
>>> Jean-Jacques Hiblot (3):
>>>        dm: usb: udc: Use SEQ_ALIAS to index the USB gadget ports
>>>        ARM: dts: define USB aliases for all omap5 platforms
>>>        dm: usb: gadget: Fix boot breakage on sunxi platforms
>> Why is udc-uclass.c compiled in when CONFIG_(SPL_)DM is enabled ?
>> This doesn't make sense, I don't want any UDC code in builds which
>> enable just DM .
> 
> It only adds the UCLASS_DRIVER definition for the USB gadget. The rest
> is compiled out inside udc-uclass.c

Why do we want it in unconditionally at all ?

-- 
Best regards,
Marek Vasut

^ permalink raw reply	[flat|nested] 6+ messages in thread

* [U-Boot] [GIT] Pull request: u-boot-dfu (04.01.2019)
  2019-01-04 18:51     ` Marek Vasut
@ 2019-01-07 14:51       ` Jean-Jacques Hiblot
  2019-01-07 14:59         ` Marek Vasut
  0 siblings, 1 reply; 6+ messages in thread
From: Jean-Jacques Hiblot @ 2019-01-07 14:51 UTC (permalink / raw)
  To: u-boot


On 04/01/2019 19:51, Marek Vasut wrote:
> On 1/4/19 4:17 PM, Jean-Jacques Hiblot wrote:
>> On 04/01/2019 16:07, Marek Vasut wrote:
>>> On 1/4/19 8:01 AM, Lukasz Majewski wrote:
>>>> Dear Marek,
>>>>
>>>> The following changes since commit
>>>> 7436f5e54d35bcad53befec90e2e67288071f74e:
>>>>
>>>>     Merge tag 'for-master-20190103' of git://git.denx.de/u-boot-rockchip
>>>>     (2019-01-03 08:39:44 -0500)
>>>>
>>>> are available in the git repository at:
>>>>
>>>>     git://git.denx.de/u-boot-dfu.git
>>>>
>>>> for you to fetch changes up to deb73d0aea6ca09ecb56ea3431d3151427fff4a4:
>>>>
>>>>     dm: usb: gadget: Fix boot breakage on sunxi platforms (2019-01-03
>>>>     22:43:02 +0100)
>>>>
>>>> ----------------------------------------------------------------
>>>> Jean-Jacques Hiblot (3):
>>>>         dm: usb: udc: Use SEQ_ALIAS to index the USB gadget ports
>>>>         ARM: dts: define USB aliases for all omap5 platforms
>>>>         dm: usb: gadget: Fix boot breakage on sunxi platforms
>>> Why is udc-uclass.c compiled in when CONFIG_(SPL_)DM is enabled ?
>>> This doesn't make sense, I don't want any UDC code in builds which
>>> enable just DM .
>> It only adds the UCLASS_DRIVER definition for the USB gadget. The rest
>> is compiled out inside udc-uclass.c
> Why do we want it in unconditionally at all ?

We don't really. I'll post something to make it conditional on USB_GADGET.

I'll take the opportunity to rename SPL_USB_GADGET_SUPPORT to 
SPL_USB_GADGET for consistency.

But this may take some time to verify (buildman and also boot check), 
and the current version seems to work for everyone so far.

JJ





>

^ permalink raw reply	[flat|nested] 6+ messages in thread

* [U-Boot] [GIT] Pull request: u-boot-dfu (04.01.2019)
  2019-01-07 14:51       ` Jean-Jacques Hiblot
@ 2019-01-07 14:59         ` Marek Vasut
  0 siblings, 0 replies; 6+ messages in thread
From: Marek Vasut @ 2019-01-07 14:59 UTC (permalink / raw)
  To: u-boot

On 1/7/19 3:51 PM, Jean-Jacques Hiblot wrote:
> 
> On 04/01/2019 19:51, Marek Vasut wrote:
>> On 1/4/19 4:17 PM, Jean-Jacques Hiblot wrote:
>>> On 04/01/2019 16:07, Marek Vasut wrote:
>>>> On 1/4/19 8:01 AM, Lukasz Majewski wrote:
>>>>> Dear Marek,
>>>>>
>>>>> The following changes since commit
>>>>> 7436f5e54d35bcad53befec90e2e67288071f74e:
>>>>>
>>>>>     Merge tag 'for-master-20190103' of
>>>>> git://git.denx.de/u-boot-rockchip
>>>>>     (2019-01-03 08:39:44 -0500)
>>>>>
>>>>> are available in the git repository at:
>>>>>
>>>>>     git://git.denx.de/u-boot-dfu.git
>>>>>
>>>>> for you to fetch changes up to
>>>>> deb73d0aea6ca09ecb56ea3431d3151427fff4a4:
>>>>>
>>>>>     dm: usb: gadget: Fix boot breakage on sunxi platforms (2019-01-03
>>>>>     22:43:02 +0100)
>>>>>
>>>>> ----------------------------------------------------------------
>>>>> Jean-Jacques Hiblot (3):
>>>>>         dm: usb: udc: Use SEQ_ALIAS to index the USB gadget ports
>>>>>         ARM: dts: define USB aliases for all omap5 platforms
>>>>>         dm: usb: gadget: Fix boot breakage on sunxi platforms
>>>> Why is udc-uclass.c compiled in when CONFIG_(SPL_)DM is enabled ?
>>>> This doesn't make sense, I don't want any UDC code in builds which
>>>> enable just DM .
>>> It only adds the UCLASS_DRIVER definition for the USB gadget. The rest
>>> is compiled out inside udc-uclass.c
>> Why do we want it in unconditionally at all ?
> 
> We don't really. I'll post something to make it conditional on USB_GADGET.
> 
> I'll take the opportunity to rename SPL_USB_GADGET_SUPPORT to
> SPL_USB_GADGET for consistency.
> 
> But this may take some time to verify (buildman and also boot check),
> and the current version seems to work for everyone so far.

That's fine, there's still ~1 week till the release.

-- 
Best regards,
Marek Vasut

^ permalink raw reply	[flat|nested] 6+ messages in thread

end of thread, other threads:[~2019-01-07 14:59 UTC | newest]

Thread overview: 6+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2019-01-04  7:01 [U-Boot] [GIT] Pull request: u-boot-dfu (04.01.2019) Lukasz Majewski
2019-01-04 15:07 ` Marek Vasut
2019-01-04 15:17   ` Jean-Jacques Hiblot
2019-01-04 18:51     ` Marek Vasut
2019-01-07 14:51       ` Jean-Jacques Hiblot
2019-01-07 14:59         ` Marek Vasut

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.