[00/11] sunxi: arm64 binary size fixes

Message ID 20171221124030.23721-1-maxime.ripard@free-electrons.com
Headers show
Series
  • sunxi: arm64 binary size fixes
Related show

Message

Maxime Ripard Dec. 21, 2017, 12:40 p.m.
Hi,

As we discussed already, this is the list of options that we will need
to disable by default.

I also added to the mix a size check, which even though it's not
really optimal, is temporary and will be removed in a future (and
hopefully close) release.

Tom, is this still something we can merge for 2018.01?

Thanks!
Maxime

Andre Heider (1):
  sunxi: imply CONFIG_OF_LIBFDT_OVERLAY

Maxime Ripard (10):
  cmd: crc32: Disable by default on sunXi
  cmd: unzip: Disable by default on sunXi
  cmd: loadb: Disable by default on sunXi
  cmd: loads: Disable by default on sunXi
  cmd: misc: Disable by default on sunXi
  video: bpp8: Disable by default on sunXi
  video: bpp16: Disable by default on sunXi
  net: regex: Disable by default on sunXi
  Makefile: Add size check to the u-boot.itb make target
  sunxi: Add limit with the MMC environment

 Makefile                       |  1 +
 arch/arm/Kconfig               |  1 +
 cmd/Kconfig                    |  5 +++++
 configs/CHIP_defconfig         |  1 -
 configs/CHIP_pro_defconfig     |  1 -
 drivers/video/Kconfig          |  2 ++
 include/configs/sunxi-common.h | 10 ++++++++++
 lib/Kconfig                    |  1 +
 8 files changed, 20 insertions(+), 2 deletions(-)

Comments

Tom Rini Dec. 21, 2017, 12:56 p.m. | #1
On Thu, Dec 21, 2017 at 01:40:19PM +0100, Maxime Ripard wrote:

> Hi,

> 

> As we discussed already, this is the list of options that we will need

> to disable by default.

> 

> I also added to the mix a size check, which even though it's not

> really optimal, is temporary and will be removed in a future (and

> hopefully close) release.

> 

> Tom, is this still something we can merge for 2018.01?


Everything looks reasonable, and we need to fix this, so yes, thanks!

-- 
Tom
Jagan Teki Jan. 4, 2018, 5:28 a.m. | #2
Maxime,

On Thu, Dec 21, 2017 at 6:10 PM, Maxime Ripard
<maxime.ripard@free-electrons.com> wrote:
> Hi,
>
> As we discussed already, this is the list of options that we will need
> to disable by default.
>
> I also added to the mix a size check, which even though it's not
> really optimal, is temporary and will be removed in a future (and
> hopefully close) release.
>
> Tom, is this still something we can merge for 2018.01?
>
> Thanks!
> Maxime
>
> Andre Heider (1):
>   sunxi: imply CONFIG_OF_LIBFDT_OVERLAY
>
> Maxime Ripard (10):
>   cmd: crc32: Disable by default on sunXi
>   cmd: unzip: Disable by default on sunXi
>   cmd: loadb: Disable by default on sunXi
>   cmd: loads: Disable by default on sunXi
>   cmd: misc: Disable by default on sunXi
>   video: bpp8: Disable by default on sunXi
>   video: bpp16: Disable by default on sunXi
>   net: regex: Disable by default on sunXi
>   Makefile: Add size check to the u-boot.itb make target
>   sunxi: Add limit with the MMC environment

based on the conversation on individual patches, I think all these
need to change only for 64-bit sunxi boards for now and we can push
this for the release if all fine.
Maxime Ripard Jan. 5, 2018, 10 a.m. | #3
On Thu, Jan 04, 2018 at 10:58:16AM +0530, Jagan Teki wrote:
> Maxime,

> 

> On Thu, Dec 21, 2017 at 6:10 PM, Maxime Ripard

> <maxime.ripard@free-electrons.com> wrote:

> > Hi,

> >

> > As we discussed already, this is the list of options that we will need

> > to disable by default.

> >

> > I also added to the mix a size check, which even though it's not

> > really optimal, is temporary and will be removed in a future (and

> > hopefully close) release.

> >

> > Tom, is this still something we can merge for 2018.01?

> >

> > Thanks!

> > Maxime

> >

> > Andre Heider (1):

> >   sunxi: imply CONFIG_OF_LIBFDT_OVERLAY

> >

> > Maxime Ripard (10):

> >   cmd: crc32: Disable by default on sunXi

> >   cmd: unzip: Disable by default on sunXi

> >   cmd: loadb: Disable by default on sunXi

> >   cmd: loads: Disable by default on sunXi

> >   cmd: misc: Disable by default on sunXi

> >   video: bpp8: Disable by default on sunXi

> >   video: bpp16: Disable by default on sunXi

> >   net: regex: Disable by default on sunXi

> >   Makefile: Add size check to the u-boot.itb make target

> >   sunxi: Add limit with the MMC environment

> 

> based on the conversation on individual patches, I think all these

> need to change only for 64-bit sunxi boards for now and we can push

> this for the release if all fine.


As far as I'm concerned, these patches can be merged as is.

Maxime

-- 
Maxime Ripard, Free Electrons
Embedded Linux and Kernel engineering
http://free-electrons.com
Jagan Teki Jan. 5, 2018, 10:16 a.m. | #4
On Fri, Jan 5, 2018 at 3:30 PM, Maxime Ripard
<maxime.ripard@free-electrons.com> wrote:
> On Thu, Jan 04, 2018 at 10:58:16AM +0530, Jagan Teki wrote:
>> Maxime,
>>
>> On Thu, Dec 21, 2017 at 6:10 PM, Maxime Ripard
>> <maxime.ripard@free-electrons.com> wrote:
>> > Hi,
>> >
>> > As we discussed already, this is the list of options that we will need
>> > to disable by default.
>> >
>> > I also added to the mix a size check, which even though it's not
>> > really optimal, is temporary and will be removed in a future (and
>> > hopefully close) release.
>> >
>> > Tom, is this still something we can merge for 2018.01?
>> >
>> > Thanks!
>> > Maxime
>> >
>> > Andre Heider (1):
>> >   sunxi: imply CONFIG_OF_LIBFDT_OVERLAY
>> >
>> > Maxime Ripard (10):
>> >   cmd: crc32: Disable by default on sunXi
>> >   cmd: unzip: Disable by default on sunXi
>> >   cmd: loadb: Disable by default on sunXi
>> >   cmd: loads: Disable by default on sunXi
>> >   cmd: misc: Disable by default on sunXi
>> >   video: bpp8: Disable by default on sunXi
>> >   video: bpp16: Disable by default on sunXi
>> >   net: regex: Disable by default on sunXi
>> >   Makefile: Add size check to the u-boot.itb make target
>> >   sunxi: Add limit with the MMC environment
>>
>> based on the conversation on individual patches, I think all these
>> need to change only for 64-bit sunxi boards for now and we can push
>> this for the release if all fine.
>
> As far as I'm concerned, these patches can be merged as is.

I've no issues with this series if it goes to next version, since we
have time to validate. But I'm thinking to push these asap to merge
them in coming release in two days, it would be good to untouch 32-bit
stuff at last minute.