Re: [powerpc:next-test 21/79] arch/powerpc/platforms/powermac/feature.c:137:19: error: unused function 'simple_feature_tweak'

2023-08-19 Thread Michael Ellerman
"Arnd Bergmann"  writes:
> On Fri, Aug 18, 2023, at 09:59, kernel test robot wrote:
>> tree:   
>> https://git.kernel.org/pub/scm/linux/kernel/git/powerpc/linux.git 
>> next-test
>> head:   c56963d48343b50998bacc6df949217c950163e3
>> commit: 54f30b83fe627453082f15d83d7820b28b2d24bb [21/79] powerpc: 
>> address missing-prototypes warnings
>
>>
>> If you fix the issue in a separate patch/commit (i.e. not just a new 
>> version of
>> the same patch/commit), kindly add following tags
>> | Reported-by: kernel test robot 
>> | Closes: 
>> https://lore.kernel.org/oe-kbuild-all/202308181501.ar5hmdwc-...@intel.com/
>>
>> All errors (new ones prefixed by >>):
>>
 arch/powerpc/platforms/powermac/feature.c:137:19: error: unused function 
 'simple_feature_tweak' [-Werror,-Wunused-function]
>>static inline int simple_feature_tweak(struct device_node *node, int type,
>>  ^
>
> This looks unrelated to my patch, but is easily addressed by marking
> the function __maybe_unused. The warning is triggered by clang specific
> behavior that treats inline functions in .c files differently, see
> 6863f5643dd7 ("kbuild: allow Clang to find unused static inline functions
> for W=1 build")

It looks like it can just move inside the nearby ifdef. Will send a
patch.

cheers


Re: [GIT PULL] Please pull powerpc/linux.git powerpc-6.5-6 tag

2023-08-19 Thread pr-tracker-bot
The pull request you sent on Sat, 19 Aug 2023 16:30:39 +1000:

> https://git.kernel.org/pub/scm/linux/kernel/git/powerpc/linux.git 
> tags/powerpc-6.5-6

has been merged into torvalds/linux.git:
https://git.kernel.org/torvalds/c/4e7ffde6984a7fa842489be7055570e5f5a4f0b5

Thank you!

-- 
Deet-doot-dot, I am a bot.
https://korg.docs.kernel.org/prtracker.html


Re: [PATCH] treewide: drop CONFIG_EMBEDDED

2023-08-19 Thread Christophe Leroy
Hi,

Le 19/08/2023 à 05:33, Jesse T a écrit :
>>>
 Should there be a warning here to update change it instead of removal?
>>>
>>> kconfig doesn't have a warning mechanism AFAIK.
>>> Do you have an idea of how this would work?
> 
> No, unfortunately. As you said without a warning it would be overlooked so
> a change would not be necessary.
> 
> A possible solution is to check in a header file with:
> 
> #ifdef CONFIG_EMBEDDED
> #warning "CONFIG_EMBEDDED has changed to CONFIG_EXPERT"
> #endif
> 
> Does anyone else have an opinion on this?

My opinion is that has happen several times in the past and will happen 
again. It is not a big deal, whoever updates to a new kernel will make a 
savedefconfig and compare with previous defconfig and see what has 
changed. Once you see that CONFIG_EMBEDDED is disappearing you look at 
kernel history to find out why CONFIG_EMBEDDED disappears, and you 
understand from the commit message that you have to select CONFIG_EXPERT 
instead.

A couple examples I have in mind from the past:
- CONFIG_FORCE_MAX_ZONEORDER became CONFIG_ARCH_FORCE_MAX_ORDER
- CONFIG_MTD_NAND became CONFIG_MTD_RAW_NAND

> Since kconfig doesn't have a warning mechanism the patch seems fine as is.

So yes the patch is fine as is IMHO.

Christophe


Re: [PATCH v4 27/28] ASoC: codecs: Add support for the framer codec

2023-08-19 Thread Christophe Leroy


Le 19/08/2023 à 01:18, Randy Dunlap a écrit :
> Hi,
> 
> On 8/18/23 09:39, Christophe Leroy wrote:
>> +config SND_SOC_FRAMER
>> +tristate "Framer codec"
>> +depends on GENERIC_FRAMER
>> +help
>> +  Enable support for the framer codec.
>> +  The framer codec uses the generic framer infrastructure to transport
>> +  some audio data over an analog E1/T1/J1 line.
>> +  This codec allows to use some of the time slots available on the TDM
>> +  bus on which the framer is connected to transport the audio data.
>> +
> 
> Just curious: what controls the slot allocations/usages?
> Is that done in userspace?

For audio, this is done in userspace through alsalib.

For IP over E1, a mask is provided with the userspace tool 'sethdlc'

For the time being there is no sharing, either the E1 line is used for 
audio either the E1 line is used for IP over E1 (HDLC)

Christophe


[Bug 216156] kmemleak: Not scanning unknown object at 0xc00000007f000000

2023-08-19 Thread bugzilla-daemon
https://bugzilla.kernel.org/show_bug.cgi?id=216156

Erhard F. (erhar...@mailbox.org) changed:

   What|Removed |Added

 Attachment #301230|0   |1
is obsolete||
 Attachment #301232|0   |1
is obsolete||
 Attachment #303178|0   |1
is obsolete||

--- Comment #8 from Erhard F. (erhar...@mailbox.org) ---
Created attachment 304914
  --> https://bugzilla.kernel.org/attachment.cgi?id=304914&action=edit
dmesg (6.5-rc6, PowerMac G5 11,2)

Kernel 6.5-rc6 still affected.

[...]
PowerMac motherboard: PowerMac G5 Dual Core
ioremap() called early from btext_map+0x6c/0xf0. Use early_ioremap() instead
ioremap() called early from iommu_init_early_dart+0x270/0x948. Use
early_ioremap() instead
kmemleak: Not scanning unknown object at 0xc0007f00
CPU: 0 PID: 0 Comm: swapper Tainted: GT  6.5.0-rc6-PMacG5-dirty
#1
Call Trace:
[c241fc70] [c0ff726c] dump_stack_lvl+0xb0/0x11c (unreliable)
[c241fcb0] [c048fdc8] kmemleak_no_scan+0x118/0x130
[c241fd20] [c2021ad0] iommu_init_early_dart+0x300/0x948
[c241fe30] [c2022bf8] pmac_probe+0x1b8/0x228
[c241fe80] [c200ec24] setup_arch+0x270/0x7d4
[c241ff30] [c2004f14] start_kernel+0xc0/0x850
[c241ffe0] [c000cb48] start_here_common+0x1c/0x20
DART table allocated at: (ptrval)
DART IOMMU initialized for U4 type chipset
Hardware name: PowerMac11,2 PPC970MP 0x440101 PowerMac
printk: bootconsole [udbg0] enabled
CPU maps initialized for 1 thread per core
[...]

-- 
You may reply to this email to add a comment.

You are receiving this mail because:
You are watching the assignee of the bug.

[Bug 216156] kmemleak: Not scanning unknown object at 0xc00000007f000000

2023-08-19 Thread bugzilla-daemon
https://bugzilla.kernel.org/show_bug.cgi?id=216156

Erhard F. (erhar...@mailbox.org) changed:

   What|Removed |Added

 Attachment #301231|0   |1
is obsolete||
 Attachment #303179|0   |1
is obsolete||

--- Comment #9 from Erhard F. (erhar...@mailbox.org) ---
Created attachment 304915
  --> https://bugzilla.kernel.org/attachment.cgi?id=304915&action=edit
kernel .config (6.5-rc6, PowerMac G5 11,2)

-- 
You may reply to this email to add a comment.

You are receiving this mail because:
You are watching the assignee of the bug.

[Bug 216902] WARNING: CPU: 0 PID: 70 at kernel/locking/irqflag-debug.c:10 at kallsyms_selftest

2023-08-19 Thread bugzilla-daemon
https://bugzilla.kernel.org/show_bug.cgi?id=216902

Erhard F. (erhar...@mailbox.org) changed:

   What|Removed |Added

 Status|NEW |RESOLVED
 Resolution|--- |OBSOLETE

--- Comment #2 from Erhard F. (erhar...@mailbox.org) ---
Have not seen this on kernel 6.4.x and latest 6.5-rc6.

Closing.

-- 
You may reply to this email to add a comment.

You are receiving this mail because:
You are watching the assignee of the bug.